[U-Boot] [PATCH v2 1/2] usb: ehci: exynos: set/reset hsic phys

[Inderpal Singh]

On 4 January 2014 12:46, Marek Vasut <marex at denx.de> wrote:

> On Friday, January 03, 2014 at 06:03:47 AM, Inderpal Singh wrote:
> > Hi Marek,
> >
> > Thanks for the review.
> >
> > On 3 January 2014 06:24, Marek Vasut <marex at denx.de> wrote:
> > > On Thursday, January 02, 2014 at 10:41:58 AM, Inderpal Singh wrote:
> > > > From: Inderpal Singh <chander.kashyap at linaro.org>
> > > >
> > > > The controller has 3 ports. The port0 is for USB 2.0 Phy, port1 and
> > > > port2 are for HSIC phys. The usb 2.0 phy is already being setup. This
> > > > patch sets up the hsic phys.
> > > >
> > > > Signed-off-by: Inderpal Singh <inderpal.singh at linaro.org>
> > > > ---
> > > >
> > > >  arch/arm/include/asm/arch-exynos/ehci.h |   14 +++++++++++
> > > >  drivers/usb/host/ehci-exynos.c          |   39
> > > >
> > > > +++++++++++++++++++++++++++++++ 2 files changed, 53 insertions(+)
> > >
> > > Is it OK to set all the ports up unconditionally ? I am not sure about
> > > exynos of
> > > course, but is it possible there are some machines which don't use the
> > > HSIC ports and this would have some kind of adverse effects on those?
> >
> > I feel it should not cause any side effect as it wont interfere with the
> > normal USB 2.0 phy port. Also, its being done along the same lines as
> > kernel driver at drivers/usb/phy/phy-samsung-usb2.c, which also sets up
> all
> > ports unconditionally.
>
> OK, I won't fight this. I would be much more inclined to being able to
> conditionally select which ports get configured. Especially, since you do
> know
> that information from DT, dont you?
>

Ok, Thanks Marek.
As of now DT is not providing port information.

Regards,
Inder


> Best regards,
> Marek Vasut
>