diff options
| author | Kamal Dasu <kdasu.kdev@gmail.com> | 2021-11-24 14:33:53 -0500 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2021-11-25 11:54:37 +0000 |
| commit | e10a6bb5f52de70c7798b720d16632d4042d2552 (patch) | |
| tree | 4c3fdc118bddadf398bd1ffbd86f176692ba45cd /drivers/spi/spi-bcm-qspi.c | |
| parent | c74526f947ab946273939757c72499c0a5b09826 (diff) | |
| download | lwn-e10a6bb5f52de70c7798b720d16632d4042d2552.tar.gz lwn-e10a6bb5f52de70c7798b720d16632d4042d2552.zip | |
spi: bcm-qspi: set transfer parameter only if they change
Check if the transfer parameters have changed from previous settings
before applying new parameters.
Signed-off-by: Kamal Dasu <kdasu.kdev@gmail.com>
Link: https://lore.kernel.org/r/20211124193353.32311-3-kdasu.kdev@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi-bcm-qspi.c')
| -rw-r--r-- | drivers/spi/spi-bcm-qspi.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/drivers/spi/spi-bcm-qspi.c b/drivers/spi/spi-bcm-qspi.c index 38e6e2cb62ca..c9a769b8594b 100644 --- a/drivers/spi/spi-bcm-qspi.c +++ b/drivers/spi/spi-bcm-qspi.c @@ -598,12 +598,24 @@ static void bcm_qspi_chip_select(struct bcm_qspi *qspi, int cs) qspi->curr_cs = cs; } +static bool bcmspi_parms_did_change(const struct bcm_qspi_parms * const cur, + const struct bcm_qspi_parms * const prev) +{ + return (cur->speed_hz != prev->speed_hz) || + (cur->mode != prev->mode) || + (cur->bits_per_word != prev->bits_per_word); +} + + /* MSPI helpers */ static void bcm_qspi_hw_set_parms(struct bcm_qspi *qspi, const struct bcm_qspi_parms *xp) { u32 spcr, spbr = 0; + if (!bcmspi_parms_did_change(xp, &qspi->last_parms)) + return; + if (!qspi->mspi_maj_rev) /* legacy controller */ spcr = MSPI_MASTER_BIT; |