On 09/28/2016 04:50 PM, Wei Yongjun wrote: > From: Wei Yongjun <weiyongjun1@xxxxxxxxxx> > > Fix to return error code -EINVAL if no CS GPIOs available > instead of 0, as done elsewhere in this function. > > Fixes: f13d4e189d20 ("spi: imx: Gracefully handle NULL master->cs_gpios") > Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx> Oops, good catch, thanks. Acked-by: Marek Vasut <marex@xxxxxxx> > --- > drivers/spi/spi-imx.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c > index 1ef5429..deb782f 100644 > --- a/drivers/spi/spi-imx.c > +++ b/drivers/spi/spi-imx.c > @@ -1270,6 +1270,7 @@ static int spi_imx_probe(struct platform_device *pdev) > > if (!master->cs_gpios) { > dev_err(&pdev->dev, "No CS GPIOs available\n"); > + ret = -EINVAL; > goto out_clk_put; > } > -- Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html