mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Joël Porquet" <joel@porquet.org>
To: Vineet Gupta <Vineet.Gupta1@synopsys.com>
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, 02 Jul 2015 14:23:10 -0400	[thread overview]
Message-ID: <5241045.u2d8ixKRcE@joel-zenbook> (raw)
In-Reply-To: <5594E229.2020506@synopsys.com>

On Thursday, July 02, 2015 12:33:05 PM Vineet Gupta wrote:
> 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

Hi Vineet (and all),

Sorry for the mistake, I hope I didn't spam anyone (too much). I realized to 
late that sending about fifty patches to 26 recipients was probably not a good 
idea, and my smtp provider would have blocked me before the end anyway.

Therefore I will follow your suggestion and wait until after 4.2-rc1. Then 
I'll resubmit a new patchset that takes into account the new intc(s) as well.

But since this patchset affects many files across several drivers and 
architectures, what would be the best way to submit it?

Would it be OK to send the cover to all the maintainers/mailing-lists involved 
in order to inform them that a patchset is affecting their respective 
subsystem, but to send the patches only on the kernel mailing-list?

And/or is there someone in particular who is in charge to integrate such a 
transversal patchset?

Thanks,
Joël

  reply	other threads:[~2015-07-02 19:45 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-01 22:32 Joel Porquet
2015-07-02  7:03 ` Vineet Gupta
2015-07-02 18:23   ` Joël Porquet [this message]
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=5241045.u2d8ixKRcE@joel-zenbook \
    --to=joel@porquet.org \
    --cc=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=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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®