[U-Boot] [U-Boot PATCH V2 9/9] rockusb: add a simple readme

[Eddie Cai]

Hi Lukasz

2017-04-20 6:39 GMT+08:00 Lukasz Majewski <lukma at denx.de>:
> Hi Eddie,
>
>> add a simple readme to introduce rockusb and tell people how to use it
>>
>> Signed-off-by: Eddie Cai <eddie.cai.linux at gmail.com>
>> ---
>>  doc/README.rockusb | 35 +++++++++++++++++++++++++++++++++++
>>  1 file changed, 35 insertions(+)
>>  create mode 100644 doc/README.rockusb
>>
>> diff --git a/doc/README.rockusb b/doc/README.rockusb
>> new file mode 100644
>> index 0000000..13893d9
>> --- /dev/null
>> +++ b/doc/README.rockusb
>> @@ -0,0 +1,35 @@
>> +Rockusb (Rockchip USB protocol)
>> +=====================================================
>> +
>> +Overview
>> +--------
>> +
>> +Rockusb protocol is widely used with Rockchip devices.
>> +It can read/write info,
>
> R/W the info is the differentiator here. What kind of information canwe
> read from the device? Do you plan to add this in the future?
SoC infor, Storage infor,  thins in eFuse like serial number. I did have plan to
add this function. But to keep things simple. I want to add minimum function
first. It will be easy for me to add new function if everybody agree
the framework.
>
> Does it allow for reading/writing the RK soc's env variables ?
What do you mean soc's env variables here? yes If you mean things
like cpu model number
>
>>image to/from devices. This
>> +document briefly describes how to use Rockusb for upgrading
>> +firmware (e.g. kernel, u-boot, rootfs, etc.).
>> +
>> +Tools
>> +--------
>> +There are many tools can support Rockusb protocol. rkdeveloptool
>> +is open source, It is maintained by Rockchip.
>
> Ok.
>
>> +
>> +Usage
>> +--------
>> +The Usage of Rockusb command is
>> +rockusb <USB_controller> [<devtype>] <devnum>
>> +e.g. rockusb 0 mmc 0
>> +rockusb 0 usb 0
>> +On your U-Boot console, type this command to enter rockusb mode.
>> +On your host PC. use lsusb command. you should see a usb device
>> +using 0x2207 as its USB verdor id. Then use rkdeveloptool
>> +sudo rkdeveloptool wl  <BeginSec> <File>
>> +to flash U-Boot image use command
>> +sudo rkdeveloptool wl  64 <U-Boot binary>
>
> Just informative - this could be achieved with UMS command with using
> dd.
yes, if you only talk about wl command. But there are many other rockusb
command can not achieved with UMS command
>
>> +There are plenty of Rockusb command. but wl(write lba) and
>> +rd(reboot) command.
>
> Reboot is also supported - but there is no unification - though each
> gadget (thor, dfu, fastboot)
>
>> These two command can let people flash
>> +image to device.
>
> IMHO reading images (for backup) would be handy as well.
I do have plan to add this function. But I would like this minimum set agreed by
everybody. then add more command.
If i implemented all the rockusb command. It will be huge work for me when
i move from V1 to V2 patch.
Please help review my V1 patch. My V3 patch will based on V1 patch.
Let me know if you have more comment.
https://lists.denx.de/pipermail/u-boot/2017-March/283730.html
>
>> +
>> +To do
>> +-----
>> +* Fully support Rockusb protocol
>
> For me it is OK, to have a minimal subset of commands supported - let's
> add the very basic support and extend it latter.
Totally agree.
>
>
> Best regards,
>
> Lukasz Majewski
>
> --
>
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de