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

[Marek Vasut]

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.

> However I do think it would be good to move to using DFU etc. if possible?

Yes.

-- 
Best regards,
Marek Vasut