diff options
| author | Akinobu Mita | 2018-05-06 16:19:17 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab | 2018-05-28 19:58:34 +0200 |
| commit | a024ee14cd36dba55fd77322f5d48ee0e55418ae (patch) | |
| tree | 553c3322f46feeded4cfca533d63e42fba0725db /drivers/media/common/videobuf2/videobuf2-core.c | |
| parent | media: dt-bindings: ov772x: add device tree binding (diff) | |
| download | kernel-qcow2-linux-a024ee14cd36dba55fd77322f5d48ee0e55418ae.tar.gz kernel-qcow2-linux-a024ee14cd36dba55fd77322f5d48ee0e55418ae.tar.xz kernel-qcow2-linux-a024ee14cd36dba55fd77322f5d48ee0e55418ae.zip | |
media: ov772x: correct setting of banding filter
The banding filter ON/OFF is controlled via bit 5 of COM8 register. It
is attempted to be enabled in ov772x_set_params() by the following line.
ret = ov772x_mask_set(client, COM8, BNDF_ON_OFF, 1);
But this unexpectedly results disabling the banding filter, because the
mask and set bits are exclusive.
On the other hand, ov772x_s_ctrl() correctly sets the bit by:
ret = ov772x_mask_set(client, COM8, BNDF_ON_OFF, BNDF_ON_OFF);
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Diffstat (limited to 'drivers/media/common/videobuf2/videobuf2-core.c')
0 files changed, 0 insertions, 0 deletions