On Monday, January 06, 2014 at 07:22:11 AM, Inderpal Singh wrote:
> On 4 January 2014 12:49, Marek Vasut <ma...@denx.de> wrote:
> > On Friday, January 03, 2014 at 06:14:38 AM, Inderpal Singh wrote:
> > > Hi Marek,
> > >
> > > Thanks for review.
> > >
> > > On 3 January 2014 06:26, Marek Vasut <ma...@denx.de> wrote:
> > > > On Thursday, January 02, 2014 at 10:41:59 AM, Inderpal Singh wrote:
> > > > > Arndale board has AX88760, which is USB 2.0 Hub & USB 2.0 Ethernet
> > > > > Combo controller, connected to HSIC Phy of USB host controller via
> > > > > USB3503 hub.
> > > > >
> > > > > This patch implements a board specific board_usb_init function in
> >
> > ehci
> >
> > > > > driver to perform reset sequence for USB3503 hub and enables the
> > > > > relevant config options for network to work.
> > > > >
> > > > > Signed-off-by: Inderpal Singh <inderpal.si...@linaro.org>
> > > > > Signed-off-by: Chander Kashyap <chander.kash...@linaro.org>
> > > > > ---
> > > > >
> > > > > board/samsung/arndale/arndale.c | 13 +++++++++++++
> > > > > drivers/usb/host/ehci-exynos.c | 10 ++++++++++
> > > > > include/configs/arndale.h | 4 ++++
> > > > > 3 files changed, 27 insertions(+)
> > > > >
> > > > > diff --git a/board/samsung/arndale/arndale.c
> > > > > b/board/samsung/arndale/arndale.c index 052fecd..deca348 100644
> > > > > --- a/board/samsung/arndale/arndale.c
> > > > > +++ b/board/samsung/arndale/arndale.c
> > > > > @@ -7,10 +7,23 @@
> > > > >
> > > > > #include <common.h>
> > > > > #include <asm/arch/pinmux.h>
> > > > > #include <asm/arch/dwmmc.h>
> > > > >
> > > > > +#include <asm/arch/gpio.h>
> > > > >
> > > > > #include <asm/arch/power.h>
> > > > >
> > > > > DECLARE_GLOBAL_DATA_PTR;
> > > > >
> > > > > +#ifdef CONFIG_USB_EHCI_EXYNOS
> > > > > +void exynos_board_usb_init(int value)
> > > > > +{
> > > > > + struct exynos5_gpio_part1 *gpio = (struct exynos5_gpio_part1
> > > > > *) +
> > > >
> > > > samsung_get_base_gpio_part1();
> > > >
> > > > > +
> > > > > + /* Configure gpios for usb 3503 hub's reset and connect */
> > > > > + s5p_gpio_direction_output(&gpio->x3, 5, value);
> > > > > + s5p_gpio_direction_output(&gpio->d1, 7, value);
> > > > > +}
> > > > > +#endif
> > > > > +
> > > > >
> > > > > int board_init(void)
> > > > > {
> > > > >
> > > > > gd->bd->bi_boot_params = (PHYS_SDRAM_1 + 0x100UL);
> > > > >
> > > > > diff --git a/drivers/usb/host/ehci-exynos.c
> > > > > b/drivers/usb/host/ehci-exynos.c index 88e6466..4be6a60 100644
> > > > > --- a/drivers/usb/host/ehci-exynos.c
> > > > > +++ b/drivers/usb/host/ehci-exynos.c
> > > > > @@ -175,6 +175,12 @@ static void reset_usb_phy(struct
> > > > > exynos_usb_phy
> > > >
> > > > *usb)
> > > >
> > > > > set_usbhost_phy_ctrl(POWER_USB_HOST_PHY_CTRL_DISABLE);
> > > > >
> > > > > }
> > > > >
> > > > > +inline void __exynos_board_usb_init(int value)
> > > > > +{
> > > > > +}
> > > > > +void exynos_board_usb_init(int)
> > > > > + __attribute__((weak, alias("__exynos_board_usb_init")));
> > > > > +
> > > >
> > > > Sorry, this is not happening. Why can you not use the existing
> > > > board_usb_init()
> > > > and instead have to invent new stuff ?
> > >
> > > I did not use board_usb_init because it has 2 parameters which may not
> > > be used as described in usb.h for my use case, which is to only
> > > configure gpios for reset and connect of usb 3503 hub.
> > >
> > > Let me know if you still feel that board_usb_init should be used. In
> > > that case I will have to utilize index parameter in my own way.
> >
> > Shall I read that as "I will abuse API" ? You know that this will not
> > happen ;-)
>
> Yes, thats an abuse and have not done that yet :-)
>
> > Why can you not reset the USB HUB in regular board_usb_init() simply by
> > toggling
> > the switch's reset gpio as done on the rest of the boards ?
>
> Ok, let me try this. If it works I will send the updated patch.
Thank you!
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
