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>
struct pci_dev *pci; /* the pci kernel structure of this card */
int irq;
unsigned long io_base;
struct pci_dev *pci; /* the pci kernel structure of this card */
int irq;
unsigned long io_base;
- unsigned long mem_base;
+ void __iomem *mem_base;
enum CHIPTYP chip_type;
enum CTCARDS model;
enum CHIPTYP chip_type;
enum CTCARDS model;
unsigned int is_uaa;
unsigned int data[4] = {0};
unsigned int io_base;
unsigned int is_uaa;
unsigned int data[4] = {0};
unsigned int io_base;
+ void __iomem *mem_base;
int i;
const u32 CTLX = CTLBITS('C', 'T', 'L', 'X');
const u32 CTL_ = CTLBITS('C', 'T', 'L', '-');
int i;
const u32 CTLX = CTLBITS('C', 'T', 'L', 'X');
const u32 CTL_ = CTLBITS('C', 'T', 'L', '-');
hw->irq = -1;
if (hw->mem_base)
hw->irq = -1;
if (hw->mem_base)
- iounmap((void *)hw->mem_base);
- hw->mem_base = (unsigned long)NULL;
if (hw->io_base)
pci_release_regions(hw->pci);
if (hw->io_base)
pci_release_regions(hw->pci);
goto error1;
hw->io_base = pci_resource_start(hw->pci, 2);
goto error1;
hw->io_base = pci_resource_start(hw->pci, 2);
- hw->mem_base = (unsigned long)ioremap(hw->io_base,
- pci_resource_len(hw->pci, 2));
+ hw->mem_base = ioremap(hw->io_base,
+ pci_resource_len(hw->pci, 2));
if (!hw->mem_base) {
err = -ENOENT;
goto error2;
if (!hw->mem_base) {
err = -ENOENT;
goto error2;
hw->irq = -1;
if (hw->mem_base)
hw->irq = -1;
if (hw->mem_base)
- iounmap((void *)hw->mem_base);
- hw->mem_base = (unsigned long)NULL;
if (hw->io_base)
pci_release_regions(hw->pci);
if (hw->io_base)
pci_release_regions(hw->pci);
static u32 hw_read_20kx(struct hw *hw, u32 reg)
{
static u32 hw_read_20kx(struct hw *hw, u32 reg)
{
- return readl((void *)(hw->mem_base + reg));
+ return readl(hw->mem_base + reg);
}
static void hw_write_20kx(struct hw *hw, u32 reg, u32 data)
{
}
static void hw_write_20kx(struct hw *hw, u32 reg, u32 data)
{
- writel(data, (void *)(hw->mem_base + reg));
+ writel(data, hw->mem_base + reg);
}
static struct hw ct20k2_preset = {
}
static struct hw ct20k2_preset = {