Dear Marek Vasut,
In message 201207271515.36085.marex@denx.de you wrote:
sprintf(cmd_buf, "%s %s %d:%d 0x%x %s %lx", command, device, ...
and set command as needed to "fatread" or "ext2load" or whatever, and device can be set to "mmc" or "usb" or ...
Given that they have slightly different syntax, this is crazy too.
Draw the line between clever and crazy, genius and idiot :-)
Well ... after this discussion, I feel like I'll just go dig me a grave again.
Why? I think the discussion has been beneficial - we isolated a poor piece of code, discussed how it should be improved, and now ry to find an intermediate solution that adds not too much work for Lukasz while still keeing the existing code clean and allowing for easy adaption to the new DM interfaces - three worthwile goals at once.
Best regards,
Wolfgang Denk