linux-kernel - Re: [PATCH V3] irqchip: Add TB10x interrupt controller driver

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

Date:	Tue, 25 Jun 2013 16:37:31 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Christian Ruppert <christian.ruppert@...lis.com>
cc:	Vineet Gupta <Vineet.Gupta1@...opsys.com>,
	Grant Likely <grant.likely@...retlab.ca>,
	Rob Herring <rob.herring@...xeda.com>,
	Rob Landley <rob@...dley.net>,
	devicetree-discuss@...ts.ozlabs.org, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Pierrick Hascoet <pierrick.hascoet@...lis.com>
Subject: Re: [PATCH V3] irqchip: Add TB10x interrupt controller driver

On Tue, 25 Jun 2013, Christian Ruppert wrote:
> On Tue, Jun 25, 2013 at 03:58:43PM +0200, Thomas Gleixner wrote:
> > On Fri, 31 May 2013, Christian Ruppert wrote:
> > > The SOC interrupt controller driver for the Abilis Systems TB10x series of
> > > SOCs based on ARC700 CPUs.
> > > 
> > > This patch depends on commits eb76bdd407d8a90e59a06cb0158886df390e5d1c and
> > > 712bc93df9e7f14b8a163148d2aa7c778e151627 from branch irq/for-arm of
> > > git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git.
> > 
> > This should go below --- as it is not part of the changelog.
> 
> OK, I will change that.
> 
> > > Signed-off-by: Christian Ruppert <christian.ruppert@...lis.com>
> > > Signed-off-by: Pierrick Hascoet <pierrick.hascoet@...lis.com>
> > 
> > Why is Pierrick in the SOB chain? 
> 
> We work on the platform port together and review and test each others'
> patches. It is planned submitting several of his patches as soon as
> basic platform is in.
> 
> > > ---
> > >  .../interrupt-controller/abilis,tb10x-ictl.txt     |   38 ++++
> > >  arch/arc/boot/dts/abilis_tb100.dtsi                |   32 ++--
> > >  arch/arc/boot/dts/abilis_tb101.dtsi                |   32 ++--
> > >  arch/arc/boot/dts/abilis_tb10x.dtsi                |   32 ++--
> > >  arch/arc/plat-tb10x/Kconfig                        |    1 +
> > >  drivers/irqchip/Kconfig                            |    5 +
> > >  drivers/irqchip/Makefile                           |    1 +
> > >  drivers/irqchip/irq-tb10x.c                        |  195 ++++++++++++++++++++
> > 
> > I can pick it up, but I'm not so fond of carrying arch/arc patches
> > which might conflict with other stuff in the arc tree. Veenet, if I
> > should pick the whole thing up, I'd like to have an acked-by at least.
> 
> Would you prefer if we separated everything in arch/arc from this patch
> and push it through Vineet's channel?

If it can be split w/o breaking the world, this would be the most
elegant solution.

Thanks,

	tglx
--
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/