Commit 3d1e4228 authored by Daniel Scally's avatar Daniel Scally Committed by Sakari Ailus

media: i2c: Fix pixel array positions in ov8865

The ov8865's datasheet gives the pixel array as 3296x2528, and the
active portion as the centre 3264x2448. This makes for a top offset
of 40 and a left offset of 16, not 32 and 80.

Fixes: acd25e22 ("media: i2c: Add .get_selection() support to ov8865")
Reported-by: default avatarJean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com>
Signed-off-by: default avatarDaniel Scally <djrscally@gmail.com>
Reviewed-by: default avatarKieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Reviewed-by: default avatarPaul Kocialkowski <paul.kocialkowski@bootlin.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
parent 3e4fcec0
...@@ -457,8 +457,8 @@ ...@@ -457,8 +457,8 @@
#define OV8865_NATIVE_WIDTH 3296 #define OV8865_NATIVE_WIDTH 3296
#define OV8865_NATIVE_HEIGHT 2528 #define OV8865_NATIVE_HEIGHT 2528
#define OV8865_ACTIVE_START_TOP 32 #define OV8865_ACTIVE_START_LEFT 16
#define OV8865_ACTIVE_START_LEFT 80 #define OV8865_ACTIVE_START_TOP 40
#define OV8865_ACTIVE_WIDTH 3264 #define OV8865_ACTIVE_WIDTH 3264
#define OV8865_ACTIVE_HEIGHT 2448 #define OV8865_ACTIVE_HEIGHT 2448
......
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