Commit f205e8d1 authored by Malcolm Priestley's avatar Malcolm Priestley Committed by Greg Kroah-Hartman

staging: vt6655: mac.c Replace VNSvInPortB byData reads with ioread8.

Removing byData variable altogether.
Signed-off-by: default avatarMalcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent c2d845de
...@@ -74,10 +74,8 @@ bool MACbIsRegBitsOn(struct vnt_private *priv, unsigned char byRegOfs, ...@@ -74,10 +74,8 @@ bool MACbIsRegBitsOn(struct vnt_private *priv, unsigned char byRegOfs,
unsigned char byTestBits) unsigned char byTestBits)
{ {
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned char byData;
VNSvInPortB(io_base + byRegOfs, &byData); return (ioread8(io_base + byRegOfs) & byTestBits) == byTestBits;
return (byData & byTestBits) == byTestBits;
} }
/* /*
...@@ -99,10 +97,8 @@ bool MACbIsRegBitsOff(struct vnt_private *priv, unsigned char byRegOfs, ...@@ -99,10 +97,8 @@ bool MACbIsRegBitsOff(struct vnt_private *priv, unsigned char byRegOfs,
unsigned char byTestBits) unsigned char byTestBits)
{ {
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned char byData;
VNSvInPortB(io_base + byRegOfs, &byData); return !(ioread8(io_base + byRegOfs) & byTestBits);
return !(byData & byTestBits);
} }
/* /*
...@@ -304,15 +300,13 @@ void MACvRestoreContext(struct vnt_private *priv, unsigned char *pbyCxtBuf) ...@@ -304,15 +300,13 @@ void MACvRestoreContext(struct vnt_private *priv, unsigned char *pbyCxtBuf)
bool MACbSoftwareReset(struct vnt_private *priv) bool MACbSoftwareReset(struct vnt_private *priv)
{ {
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned char byData;
unsigned short ww; unsigned short ww;
/* turn on HOSTCR_SOFTRST, just write 0x01 to reset */ /* turn on HOSTCR_SOFTRST, just write 0x01 to reset */
VNSvOutPortB(io_base + MAC_REG_HOSTCR, 0x01); VNSvOutPortB(io_base + MAC_REG_HOSTCR, 0x01);
for (ww = 0; ww < W_MAX_TIMEOUT; ww++) { for (ww = 0; ww < W_MAX_TIMEOUT; ww++) {
VNSvInPortB(io_base + MAC_REG_HOSTCR, &byData); if (!(ioread8(io_base + MAC_REG_HOSTCR) & HOSTCR_SOFTRST))
if (!(byData & HOSTCR_SOFTRST))
break; break;
} }
if (ww == W_MAX_TIMEOUT) if (ww == W_MAX_TIMEOUT)
...@@ -370,7 +364,6 @@ bool MACbSafeRxOff(struct vnt_private *priv) ...@@ -370,7 +364,6 @@ bool MACbSafeRxOff(struct vnt_private *priv)
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned short ww; unsigned short ww;
unsigned long dwData; unsigned long dwData;
unsigned char byData;
/* turn off wow temp for turn off Rx safely */ /* turn off wow temp for turn off Rx safely */
...@@ -400,8 +393,7 @@ bool MACbSafeRxOff(struct vnt_private *priv) ...@@ -400,8 +393,7 @@ bool MACbSafeRxOff(struct vnt_private *priv)
MACvRegBitsOff(io_base, MAC_REG_HOSTCR, HOSTCR_RXON); MACvRegBitsOff(io_base, MAC_REG_HOSTCR, HOSTCR_RXON);
/* W_MAX_TIMEOUT is the timeout period */ /* W_MAX_TIMEOUT is the timeout period */
for (ww = 0; ww < W_MAX_TIMEOUT; ww++) { for (ww = 0; ww < W_MAX_TIMEOUT; ww++) {
VNSvInPortB(io_base + MAC_REG_HOSTCR, &byData); if (!(ioread8(io_base + MAC_REG_HOSTCR) & HOSTCR_RXONST))
if (!(byData & HOSTCR_RXONST))
break; break;
} }
if (ww == W_MAX_TIMEOUT) { if (ww == W_MAX_TIMEOUT) {
...@@ -429,7 +421,6 @@ bool MACbSafeTxOff(struct vnt_private *priv) ...@@ -429,7 +421,6 @@ bool MACbSafeTxOff(struct vnt_private *priv)
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned short ww; unsigned short ww;
unsigned long dwData; unsigned long dwData;
unsigned char byData;
/* Clear TX DMA */ /* Clear TX DMA */
/* Tx0 */ /* Tx0 */
...@@ -461,8 +452,7 @@ bool MACbSafeTxOff(struct vnt_private *priv) ...@@ -461,8 +452,7 @@ bool MACbSafeTxOff(struct vnt_private *priv)
/* W_MAX_TIMEOUT is the timeout period */ /* W_MAX_TIMEOUT is the timeout period */
for (ww = 0; ww < W_MAX_TIMEOUT; ww++) { for (ww = 0; ww < W_MAX_TIMEOUT; ww++) {
VNSvInPortB(io_base + MAC_REG_HOSTCR, &byData); if (!(ioread8(io_base + MAC_REG_HOSTCR) & HOSTCR_TXONST))
if (!(byData & HOSTCR_TXONST))
break; break;
} }
if (ww == W_MAX_TIMEOUT) { if (ww == W_MAX_TIMEOUT) {
...@@ -584,7 +574,6 @@ void MACvSetCurrRx0DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd ...@@ -584,7 +574,6 @@ void MACvSetCurrRx0DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd
{ {
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned short ww; unsigned short ww;
unsigned char byData;
unsigned char byOrgDMACtl; unsigned char byOrgDMACtl;
VNSvInPortB(io_base + MAC_REG_RXDMACTL0, &byOrgDMACtl); VNSvInPortB(io_base + MAC_REG_RXDMACTL0, &byOrgDMACtl);
...@@ -592,8 +581,7 @@ void MACvSetCurrRx0DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd ...@@ -592,8 +581,7 @@ void MACvSetCurrRx0DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd
VNSvOutPortB(io_base + MAC_REG_RXDMACTL0+2, DMACTL_RUN); VNSvOutPortB(io_base + MAC_REG_RXDMACTL0+2, DMACTL_RUN);
for (ww = 0; ww < W_MAX_TIMEOUT; ww++) { for (ww = 0; ww < W_MAX_TIMEOUT; ww++) {
VNSvInPortB(io_base + MAC_REG_RXDMACTL0, &byData); if (!(ioread8(io_base + MAC_REG_RXDMACTL0) & DMACTL_RUN))
if (!(byData & DMACTL_RUN))
break; break;
} }
...@@ -620,7 +608,6 @@ void MACvSetCurrRx1DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd ...@@ -620,7 +608,6 @@ void MACvSetCurrRx1DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd
{ {
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned short ww; unsigned short ww;
unsigned char byData;
unsigned char byOrgDMACtl; unsigned char byOrgDMACtl;
VNSvInPortB(io_base + MAC_REG_RXDMACTL1, &byOrgDMACtl); VNSvInPortB(io_base + MAC_REG_RXDMACTL1, &byOrgDMACtl);
...@@ -628,8 +615,7 @@ void MACvSetCurrRx1DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd ...@@ -628,8 +615,7 @@ void MACvSetCurrRx1DescAddr(struct vnt_private *priv, unsigned long dwCurrDescAd
VNSvOutPortB(io_base + MAC_REG_RXDMACTL1+2, DMACTL_RUN); VNSvOutPortB(io_base + MAC_REG_RXDMACTL1+2, DMACTL_RUN);
for (ww = 0; ww < W_MAX_TIMEOUT; ww++) { for (ww = 0; ww < W_MAX_TIMEOUT; ww++) {
VNSvInPortB(io_base + MAC_REG_RXDMACTL1, &byData); if (!(ioread8(io_base + MAC_REG_RXDMACTL1) & DMACTL_RUN))
if (!(byData & DMACTL_RUN))
break; break;
} }
...@@ -658,7 +644,6 @@ void MACvSetCurrTx0DescAddrEx(struct vnt_private *priv, ...@@ -658,7 +644,6 @@ void MACvSetCurrTx0DescAddrEx(struct vnt_private *priv,
{ {
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned short ww; unsigned short ww;
unsigned char byData;
unsigned char byOrgDMACtl; unsigned char byOrgDMACtl;
VNSvInPortB(io_base + MAC_REG_TXDMACTL0, &byOrgDMACtl); VNSvInPortB(io_base + MAC_REG_TXDMACTL0, &byOrgDMACtl);
...@@ -666,8 +651,7 @@ void MACvSetCurrTx0DescAddrEx(struct vnt_private *priv, ...@@ -666,8 +651,7 @@ void MACvSetCurrTx0DescAddrEx(struct vnt_private *priv,
VNSvOutPortB(io_base + MAC_REG_TXDMACTL0+2, DMACTL_RUN); VNSvOutPortB(io_base + MAC_REG_TXDMACTL0+2, DMACTL_RUN);
for (ww = 0; ww < W_MAX_TIMEOUT; ww++) { for (ww = 0; ww < W_MAX_TIMEOUT; ww++) {
VNSvInPortB(io_base + MAC_REG_TXDMACTL0, &byData); if (!(ioread8(io_base + MAC_REG_TXDMACTL0) & DMACTL_RUN))
if (!(byData & DMACTL_RUN))
break; break;
} }
...@@ -696,7 +680,6 @@ void MACvSetCurrAC0DescAddrEx(struct vnt_private *priv, ...@@ -696,7 +680,6 @@ void MACvSetCurrAC0DescAddrEx(struct vnt_private *priv,
{ {
void __iomem *io_base = priv->PortOffset; void __iomem *io_base = priv->PortOffset;
unsigned short ww; unsigned short ww;
unsigned char byData;
unsigned char byOrgDMACtl; unsigned char byOrgDMACtl;
VNSvInPortB(io_base + MAC_REG_AC0DMACTL, &byOrgDMACtl); VNSvInPortB(io_base + MAC_REG_AC0DMACTL, &byOrgDMACtl);
...@@ -704,8 +687,7 @@ void MACvSetCurrAC0DescAddrEx(struct vnt_private *priv, ...@@ -704,8 +687,7 @@ void MACvSetCurrAC0DescAddrEx(struct vnt_private *priv,
VNSvOutPortB(io_base + MAC_REG_AC0DMACTL+2, DMACTL_RUN); VNSvOutPortB(io_base + MAC_REG_AC0DMACTL+2, DMACTL_RUN);
for (ww = 0; ww < W_MAX_TIMEOUT; ww++) { for (ww = 0; ww < W_MAX_TIMEOUT; ww++) {
VNSvInPortB(io_base + MAC_REG_AC0DMACTL, &byData); if (!(ioread8(io_base + MAC_REG_AC0DMACTL) & DMACTL_RUN))
if (!(byData & DMACTL_RUN))
break; break;
} }
if (ww == W_MAX_TIMEOUT) if (ww == W_MAX_TIMEOUT)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment