GitHub-Mirror/Star64_linux
2
0
Fork 0
mirror of https://github.com/Fishwaldo/Star64_linux.git synced 2025-04-23 06:44:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

spi: dw: Remove needless if statements

Browse source 
SPI core validates both bits_per_word and speed_hz transfer parameters and
defaults to spi->bits_per_word and spi->max_speed_hz in case these per
transfer parameters are not set. This allows to remove related if
statements as they evaluate always to true and reduce indentation.

Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
Jarkko Nikula 2015-09-15 16:26:23 +03:00 • committed by Mark Brown
parent 6ff33f3902
commit 0ed36990a9
1 changed files with 18 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

41
drivers/spi/spi-dw.c
View file

@ -309,34 +309,29 @@ static int dw_spi_transfer_one(struct spi_master *master,
cr0 = chip->cr0; cr0 = chip->cr0;
/* Handle per transfer options for bpw and speed */ /* Handle per transfer options for bpw and speed */
if (transfer->speed_hz) { speed = chip->speed_hz;
speed = chip->speed_hz; if ((transfer->speed_hz != speed) || !chip->clk_div) {
speed = transfer->speed_hz;
if ((transfer->speed_hz != speed) || !chip->clk_div) { /* clk_div doesn't support odd number */
speed = transfer->speed_hz; clk_div = (dws->max_freq / speed + 1) & 0xfffe;
/* clk_div doesn't support odd number */ chip->speed_hz = speed;
clk_div = (dws->max_freq / speed + 1) & 0xfffe; chip->clk_div = clk_div;
chip->speed_hz = speed; spi_set_clk(dws, chip->clk_div);
chip->clk_div = clk_div;
spi_set_clk(dws, chip->clk_div);
}
} }
if (transfer->bits_per_word) { if (transfer->bits_per_word == 8) {
if (transfer->bits_per_word == 8) { dws->n_bytes = 1;
dws->n_bytes = 1; dws->dma_width = 1;
dws->dma_width = 1; } else if (transfer->bits_per_word == 16) {
} else if (transfer->bits_per_word == 16) { dws->n_bytes = 2;
dws->n_bytes = 2; dws->dma_width = 2;
dws->dma_width = 2;
}
cr0 = (transfer->bits_per_word - 1)
| (chip->type << SPI_FRF_OFFSET)
| (spi->mode << SPI_MODE_OFFSET)
| (chip->tmode << SPI_TMOD_OFFSET);
} }
cr0 = (transfer->bits_per_word - 1)
| (chip->type << SPI_FRF_OFFSET)
| (spi->mode << SPI_MODE_OFFSET)
| (chip->tmode << SPI_TMOD_OFFSET);
/* /*
* Adjust transfer mode if necessary. Requires platform dependent * Adjust transfer mode if necessary. Requires platform dependent