提交 a7eee06b 编写于 作者: C Christian Lamparter 提交者: John W. Linville

p54: two endian fixes

This patch fixes all CHECK_ENDIAN complains:

1. p54/fwio.c:296:6: warning: restricted __le32 degrades to integer
    p54/fwio.c:296:6: warning: restricted __le32 degrades to integer

2. p54/p54spi.c:172:32: warning: incorrect type in initializer
   p54spi.c:172:32:    expected restricted __le32 [usertype] buffer
   p54/p54spi.c:172:32:    got unsigned int
Signed-off-by: NChristian Lamparter <chunkeey@web.de>
Signed-off-by: NJohn W. Linville <linville@tuxdriver.com>
上级 6c230c02
...@@ -292,8 +292,9 @@ int p54_tx_cancel(struct p54_common *priv, __le32 req_id) ...@@ -292,8 +292,9 @@ int p54_tx_cancel(struct p54_common *priv, __le32 req_id)
{ {
struct sk_buff *skb; struct sk_buff *skb;
struct p54_txcancel *cancel; struct p54_txcancel *cancel;
u32 _req_id = le32_to_cpu(req_id);
if (unlikely(req_id < priv->rx_start || req_id > priv->rx_end)) if (unlikely(_req_id < priv->rx_start || _req_id > priv->rx_end))
return -EINVAL; return -EINVAL;
skb = p54_alloc_skb(priv, P54_HDR_FLAG_CONTROL_OPSET, sizeof(*cancel), skb = p54_alloc_skb(priv, P54_HDR_FLAG_CONTROL_OPSET, sizeof(*cancel),
......
...@@ -164,12 +164,12 @@ static const struct p54spi_spi_reg p54spi_registers_array[] = ...@@ -164,12 +164,12 @@ static const struct p54spi_spi_reg p54spi_registers_array[] =
{ SPI_ADRS_DMA_WRITE_BASE, 32, "DMA_RD_BASE " } { SPI_ADRS_DMA_WRITE_BASE, 32, "DMA_RD_BASE " }
}; };
static int p54spi_wait_bit(struct p54s_priv *priv, u16 reg, __le32 bits) static int p54spi_wait_bit(struct p54s_priv *priv, u16 reg, u32 bits)
{ {
int i; int i;
for (i = 0; i < 2000; i++) { for (i = 0; i < 2000; i++) {
__le32 buffer = p54spi_read32(priv, reg); u32 buffer = p54spi_read32(priv, reg);
if ((buffer & bits) == bits) if ((buffer & bits) == bits)
return 1; return 1;
} }
...@@ -179,8 +179,7 @@ static int p54spi_wait_bit(struct p54s_priv *priv, u16 reg, __le32 bits) ...@@ -179,8 +179,7 @@ static int p54spi_wait_bit(struct p54s_priv *priv, u16 reg, __le32 bits)
static int p54spi_spi_write_dma(struct p54s_priv *priv, __le32 base, static int p54spi_spi_write_dma(struct p54s_priv *priv, __le32 base,
const void *buf, size_t len) const void *buf, size_t len)
{ {
if (!p54spi_wait_bit(priv, SPI_ADRS_DMA_WRITE_CTRL, if (!p54spi_wait_bit(priv, SPI_ADRS_DMA_WRITE_CTRL, HOST_ALLOWED)) {
cpu_to_le32(HOST_ALLOWED))) {
dev_err(&priv->spi->dev, "spi_write_dma not allowed " dev_err(&priv->spi->dev, "spi_write_dma not allowed "
"to DMA write.\n"); "to DMA write.\n");
return -EAGAIN; return -EAGAIN;
...@@ -333,7 +332,7 @@ static int p54spi_wakeup(struct p54s_priv *priv) ...@@ -333,7 +332,7 @@ static int p54spi_wakeup(struct p54s_priv *priv)
/* And wait for the READY interrupt */ /* And wait for the READY interrupt */
if (!p54spi_wait_bit(priv, SPI_ADRS_HOST_INTERRUPTS, if (!p54spi_wait_bit(priv, SPI_ADRS_HOST_INTERRUPTS,
cpu_to_le32(SPI_HOST_INT_READY))) { SPI_HOST_INT_READY)) {
dev_err(&priv->spi->dev, "INT_READY timeout\n"); dev_err(&priv->spi->dev, "INT_READY timeout\n");
return -EBUSY; return -EBUSY;
} }
...@@ -444,7 +443,7 @@ static int p54spi_tx_frame(struct p54s_priv *priv, struct sk_buff *skb) ...@@ -444,7 +443,7 @@ static int p54spi_tx_frame(struct p54s_priv *priv, struct sk_buff *skb)
goto out; goto out;
if (!p54spi_wait_bit(priv, SPI_ADRS_HOST_INTERRUPTS, if (!p54spi_wait_bit(priv, SPI_ADRS_HOST_INTERRUPTS,
cpu_to_le32(SPI_HOST_INT_WR_READY))) { SPI_HOST_INT_WR_READY)) {
dev_err(&priv->spi->dev, "WR_READY timeout\n"); dev_err(&priv->spi->dev, "WR_READY timeout\n");
ret = -EAGAIN; ret = -EAGAIN;
goto out; goto out;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册