linux-kernel - Re: [PATCH] of/irq: add empty irq_of_parse_and

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] [day] [month] [year] [list]

Date:	Tue, 27 Mar 2012 16:28:27 -0600
From:	Grant Likely <grant.likely@...retlab.ca>
To:	Thomas Abraham <thomas.abraham@...aro.org>
Cc:	Rob Herring <robherring2@...il.com>,
	devicetree-discuss@...ts.ozlabs.org, patches@...aro.org,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] of/irq: add empty irq_of_parse_and_map() for non-dt builds

On Sun, 25 Mar 2012 20:26:15 +0530, Thomas Abraham <thomas.abraham@...aro.org> wrote:
> On 25 March 2012 06:12, Grant Likely <grant.likely@...retlab.ca> wrote:
> > On Sat, 24 Mar 2012 08:12:39 -0500, Rob Herring <robherring2@...il.com> wrote:
> >> On 03/24/2012 04:27 AM, Thomas Abraham wrote:
> >> > Add a empty irq_of_parse_and_map() that returns 0 for non-dt builds and
> >> > avoid having #ifdef CONFIG_OF around all calls to irq_of_parse_and_map().
> >> >
> >> > Suggested-by: Grant Likely <grant.likely@...retlab.ca>
> >> > Signed-off-by: Thomas Abraham <thomas.abraham@...aro.org>
> >> > ---
> >>
> >> Acked-by: Rob Herring <rob.herring@...xeda.com>
> >>
> >> Go ahead and merge with the rest of your series.
> >
> > Actually, it's not quite fully baked; the forward declaration of
> > irq_of_parse_and_map() needs to be moved under the #if
> > defined(CONFIG_OF_IRQ) block (just move the #if statement up 7 lines).
> > After doing that you can add my acked-by and merge it with the rest of
> > your series.  There is no sense merging it through my tree when you
> > are the only user depending on it.
> >
> > Acked-by: Grant Likely <grant.likely@...retlab.ca>
> 
> Thanks. I will do that change which you have suggested and take it
> through the Samsung tree. But I was thinking that if it goes through
> your tree, there would be less chances of a merge conflict in
> include/linux/of_irq.h, a file that might see updates from other
> sources.

Hmmm... is this going to be merged for v3.4 or v3.5?  If it is v3.5,
then yes it probably does need to go through my tree since there may
be others that need to pull it into their trees.  I was thinking that
you needed this immediately.

g.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/