Commit 9f918a72 authored by Lukasz Majewski's avatar Lukasz Majewski Committed by Mark Brown

spi: Add call to spi_slave_abort() function when spidev driver is released

This change is necessary for spidev devices (e.g. /dev/spidev3.0) working
in the slave mode (like NXP's dspi driver for Vybrid SoC).

When SPI HW works in this mode - the master is responsible for providing
CS and CLK signals. However, when some fault happens - like for example
distortion on SPI lines - the SPI Linux driver needs a chance to recover
from this abnormal situation and prepare itself for next (correct)
transmission.

This change doesn't pose any threat on drivers working in master mode as
spi_slave_abort() function checks if SPI slave mode is supported.
Signed-off-by: default avatarLukasz Majewski <lukma@denx.de>
Link: https://lore.kernel.org/r/20190924110547.14770-2-lukma@denx.deSigned-off-by: default avatarMark Brown <broonie@kernel.org>
Reported-by: default avatarkbuild test robot <lkp@intel.com>
Link: https://lore.kernel.org/r/20190925091143.15468-2-lukma@denx.deSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 6fe7ab38
...@@ -627,6 +627,9 @@ static int spidev_release(struct inode *inode, struct file *filp) ...@@ -627,6 +627,9 @@ static int spidev_release(struct inode *inode, struct file *filp)
if (dofree) if (dofree)
kfree(spidev); kfree(spidev);
} }
#ifdef CONFIG_SPI_SLAVE
spi_slave_abort(spidev->spi);
#endif
mutex_unlock(&device_list_lock); mutex_unlock(&device_list_lock);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment