On Thu, Nov 5, 2015 at 5:16 PM, Andy Shevchenko <[email protected]> wrote: > On Thu, Nov 5, 2015 at 7:50 AM, Nicolas Boichat <[email protected]> wrote: [...] >> diff --git a/drivers/spi/spi-mt65xx.c b/drivers/spi/spi-mt65xx.c >> index 563954a..f694031 100644 >> --- a/drivers/spi/spi-mt65xx.c >> +++ b/drivers/spi/spi-mt65xx.c >> @@ -410,7 +410,7 @@ static int mtk_spi_setup(struct spi_device *spi) >> if (!spi->controller_data) >> spi->controller_data = (void *)&mtk_default_chip_info; >> >> - if (mdata->dev_comp->need_pad_sel) >> + if (mdata->dev_comp->need_pad_sel && spi->cs_gpio >= 0) > > gpio_is_valid() ?
Yes, thanks! Will fix in next round. (there is a few other ones in driver/spi/* that should probably be fixed too...) Best, -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
