spi: sh-msiof: Avoid writing to registers from spi_master.setup()
authorGeert Uytterhoeven <geert+renesas@glider.be>
Wed, 13 Dec 2017 19:05:10 +0000 (20:05 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 14 Dec 2017 11:46:02 +0000 (11:46 +0000)
commit7ff0b53c4051145d1cf992d2f60987e6447eed4f
tree810a911241204673b058aa8832bf44c6c0708c94
parent5d8e614f6cf8850657edbd1859391a2ae45b4488
spi: sh-msiof: Avoid writing to registers from spi_setup()

The spi_master.setup() callback must not change configuration registers,
as that could corrupt I/O that is in progress for other SPI slaves.

The only exception is the configuration of the native chip select
polarity in SPI master mode, as a wrong chip select polarity will cause
havoc during all future transfers to any other SPI slave.

Hence stop writing to registers in sh_msiof_spi_setup(), unless it is
the first call for a controller using a native chip select, or unless
native chip select polarity has changed (note that you'll loose anyway
if I/O is in progress).  Even then, only do what is strictly necessary,
instead of calling sh_msiof_spi_set_pin_regs().

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-sh-msiof.c