Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261604AbVCKUyZ (ORCPT ); Fri, 11 Mar 2005 15:54:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261284AbVCKUu4 (ORCPT ); Fri, 11 Mar 2005 15:50:56 -0500 Received: from outmx017.isp.belgacom.be ([195.238.2.116]:26597 "EHLO outmx017.isp.belgacom.be") by vger.kernel.org with ESMTP id S261430AbVCKUrm (ORCPT ); Fri, 11 Mar 2005 15:47:42 -0500 Message-ID: <423203EC.1070003@246tNt.com> Date: Fri, 11 Mar 2005 21:47:40 +0100 From: Sylvain Munaut User-Agent: Mozilla Thunderbird 0.7.3 (X11/20040816) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Kumar Gala Cc: Tom Rini , LKML , Embedded PPC Linux list Subject: Re: [PATCH 1/2] MPC52xx updates : sparse clean-ups References: <4231F9F9.5080506@246tNt.com> In-Reply-To: X-Enigmail-Version: 0.85.0.0 X-Enigmail-Supports: pgp-inline, pgp-mime Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2798 Lines: 84 Kumar Gala wrote: >> >> >> diff -Nru a/arch/ppc/syslib/mpc52xx_pic.c >> b/arch/ppc/syslib/mpc52xx_pic.c >> --- a/arch/ppc/syslib/mpc52xx_pic.c 2005-03-11 20:41:36 +01:00 >> +++ b/arch/ppc/syslib/mpc52xx_pic.c 2005-03-11 20:41:36 +01:00 >> @@ -33,8 +33,8 @@ >> #include >> >> >> >> -static struct mpc52xx_intr *intr; >> -static struct mpc52xx_sdma *sdma; >> +static struct mpc52xx_intr __iomem *intr; >> +static struct mpc52xx_sdma __iomem *sdma; >> >> static void >> mpc52xx_ic_disable(unsigned int irq) >> @@ -173,7 +173,7 @@ >> mpc52xx_ic_disable, /* disable(irq) */ >> mpc52xx_ic_disable_and_ack, /* disable_and_ack(irq) */ >> mpc52xx_ic_end, /* end(irq) */ >> - 0 /* set_affinity(irq, cpumask) >> SMP. */ >> + NULL /* set_affinity(irq, cpumask) >> SMP. */ >> }; > > > It looks like others have moved to a C99 initialization style for > hw_interrupt_type, see syslib/ipic.c for an example. > Indeed. Here's a third patch ;) It has been added to the bk tree as well. # This is a BitKeeper generated diff -Nru style patch. # # ChangeSet # 2005/03/11 21:37:08+01:00 tnt@246tNt.com # ppc32: Change to a C99 initialization style for hw_interrupt_type # in MPC52xx interrupt controller # # arch/ppc/syslib/mpc52xx_pic.c # 2005/03/11 21:36:54+01:00 tnt@246tNt.com +5 -8 # ppc32: Change to a C99 initialization style for hw_interrupt_type # in MPC52xx interrupt controller # diff -Nru a/arch/ppc/syslib/mpc52xx_pic.c b/arch/ppc/syslib/mpc52xx_pic.c --- a/arch/ppc/syslib/mpc52xx_pic.c 2005-03-11 21:45:50 +01:00 +++ b/arch/ppc/syslib/mpc52xx_pic.c 2005-03-11 21:45:50 +01:00 @@ -166,14 +166,11 @@ } static struct hw_interrupt_type mpc52xx_ic = { - "MPC52xx", - NULL, /* startup(irq) */ - NULL, /* shutdown(irq) */ - mpc52xx_ic_enable, /* enable(irq) */ - mpc52xx_ic_disable, /* disable(irq) */ - mpc52xx_ic_disable_and_ack, /* disable_and_ack(irq) */ - mpc52xx_ic_end, /* end(irq) */ - NULL /* set_affinity(irq, cpumask) SMP. */ + .typename = "MPC52xx", + .enable = mpc52xx_ic_enable, + .disable = mpc52xx_ic_disable, + .ack = mpc52xx_ic_disable_and_ack, + .end = mpc52xx_ic_end, }; void __init - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/