Skip to content

Commit

Permalink
[PATCH] cs89x0: use u16 for device register data
Browse files Browse the repository at this point in the history
cs89x0 inconsistently used 'int' and 'u32' for device register data.
As the cs89x0 is a 16-bit chip, change the I/O accessors over to 'u16'.
(Spotted by Deepak Saxena.)

Signed-off-by: Lennert Buytenhek <buytenh@wantstofly.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
  • Loading branch information
Lennert Buytenhek authored and Linus Torvalds committed Jan 15, 2006
1 parent 580d7b8 commit a07f0db
Showing 1 changed file with 19 additions and 19 deletions.
38 changes: 19 additions & 19 deletions drivers/net/cs89x0.c
Original file line number Diff line number Diff line change
Expand Up @@ -342,38 +342,38 @@ struct net_device * __init cs89x0_probe(int unit)
#endif

#if defined(CONFIG_ARCH_IXDP2X01)
static int
static u16
readword(unsigned long base_addr, int portno)
{
return (u16)__raw_readl(base_addr + (portno << 1));
return __raw_readl(base_addr + (portno << 1));
}

static void
writeword(unsigned long base_addr, int portno, int value)
writeword(unsigned long base_addr, int portno, u16 value)
{
__raw_writel((u16)value, base_addr + (portno << 1));
__raw_writel(value, base_addr + (portno << 1));
}
#elif defined(CONFIG_ARCH_PNX010X)
static int
static u16
readword(unsigned long base_addr, int portno)
{
return inw(base_addr + (portno << 1));
}

static void
writeword(unsigned long base_addr, int portno, int value)
writeword(unsigned long base_addr, int portno, u16 value)
{
outw(value, base_addr + (portno << 1));
}
#else
static int
static u16
readword(unsigned long base_addr, int portno)
{
return inw(base_addr + portno);
}

static void
writeword(unsigned long base_addr, int portno, int value)
writeword(unsigned long base_addr, int portno, u16 value)
{
outw(value, base_addr + portno);
}
Expand All @@ -385,11 +385,11 @@ readwords(unsigned long base_addr, int portno, void *buf, int length)
u8 *buf8 = (u8 *)buf;

do {
u32 tmp32;
u16 tmp16;

tmp32 = readword(base_addr, portno);
*buf8++ = (u8)tmp32;
*buf8++ = (u8)(tmp32 >> 8);
tmp16 = readword(base_addr, portno);
*buf8++ = (u8)tmp16;
*buf8++ = (u8)(tmp16 >> 8);
} while (--length);
}

Expand All @@ -399,23 +399,23 @@ writewords(unsigned long base_addr, int portno, void *buf, int length)
u8 *buf8 = (u8 *)buf;

do {
u32 tmp32;
u16 tmp16;

tmp32 = *buf8++;
tmp32 |= (*buf8++) << 8;
writeword(base_addr, portno, tmp32);
tmp16 = *buf8++;
tmp16 |= (*buf8++) << 8;
writeword(base_addr, portno, tmp16);
} while (--length);
}

static int
readreg(struct net_device *dev, int regno)
static u16
readreg(struct net_device *dev, u16 regno)
{
writeword(dev->base_addr, ADD_PORT, regno);
return readword(dev->base_addr, DATA_PORT);
}

static void
writereg(struct net_device *dev, int regno, int value)
writereg(struct net_device *dev, u16 regno, u16 value)
{
writeword(dev->base_addr, ADD_PORT, regno);
writeword(dev->base_addr, DATA_PORT, value);
Expand Down

0 comments on commit a07f0db

Please sign in to comment.