[U-Boot] twister: usb host support

Stefano Babic sbabic at denx.de
Thu Jul 11 10:21:48 CEST 2013


Hi Jeroen,

On 10/07/2013 18:04, Jeroen Hofstee wrote:

>> @@ -92,9 +92,6 @@ int board_init(void)
>>       enable_gpmc_cs_config(gpmc_XR16L2751, &gpmc_cfg->cs[3],
>>           XR16L2751_UART2_BASE, GPMC_SIZE_16M);
>>
>> -    gpio_request(CONFIG_OMAP_EHCI_PHY1_RESET_GPIO, "USB_PHY1_RESET");
>> -    gpio_direction_output(CONFIG_OMAP_EHCI_PHY1_RESET_GPIO, 1);
>> -
>> See my concerns above. Do not reset the hub in the kernel ?
> 
> I don't get the last part, but feedback from Technexion is
> needed first to remove all the guess, maybe etc. If it has a
> valid function, not setting it's value might not be such a
> good idea...

I think we could safe drop the two lines. In fact, this code is
redundant and it is already called inside the omap_ehci_phy_reset()
function. No changes in behavior if we remove this lines from board_init().

Regards,
Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list