On Thu, Mar 05, 2015 at 12:36:57AM +0530, Dileep Katta wrote: > On 4 March 2015 at 23:59, Tom Rini <tr...@konsulko.com> wrote: > > > On Wed, Mar 04, 2015 at 05:05:11PM +0000, Stegmaier, Angela wrote: > > > Hi Dileep, > > > > > > > -----Original Message----- > > > > From: Dileep Katta [mailto:dileep.ka...@linaro.org] > > > > Sent: Tuesday, March 03, 2015 5:11 AM > > > > To: u-boot@lists.denx.de; robherri...@gmail.com; Rini, Tom; > > > > rob.herr...@linaro.org; s...@broadcom.com; l.majew...@samsung.com; > > > > Stegmaier, Angela > > > > Cc: Dileep Katta > > > > Subject: [U-Boot][PATCH v2 2/4] fastboot: call board_usb_init() to > > enable > > > > usb > > > > > > > > g_dnl_gadget_register() fails for dra7xx dwc3 gadget when running > > > > cmd_fastboot. > > > > Calling board_usb_init() fixes this. > > > > > > > > Signed-off-by: Angela Stegmaier <angelaba...@ti.com> > > > > Signed-off-by: Dileep Katta <dileep.ka...@linaro.org> > > > > --- > > > > common/cmd_fastboot.c | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > diff --git a/common/cmd_fastboot.c b/common/cmd_fastboot.c index > > > > 346ab80..d7b8f6d 100644 > > > > --- a/common/cmd_fastboot.c > > > > +++ b/common/cmd_fastboot.c > > > > @@ -10,11 +10,13 @@ > > > > #include <common.h> > > > > #include <command.h> > > > > #include <g_dnl.h> > > > > +#include <usb.h> > > > > > > > > static int do_fastboot(cmd_tbl_t *cmdtp, int flag, int argc, char > > *const > > > > argv[]) { > > > > int ret; > > > > > > > > + board_usb_init(0, USB_INIT_DEVICE); > > > > > > The controller index should be configurable. Also, should > > board_usb_cleanup be called at the end? > > > > I _really_ _really_ want to see if we can modify things so that we don't > > have to sprinkle ever gadget driver with a cleanup() and every gadget > > too if we don't have to. Thanks! > > Most of the boards have board_usb_cleanup() as just a stub. For some, the > function is not present, and adding board_usb_cleanup() here breaks the > build for others with fastboot enabled.
That's a problem that needs fixing since those boards without a functional board_usb_cleanup() are likely causing breakage in Linux. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot