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 23e9eda5a37787f583ac0226e1436617b45d9167..ec23ab473620d3c69d0ae1e952a754b2ec3f12c9 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 6b8331bf8cf3dea8ba5306111559fdf82ccbf5d2..68ed58a48252948ee756c544a97d002d0ded8082 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,
This page took 0.029747 seconds and 5 git commands to generate.