diff options
author | Jonas Aaberg <jonas.aberg@stericsson.com> | 2010-08-09 08:07:54 -0400 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2010-09-22 17:53:45 -0400 |
commit | 1b00348d5d0b3423fe21f499bf30d40a4d1dc594 (patch) | |
tree | 6fb57150aaa04d582b860178572405492a7c3321 | |
parent | be8cb7dfd92420b12dfd6831c2d638f4f46bdfdf (diff) |
DMAENGINE: ste_dma40: No need reading, masking and setting a set register
Removes an unnecessary register read and a few lines of code.
Signed-off-by: Jonas Aaberg <jonas.aberg@stericsson.com>
Signed-off-by: Linus Walleij <linus.walleij@stericsson.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
-rw-r--r-- | drivers/dma/ste_dma40.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c index 8aff86594141..538c35d05a57 100644 --- a/drivers/dma/ste_dma40.c +++ b/drivers/dma/ste_dma40.c | |||
@@ -915,7 +915,6 @@ static irqreturn_t d40_handle_interrupt(int irq, void *data) | |||
915 | 915 | ||
916 | int i; | 916 | int i; |
917 | u32 regs[ARRAY_SIZE(il)]; | 917 | u32 regs[ARRAY_SIZE(il)]; |
918 | u32 tmp; | ||
919 | u32 idx; | 918 | u32 idx; |
920 | u32 row; | 919 | u32 row; |
921 | long chan = -1; | 920 | long chan = -1; |
@@ -942,9 +941,7 @@ static irqreturn_t d40_handle_interrupt(int irq, void *data) | |||
942 | idx = chan & (BITS_PER_LONG - 1); | 941 | idx = chan & (BITS_PER_LONG - 1); |
943 | 942 | ||
944 | /* ACK interrupt */ | 943 | /* ACK interrupt */ |
945 | tmp = readl(base->virtbase + il[row].clr); | 944 | writel(1 << idx, base->virtbase + il[row].clr); |
946 | tmp |= 1 << idx; | ||
947 | writel(tmp, base->virtbase + il[row].clr); | ||
948 | 945 | ||
949 | if (il[row].offset == D40_PHY_CHAN) | 946 | if (il[row].offset == D40_PHY_CHAN) |
950 | d40c = base->lookup_phy_chans[idx]; | 947 | d40c = base->lookup_phy_chans[idx]; |