diff options
| author | Javier Martinez Canillas <javier@osg.samsung.com> | 2016-02-05 17:09:53 -0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2016-02-10 09:38:20 -0200 |
| commit | 2bd5e4375aec8731c9818b2ac86ac035d3aebe86 (patch) | |
| tree | dbbc85534c1f174f694438f2bc540a5278c32776 /drivers/media | |
| parent | 7c3e1ec1f772c1f740fce3180bf419018582977f (diff) | |
| download | op-kernel-dev-2bd5e4375aec8731c9818b2ac86ac035d3aebe86.zip op-kernel-dev-2bd5e4375aec8731c9818b2ac86ac035d3aebe86.tar.gz | |
[media] tvp5150: put endpoint node on error
If the parallel mbus configuration is not correct, the endpoint
device node isn't currently put again in the error path. Fix it.
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media')
| -rw-r--r-- | drivers/media/i2c/tvp5150.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/media/i2c/tvp5150.c b/drivers/media/i2c/tvp5150.c index 19b5273..c7eeb59 100644 --- a/drivers/media/i2c/tvp5150.c +++ b/drivers/media/i2c/tvp5150.c @@ -1268,8 +1268,10 @@ static int tvp5150_parse_dt(struct tvp5150 *decoder, struct device_node *np) if (bus_cfg.bus_type == V4L2_MBUS_PARALLEL && !(flags & V4L2_MBUS_HSYNC_ACTIVE_HIGH && flags & V4L2_MBUS_VSYNC_ACTIVE_HIGH && - flags & V4L2_MBUS_FIELD_EVEN_LOW)) - return -EINVAL; + flags & V4L2_MBUS_FIELD_EVEN_LOW)) { + ret = -EINVAL; + goto err; + } decoder->mbus_type = bus_cfg.bus_type; |
