On Mon, Nov 07, 2011 at 03:07:28PM -0800, Tony Lindgren wrote: > * Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx> [111107 14:20]: > > On Mon, Nov 07, 2011 at 02:51:57PM -0800, Tony Lindgren wrote: > > > Here's what I got. Looks like the removal of the sr[12]_hwmod > > > part is no longer needed, so only the r value check part is needed. > > > > Err. So do you have anything in your git tree which you're pushing out > > this evening which removes the sr[12]_hwmod from the omap_3xxx_hwmod > > list? > > > > If not, how does that bug get fixed in mainline? > > Ah sorry, there's also the following commit there that deals > with other related issues: Great, so it looks like everything that I've found over the last two days should be solved completely by what you have queued for Arnd. One last issue though... We had this patch: http://lists.arm.linux.org.uk/lurker/message/20111006.232441.227dfcf6.en.html which is from Paul, post my complaint about IS_ERR_VALUE being used. This removed sr1_hwmod and sr2_hwmod, and the commit message explicitly talks about doing this. There's also the patch below. Somehow, this resulted in that bit of Paul's patch being removed (possibly automatically by git) but the commit comments are retained, which are now stale and no longer reflect what the patch is doing... That's rather unfortunate, but at this point its probably too late to do something about it. > From: Abhilash K V <abhilash.kv@xxxxxx> > Date: Fri, 7 Oct 2011 03:08:56 -0600 > Subject: [PATCH] ARM: OMAP: AM35x: remove hwmods that aren't generic > > Removing modules iva, sr1_hwmod, sr2_hwmod, mailbox from > the base omap3xxx_hwmods list, so that they can be excluded > for am35x. This removes quite a few warnings on boot for AM35x. > > Signed-off-by: Abhilash K V <abhilash.kv@xxxxxx> > [paul@xxxxxxxxx: dropped 'mailbox class' comments; updated changelog] > Signed-off-by: Paul Walmsley <paul@xxxxxxxxx> > > --- a/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c > +++ b/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c > @@ -3132,7 +3132,6 @@ static __initdata struct omap_hwmod *omap3xxx_hwmods[] = { > &omap3xxx_mmc2_hwmod, > &omap3xxx_mmc3_hwmod, > &omap3xxx_mpu_hwmod, > - &omap3xxx_iva_hwmod, > > &omap3xxx_timer1_hwmod, > &omap3xxx_timer2_hwmod, > @@ -3161,8 +3160,6 @@ static __initdata struct omap_hwmod *omap3xxx_hwmods[] = { > &omap3xxx_i2c1_hwmod, > &omap3xxx_i2c2_hwmod, > &omap3xxx_i2c3_hwmod, > - &omap34xx_sr1_hwmod, > - &omap34xx_sr2_hwmod, > > /* gpio class */ > &omap3xxx_gpio1_hwmod, > @@ -3184,8 +3181,6 @@ static __initdata struct omap_hwmod *omap3xxx_hwmods[] = { > &omap3xxx_mcbsp2_sidetone_hwmod, > &omap3xxx_mcbsp3_sidetone_hwmod, > > - /* mailbox class */ > - &omap3xxx_mailbox_hwmod, > > /* mcspi class */ > &omap34xx_mcspi1, > @@ -3198,31 +3193,39 @@ static __initdata struct omap_hwmod *omap3xxx_hwmods[] = { > > /* 3430ES1-only hwmods */ > static __initdata struct omap_hwmod *omap3430es1_hwmods[] = { > + &omap3xxx_iva_hwmod, > &omap3430es1_dss_core_hwmod, > + &omap3xxx_mailbox_hwmod, > NULL > }; > > /* 3430ES2+-only hwmods */ > static __initdata struct omap_hwmod *omap3430es2plus_hwmods[] = { > + &omap3xxx_iva_hwmod, > &omap3xxx_dss_core_hwmod, > &omap3xxx_usbhsotg_hwmod, > + &omap3xxx_mailbox_hwmod, > NULL > }; > > /* 34xx-only hwmods (all ES revisions) */ > static __initdata struct omap_hwmod *omap34xx_hwmods[] = { > + &omap3xxx_iva_hwmod, > &omap34xx_sr1_hwmod, > &omap34xx_sr2_hwmod, > + &omap3xxx_mailbox_hwmod, > NULL > }; > > /* 36xx-only hwmods (all ES revisions) */ > static __initdata struct omap_hwmod *omap36xx_hwmods[] = { > + &omap3xxx_iva_hwmod, > &omap3xxx_uart4_hwmod, > &omap3xxx_dss_core_hwmod, > &omap36xx_sr1_hwmod, > &omap36xx_sr2_hwmod, > &omap3xxx_usbhsotg_hwmod, > + &omap3xxx_mailbox_hwmod, > NULL > }; > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
