Dear Lukasz Majewski,

> Support for MMC storage devices to work with DFU framework.
> 
> Signed-off-by: Lukasz Majewski <l.majew...@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
> Cc: Marek Vasut <ma...@denx.de>
> ---

[...]

> +
> +#include <common.h>
> +#include <malloc.h>
> +#include <dfu.h>
> +
> +int dfu_write_medium_mmc(struct dfu_entity *dfu, void *buf, long *len)
> +{
> +     ALLOC_CACHE_ALIGN_BUFFER(char, cmd_buf, DFU_CMD_BUF_SIZE);
> +
> +     memset(cmd_buf, '\0', sizeof(cmd_buf));
> +
> +     switch (dfu->layout) {
> +     case RAW_ADDR:
> +             sprintf(cmd_buf, "mmc write 0x%x %x %x", (unsigned int) buf,
> +                     dfu->data.mmc.lba_start, dfu->data.mmc.lba_size);
> +             break;
> +     case FAT:
> +             sprintf(cmd_buf, "fatwrite mmc %d:%d 0x%x %s %lx",
> +                     dfu->data.mmc.dev, dfu->data.mmc.part,
> +                     (unsigned int) buf, dfu->name, *len);
> +             break;
> +     default:
> +             printf("%s: Wrong layout!\n", __func__);
> +     }
> +
> +     debug("%s: %s 0x%p\n", __func__, cmd_buf, cmd_buf);
> +     run_command(cmd_buf, 0);

Holy Moly ... can we not make this into simple calls to those subsystems ? 
Instead invoking command is crazy ;-)

[...]

Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to