diff options
author | Geert Uytterhoeven <geert+renesas@linux-m68k.org> | 2013-12-24 04:49:31 -0500 |
---|---|---|
committer | Mark Brown <broonie@linaro.org> | 2013-12-24 08:19:55 -0500 |
commit | c132f0949ab62ba95474f6eda44e9e1b5780b973 (patch) | |
tree | f649e087596e3806d0391508e9c86be8854d21d6 /drivers/spi | |
parent | fc671a900b698e5aa35109100bb9a1598e914466 (diff) |
spi: rspi: Remove casts
Remove useless casts, and do not cast away const.
Signed-off-by: Geert Uytterhoeven <geert+renesas@linux-m68k.org>
Signed-off-by: Mark Brown <broonie@linaro.org>
Diffstat (limited to 'drivers/spi')
-rw-r--r-- | drivers/spi/spi-rspi.c | 29 |
1 files changed, 14 insertions, 15 deletions
diff --git a/drivers/spi/spi-rspi.c b/drivers/spi/spi-rspi.c index e6aa42402850..73f1a29b22c3 100644 --- a/drivers/spi/spi-rspi.c +++ b/drivers/spi/spi-rspi.c | |||
@@ -330,9 +330,7 @@ static int rspi_send_pio(struct rspi_data *rspi, struct spi_message *mesg, | |||
330 | struct spi_transfer *t) | 330 | struct spi_transfer *t) |
331 | { | 331 | { |
332 | int remain = t->len; | 332 | int remain = t->len; |
333 | u8 *data; | 333 | const u8 *data = t->tx_buf; |
334 | |||
335 | data = (u8 *)t->tx_buf; | ||
336 | while (remain > 0) { | 334 | while (remain > 0) { |
337 | rspi_write8(rspi, rspi_read8(rspi, RSPI_SPCR) | SPCR_TXMD, | 335 | rspi_write8(rspi, rspi_read8(rspi, RSPI_SPCR) | SPCR_TXMD, |
338 | RSPI_SPCR); | 336 | RSPI_SPCR); |
@@ -358,12 +356,11 @@ static int qspi_send_pio(struct rspi_data *rspi, struct spi_message *mesg, | |||
358 | struct spi_transfer *t) | 356 | struct spi_transfer *t) |
359 | { | 357 | { |
360 | int remain = t->len; | 358 | int remain = t->len; |
361 | u8 *data; | 359 | const u8 *data = t->tx_buf; |
362 | 360 | ||
363 | rspi_write8(rspi, SPBFCR_TXRST, QSPI_SPBFCR); | 361 | rspi_write8(rspi, SPBFCR_TXRST, QSPI_SPBFCR); |
364 | rspi_write8(rspi, 0x00, QSPI_SPBFCR); | 362 | rspi_write8(rspi, 0x00, QSPI_SPBFCR); |
365 | 363 | ||
366 | data = (u8 *)t->tx_buf; | ||
367 | while (remain > 0) { | 364 | while (remain > 0) { |
368 | 365 | ||
369 | if (rspi_wait_for_interrupt(rspi, SPSR_SPTEF, SPCR_SPTIE) < 0) { | 366 | if (rspi_wait_for_interrupt(rspi, SPSR_SPTEF, SPCR_SPTIE) < 0) { |
@@ -399,8 +396,8 @@ static void rspi_dma_complete(void *arg) | |||
399 | wake_up_interruptible(&rspi->wait); | 396 | wake_up_interruptible(&rspi->wait); |
400 | } | 397 | } |
401 | 398 | ||
402 | static int rspi_dma_map_sg(struct scatterlist *sg, void *buf, unsigned len, | 399 | static int rspi_dma_map_sg(struct scatterlist *sg, const void *buf, |
403 | struct dma_chan *chan, | 400 | unsigned len, struct dma_chan *chan, |
404 | enum dma_transfer_direction dir) | 401 | enum dma_transfer_direction dir) |
405 | { | 402 | { |
406 | sg_init_table(sg, 1); | 403 | sg_init_table(sg, 1); |
@@ -440,12 +437,13 @@ static void rspi_memory_from_8bit(void *buf, const void *data, unsigned len) | |||
440 | static int rspi_send_dma(struct rspi_data *rspi, struct spi_transfer *t) | 437 | static int rspi_send_dma(struct rspi_data *rspi, struct spi_transfer *t) |
441 | { | 438 | { |
442 | struct scatterlist sg; | 439 | struct scatterlist sg; |
443 | void *buf = NULL; | 440 | const void *buf = NULL; |
444 | struct dma_async_tx_descriptor *desc; | 441 | struct dma_async_tx_descriptor *desc; |
445 | unsigned len; | 442 | unsigned len; |
446 | int ret = 0; | 443 | int ret = 0; |
447 | 444 | ||
448 | if (rspi->dma_width_16bit) { | 445 | if (rspi->dma_width_16bit) { |
446 | void *tmp; | ||
449 | /* | 447 | /* |
450 | * If DMAC bus width is 16-bit, the driver allocates a dummy | 448 | * If DMAC bus width is 16-bit, the driver allocates a dummy |
451 | * buffer. And, the driver converts original data into the | 449 | * buffer. And, the driver converts original data into the |
@@ -454,13 +452,14 @@ static int rspi_send_dma(struct rspi_data *rspi, struct spi_transfer *t) | |||
454 | * DMAC data: 1st byte, dummy, 2nd byte, dummy ... | 452 | * DMAC data: 1st byte, dummy, 2nd byte, dummy ... |
455 | */ | 453 | */ |
456 | len = t->len * 2; | 454 | len = t->len * 2; |
457 | buf = kmalloc(len, GFP_KERNEL); | 455 | tmp = kmalloc(len, GFP_KERNEL); |
458 | if (!buf) | 456 | if (!tmp) |
459 | return -ENOMEM; | 457 | return -ENOMEM; |
460 | rspi_memory_to_8bit(buf, t->tx_buf, t->len); | 458 | rspi_memory_to_8bit(tmp, t->tx_buf, t->len); |
459 | buf = tmp; | ||
461 | } else { | 460 | } else { |
462 | len = t->len; | 461 | len = t->len; |
463 | buf = (void *)t->tx_buf; | 462 | buf = t->tx_buf; |
464 | } | 463 | } |
465 | 464 | ||
466 | if (!rspi_dma_map_sg(&sg, buf, len, rspi->chan_tx, DMA_TO_DEVICE)) { | 465 | if (!rspi_dma_map_sg(&sg, buf, len, rspi->chan_tx, DMA_TO_DEVICE)) { |
@@ -528,7 +527,7 @@ static int rspi_receive_pio(struct rspi_data *rspi, struct spi_message *mesg, | |||
528 | 527 | ||
529 | rspi_receive_init(rspi); | 528 | rspi_receive_init(rspi); |
530 | 529 | ||
531 | data = (u8 *)t->rx_buf; | 530 | data = t->rx_buf; |
532 | while (remain > 0) { | 531 | while (remain > 0) { |
533 | rspi_write8(rspi, rspi_read8(rspi, RSPI_SPCR) & ~SPCR_TXMD, | 532 | rspi_write8(rspi, rspi_read8(rspi, RSPI_SPCR) & ~SPCR_TXMD, |
534 | RSPI_SPCR); | 533 | RSPI_SPCR); |
@@ -575,7 +574,7 @@ static int qspi_receive_pio(struct rspi_data *rspi, struct spi_message *mesg, | |||
575 | 574 | ||
576 | qspi_receive_init(rspi); | 575 | qspi_receive_init(rspi); |
577 | 576 | ||
578 | data = (u8 *)t->rx_buf; | 577 | data = t->rx_buf; |
579 | while (remain > 0) { | 578 | while (remain > 0) { |
580 | 579 | ||
581 | if (rspi_wait_for_interrupt(rspi, SPSR_SPTEF, SPCR_SPTIE) < 0) { | 580 | if (rspi_wait_for_interrupt(rspi, SPSR_SPTEF, SPCR_SPTIE) < 0) { |
@@ -802,7 +801,7 @@ static void rspi_cleanup(struct spi_device *spi) | |||
802 | 801 | ||
803 | static irqreturn_t rspi_irq(int irq, void *_sr) | 802 | static irqreturn_t rspi_irq(int irq, void *_sr) |
804 | { | 803 | { |
805 | struct rspi_data *rspi = (struct rspi_data *)_sr; | 804 | struct rspi_data *rspi = _sr; |
806 | unsigned long spsr; | 805 | unsigned long spsr; |
807 | irqreturn_t ret = IRQ_NONE; | 806 | irqreturn_t ret = IRQ_NONE; |
808 | unsigned char disable_irq = 0; | 807 | unsigned char disable_irq = 0; |