>Subject: Re: [U-Boot] [PATCH] powerpc/85xx: enanle USB2 gadget mode for
>corenet ds board
>
>1. fix commit message 'enanle' -> enable 2. add some details about how
>this change addresses the issue in commit message
>
[Xie Shaohui] OK, Thanks.
>On Jul 13, 2011, at 9:42 PM, Shaohui Xie wrote:
>
>> Signed-off-by: Shaohui Xie <shaohui....@freescale.com>
>> ---
>> arch/powerpc/cpu/mpc8xxx/fdt.c | 3 ++-
>> board/freescale/corenet_ds/corenet_ds.c | 1 +
>> include/configs/corenet_ds.h | 2 ++
>> 3 files changed, 5 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/powerpc/cpu/mpc8xxx/fdt.c
>> b/arch/powerpc/cpu/mpc8xxx/fdt.c index d9e3e7e..60cb210 100644
>> --- a/arch/powerpc/cpu/mpc8xxx/fdt.c
>> +++ b/arch/powerpc/cpu/mpc8xxx/fdt.c
>> @@ -164,7 +164,8 @@ void fdt_fixup_dr_usb(void *blob, bd_t *bd)
>> if (mode_idx < 0 && phy_idx < 0)
>> printf("WARNING: invalid phy or mode\n");
>> } else {
>> - break;
>> + if (i > CONFIG_SYS_USB_DEVICE)
>> + break;
>> }
>
>I'm not sure that CONFIG_SYS_USB_DEVICE is the right way / #define to use
>for this purpose.
>
[Xie Shaohui] I'm not sure about this too. The CONFIG_SYS_USB_DEVICE is used to
tell driver
How many USB controllers the board has, then the driver will not break until it
scanned all the controllers.
>Do we need the 'else' clause at all?
>
[Xie Shaohui] There is a macro USB_MAX_DEVICE which is defined in
include/usb.h, it's value is 32,
Without else, driver will scan 32 times in 'hwconfig' looking for 'usbx'
(x->1..32).
Best Regards,
Shaohui Xie
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
