Commit 7f6cd6c4 authored by Javier Martinez Canillas's avatar Javier Martinez Canillas Committed by Mauro Carvalho Chehab

[media] adv7604: Check v4l2_of_parse_endpoint() return value

The v4l2_of_parse_endpoint() function can fail so check the return value.
Signed-off-by: default avatarJavier Martinez Canillas <javier@osg.samsung.com>
Acked-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent f7842cfd
...@@ -2820,6 +2820,7 @@ static int adv76xx_parse_dt(struct adv76xx_state *state) ...@@ -2820,6 +2820,7 @@ static int adv76xx_parse_dt(struct adv76xx_state *state)
struct device_node *endpoint; struct device_node *endpoint;
struct device_node *np; struct device_node *np;
unsigned int flags; unsigned int flags;
int ret;
u32 v; u32 v;
np = state->i2c_clients[ADV76XX_PAGE_IO]->dev.of_node; np = state->i2c_clients[ADV76XX_PAGE_IO]->dev.of_node;
...@@ -2829,7 +2830,11 @@ static int adv76xx_parse_dt(struct adv76xx_state *state) ...@@ -2829,7 +2830,11 @@ static int adv76xx_parse_dt(struct adv76xx_state *state)
if (!endpoint) if (!endpoint)
return -EINVAL; return -EINVAL;
v4l2_of_parse_endpoint(endpoint, &bus_cfg); ret = v4l2_of_parse_endpoint(endpoint, &bus_cfg);
if (ret) {
of_node_put(endpoint);
return ret;
}
if (!of_property_read_u32(endpoint, "default-input", &v)) if (!of_property_read_u32(endpoint, "default-input", &v))
state->pdata.default_input = v; state->pdata.default_input = v;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment