On Sun, 2015-03-08 at 20:01 +0000, Mark Brown wrote: > On Fri, Mar 06, 2015 at 05:45:15PM +0200, Ivan T. Ivanov wrote: > > > if (spi->master->setup) > > status = spi->master->setup(spi); > > + else > > + spi_set_cs(spi, false); > > Why would this be conditional on there not being a setup operation? > It's a good idea, I just don't see why the backstop isn't always useful. I was thinking that this is expected to be handled inside setup operation, nothing more. Ivan -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html