linux-kernel - Re: [PATCH 2/5 v14] arm: omap: usb: ehci and ohci hwmod structures for omap3

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.00.1110140205460.16865@utopia.booyaka.com>
Date:	Fri, 14 Oct 2011 02:11:43 -0600 (MDT)
From:	Paul Walmsley <paul@...an.com>
To:	"Munegowda, Keshava" <keshava_mgowda@...com>
cc:	linux-usb@...r.kernel.org, linux-omap@...r.kernel.org,
	linux-kernel@...r.kernel.org, balbi@...com, khilman@...com,
	b-cousson@...com, gadiyar@...com, sameo@...ux.intel.com,
	parthab@...ia.ti.com, tony@...mide.com, johnstul@...ibm.com,
	vishwanath.bs@...com
Subject: Re: [PATCH 2/5 v14] arm: omap: usb: ehci and ohci hwmod structures
 for omap3

Hi

On Fri, 14 Oct 2011, Munegowda, Keshava wrote:

> On Fri, Oct 14, 2011 at 6:33 AM, Paul Walmsley <paul@...an.com> wrote:

> >  static __initdata struct omap_hwmod *omap3xxx_hwmods[] = {
> >        &omap3xxx_l3_main_hwmod,
> >        &omap3xxx_l4_core_hwmod,
> > @@ -3151,6 +3365,7 @@ static __initdata struct omap_hwmod *omap3xxx_hwmods[] = {
> >        &omap3xxx_uart1_hwmod,
> >        &omap3xxx_uart2_hwmod,
> >        &omap3xxx_uart3_hwmod,
> > +
> 
> why you have removed  &omap3xxx_usb_host_hs_hwmod,
> and       &omap3xxx_usb_tll_hs_hwmod.?

I have removed them because, according to the OMAP3430 ES1.0 Revision A 
TRM (SWPU100A), the HS USBHOST subsystem is not present on OMAP3430 ES1.0.  
Therefore the HS USBHOST modules must not be marked as being common to all 
OMAP3xxx chips in the hwmod data.

If you have information that indicates that it's present on OMAP3430 
ES1.0, please let me know ASAP.

> I think it is required here, please let me know.
> 
> regards
> keshava
> 
> 
> 
> 
> >        /* dss class */
> >        &omap3xxx_dss_dispc_hwmod,
> >        &omap3xxx_dss_dsi1_hwmod,
> > @@ -3206,6 +3421,8 @@ static __initdata struct omap_hwmod *omap3430es1_hwmods[] = {
> >  static __initdata struct omap_hwmod *omap3430es2plus_hwmods[] = {
> >        &omap3xxx_dss_core_hwmod,
> >        &omap3xxx_usbhsotg_hwmod,
> > +       &omap3xxx_usb_host_hs_hwmod,
> > +       &omap3xxx_usb_tll_hs_hwmod,
> >        NULL
> >  };
> >
> > @@ -3223,12 +3440,16 @@ static __initdata struct omap_hwmod *omap36xx_hwmods[] = {
> >        &omap36xx_sr1_hwmod,
> >        &omap36xx_sr2_hwmod,
> >        &omap3xxx_usbhsotg_hwmod,
> > +       &omap3xxx_usb_host_hs_hwmod,
> > +       &omap3xxx_usb_tll_hs_hwmod,
> >        NULL
> >  };
> >
> >  static __initdata struct omap_hwmod *am35xx_hwmods[] = {
> >        &omap3xxx_dss_core_hwmod, /* XXX ??? */
> >        &am35xx_usbhsotg_hwmod,
> > +       &omap3xxx_usb_host_hs_hwmod,
> > +       &omap3xxx_usb_tll_hs_hwmod,
> >        NULL
> >  };
> >
> > --
> > 1.7.7
> >
> >
> 


- Paul