[PATCH v2 02/12] usb: chipidea: permit driver bindings pass phy pointer
Marek Vasut
marex at denx.de
Tue May 22 10:07:31 EDT 2012
Dear Alexander Shishkin,
> Richard Zhao <richard.zhao at freescale.com> writes:
>
> Hi,
>
> > Sometimes, the driver bindings may know what phy they use.
> > For example, when using device tree, the usb controller may have a
> > phandler pointing to usb phy.
>
> Some comments below:
>
> [snip]
>
> > ci->dev = dev;
> > ci->udc_driver = dev->platform_data;
> >
> > + if (ci->udc_driver->phy)
> > + ci->transceiver = ci->udc_driver->phy;
> > + else
> > + ci->global_phy = 1;
>
> Nitpick: since it's a bool, don't you want to use "true" instead?
>
> [snip]
>
> > +#include <linux/usb/otg.h>
> > +
> >
> > struct ci13xxx;
> > struct ci13xxx_udc_driver {
> >
> > const char *name;
> > /* offset of the capability registers */
> > uintptr_t capoffset;
> > unsigned power_budget;
> >
> > + struct usb_phy *phy;
>
> This structure is really platform information, not per-device
> instance. Consider, for example, situation when you have 2 controllers
> in one system, they'll be using the same platform structure, but they'll
> have different phys. Right?
Not really, won't you allocate separate structure for them?
>
> Regards,
> --
> Alex
Best regards,
Marek Vasut
More information about the linux-arm-kernel
mailing list