Re: [PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Mark Brown
On Wed, May 13, 2015 at 09:34:41AM -, Michal Suchanek wrote: > Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW is > set. Rename spidev devices to avoid sysfs conflict. To repeat my previous feedback I don't see any way in which this is sane, having two drivers

Re: [PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Michal Suchanek
On 13 May 2015 at 12:16, Maxime Ripard wrote: > Hi, > > On Wed, May 13, 2015 at 09:34:41AM -, Michal Suchanek wrote: >> Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW >> is >> set. Rename spidev devices to avoid sysfs conflict. >> >> This allows dynamically

Re: [PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Maxime Ripard
Hi, On Wed, May 13, 2015 at 09:34:41AM -, Michal Suchanek wrote: > Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW is > set. Rename spidev devices to avoid sysfs conflict. > > This allows dynamically loading SPI device overlays or communicating > with SPI devices

[PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Michal Suchanek
Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW is set. Rename spidev devices to avoid sysfs conflict. This allows dynamically loading SPI device overlays or communicating with SPI devices configured by a kernel driver from userspace. Signed-off-by: Michal Suchanek

Re: [PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Michal Suchanek
On 13 May 2015 at 12:16, Maxime Ripard maxime.rip...@free-electrons.com wrote: Hi, On Wed, May 13, 2015 at 09:34:41AM -, Michal Suchanek wrote: Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW is set. Rename spidev devices to avoid sysfs conflict. This

Re: [PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Maxime Ripard
Hi, On Wed, May 13, 2015 at 09:34:41AM -, Michal Suchanek wrote: Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW is set. Rename spidev devices to avoid sysfs conflict. This allows dynamically loading SPI device overlays or communicating with SPI devices

[PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Michal Suchanek
Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW is set. Rename spidev devices to avoid sysfs conflict. This allows dynamically loading SPI device overlays or communicating with SPI devices configured by a kernel driver from userspace. Signed-off-by: Michal Suchanek

Re: [PATCH] spi: Add option to bind spidev to all chipselects

2015-05-13 Thread Mark Brown
On Wed, May 13, 2015 at 09:34:41AM -, Michal Suchanek wrote: Bypass the check if CS is in use for spidev devices if CONFIG_SPIDEV_SHADOW is set. Rename spidev devices to avoid sysfs conflict. To repeat my previous feedback I don't see any way in which this is sane, having two drivers