s5k6a3 sensor has actual pixel resolution of 1408x1402 against
the active resolution 1392x1392. The real resolution is needed
when raw sensor SRGB data is dumped to memory by fimc-lite.

Signed-off-by: Arun Kumar K <arun...@samsung.com>
Reviewed-by: Sylwester Nawrocki <s.nawro...@samsung.com>
---
 drivers/media/i2c/s5k6a3.c |   19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/drivers/media/i2c/s5k6a3.c b/drivers/media/i2c/s5k6a3.c
index ccbb4fc..34c3165 100644
--- a/drivers/media/i2c/s5k6a3.c
+++ b/drivers/media/i2c/s5k6a3.c
@@ -25,10 +25,12 @@
 #include <media/v4l2-async.h>
 #include <media/v4l2-subdev.h>
 
-#define S5K6A3_SENSOR_MAX_WIDTH                1392
-#define S5K6A3_SENSOR_MAX_HEIGHT       1392
-#define S5K6A3_SENSOR_MIN_WIDTH                32
-#define S5K6A3_SENSOR_MIN_HEIGHT       32
+#define S5K6A3_SENSOR_MAX_WIDTH                1408
+#define S5K6A3_SENSOR_MAX_HEIGHT       1402
+#define S5K6A3_SENSOR_ACTIVE_WIDTH     1392
+#define S5K6A3_SENSOR_ACTIVE_HEIGHT    1392
+#define S5K6A3_SENSOR_MIN_WIDTH                (32 + 16)
+#define S5K6A3_SENSOR_MIN_HEIGHT       (32 + 10)
 
 #define S5K6A3_DEF_PIX_WIDTH           1296
 #define S5K6A3_DEF_PIX_HEIGHT          732
@@ -107,10 +109,11 @@ static void s5k6a3_try_format(struct v4l2_mbus_framefmt 
*mf)
 
        fmt = find_sensor_format(mf);
        mf->code = fmt->code;
-       v4l_bound_align_image(&mf->width, S5K6A3_SENSOR_MIN_WIDTH,
-                             S5K6A3_SENSOR_MAX_WIDTH, 0,
-                             &mf->height, S5K6A3_SENSOR_MIN_HEIGHT,
-                             S5K6A3_SENSOR_MAX_HEIGHT, 0, 0);
+       v4l_bound_align_image(&mf->width,
+                       S5K6A3_SENSOR_MIN_WIDTH, S5K6A3_SENSOR_MAX_WIDTH, 0,
+                       &mf->height,
+                       S5K6A3_SENSOR_MIN_HEIGHT, S5K6A3_SENSOR_MAX_HEIGHT, 0,
+                       0);
 }
 
 static struct v4l2_mbus_framefmt *__s5k6a3_get_format(
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to