diff options
author | Martin Kaiser <martin@kaiser.cx> | 2016-09-01 16:38:40 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-09-14 13:06:11 -0400 |
commit | 2636ba8fa39915c7b8d73166961ebbb4c14251cd (patch) | |
tree | 5c6e9651bc9131ca8ce972f59808509940ff1704 | |
parent | 29b4817d4018df78086157ea3a55c1d9424a7cfc (diff) |
spi: imx: set spi_bus_clk for mx1, mx31 and mx35
Modify spi_imx_clkdiv_2() to return the resulting bus clock frequency
when the selected clock divider is applied. Set spi_imx->spi_bus_clk to
this frequency.
If spi_bus_clk is unset, spi_imx_calculate_timeout() causes a
division by 0.
Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | drivers/spi/spi-imx.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c index f63cb30f9010..5cc72be30744 100644 --- a/drivers/spi/spi-imx.c +++ b/drivers/spi/spi-imx.c | |||
@@ -186,17 +186,19 @@ static unsigned int spi_imx_clkdiv_1(unsigned int fin, | |||
186 | 186 | ||
187 | /* MX1, MX31, MX35, MX51 CSPI */ | 187 | /* MX1, MX31, MX35, MX51 CSPI */ |
188 | static unsigned int spi_imx_clkdiv_2(unsigned int fin, | 188 | static unsigned int spi_imx_clkdiv_2(unsigned int fin, |
189 | unsigned int fspi) | 189 | unsigned int fspi, unsigned int *fres) |
190 | { | 190 | { |
191 | int i, div = 4; | 191 | int i, div = 4; |
192 | 192 | ||
193 | for (i = 0; i < 7; i++) { | 193 | for (i = 0; i < 7; i++) { |
194 | if (fspi * div >= fin) | 194 | if (fspi * div >= fin) |
195 | return i; | 195 | goto out; |
196 | div <<= 1; | 196 | div <<= 1; |
197 | } | 197 | } |
198 | 198 | ||
199 | return 7; | 199 | out: |
200 | *fres = fin / div; | ||
201 | return i; | ||
200 | } | 202 | } |
201 | 203 | ||
202 | static int spi_imx_bytes_per_word(const int bpw) | 204 | static int spi_imx_bytes_per_word(const int bpw) |
@@ -482,9 +484,11 @@ static int mx31_config(struct spi_device *spi, struct spi_imx_config *config) | |||
482 | { | 484 | { |
483 | struct spi_imx_data *spi_imx = spi_master_get_devdata(spi->master); | 485 | struct spi_imx_data *spi_imx = spi_master_get_devdata(spi->master); |
484 | unsigned int reg = MX31_CSPICTRL_ENABLE | MX31_CSPICTRL_MASTER; | 486 | unsigned int reg = MX31_CSPICTRL_ENABLE | MX31_CSPICTRL_MASTER; |
487 | unsigned int clk; | ||
485 | 488 | ||
486 | reg |= spi_imx_clkdiv_2(spi_imx->spi_clk, config->speed_hz) << | 489 | reg |= spi_imx_clkdiv_2(spi_imx->spi_clk, config->speed_hz, &clk) << |
487 | MX31_CSPICTRL_DR_SHIFT; | 490 | MX31_CSPICTRL_DR_SHIFT; |
491 | spi_imx->spi_bus_clk = clk; | ||
488 | 492 | ||
489 | if (is_imx35_cspi(spi_imx)) { | 493 | if (is_imx35_cspi(spi_imx)) { |
490 | reg |= (config->bpw - 1) << MX35_CSPICTRL_BL_SHIFT; | 494 | reg |= (config->bpw - 1) << MX35_CSPICTRL_BL_SHIFT; |
@@ -625,9 +629,12 @@ static int mx1_config(struct spi_device *spi, struct spi_imx_config *config) | |||
625 | { | 629 | { |
626 | struct spi_imx_data *spi_imx = spi_master_get_devdata(spi->master); | 630 | struct spi_imx_data *spi_imx = spi_master_get_devdata(spi->master); |
627 | unsigned int reg = MX1_CSPICTRL_ENABLE | MX1_CSPICTRL_MASTER; | 631 | unsigned int reg = MX1_CSPICTRL_ENABLE | MX1_CSPICTRL_MASTER; |
632 | unsigned int clk; | ||
628 | 633 | ||
629 | reg |= spi_imx_clkdiv_2(spi_imx->spi_clk, config->speed_hz) << | 634 | reg |= spi_imx_clkdiv_2(spi_imx->spi_clk, config->speed_hz, &clk) << |
630 | MX1_CSPICTRL_DR_SHIFT; | 635 | MX1_CSPICTRL_DR_SHIFT; |
636 | spi_imx->spi_bus_clk = clk; | ||
637 | |||
631 | reg |= config->bpw - 1; | 638 | reg |= config->bpw - 1; |
632 | 639 | ||
633 | if (spi->mode & SPI_CPHA) | 640 | if (spi->mode & SPI_CPHA) |