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 /drivers/spi | |
| 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>
Diffstat (limited to 'drivers/spi')
| -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) |
