On 11/08/2018 14:04, Lukasz Majewski wrote:
Hi Jean-Jacques,

This series implements 2 fixes to be able to use USB Ethernet gadget
with the dwc3 driver.
It also adds new commands to bind/unbind a device to/from a driver and
update the 'dm tree' command to make it easier to use those new
commands. The bind/unbind commands can be used to bind the DWC3 USB
gadget to the usb_ether driver from the command line instead of
relying on platform code.

Changes in v4:
- Fixed compiler warning: "cmd/bind.c:236:5: warning: ‘ret’ may be
used uninitialized in this function [-Wmaybe-uninitialized]"
With the v3 Michal Simek asked (I was out of the office) if we shall
wait for sandbox tests or not?
There are sandbox tests for the bind/unbind commands provided in test/py/tests/test_bind.py.

JJ


Is there any follow up plan for adding tests to sandbox?

Changes in v3:
- update commit log
- fixed problem with the function name
- update commit log
- new commit
- new
- factorize code based on comments from ML
- remove the devices before unbinding them
- use device_find_global_by_ofnode() to get a device by its node.

Changes in v2:
- Make the bind/unbind command generic, not specific to usb device.
- Update the API to be able to bind/unbind based on DTS node path
- Add a Kconfig option to select the bind/unbind commands

Jean-Jacques Hiblot (7):
   usb: gadget: Add bcdDevice for the DWC3 USB Gadget Controller
   net: eth-uclass: Fix for DM USB ethernet support
   uclass: Add dev_get_uclass_index() to get the uclass/index of a
device dm: print the index of the device when dumping the dm tree
   dm: convert device_get_global_by_of_offset() to
     device_get_global_by_ofnode()
   device: expose the functions used to remove and unbind children of a
     device
   cmd: Add bind/unbind commands to bind a device to a driver from the
     command line

  arch/arm/mach-rockchip/rk3188-board-spl.c |   2 +-
  arch/arm/mach-rockchip/rk3288-board-spl.c |   2 +-
  arch/sandbox/dts/test.dts                 |  11 ++
  cmd/Kconfig                               |   9 ++
  cmd/Makefile                              |   1 +
  cmd/bind.c                                | 255
++++++++++++++++++++++++++++++
configs/sandbox_defconfig                 |   1 +
drivers/core/device-remove.c              |  30 ++--
drivers/core/device.c                     |  19 ++-
drivers/core/dump.c                       |  16 +-
drivers/core/uclass.c                     |  21 +++
drivers/usb/gadget/gadget_chips.h         |   2 +
include/dm/device-internal.h              |  38 +++++
include/dm/device.h                       |  23 ++-
include/dm/uclass-internal.h              |  11 ++
net/eth-uclass.c                          |   3 +-
test/py/tests/test_bind.py                | 178 +++++++++++++++++++++
17 files changed, 584 insertions(+), 38 deletions(-) create mode
100644 cmd/bind.c create mode 100644 test/py/tests/test_bind.py




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: w...@denx.de

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to