* David Brownell <[EMAIL PROTECTED]> [080424 12:07]:
> Looks better to me, but:
> 
> On Thursday 24 April 2008, Felipe Balbi wrote:
> > +static void setup_ehci_io_mux(void)
> > +{
> > +#ifdef CONFIG_OMAP_EHCI_PHY_MODE
> 
> This PHY_MODE thing should probably be provided by the board
> data not Kconfig, and
> 
> > +       /* PHY mode of operation for board: 750-2083-001
> > +        * ISP1504 connected to Port1 and Port2
> > +        * Do Func Mux setting for 12-pin ULPI PHY mode
> > +        */
> > +
> > +       /* Port1 */
> > +       omap_cfg_reg(Y9_3430_USB1HS_PHY_STP);
> > +       omap_cfg_reg(Y8_3430_USB1HS_PHY_CLK);
> >     ...
> 
> All those MUX options should be driven by the board data too.
> 
> Unless it's for some reason not possible to configure only one
> of the three EHCI ports at a time...
> 
> This patch being cleanup-only, I don't suggest you fix that
> at this time.

OK, I'll push Felipe's patch today.

Tony



> - Dave
> 
> 
> 
> > +
> > +#else
> > +       /* Set Func mux for :
> > +        * TLL mode of operation
> > +        * 12-pin ULPI SDR TLL mode for Port1/2/3
> > +        */
> > +
> > +       /* Port1 */
> > +       omap_cfg_reg(Y9_3430_USB1HS_TLL_STP);
> > +       omap_cfg_reg(Y8_3430_USB1HS_TLL_CLK);
> > +           ...
> > +#endif /* CONFIG_OMAP_EHCI_PHY_MODE */
> > +
> > +       return;
> > +}
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to [EMAIL PROTECTED]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to