diff options
author | Pratyush Yadav <p.yadav@ti.com> | 2020-12-23 00:14:19 +0530 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2021-01-06 13:08:41 +0000 |
commit | 3a5c09c8d1ed9a7323f0e5c435021531f0865c16 (patch) | |
tree | 80c77198c6104f2ecab237f7762f0e772f8cf356 /drivers/spi/spi-cadence-quadspi.c | |
parent | 74523a5dae0c96d6503fe72da66ee37fd23eb8f5 (diff) | |
download | lwn-3a5c09c8d1ed9a7323f0e5c435021531f0865c16.tar.gz lwn-3a5c09c8d1ed9a7323f0e5c435021531f0865c16.zip |
spi: cadence-quadspi: Set master max_speed_hz
As of commit 9326e4f1e5dd ("spi: Limit the spi device max speed to
controller's max speed"), the SPI device max speed is set to the
controller's max speed if it is larger. The Cadence QSPI controller does
not set the controller's max speed so it is left at its initial value of
0. This means the SPI device max speed is always set to 0.
The SPI device max speed is used to calculate the baud rate divider when
performing an operation. If this speed is 0, the default divider of 32
is used. No matter what speed is specified by the device tree property
'spi-max-frequency', the device will always operate at ref_clk / 32.
Fix this by setting master->max_speed_hz to the ref clock speed so the
correct divider can be calculated.
Signed-off-by: Pratyush Yadav <p.yadav@ti.com>
Link: https://lore.kernel.org/r/20201222184425.7028-2-p.yadav@ti.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi-cadence-quadspi.c')
-rw-r--r-- | drivers/spi/spi-cadence-quadspi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/spi/spi-cadence-quadspi.c b/drivers/spi/spi-cadence-quadspi.c index ba7d40c2922f..ea3890c7d9ff 100644 --- a/drivers/spi/spi-cadence-quadspi.c +++ b/drivers/spi/spi-cadence-quadspi.c @@ -1279,6 +1279,7 @@ static int cqspi_probe(struct platform_device *pdev) reset_control_deassert(rstc_ocp); cqspi->master_ref_clk_hz = clk_get_rate(cqspi->clk); + master->max_speed_hz = cqspi->master_ref_clk_hz; ddata = of_device_get_match_data(dev); if (ddata) { if (ddata->quirks & CQSPI_NEEDS_WR_DELAY) |