linux-kernel - Re: [PATCH v2] irqdomain: Initialize number of IRQs for simple domains

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20120110104435.GA10617@avionic-0098.mockup.avionic-design.de>
Date:	Tue, 10 Jan 2012 11:44:35 +0100
From:	Thierry Reding <thierry.reding@...onic-design.de>
To:	Jamie Iles <jamie@...ieiles.com>
Cc:	devicetree-discuss@...ts.ozlabs.org,
	Andrew Victor <linux@...im.org.za>,
	Nicolas Ferre <nicolas.ferre@...el.com>,
	Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>,
	Russell King <linux@....linux.org.uk>,
	Sascha Hauer <kernel@...gutronix.de>,
	Shawn Guo <shawn.guo@...aro.org>,
	David Brown <davidb@...eaurora.org>,
	Daniel Walker <dwalker@...o99.com>,
	Bryan Huntsman <bryanh@...eaurora.org>,
	Tony Lindgren <tony@...mide.com>,
	Barry Song <baohua.song@....com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Catalin Marinas <catalin.marinas@....com>,
	David Woodhouse <dwmw2@...radead.org>,
	Grant Likely <grant.likely@...retlab.ca>,
	Uwe Kleine-König 
	<u.kleine-koenig@...gutronix.de>,
	Rob Herring <rob.herring@...xeda.com>,
	"open list:ARM/ATMEL AT91RM9..." 
	<linux-arm-kernel@...ts.infradead.org>,
	open list <linux-kernel@...r.kernel.org>,
	"open list:ARM/QUALCOMM MSM..." <linux-arm-msm@...r.kernel.org>,
	"open list:OMAP SUPPORT" <linux-omap@...r.kernel.org>
Subject: Re: [PATCH v2] irqdomain: Initialize number of IRQs for simple
 domains

* Jamie Iles wrote:
> Hi Thierry,
> 
> On Tue, Jan 10, 2012 at 11:24:11AM +0100, Thierry Reding wrote:
> > The irq_domain_add() function needs the number of interrupts in the
> > domain to properly initialize them. In addition the allocated domain
> > is now returned by the irq_domain_{add,generate}_simple() helpers.
> > 
> > Signed-off-by: Thierry Reding <thierry.reding@...onic-design.de>
> > Acked-by: Nicolas Ferre <nicolas.ferre@...el.com>
> > Acked-by: David Brown <davidb@...eaurora.org>
> > ---
> > Note: This patch should not be applied yet. There is a conflict with a
> >       pending patch[0] by Benoit Cousson that will obsolete the OMAP2
> >       hunk in this patch.
> > 
> >       [0]: http://www.spinics.net/lists/linux-omap/msg62124.html
> 
> I also have a patch in Russell's tracker to remove one of the calls for 
> versatile (for the VIC) that is no longer needed.

Okay, I'll take a note. I guess it is okay for this patch to go in only after
the conflicting patches have all been applied. I'm assuming that sooner or
later those patches will land in linux-next and I can rebase this patch on
top, right?

> > diff --git a/arch/arm/mach-imx/imx51-dt.c b/arch/arm/mach-imx/imx51-dt.c
> > index e6bad17..811ce88 100644
> > --- a/arch/arm/mach-imx/imx51-dt.c
> > +++ b/arch/arm/mach-imx/imx51-dt.c
> > @@ -47,7 +47,12 @@ static const struct of_dev_auxdata imx51_auxdata_lookup[] __initconst = {
> >  static int __init imx51_tzic_add_irq_domain(struct device_node *np,
> >  				struct device_node *interrupt_parent)
> >  {
> > -	irq_domain_add_simple(np, 0);
> > +	struct irq_domain *domain;
> > +
> > +	domain = irq_domain_add_simple(np, 0, TZIC_NUM_IRQS);
> > +	if (!domain)
> > +		return -ENOMEM;
> > +
> >  	return 0;
> >  }
> 
> A real nit, but could this be written as:
> 
> 	return irq_domain_add_simple(np, 0, TZIC_NUM_IRQS) ? 0 : -ENOMEM;
> 
> to make it a little more concise (and the same for the others)?  Not a 
> big deal though, so either way:
> 
> Reviewed-by: Jamie Iles <jamie@...ieiles.com>

Yes, I can do that.

Thanks,
Thierry

Content of type "application/pgp-signature" skipped