diff options
| author |   Matthias Schiffer <matthias.schiffer@ew.tq-group.com> | 2022-04-20 17:56:15 +0200 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2022-04-25 14:01:04 +0100 |
| commit | 28ac902aedd18abf4faf8816b1bea6623d0e9509 (patch) | |
| tree | 1a5df13714d3cdd76389d763413043251125a133 /drivers/spi/spi-omap2-mcspi.c | |
| parent | spi: spi-tegra20-sflash: using pm_runtime_resume_and_get instead of pm_runtime_get_sync (diff) | |
| download | linux-dev-28ac902aedd18abf4faf8816b1bea6623d0e9509.tar.xz linux-dev-28ac902aedd18abf4faf8816b1bea6623d0e9509.zip | |
spi: cadence-quadspi: drop cqspi_set_protocol()
As suggested, this removes the whole cqspi_set_protocol() function, as it
is not actually needed:
- Checks for unsupported operations are already handled by supports_op(),
removing the need to distinguish DTR and non-DTR modes in the buswidth
setup
- supports_op() ensures that the DTR flags match for all relevant parts of
an operation, so op->cmd.dtr can be used instead of copying the flag to
the cqspi_flash_pdata
- The logic in cqspi_set_protocol() is moved to cqspi_calc_rdreg() and
cqspi_write_setup() (with a helper macro CQSPI_OP_WIDTH())
The helper macro checks nbytes instead of buswidth for 0, for consistency
with supports_op() etc.
Suggested-by: Pratyush Yadav <p.yadav@ti.com>
Signed-off-by: Matthias Schiffer <matthias.schiffer@ew.tq-group.com>
Link: https://lore.kernel.org/r/20220420155616.281730-1-matthias.schiffer@ew.tq-group.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi-omap2-mcspi.c')
0 files changed, 0 insertions, 0 deletions