From: Vineet Gupta <Vineet.Gupta1@synopsys.com>
To: Joel Porquet <joel@porquet.org>
Cc: <linux@arm.linux.org.uk>, <kgene@kernel.org>,
<k.kozlowski@samsung.com>, <shawn.guo@linaro.org>,
<kernel@pengutronix.de>, <tony@atomide.com>, <monstr@monstr.eu>,
<cernekee@gmail.com>, <f.fainelli@gmail.com>,
<ralf@linux-mips.org>, <tglx@linutronix.de>,
<jason@lakedaemon.net>, <swarren@wwwdotorg.org>, <lee@kernel.org>,
<shc_work@mail.ru>, <baruch@tkos.co.il>, <matthias.bgg@gmail.com>,
<baohua@kernel.org>, <maxime.ripard@free-electrons.com>,
<thierry.reding@gmail.com>, <gnurou@gmail.com>,
<chris@zankel.net>, <jcmvbkbc@gmail.com>,
<soren.brinkmann@xilinx.com>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 00/48] Make the IRQCHIP_DECLARE macro globally accessible
Date: Thu, 2 Jul 2015 12:33:05 +0530 [thread overview]
Message-ID: <5594E229.2020506@synopsys.com> (raw)
In-Reply-To: <1435790000-15720-1-git-send-email-joel@porquet.org>
On Thursday 02 July 2015 04:02 AM, Joel Porquet wrote:
> At the moment the IRQCHIP_DECLARE macro is only declared locally in
> 'drivers/irqchip/irqchip.h'. That prevents from using it directly in arch/*
> directories whenever irqchip drivers only exist there, which happens in a few
> cases (e.g. arc, arm, microblaze and mips).
>
> This patch makes the macro to be globally defined, in include/linux/irqchip.h,
> and thus usable for arch-specific declarations of irqchip drivers. In this way,
> it is very similar to what clocksource does (ie CLOCKSOURCE_OF_DECLARE is
> defined in include/linux/clocksource.h).
>
> I split up everything into patches to make the integration easier. Please let me
> know if it's not, and in such case how to make it better.
>
> For now, patch 01 of this series transfers the declaration of the macro
> IRQCHIP_DECLARE to the global header 'include/linux/irqchip.h'. The following
> patches, from 02 to 47, modify all the irqchip drivers that use IRQCHIP_DECLARE,
> one by one. And finally, the last patch 48 removes the private and now useless
> header 'drivers/irqchip/irqchip.h'.
Hi Joel,
I don't see the rest of series on lkml and/or the patch which touches arch/arc.
Also, you may wanna redo this after 4.2-rc1 anyways. For ARC atleast, there's a
new intc which would also require similar fixup. There might be others ....
Thx,
-Vineet
next prev parent reply other threads:[~2015-07-02 7:05 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-01 22:32 [PATCH 00/48] Make the IRQCHIP_DECLARE macro globally accessible Joel Porquet
2015-07-02 7:03 ` Vineet Gupta [this message]
2015-07-02 18:23 ` Joël Porquet
2015-07-02 18:34 ` Matthias Brugger
2015-07-02 19:06 ` Thomas Gleixner
2015-07-02 19:42 ` Joël Porquet
2015-07-02 20:29 ` Thomas Gleixner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5594E229.2020506@synopsys.com \
--to=vineet.gupta1@synopsys.com \
--cc=baohua@kernel.org \
--cc=baruch@tkos.co.il \
--cc=cernekee@gmail.com \
--cc=chris@zankel.net \
--cc=f.fainelli@gmail.com \
--cc=gnurou@gmail.com \
--cc=jason@lakedaemon.net \
--cc=jcmvbkbc@gmail.com \
--cc=joel@porquet.org \
--cc=k.kozlowski@samsung.com \
--cc=kernel@pengutronix.de \
--cc=kgene@kernel.org \
--cc=lee@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=matthias.bgg@gmail.com \
--cc=maxime.ripard@free-electrons.com \
--cc=monstr@monstr.eu \
--cc=ralf@linux-mips.org \
--cc=shawn.guo@linaro.org \
--cc=shc_work@mail.ru \
--cc=soren.brinkmann@xilinx.com \
--cc=swarren@wwwdotorg.org \
--cc=tglx@linutronix.de \
--cc=thierry.reding@gmail.com \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.