u-boot.git/cmd/usb.c, branch v2016.07

usb: dm: Make "usb info" use usb_for_each_root_dev()

2016-07-05T12:14:11+00:00

The old dm "usb info" implementation has several issues:

1) NULL pointer deref when a bus has no children
2) Not showing usb devices on busses without an emulated root-hub (otg host)
3) Attempting to show devices on inactive busses
4) "usb info" Would cause some hosts to get re-probed something which only
   "usb reset" should do

TL;DR: proper iterating over usb bus root devs is hard, use the helper
for it.

Reported-by: Bernhard Nortmann 
Signed-off-by: Hans de Goede

usb: dm: Add a usb_for_each_root_dev() helper function

2016-07-05T12:14:11+00:00

Iterating over usb-root devs and doing something for all of them is
a bit tricky with dm, factor out the proven usb_show_tree() for this
into a helper function.

Signed-off-by: Hans de Goede

cmd: usb: check if_type before using this device

2016-06-17T22:48:31+00:00

For legacy usb storage driver, USB_MAX_STOR_DEV is defined as 7.
If we only have one usb disk on board, `usb dev 0` is ok.
But if `usb dev 1`, still ok, then `usb read xxx` will trigger
system fault and reboot.

So check if_type before using this device.

Signed-off-by: Peng Fan 
Cc: Simon Glass 
Cc: Hans de Goede 
Cc: Bin Meng 
Cc: Marek Vasut 
Cc: Stefan Roese 
Cc: Marcel Ziswiler 
Cc: Peng Fan 
Cc: Stephen Warren

dm: usb: Drop the get_dev() function

2016-05-17T15:54:43+00:00

This function is implemented by the legacy block functions now. Drop it.

Signed-off-by: Simon Glass

dm: usb: Do not reprobe usb hosts on "usb tree" command

2016-04-10T15:18:43+00:00

Some usb hosts may have failed to probe on "usb start", i.e. an otg
host without an otg-host cable plugged in.

"usb tree" would cause the probe method of these hosts to get called
again, something which should only happen on "usb reset".

This commit fixes this.

Signed-off-by: Hans de Goede

usb: kbd: Do not deregister usbkbd twice when using dm

2016-04-10T15:18:42+00:00

The dm usb_kbd_remove function() will deregister the usb keyboard for
us on a "usb reset" / "usb stop" so there is no need to manually call
usb_kbd_deregister() in the dm case.

This commit removes usb_kbd_deregister() in the dm case fixing the
following "usb reset" errors:

usb_kbd_remove: warning, ret=-6
device_remove: Device 'usb_kbd' failed to remove, but children are gone

Signed-off-by: Hans de Goede

dm: block: Adjust device calls to go through helpers function

2016-03-14T21:34:50+00:00

To ease conversion to driver model, add helper functions which deal with
calling each block device method. With driver model we can reimplement these
functions with the same arguments.

Use inline functions to avoid increasing code size on some boards.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Tested-by: Stephen Warren

dm: part: Rename some partition functions

2016-03-14T21:34:50+00:00

Rename three partition functions so that they start with part_. This makes
it clear what they relate to.

Signed-off-by: Simon Glass 
Tested-by: Stephen Warren

dm: Drop the block_dev_desc_t typedef

2016-03-14T21:34:50+00:00

Use 'struct' instead of a typdef. Also since 'struct block_dev_desc' is long
and causes 80-column violations, rename it to struct blk_desc.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Tested-by: Stephen Warren

Remove the cmd_ prefix from command files

2016-01-25T15:39:43+00:00

Now that they are in their own directory, we can remove this prefix.
This makes it easier to find a file since the prefix does not get in the
way.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Reviewed-by: Heiko Schocher 
Acked-by: Stefan Roese 
Acked-by: Przemyslaw Marczak