media: adv7180: Only print 'chip found' message on successful probe
Currently the "chip found" message is shown even in the case where the I2C address is wrongly passed in the device tree, or also in the case of probe failure, which is misleading. To avoid such problem, move this message after real I2C transactions have been successfully made and we are certain that the adv7180 is really present and probed. Signed-off-by: Fabio Estevam <festevam@gmail.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
This commit is contained in:
parent
2289adbfa5
commit
b19c25f467
|
@ -1309,9 +1309,6 @@ static int adv7180_probe(struct i2c_client *client,
|
||||||
if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA))
|
if (!i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE_DATA))
|
||||||
return -EIO;
|
return -EIO;
|
||||||
|
|
||||||
v4l_info(client, "chip found @ 0x%02x (%s)\n",
|
|
||||||
client->addr, client->adapter->name);
|
|
||||||
|
|
||||||
state = devm_kzalloc(&client->dev, sizeof(*state), GFP_KERNEL);
|
state = devm_kzalloc(&client->dev, sizeof(*state), GFP_KERNEL);
|
||||||
if (state == NULL)
|
if (state == NULL)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
@ -1382,6 +1379,9 @@ static int adv7180_probe(struct i2c_client *client,
|
||||||
if (ret)
|
if (ret)
|
||||||
goto err_free_irq;
|
goto err_free_irq;
|
||||||
|
|
||||||
|
v4l_info(client, "chip found @ 0x%02x (%s)\n",
|
||||||
|
client->addr, client->adapter->name);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
err_free_irq:
|
err_free_irq:
|
||||||
|
|
Loading…
Reference in a new issue