davinci: da8xx: sparse cleanup: remove duplicate entries in irq priorities
authorSekhar Nori <nsekhar@ti.com>
Mon, 28 Jun 2010 11:46:37 +0000 (17:16 +0530)
committerKevin Hilman <khilman@deeprootsystems.com>
Thu, 5 Aug 2010 16:58:23 +0000 (09:58 -0700)
This patch helps get rid of the following sparse warnings
of the type:

  CHECK   arch/arm/mach-davinci/da830.c
arch/arm/mach-davinci/da830.c:1026:3: warning: Initializer entry defined twice
arch/arm/mach-davinci/da830.c:1027:3:   also defined here

coming from the irq priorities array init.

Apart from one instance of genuinie repetition, most are are instances
of multiple #defines of the same interrupt number. I have not
removed the multiple definitions from the irq.h file in the hope
that someone might decide to use them as shared interrupts at some
point of time. The priority initialization however needs to be done
only once and hence has been corrected.

Signed-off-by: Sekhar Nori <nsekhar@ti.com>
Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>
arch/arm/mach-davinci/da830.c
arch/arm/mach-davinci/da850.c

index 23e9eda..ec23ab4 100644 (file)
@@ -1024,7 +1024,6 @@ static u8 da830_default_priorities[DA830_N_CP_INTC_IRQ] = {
        [IRQ_DA8XX_EVTOUT4]             = 7,
        [IRQ_DA8XX_EVTOUT5]             = 7,
        [IRQ_DA8XX_EVTOUT6]             = 7,
-       [IRQ_DA8XX_EVTOUT6]             = 7,
        [IRQ_DA8XX_EVTOUT7]             = 7,
        [IRQ_DA8XX_CCINT0]              = 7,
        [IRQ_DA8XX_CCERRINT]            = 7,
@@ -1042,11 +1041,7 @@ static u8 da830_default_priorities[DA830_N_CP_INTC_IRQ] = {
        [IRQ_DA8XX_TINT34_1]            = 7,
        [IRQ_DA8XX_UARTINT0]            = 7,
        [IRQ_DA8XX_KEYMGRINT]           = 7,
-       [IRQ_DA8XX_SECINT]              = 7,
-       [IRQ_DA8XX_SECKEYERR]           = 7,
        [IRQ_DA830_MPUERR]              = 7,
-       [IRQ_DA830_IOPUERR]             = 7,
-       [IRQ_DA830_BOOTCFGERR]          = 7,
        [IRQ_DA8XX_CHIPINT0]            = 7,
        [IRQ_DA8XX_CHIPINT1]            = 7,
        [IRQ_DA8XX_CHIPINT2]            = 7,
index 6b8331b..68ed58a 100644 (file)
@@ -643,7 +643,6 @@ static u8 da850_default_priorities[DA850_N_CP_INTC_IRQ] = {
        [IRQ_DA8XX_EVTOUT4]             = 7,
        [IRQ_DA8XX_EVTOUT5]             = 7,
        [IRQ_DA8XX_EVTOUT6]             = 7,
-       [IRQ_DA8XX_EVTOUT6]             = 7,
        [IRQ_DA8XX_EVTOUT7]             = 7,
        [IRQ_DA8XX_CCINT0]              = 7,
        [IRQ_DA8XX_CCERRINT]            = 7,
@@ -661,27 +660,7 @@ static u8 da850_default_priorities[DA850_N_CP_INTC_IRQ] = {
        [IRQ_DA8XX_TINT34_1]            = 7,
        [IRQ_DA8XX_UARTINT0]            = 7,
        [IRQ_DA8XX_KEYMGRINT]           = 7,
-       [IRQ_DA8XX_SECINT]              = 7,
-       [IRQ_DA8XX_SECKEYERR]           = 7,
        [IRQ_DA850_MPUADDRERR0]         = 7,
-       [IRQ_DA850_MPUPROTERR0]         = 7,
-       [IRQ_DA850_IOPUADDRERR0]        = 7,
-       [IRQ_DA850_IOPUPROTERR0]        = 7,
-       [IRQ_DA850_IOPUADDRERR1]        = 7,
-       [IRQ_DA850_IOPUPROTERR1]        = 7,
-       [IRQ_DA850_IOPUADDRERR2]        = 7,
-       [IRQ_DA850_IOPUPROTERR2]        = 7,
-       [IRQ_DA850_BOOTCFG_ADDR_ERR]    = 7,
-       [IRQ_DA850_BOOTCFG_PROT_ERR]    = 7,
-       [IRQ_DA850_MPUADDRERR1]         = 7,
-       [IRQ_DA850_MPUPROTERR1]         = 7,
-       [IRQ_DA850_IOPUADDRERR3]        = 7,
-       [IRQ_DA850_IOPUPROTERR3]        = 7,
-       [IRQ_DA850_IOPUADDRERR4]        = 7,
-       [IRQ_DA850_IOPUPROTERR4]        = 7,
-       [IRQ_DA850_IOPUADDRERR5]        = 7,
-       [IRQ_DA850_IOPUPROTERR5]        = 7,
-       [IRQ_DA850_MIOPU_BOOTCFG_ERR]   = 7,
        [IRQ_DA8XX_CHIPINT0]            = 7,
        [IRQ_DA8XX_CHIPINT1]            = 7,
        [IRQ_DA8XX_CHIPINT2]            = 7,
@@ -722,8 +701,6 @@ static u8 da850_default_priorities[DA850_N_CP_INTC_IRQ] = {
        [IRQ_DA8XX_EHRPWM1]             = 7,
        [IRQ_DA8XX_EHRPWM1TZ]           = 7,
        [IRQ_DA850_SATAINT]             = 7,
-       [IRQ_DA850_TINT12_2]            = 7,
-       [IRQ_DA850_TINT34_2]            = 7,
        [IRQ_DA850_TINTALL_2]           = 7,
        [IRQ_DA8XX_ECAP0]               = 7,
        [IRQ_DA8XX_ECAP1]               = 7,
@@ -751,8 +728,6 @@ static u8 da850_default_priorities[DA850_N_CP_INTC_IRQ] = {
        [IRQ_DA850_CCINT1]              = 7,
        [IRQ_DA850_CCERRINT1]           = 7,
        [IRQ_DA850_TCERRINT2]           = 7,
-       [IRQ_DA850_TINT12_3]            = 7,
-       [IRQ_DA850_TINT34_3]            = 7,
        [IRQ_DA850_TINTALL_3]           = 7,
        [IRQ_DA850_MCBSP0RINT]          = 7,
        [IRQ_DA850_MCBSP0XINT]          = 7,