Hi Anton, On Sat, Feb 06, 2016 at 11:27:32PM -0500, Anton Protopopov wrote: > The adp1653_probe() function may return positive value EINVAL > which is obviously wrong. > > Signed-off-by: Anton Protopopov <[email protected]> > --- > drivers/media/i2c/adp1653.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/i2c/adp1653.c b/drivers/media/i2c/adp1653.c > index 7e9cbf7..fb7ed73 100644 > --- a/drivers/media/i2c/adp1653.c > +++ b/drivers/media/i2c/adp1653.c > @@ -497,7 +497,7 @@ static int adp1653_probe(struct i2c_client *client, > if (!client->dev.platform_data) { > dev_err(&client->dev, > "Neither DT not platform data provided\n"); > - return EINVAL; > + return -EINVAL; > } > flash->platform_data = client->dev.platform_data; > }
Thanks! Applied to my tree. -- Kind regards, Sakari Ailus e-mail: [email protected] XMPP: [email protected]
