From mboxrd@z Thu Jan  1 00:00:00 1970
From: Hans-Christian Egtvedt <egtvedt@samfundet.no>
Subject: Re: [RFC v1 20/25] genirq: Kill the first parameter 'irq' of
 irq_flow_handler_t
Date: Wed, 20 May 2015 12:16:46 +0200
Message-ID: <20150520101646.GB12898@samfundet.no>
References: <1432116013-25902-1-git-send-email-jiang.liu@linux.intel.com>
 <1432116013-25902-21-git-send-email-jiang.liu@linux.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Return-path: <sparclinux-owner@vger.kernel.org>
Content-Disposition: inline
In-Reply-To: <1432116013-25902-21-git-send-email-jiang.liu@linux.intel.com>
Sender: sparclinux-owner@vger.kernel.org
To: Jiang Liu <jiang.liu@linux.intel.com>
Cc: Thomas Gleixner <tglx@linutronix.de>, Bjorn Helgaas <bhelgaas@google.com>, Benjamin Herrenschmidt <benh@kernel.crashing.org>, Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>, "Rafael J. Wysocki" <rjw@rjwysocki.net>, Randy Dunlap <rdunlap@infradead.org>, Yinghai Lu <yinghai@kernel.org>, Borislav Petkov <bp@alien8.de>, Richard Henderson <rth@twiddle.net>, Ivan Kokshaysky <ink@jurassic.park.msu.ru>, Matt Turner <mattst88@gmail.com>, Haavard Skinnemoen <hskinnemoen@gmail.com>, Steven Miao <realmz6@gmail.com>, Mark Salter <msalter@redhat.com>, Aurelien Jacquiot <a-jacquiot@ti.com>, "David S. Miller" <davem@davemloft.net>, Chris Metcalf <cmetcalf@ezchip.com>, Guan Xuetao <gxt@mprc.pku.edu.cn>, x86@kernel.org, Rusty Russell <rusty@rustcorp.com.au>, Andreas Larsson <andreas@gaisler.com>, Joe Perches <joe@perches.com>, Christoph Lameter <cl@linux.com>
List-Id: linux-acpi@vger.kernel.org

Around Wed 20 May 2015 18:00:08 +0800 or thereabout, Jiang Liu wrote:
> Now most IRQ flow handlers make no use of the first parameter 'irq'.
> And for those who do make use of 'irq', we could easily get the irq
> number through irq_desc->irq_data->irq. So kill the first parameter
> 'irq' of irq_flow_handler_t.
> 
> To ease review, I have split the changes into several parts, though
> they should be merge as one to support bisecting.
> 
> Signed-off-by: Jiang Liu <jiang.liu@linux.intel.com>
> ---
>  arch/alpha/kernel/irq.c             |    2 +-
>  arch/avr32/mach-at32ap/extint.c     |    2 +-
>  arch/avr32/mach-at32ap/pio.c        |    2 +-

For the AVR32 related bits

Acked-by: Hans-Christian Egtvedt <egtvedt@samfundet.no>

>  arch/blackfin/kernel/ipipe.c        |    4 ++--
>  arch/c6x/platforms/megamod-pic.c    |    3 ++-
>  arch/sparc/kernel/leon_kernel.c     |    2 +-
>  arch/sparc/kernel/leon_pci_grpci1.c |    2 +-
>  arch/sparc/kernel/leon_pci_grpci2.c |    2 +-
>  arch/tile/kernel/pci_gx.c           |    4 ++--
>  arch/unicore32/kernel/irq.c         |    3 ++-
>  arch/x86/kernel/irq_64.c            |    2 +-
>  arch/x86/lguest/boot.c              |    2 +-
>  12 files changed, 16 insertions(+), 14 deletions(-)

<snipp diff>

-- 
HcE

From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1753213AbbETKUy (ORCPT <rfc822;w@1wt.eu>);
	Wed, 20 May 2015 06:20:54 -0400
Received: from cassarossa.samfundet.no ([193.35.52.29]:48787 "EHLO
	cassarossa.samfundet.no" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1752517AbbETKUs (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 20 May 2015 06:20:48 -0400
Date: Wed, 20 May 2015 12:16:46 +0200
From: Hans-Christian Egtvedt <egtvedt@samfundet.no>
To: Jiang Liu <jiang.liu@linux.intel.com>
Cc: Thomas Gleixner <tglx@linutronix.de>, Bjorn Helgaas <bhelgaas@google.com>,
        Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
        "Rafael J. Wysocki" <rjw@rjwysocki.net>,
        Randy Dunlap <rdunlap@infradead.org>, Yinghai Lu <yinghai@kernel.org>,
        Borislav Petkov <bp@alien8.de>, Richard Henderson <rth@twiddle.net>,
        Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
        Matt Turner <mattst88@gmail.com>,
        Haavard Skinnemoen <hskinnemoen@gmail.com>,
        Steven Miao <realmz6@gmail.com>, Mark Salter <msalter@redhat.com>,
        Aurelien Jacquiot <a-jacquiot@ti.com>,
        "David S. Miller" <davem@davemloft.net>,
        Chris Metcalf <cmetcalf@ezchip.com>, Guan Xuetao <gxt@mprc.pku.edu.cn>,
        x86@kernel.org, Rusty Russell <rusty@rustcorp.com.au>,
        Andreas Larsson <andreas@gaisler.com>, Joe Perches <joe@perches.com>,
        Christoph Lameter <cl@linux.com>, Brian Gerst <brgerst@gmail.com>,
        Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
        Tony Luck <tony.luck@intel.com>, linux-kernel@vger.kernel.org,
        linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org,
        Ingo Molnar <mingo@kernel.org>, linux-alpha@vger.kernel.org,
        adi-buildroot-devel@lists.sourceforge.net, linux-c6x-dev@linux-c6x.org,
        sparclinux@vger.kernel.org, lguest@lists.ozlabs.org
Subject: Re: [RFC v1 20/25] genirq: Kill the first parameter 'irq' of
 irq_flow_handler_t
Message-ID: <20150520101646.GB12898@samfundet.no>
References: <1432116013-25902-1-git-send-email-jiang.liu@linux.intel.com>
 <1432116013-25902-21-git-send-email-jiang.liu@linux.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1432116013-25902-21-git-send-email-jiang.liu@linux.intel.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Around Wed 20 May 2015 18:00:08 +0800 or thereabout, Jiang Liu wrote:
> Now most IRQ flow handlers make no use of the first parameter 'irq'.
> And for those who do make use of 'irq', we could easily get the irq
> number through irq_desc->irq_data->irq. So kill the first parameter
> 'irq' of irq_flow_handler_t.
> 
> To ease review, I have split the changes into several parts, though
> they should be merge as one to support bisecting.
> 
> Signed-off-by: Jiang Liu <jiang.liu@linux.intel.com>
> ---
>  arch/alpha/kernel/irq.c             |    2 +-
>  arch/avr32/mach-at32ap/extint.c     |    2 +-
>  arch/avr32/mach-at32ap/pio.c        |    2 +-

For the AVR32 related bits

Acked-by: Hans-Christian Egtvedt <egtvedt@samfundet.no>

>  arch/blackfin/kernel/ipipe.c        |    4 ++--
>  arch/c6x/platforms/megamod-pic.c    |    3 ++-
>  arch/sparc/kernel/leon_kernel.c     |    2 +-
>  arch/sparc/kernel/leon_pci_grpci1.c |    2 +-
>  arch/sparc/kernel/leon_pci_grpci2.c |    2 +-
>  arch/tile/kernel/pci_gx.c           |    4 ++--
>  arch/unicore32/kernel/irq.c         |    3 ++-
>  arch/x86/kernel/irq_64.c            |    2 +-
>  arch/x86/lguest/boot.c              |    2 +-
>  12 files changed, 16 insertions(+), 14 deletions(-)

<snipp diff>

-- 
HcE

From mboxrd@z Thu Jan  1 00:00:00 1970
From: Hans-Christian Egtvedt <egtvedt@samfundet.no>
Date: Wed, 20 May 2015 10:16:46 +0000
Subject: Re: [RFC v1 20/25] genirq: Kill the first parameter 'irq' of irq_flow_handler_t
Message-Id: <20150520101646.GB12898@samfundet.no>
List-Id: <sparclinux.vger.kernel.org>
References: <1432116013-25902-1-git-send-email-jiang.liu@linux.intel.com>
 <1432116013-25902-21-git-send-email-jiang.liu@linux.intel.com>
In-Reply-To: <1432116013-25902-21-git-send-email-jiang.liu@linux.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: Jiang Liu <jiang.liu@linux.intel.com>
Cc: Thomas Gleixner <tglx@linutronix.de>, Bjorn Helgaas <bhelgaas@google.com>, Benjamin Herrenschmidt <benh@kernel.crashing.org>, Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>, "Rafael J. Wysocki" <rjw@rjwysocki.net>, Randy Dunlap <rdunlap@infradead.org>, Yinghai Lu <yinghai@kernel.org>, Borislav Petkov <bp@alien8.de>, Richard Henderson <rth@twiddle.net>, Ivan Kokshaysky <ink@jurassic.park.msu.ru>, Matt Turner <mattst88@gmail.com>, Haavard Skinnemoen <hskinnemoen@gmail.com>, Steven Miao <realmz6@gmail.com>, Mark Salter <msalter@redhat.com>, Aurelien Jacquiot <a-jacquiot@ti.com>, "David S. Miller" <davem@davemloft.net>, Chris Metcalf <cmetcalf@ezchip.com>, Guan Xuetao <gxt@mprc.pku.edu.cn>, x86@kernel.org, Rusty Russell <rusty@rustcorp.com.au>, Andreas Larsson <andreas@gaisler.com>, Joe Perches <joe@perches.com>, Christoph Lameter <cl@linux.com>

Around Wed 20 May 2015 18:00:08 +0800 or thereabout, Jiang Liu wrote:
> Now most IRQ flow handlers make no use of the first parameter 'irq'.
> And for those who do make use of 'irq', we could easily get the irq
> number through irq_desc->irq_data->irq. So kill the first parameter
> 'irq' of irq_flow_handler_t.
> 
> To ease review, I have split the changes into several parts, though
> they should be merge as one to support bisecting.
> 
> Signed-off-by: Jiang Liu <jiang.liu@linux.intel.com>
> ---
>  arch/alpha/kernel/irq.c             |    2 +-
>  arch/avr32/mach-at32ap/extint.c     |    2 +-
>  arch/avr32/mach-at32ap/pio.c        |    2 +-

For the AVR32 related bits

Acked-by: Hans-Christian Egtvedt <egtvedt@samfundet.no>

>  arch/blackfin/kernel/ipipe.c        |    4 ++--
>  arch/c6x/platforms/megamod-pic.c    |    3 ++-
>  arch/sparc/kernel/leon_kernel.c     |    2 +-
>  arch/sparc/kernel/leon_pci_grpci1.c |    2 +-
>  arch/sparc/kernel/leon_pci_grpci2.c |    2 +-
>  arch/tile/kernel/pci_gx.c           |    4 ++--
>  arch/unicore32/kernel/irq.c         |    3 ++-
>  arch/x86/kernel/irq_64.c            |    2 +-
>  arch/x86/lguest/boot.c              |    2 +-
>  12 files changed, 16 insertions(+), 14 deletions(-)

<snipp diff>

-- 
HcE