diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2014-09-23 15:00:24 +0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2014-09-23 18:00:31 +0400 |
commit | 7a7686bd0d153c0d6e120da6712c9339aaeaa2f9 (patch) | |
tree | a170a1a9e2ac98edaf93dfad35911d40c2f43f88 /sound/pci/ctxfi/cthw20k1.c | |
parent | 95f72cf2cdf0e612aeaf36d8af51689882fd64db (diff) | |
download | linux-7a7686bd0d153c0d6e120da6712c9339aaeaa2f9.tar.xz |
ALSA: ctxfi: sparse warning
fixed sparse warning of incorrect type (different address spaces) in
cthw20k1.c and cthw20k2.c which was being actually caused as mem_base
was of the type unsigned long.
Again as mem_base was previously unsigned long , so it required many
typecasts in the code to convert interger to pointer.
Now after giving the correct type of mem_base as void __iomem *
we can also remove those typecasts maintaining the same functionality
and logic of the code.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/ctxfi/cthw20k1.c')
-rw-r--r-- | sound/pci/ctxfi/cthw20k1.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sound/pci/ctxfi/cthw20k1.c b/sound/pci/ctxfi/cthw20k1.c index 71d496f780e3..8fc524fbaeab 100644 --- a/sound/pci/ctxfi/cthw20k1.c +++ b/sound/pci/ctxfi/cthw20k1.c @@ -1802,7 +1802,7 @@ static int uaa_to_xfi(struct pci_dev *pci) unsigned int is_uaa; unsigned int data[4] = {0}; unsigned int io_base; - void *mem_base; + void __iomem *mem_base; int i; const u32 CTLX = CTLBITS('C', 'T', 'L', 'X'); const u32 CTL_ = CTLBITS('C', 'T', 'L', '-'); @@ -1984,9 +1984,9 @@ static int hw_card_shutdown(struct hw *hw) hw->irq = -1; if (hw->mem_base) - iounmap((void *)hw->mem_base); + iounmap(hw->mem_base); - hw->mem_base = (unsigned long)NULL; + hw->mem_base = NULL; if (hw->io_base) pci_release_regions(hw->pci); |