[U-Boot] [PATCH v3] board: sun50i: Add Amarula A64-Relic initial support

[Maxime Ripard]

On Thu, May 31, 2018 at 03:13:15PM +0530, Jagan Teki wrote:
> On Thu, May 31, 2018 at 3:07 PM, Maxime Ripard
> <maxime.ripard at bootlin.com> wrote:
> > On Tue, May 29, 2018 at 09:50:39PM +0530, Jagan Teki wrote:
> >> On Tue, May 29, 2018 at 9:44 PM, Maxime Ripard
> >> <maxime.ripard at bootlin.com> wrote:
> >> > On Mon, May 28, 2018 at 04:34:43PM +0530, Jagan Teki wrote:
> >> >> +CONFIG_CMD_USB_MASS_STORAGE=y
> >> >
> >> > You still haven't explained why this is useful to add it by default on
> >> > this particular board.
> >>
> >> This board does have only eMMC and no external storage
> >> possibilities(like eMMC or Ethernet) so to copy images between eMMC vs
> >> host intermediate way (other than fastboot) ums can be the possible
> >> way.
> >
> > We have fastboot and DFU to address that already, and a number of
> > other boards accomodate that just fine (the CHIP, the NES Classic,
> > etc.).
> 
> Yes, but ums is different where we can simply mount the flash on host
> and update the partitions data directly instead of create partitions
> with new data to write again. what's wrong with using ums. I always
> believe fastboot, DFU is different than ums where former need to
> create image file and later simply copy.

I'm not arguing the usefulness of the UMS gadget itself, I don't have
any objections to that, and I totally get why and how it can be
useful. What I'm arguing for is why you should enable it *by default*.

You can very easily do that with menuconfig already, and we won't
enable each and every command and feature someone finds
interesting. We've already been bitten hard by this, and I have no
intention of doing that kind of thing a second time.

So yeah, you can definitely use UMS if you wish so. But enabling it by
default when we have already two alternatives for the same use case
enabled is pointless.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180601/32af04a4/attachment.sig>