* Latest OMAP randconfig build error @ 2012-02-22 8:51 Russell King - ARM Linux 2012-02-22 10:01 ` Ohad Ben-Cohen 0 siblings, 1 reply; 9+ messages in thread From: Russell King - ARM Linux @ 2012-02-22 8:51 UTC (permalink / raw) To: linux-omap arch/arm/mach-omap2/mailbox.c: In function 'omap2_mbox_probe': arch/arm/mach-omap2/mailbox.c:354: error: 'omap2_mboxes' undeclared (first use in this function) arch/arm/mach-omap2/mailbox.c:354: error: (Each undeclared identifier is reported only once arch/arm/mach-omap2/mailbox.c:354: error: for each function it appears in.) There's also these warnings: arch/arm/mach-omap2/omap-wakeupgen.c:181: warning: 'wakeupgen_irqmask_all' defined but not used WARNING: arch/arm/mach-omap2/built-in.o(.text+0x7878): Section mismatch in reference from the function sr_dev_init() to the function .init.text:sr_set_nvalues() The function sr_dev_init() references the function __init sr_set_nvalues(). This is often because sr_dev_init lacks a __init annotation or the annotation of sr_set_nvalues is wrong. Config is at the usual place. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 8:51 Latest OMAP randconfig build error Russell King - ARM Linux @ 2012-02-22 10:01 ` Ohad Ben-Cohen 2012-02-22 17:58 ` Tony Lindgren 0 siblings, 1 reply; 9+ messages in thread From: Ohad Ben-Cohen @ 2012-02-22 10:01 UTC (permalink / raw) To: Russell King - ARM Linux; +Cc: linux-omap, Tony Lindgren, Anna, Suman + Tony, Suman On Wed, Feb 22, 2012 at 10:51 AM, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote: > arch/arm/mach-omap2/mailbox.c: In function 'omap2_mbox_probe': > arch/arm/mach-omap2/mailbox.c:354: error: 'omap2_mboxes' undeclared (first use in this function) > arch/arm/mach-omap2/mailbox.c:354: error: (Each undeclared identifier is reported only once > arch/arm/mach-omap2/mailbox.c:354: error: for each function it appears in.) The below should trivially solve this, but I wonder if there was any other merit in explicitly using CONFIG_SOC_OMAP2420 there (any different between 2420 and 2430 in that respect ?). diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c index 609ea2d..e61d275 100644 --- a/arch/arm/mach-omap2/mailbox.c +++ b/arch/arm/mach-omap2/mailbox.c @@ -258,7 +258,7 @@ struct omap_mbox mbox_dsp_info = { struct omap_mbox *omap3_mboxes[] = { &mbox_dsp_info, NULL }; #endif -#if defined(CONFIG_SOC_OMAP2420) +#if defined(CONFIG_ARCH_OMAP2) /* IVA */ static struct omap_mbox2_priv omap2_mbox_iva_priv = { .tx_fifo = { ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 10:01 ` Ohad Ben-Cohen @ 2012-02-22 17:58 ` Tony Lindgren 2012-02-22 19:40 ` Ramirez Luna, Omar 2012-02-22 19:55 ` Ohad Ben-Cohen 0 siblings, 2 replies; 9+ messages in thread From: Tony Lindgren @ 2012-02-22 17:58 UTC (permalink / raw) To: Ohad Ben-Cohen; +Cc: Russell King - ARM Linux, linux-omap, Anna, Suman * Ohad Ben-Cohen <ohad@wizery.com> [120222 01:30]: > + Tony, Suman > > On Wed, Feb 22, 2012 at 10:51 AM, Russell King - ARM Linux > <linux@arm.linux.org.uk> wrote: > > arch/arm/mach-omap2/mailbox.c: In function 'omap2_mbox_probe': > > arch/arm/mach-omap2/mailbox.c:354: error: 'omap2_mboxes' undeclared (first use in this function) > > arch/arm/mach-omap2/mailbox.c:354: error: (Each undeclared identifier is reported only once > > arch/arm/mach-omap2/mailbox.c:354: error: for each function it appears in.) > > The below should trivially solve this, but I wonder if there was any > other merit in explicitly using CONFIG_SOC_OMAP2420 there (any > different between 2420 and 2430 in that respect ?). > > diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c > index 609ea2d..e61d275 100644 > --- a/arch/arm/mach-omap2/mailbox.c > +++ b/arch/arm/mach-omap2/mailbox.c > @@ -258,7 +258,7 @@ struct omap_mbox mbox_dsp_info = { > struct omap_mbox *omap3_mboxes[] = { &mbox_dsp_info, NULL }; > #endif > > -#if defined(CONFIG_SOC_OMAP2420) > +#if defined(CONFIG_ARCH_OMAP2) > /* IVA */ > static struct omap_mbox2_priv omap2_mbox_iva_priv = { > .tx_fifo = { 2430 is like omap3 for the mailbox. So the code we have seems wrong trying to initialize it like 2420 mailbox. So we either need a new entry for omap2430_mboxes[], or should just bail out from the probe for 2430 for the fix. Regards, Tony ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 17:58 ` Tony Lindgren @ 2012-02-22 19:40 ` Ramirez Luna, Omar 2012-02-22 19:55 ` Ohad Ben-Cohen 1 sibling, 0 replies; 9+ messages in thread From: Ramirez Luna, Omar @ 2012-02-22 19:40 UTC (permalink / raw) To: Tony Lindgren Cc: Ohad Ben-Cohen, Russell King - ARM Linux, linux-omap, Anna, Suman On Wed, Feb 22, 2012 at 11:58 AM, Tony Lindgren <tony@atomide.com> wrote: > * Ohad Ben-Cohen <ohad@wizery.com> [120222 01:30]: >> + Tony, Suman >> >> On Wed, Feb 22, 2012 at 10:51 AM, Russell King - ARM Linux >> <linux@arm.linux.org.uk> wrote: >> > arch/arm/mach-omap2/mailbox.c: In function 'omap2_mbox_probe': >> > arch/arm/mach-omap2/mailbox.c:354: error: 'omap2_mboxes' undeclared (first use in this function) >> > arch/arm/mach-omap2/mailbox.c:354: error: (Each undeclared identifier is reported only once >> > arch/arm/mach-omap2/mailbox.c:354: error: for each function it appears in.) >> >> The below should trivially solve this, but I wonder if there was any >> other merit in explicitly using CONFIG_SOC_OMAP2420 there (any >> different between 2420 and 2430 in that respect ?). >> >> diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c >> index 609ea2d..e61d275 100644 >> --- a/arch/arm/mach-omap2/mailbox.c >> +++ b/arch/arm/mach-omap2/mailbox.c >> @@ -258,7 +258,7 @@ struct omap_mbox mbox_dsp_info = { >> struct omap_mbox *omap3_mboxes[] = { &mbox_dsp_info, NULL }; >> #endif >> >> -#if defined(CONFIG_SOC_OMAP2420) >> +#if defined(CONFIG_ARCH_OMAP2) >> /* IVA */ >> static struct omap_mbox2_priv omap2_mbox_iva_priv = { >> .tx_fifo = { > > 2430 is like omap3 for the mailbox. So the code we have seems > wrong trying to initialize it like 2420 mailbox. So we either > need a new entry for omap2430_mboxes[], or should just bail > out from the probe for 2430 for the fix. Yes, current code tries to configure both mboxes in a 2430, however it shouldn't be assigning an irq line for the iva mbox, and any request for iva mbox should fail due to that. Code is wrong to register both in 2430 though. Regards, Omar -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 17:58 ` Tony Lindgren 2012-02-22 19:40 ` Ramirez Luna, Omar @ 2012-02-22 19:55 ` Ohad Ben-Cohen 2012-02-22 20:12 ` Russell King - ARM Linux 1 sibling, 1 reply; 9+ messages in thread From: Ohad Ben-Cohen @ 2012-02-22 19:55 UTC (permalink / raw) To: Tony Lindgren Cc: Russell King - ARM Linux, linux-omap, Anna, Suman, Omar Ramirez Luna On Wed, Feb 22, 2012 at 7:58 PM, Tony Lindgren <tony@atomide.com> wrote: > 2430 is like omap3 for the mailbox. Gotcha, thanks. This one below isn't pretty, but it should satisfy all build permutations and still be correct hw-wise. If it looks good to you I'll submit it properly. diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c index 609ea2d..6f0f228 100644 --- a/arch/arm/mach-omap2/mailbox.c +++ b/arch/arm/mach-omap2/mailbox.c @@ -281,8 +281,14 @@ static struct omap_mbox mbox_iva_info = { .ops = &omap2_mbox_ops, .priv = &omap2_mbox_iva_priv, }; +#endif -struct omap_mbox *omap2_mboxes[] = { &mbox_dsp_info, &mbox_iva_info, NULL }; +#if defined(CONFIG_ARCH_OMAP2) +struct omap_mbox *omap2_mboxes[] = { &mbox_dsp_info, +#if defined(CONFIG_SOC_OMAP2420) + &mbox_iva_info, +#endif + NULL }; #endif #if defined(CONFIG_ARCH_OMAP4) -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 19:55 ` Ohad Ben-Cohen @ 2012-02-22 20:12 ` Russell King - ARM Linux 2012-02-22 20:22 ` Ohad Ben-Cohen 0 siblings, 1 reply; 9+ messages in thread From: Russell King - ARM Linux @ 2012-02-22 20:12 UTC (permalink / raw) To: Ohad Ben-Cohen; +Cc: Tony Lindgren, linux-omap, Anna, Suman, Omar Ramirez Luna On Wed, Feb 22, 2012 at 09:55:56PM +0200, Ohad Ben-Cohen wrote: > On Wed, Feb 22, 2012 at 7:58 PM, Tony Lindgren <tony@atomide.com> wrote: > > 2430 is like omap3 for the mailbox. > > Gotcha, thanks. > > This one below isn't pretty, but it should satisfy all build > permutations and still be correct hw-wise. > > If it looks good to you I'll submit it properly. > > diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c > index 609ea2d..6f0f228 100644 > --- a/arch/arm/mach-omap2/mailbox.c > +++ b/arch/arm/mach-omap2/mailbox.c > @@ -281,8 +281,14 @@ static struct omap_mbox mbox_iva_info = { > .ops = &omap2_mbox_ops, > .priv = &omap2_mbox_iva_priv, > }; > +#endif > > -struct omap_mbox *omap2_mboxes[] = { &mbox_dsp_info, &mbox_iva_info, NULL }; > +#if defined(CONFIG_ARCH_OMAP2) > +struct omap_mbox *omap2_mboxes[] = { &mbox_dsp_info, > +#if defined(CONFIG_SOC_OMAP2420) > + &mbox_iva_info, > +#endif > + NULL }; Better would be: +#ifdef CONFIG_ARCH_OMAP2 +struct omap_mbox *omap2_mboxes[] = { + &mbox_dsp_info, +#ifdef CONFIG_SOC_OMAP2420 + &mbox_iva_info, +#endif + NULL +}; #endif There's no point in typing any more than you have to with #if defined(). The only place where using #if defined() makes sense is if you want to subsequently do #elif defined xxx because there isn't a #elifdef. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 20:12 ` Russell King - ARM Linux @ 2012-02-22 20:22 ` Ohad Ben-Cohen 2012-02-22 22:56 ` Tony Lindgren 0 siblings, 1 reply; 9+ messages in thread From: Ohad Ben-Cohen @ 2012-02-22 20:22 UTC (permalink / raw) To: Russell King - ARM Linux Cc: Tony Lindgren, linux-omap, Anna, Suman, Omar Ramirez Luna On Wed, Feb 22, 2012 at 10:12 PM, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote: > +#ifdef CONFIG_ARCH_OMAP2 > +struct omap_mbox *omap2_mboxes[] = { > + &mbox_dsp_info, > +#ifdef CONFIG_SOC_OMAP2420 > + &mbox_iva_info, > +#endif > + NULL > +}; > #endif Beautiful. Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 20:22 ` Ohad Ben-Cohen @ 2012-02-22 22:56 ` Tony Lindgren 2012-02-23 8:02 ` Ohad Ben-Cohen 0 siblings, 1 reply; 9+ messages in thread From: Tony Lindgren @ 2012-02-22 22:56 UTC (permalink / raw) To: Ohad Ben-Cohen Cc: Russell King - ARM Linux, linux-omap, Anna, Suman, Omar Ramirez Luna * Ohad Ben-Cohen <ohad@wizery.com> [120222 11:51]: > On Wed, Feb 22, 2012 at 10:12 PM, Russell King - ARM Linux > <linux@arm.linux.org.uk> wrote: > > +#ifdef CONFIG_ARCH_OMAP2 > > +struct omap_mbox *omap2_mboxes[] = { > > + &mbox_dsp_info, > > +#ifdef CONFIG_SOC_OMAP2420 > > + &mbox_iva_info, > > +#endif > > + NULL > > +}; > > #endif > > Beautiful. Thanks! Care to post an updated patch for me to apply into fixes? Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Latest OMAP randconfig build error 2012-02-22 22:56 ` Tony Lindgren @ 2012-02-23 8:02 ` Ohad Ben-Cohen 0 siblings, 0 replies; 9+ messages in thread From: Ohad Ben-Cohen @ 2012-02-23 8:02 UTC (permalink / raw) To: Tony Lindgren Cc: Russell King - ARM Linux, linux-omap, Anna, Suman, Omar Ramirez Luna On Thu, Feb 23, 2012 at 12:56 AM, Tony Lindgren <tony@atomide.com> wrote: > Care to post an updated patch for me to apply into fixes? Coming right up! ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2012-02-23 8:02 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-02-22 8:51 Latest OMAP randconfig build error Russell King - ARM Linux 2012-02-22 10:01 ` Ohad Ben-Cohen 2012-02-22 17:58 ` Tony Lindgren 2012-02-22 19:40 ` Ramirez Luna, Omar 2012-02-22 19:55 ` Ohad Ben-Cohen 2012-02-22 20:12 ` Russell King - ARM Linux 2012-02-22 20:22 ` Ohad Ben-Cohen 2012-02-22 22:56 ` Tony Lindgren 2012-02-23 8:02 ` Ohad Ben-Cohen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).