[U-Boot] [U-Boot 0/3] introduce Rockchip rockusb

[Tom Rini]

On Tue, Mar 21, 2017 at 01:17:18PM +0800, Kever Yang wrote:
> Hi Marek,
> 
> On 03/21/2017 12:33 AM, Marek Vasut wrote:
> >On 03/20/2017 05:01 PM, Simon Glass wrote:
> >>Hi Marek,
> >>
> >>On 20 March 2017 at 06:24, Marek Vasut <marex at denx.de> wrote:
> >>>On 03/20/2017 02:53 AM, Eddie Cai wrote:
> >>>>2017-03-17 5:26 GMT+08:00 Marek Vasut <marex at denx.de>:
> >>>>
> >>>>>On 03/15/2017 08:56 AM, Eddie Cai wrote:
> >>>>>>rockusb is a protocol run between host pc and device. it help people get
> >>>>>device
> >>>>>>info, flash image to device. this patch implement rockusb on device side.
> >>>>>What is the benefit of this yet-another NIH protocol compared to ie.
> >>>>>DFU/UMS/Thor/Fastboot ?
> >>>>>
> >>>>This is mostly  used on Rockchip chipset. All the rockchip SoC boot rom and
> >>>>develop tools implement this protocol.
> >>>>So we want to use the same protocol which compatible with current tools.
> >>>This does not really answer my question. We have protocols which
> >>>implement exactly this functionality already, why do we need another one ?
> >>I think the point is that this is widely used with Rockchip devices,
> >>and for mainline U-Boot to be fully useful it needs to support it.
> >OK, I'll leave that decision to Lukasz . I'm not a big fan of adding
> >redundant functionality only because someone thought NIH is good before
> >looking at standards.
> 
> I don't know how you define "redundant functionality", from the
> gadget framework view,
> yes, it's very similar to other storage based gadget, we may able to
> re-use the common part of the framework.
> But it's definitely a different USB class driver which is used in
> Rockchip bootrom
> and Rockchip loader, and some commands like RD(Reset device),
> DB(download boot)
> are not able to use in ums or fastboot, but they are useful in
> Rockchip platform.

So here's my two cents.  At the back end side of things, DFU, fastboot,
UMS, Thor, rockusb and whatever comes down the pipe next should all be
able to share a good bit of back end code, in a common way (and maybe we
can even address DFU using the CLI rather than API to do things) as it
all boils down to "take data via USB, write to defined location" or "do
something then reset the board" and so on.  And by that same token,
while I wish more vendors used the same protocols, we need to support
the world that exists, not the world we wish we had.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170321/16ddc6b8/attachment.sig>