From patchwork Wed Apr 12 21:11:59 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Pavel Machek X-Patchwork-Id: 40785 Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cyPZI-0007A3-Fx; Wed, 12 Apr 2017 21:12:48 +0000 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.84_2/mailfrontend-5) with esmtp id 1cyPZF-0000fu-80; Wed, 12 Apr 2017 23:12:48 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754914AbdDLVME (ORCPT + 1 other); Wed, 12 Apr 2017 17:12:04 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:52568 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752851AbdDLVMD (ORCPT ); Wed, 12 Apr 2017 17:12:03 -0400 Received: by atrey.karlin.mff.cuni.cz (Postfix, from userid 512) id 3B73B81D3E; Wed, 12 Apr 2017 23:12:00 +0200 (CEST) Date: Wed, 12 Apr 2017 23:11:59 +0200 From: Pavel Machek To: pali.rohar@gmail.com, sre@kernel.org, kernel list , linux-arm-kernel , linux-omap@vger.kernel.org, tony@atomide.com, khilman@kernel.org, aaro.koskinen@iki.fi, ivo.g.dimitrov.75@gmail.com, patrikbachan@gmail.com, serge@hallyn.com, abcloriens@gmail.com, sakari.ailus@iki.fi, mchehab@kernel.org, linux-media@vger.kernel.org Subject: et8ek8 camera on Nokia N900: trying to understand what is going on with modes Message-ID: <20170412211159.GA2313@amd> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2017.4.12.210316 X-PMX-Spam: Gauge=IIIIIIIII, Probability=9%, Report=' MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, LINES_OF_YELLING_3 0.05, MSGID_ADDED_BY_MTA 0.05, BODY_SIZE_10000_PLUS 0, INVALID_MSGID_NO_FQDN 0, LEGITIMATE_SIGNS 0, MULTIPLE_REAL_RCPTS 0, URI_WITH_PATH_ONLY 0, __ANY_URI 0, __ATTACHMENT_SIZE_0_10K 0, __ATTACHMENT_SIZE_10_25K 0, __CD 0, __CP_MEDIA_BODY 0, __CP_URI_IN_BODY 0, __CT 0, __CTYPE_HAS_BOUNDARY 0, __CTYPE_MULTIPART 0, __FRAUD_MONEY_BIG_COIN 0, __FRAUD_MONEY_BIG_COIN_DIG 0, __HAS_ATTACHMENT 0, __HAS_ATTACHMENT1 0, __HAS_ATTACHMENT2 0, __HAS_FROM 0, __HAS_LIST_ID 0, __HAS_MSGID 0, __HAS_X_MAILING_LIST 0, __HTTPS_URI 0, __LEGIT_LIST_HEADER 0, __LINES_OF_YELLING 0, __MIME_TEXT_P 0, __MIME_TEXT_P1 0, __MIME_TEXT_P2 0, __MIME_VERSION 0, __MULTIPLE_RCPTS_TO_X5 0, __MULTIPLE_URI_TEXT 0, __NO_HTML_TAG_RAW 0, __SANE_MSGID 0, __SUBJ_ALPHA_END 0, __SUBJ_ALPHA_START 0, __SUBJ_ALPHA_START_END 0, __TO_MALFORMED_2 0, __TO_NAME 0, __TO_NAME_DIFF_FROM_ACC 0, __TO_NO_NAME 0, __TO_REAL_NAMES 0, __URI_IN_BODY 0, __URI_NOT_IMG 0, __URI_NO_MAILTO 0, __URI_NO_WWW 0, __URI_NS , __URI_WITH_PATH 0, __USER_AGENT 0' Hi! 5Mpix mode does not work on N900, which is something I'd like to understand. et8ek8_mode contains huge tables of register settings and parameter values, but it seems that they are not really independend. To test that theory, I started with checking values against each other. This is the work so far, it is neither complete nor completely working at the moment. Perhaps someone wants to play... Pavel diff --git a/drivers/media/i2c/et8ek8/et8ek8_driver.c b/drivers/media/i2c/et8ek8/et8ek8_driver.c index 6296f6f..ca2f648 100644 --- a/drivers/media/i2c/et8ek8/et8ek8_driver.c +++ b/drivers/media/i2c/et8ek8/et8ek8_driver.c @@ -798,6 +798,8 @@ static void et8ek8_update_controls(struct et8ek8_sensor *sensor) u32 min, max, pixel_rate; static const int S = 8; + printk("Updating controls for %d x %d @ %d mode -- %s\n", mode->width, mode->height, mode->pixel_clock, mode->name); + ctrl = sensor->exposure; #ifdef COMPATIBLE @@ -820,6 +822,127 @@ static void et8ek8_update_controls(struct et8ek8_sensor *sensor) __v4l2_ctrl_s_ctrl_int64(sensor->pixel_rate, pixel_rate << S); } +static int read_8(struct i2c_client *client, unsigned long addr) +{ + int val; + et8ek8_i2c_read_reg(client, ET8EK8_REG_8BIT, addr, &val); + return val; +} + +static int read_16(struct i2c_client *client, unsigned long addr) +{ + return read_8(client, addr); +} + +static void assert_value(struct i2c_client *client, unsigned long addr, unsigned long val) +{ + int val2 = read_8(client, addr); + if (val != val2) + printk("et8ek8: assertion check %lx / should be %lx is %lx\n", addr, val, val2); +} + +static void assert(struct i2c_client *client, int v, char *msg) +{ + if (!v) + printk("et8ek8: assertion: %s\n", msg); +} + +static void assert_eq(struct i2c_client *client, int v1, int v2, char *msg) +{ + if (v1 != v2) + printk("et8ek8: assertion: %d == %d %s\n", v1, v2, msg); +} + +static void et8ek8_check(struct et8ek8_sensor *sensor) +{ + /* + 1239 4F # CKVAR_DIV + 1238 02 # CKVAR_DIV[8] CKREF_DIV + 123B 70 # MRCK_DIV LVDSCK_DIV + 123A 05 # VCO_DIV SPCK_DIV + 121B 63 # PIC_SIZE MONI_MODE + 1220 85 # H_COUNT + 1221 00 # H_COUNT[10:8] + 1222 58 # V_COUNT + 1223 00 # V_COUNT[12:8] + 121D 63 # H_SIZE H_INTERMIT + 125D 83 # CCP_LVDS_MODE/ _/ _/ _/ _/ CCP_COMP_MODE[2-0] + */ + struct et8ek8_reglist *r = sensor->current_reglist; + struct v4l2_subdev *subdev = &sensor->subdev; + struct i2c_client *client = v4l2_get_subdevdata(subdev); + int vco; + + printk("Mode validation:\n"); + + assert_value(client, 0x1220, (r->mode.width / 24) & 0xff); + assert_value(client, 0x1221, (r->mode.width / 24) >> 8); + + assert_value(client, 0x1222, (r->mode.height / 24) & 0xff); + assert_value(client, 0x1223, (r->mode.height / 24) >> 8); + + { + int ckref_div = read_16(client, 0x1238) & 0xf; + int ckvar_div = ((read_16(client, 0x1238) & 0x80) >> 7) | (read_16(client, 0x1239) << 1); + int vco_div = read_16(client, 0x123A) >> 4; + int spck_div = read_16(client, 0x123A) & 0xf; + int mrck_div = read_16(client, 0x123B) >> 4; + int lvdsck_div = read_16(client, 0x123B) & 0xf; + + vco = (r->mode.ext_clock * ckvar_div) / (ckref_div + 1); + printk("Vco is %d, %d %d %d\n", vco, r->mode.ext_clock, ckvar_div, ckref_div); + int ccp2 = vco / ((lvdsck_div + 1) * (vco_div + 1)); + int spck = vco / ((spck_div + 1) * (vco_div + 1)); + + assert_eq(client, r->mode.pixel_clock, spck, "spck"); + } + + assert_eq(client, r->mode.max_exp, r->mode.height - 4, "max_exp"); + + assert(client, !(r->mode.sensor_window_width % r->mode.window_width), "window_width"); + switch(r->mode.sensor_window_width / r->mode.window_width) { + case 1: assert_value(client, 0x121d, 0x64); + break; + case 2: assert_value(client, 0x121d, 0x63); + break; + case 3: assert_value(client, 0x121d, 0x62); + break; + default: + assert(client, 0, "bad window_width"); + } + + assert(client, !(r->mode.sensor_window_height % r->mode.window_height), "window_width"); + switch(r->mode.sensor_window_height / r->mode.window_height) { + case 1: assert_value(client, 0x121b, 0x64); + break; + case 2: assert_value(client, 0x121b, 0x63); + break; + case 3: assert_value(client, 0x121b, 0x62); + break; + default: + assert(client, 0, "bad window_height"); + } + + //assert(r->mode.height * r->mode.width * fps == r->mode.pixel_clock); + + switch (r->mode.bus_format) { + case MEDIA_BUS_FMT_SGRBG10_1X10: + assert_value(client, 0x125D, 0x88); + assert_eq(client, vco, r->mode.pixel_clock * 8, "vco_clock"); + break; + case MEDIA_BUS_FMT_SGRBG10_DPCM8_1X8: + assert_value(client, 0x125D, 0x83); + assert_eq(client, vco, r->mode.pixel_clock * 6, "vco_clock"); + break; + default: + assert(client, 0, "unexpected bus format"); + + /* There are more possibilities, see + https://github.com/maemo-foss/omap3camera-firmware/blob/master/makemodes-et8ek8.pl + */ + } +} + static int et8ek8_configure(struct et8ek8_sensor *sensor) { struct v4l2_subdev *subdev = &sensor->subdev; @@ -872,6 +995,8 @@ static int et8ek8_s_stream(struct v4l2_subdev *subdev, int streaming) if (ret < 0) return ret; + et8ek8_check(sensor); + return et8ek8_stream_on(sensor); } diff --git a/drivers/media/i2c/et8ek8/et8ek8_mode.c b/drivers/media/i2c/et8ek8/et8ek8_mode.c index a79882a..045d361 100644 --- a/drivers/media/i2c/et8ek8/et8ek8_mode.c +++ b/drivers/media/i2c/et8ek8/et8ek8_mode.c @@ -22,6 +22,11 @@ * Stingray sensor mode settings for Scooby */ +/* https://github.com/maemo-foss/omap3camera-firmware/blob/master/makemodes-et8ek8.pl + + /data/l/maemo/kernel-power/kernel-power-2.6.28/drivers/media/video/et8ek8-modes.h +*/ + /* Mode1_poweron_Mode2_16VGA_2592x1968_12.07fps */ static struct et8ek8_reglist mode1_poweron_mode2_16vga_2592x1968_12_07fps = { /* (without the +1) @@ -39,18 +44,20 @@ static struct et8ek8_reglist mode1_poweron_mode2_16vga_2592x1968_12_07fps = { */ .type = ET8EK8_REGLIST_POWERON, .mode = { - .sensor_width = 2592, - .sensor_height = 1968, + .name = "mode1_poweron_mode2_16vga_2592x1968_12_07fps", + + .sensor_width = 259, + .sensor_height = 196, .sensor_window_origin_x = 0, .sensor_window_origin_y = 0, - .sensor_window_width = 2592, - .sensor_window_height = 1968, - .width = 3288, - .height = 2016, + .sensor_window_width = 259, + .sensor_window_height = 196, + .width = 328, + .height = 201, .window_origin_x = 0, .window_origin_y = 0, - .window_width = 2592, - .window_height = 1968, + .window_width = 259, + .window_height = 196, .pixel_clock = 80000000, .ext_clock = 9600000, .timeperframe = { @@ -108,6 +115,65 @@ static struct et8ek8_reglist mode1_poweron_mode2_16vga_2592x1968_12_07fps = { { ET8EK8_REG_8BIT, 0x1648, 0x00 }, { ET8EK8_REG_8BIT, 0x113E, 0x01 }, { ET8EK8_REG_8BIT, 0x113F, 0x22 }, + /* Settings from here on seem to for the 2592x1968 mode. */ + { ET8EK8_REG_8BIT, 0x1239, 0x64 }, + { ET8EK8_REG_8BIT, 0x1238, 0x02 }, + { ET8EK8_REG_8BIT, 0x123B, 0x70 }, + { ET8EK8_REG_8BIT, 0x123A, 0x07 }, + { ET8EK8_REG_8BIT, 0x121B, 0x64 }, + { ET8EK8_REG_8BIT, 0x121D, 0x64 }, + { ET8EK8_REG_8BIT, 0x1221, 0x00 }, + { ET8EK8_REG_8BIT, 0x1220, 0x89 }, + { ET8EK8_REG_8BIT, 0x1223, 0x00 }, + { ET8EK8_REG_8BIT, 0x1222, 0x54 }, + { ET8EK8_REG_8BIT, 0x125D, 0x88 }, /* CCP_LVDS_MODE/ */ + { ET8EK8_REG_TERM, 0, 0} + } +}; + +static struct et8ek8_reglist mode2_16vga_2592x1968_12_07fps = { +/* (without the +1) + * SPCK = 80 MHz + * CCP2 = 640 MHz + * VCO = 640 MHz + * VCOUNT = 84 (2016) + * HCOUNT = 137 (3288) + * CKREF_DIV = 2 + * CKVAR_DIV = 200 + * VCO_DIV = 0 + * SPCK_DIV = 7 + * MRCK_DIV = 7 + * LVDSCK_DIV = 0 + */ + .type = ET8EK8_REGLIST_MODE, + .mode = { + .name = "mode2_16vga_2592x1968_12_07fps", + + .sensor_width = 2592, + .sensor_height = 1968, + .sensor_window_origin_x = 0, + .sensor_window_origin_y = 0, + .sensor_window_width = 2592, + .sensor_window_height = 1968, + .width = 3288, + .height = 2016, + .window_origin_x = 0, + .window_origin_y = 0, + .window_width = 2592, + .window_height = 1968, + .pixel_clock = 80000000, + .ext_clock = 9600000, + .timeperframe = { + .numerator = 100, + .denominator = 1207 + }, + .max_exp = 2012, + /* .max_gain = 0, */ + .bus_format = MEDIA_BUS_FMT_SGRBG10_1X10, + .sensitivity = 65536 + }, + .regs = { + /* Settings from here on seem to for the 2592x1968 mode. */ { ET8EK8_REG_8BIT, 0x1239, 0x64 }, { ET8EK8_REG_8BIT, 0x1238, 0x02 }, { ET8EK8_REG_8BIT, 0x123B, 0x70 }, @@ -123,6 +189,7 @@ static struct et8ek8_reglist mode1_poweron_mode2_16vga_2592x1968_12_07fps = { } }; + /* Mode1_16VGA_2592x1968_13.12fps_DPCM10-8 */ static struct et8ek8_reglist mode1_16vga_2592x1968_13_12fps_dpcm10_8 = { /* (without the +1) @@ -140,6 +207,7 @@ static struct et8ek8_reglist mode1_16vga_2592x1968_13_12fps_dpcm10_8 = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode1_16vga_2592x1968_13_12fps_dpcm10_8", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -196,6 +264,7 @@ static struct et8ek8_reglist mode3_4vga_1296x984_29_99fps_dpcm10_8 = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode3_4vga_1296x984_29_99fps_dpcm10_8", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -252,6 +321,7 @@ static struct et8ek8_reglist mode4_svga_864x656_29_88fps = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode4_svga_864x656_29_88fps", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -308,6 +378,7 @@ static struct et8ek8_reglist mode5_vga_648x492_29_93fps = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode5_vga_648x492_29_93fps", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -364,6 +435,7 @@ static struct et8ek8_reglist mode2_16vga_2592x1968_3_99fps = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode2_16vga_2592x1968_3_99fps", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -398,6 +470,7 @@ static struct et8ek8_reglist mode2_16vga_2592x1968_3_99fps = { { ET8EK8_REG_8BIT, 0x1220, 0x89 }, { ET8EK8_REG_8BIT, 0x1223, 0x00 }, { ET8EK8_REG_8BIT, 0x1222, 0xFE }, + { ET8EK8_REG_8BIT, 0x125D, 0x88 }, /* CCP_LVDS_MODE/ */ { ET8EK8_REG_TERM, 0, 0} } }; @@ -419,6 +492,7 @@ static struct et8ek8_reglist mode_648x492_5fps = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode_648x492_5fps", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -475,6 +549,7 @@ static struct et8ek8_reglist mode3_4vga_1296x984_5fps = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode3_4vga_1296x984_5fps", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -531,6 +606,7 @@ static struct et8ek8_reglist mode_4vga_1296x984_25fps_dpcm10_8 = { */ .type = ET8EK8_REGLIST_MODE, .mode = { + .name = "mode_4vga_1296x984_25fps_dpcm10_8", .sensor_width = 2592, .sensor_height = 1968, .sensor_window_origin_x = 0, @@ -573,15 +649,23 @@ static struct et8ek8_reglist mode_4vga_1296x984_25fps_dpcm10_8 = { struct et8ek8_meta_reglist meta_reglist = { .version = "V14 03-June-2008", .reglist = { + /* power on mode; strange & special */ { .ptr = &mode1_poweron_mode2_16vga_2592x1968_12_07fps }, - { .ptr = &mode1_16vga_2592x1968_13_12fps_dpcm10_8 }, - { .ptr = &mode3_4vga_1296x984_29_99fps_dpcm10_8 }, - { .ptr = &mode4_svga_864x656_29_88fps }, - { .ptr = &mode5_vga_648x492_29_93fps }, - { .ptr = &mode2_16vga_2592x1968_3_99fps }, - { .ptr = &mode_648x492_5fps }, - { .ptr = &mode3_4vga_1296x984_5fps }, + /* dpcm10/8 modes */ +#if 0 + { .ptr = &mode1_16vga_2592x1968_13_12fps_dpcm10_8 }, /* No luck */ + { .ptr = &mode3_4vga_1296x984_29_99fps_dpcm10_8 }, /* No luck */ { .ptr = &mode_4vga_1296x984_25fps_dpcm10_8 }, +#endif + /* "normal" modes */ +#if 1 + { .ptr = &mode2_16vga_2592x1968_12_07fps }, /* My hacks. */ + { .ptr = &mode4_svga_864x656_29_88fps }, /* Works, AFAICT */ + { .ptr = &mode5_vga_648x492_29_93fps }, /* Does not seem to work ? */ +// { .ptr = &mode2_16vga_2592x1968_3_99fps }, /* Does not seem to work: scrolling */ + { .ptr = &mode_648x492_5fps }, /* Does not seem to work ? */ + { .ptr = &mode3_4vga_1296x984_5fps }, /* Works, AFAICT */ +#endif { .ptr = NULL } } }; diff --git a/drivers/media/i2c/et8ek8/et8ek8_reg.h b/drivers/media/i2c/et8ek8/et8ek8_reg.h index 07f1873..60ab305 100644 --- a/drivers/media/i2c/et8ek8/et8ek8_reg.h +++ b/drivers/media/i2c/et8ek8/et8ek8_reg.h @@ -37,19 +37,21 @@ struct et8ek8_mode { u16 sensor_window_height; /* Image data coming from sensor (after scaling) */ - u16 width; + u16 width; /* u */ u16 height; u16 window_origin_x; u16 window_origin_y; - u16 window_width; - u16 window_height; + u16 window_width; /* u */ + u16 window_height; /* u */ - u32 pixel_clock; /* in Hz */ - u32 ext_clock; /* in Hz */ - struct v4l2_fract timeperframe; - u32 max_exp; /* Maximum exposure value */ - u32 bus_format; /* MEDIA_BUS_FMT_ */ + u32 pixel_clock; /* u in Hz */ + u32 ext_clock; /* u in Hz */ + struct v4l2_fract timeperframe; /* u */ + u32 max_exp; /* u Maximum exposure value */ + u32 bus_format; /* u MEDIA_BUS_FMT_ */ u32 sensitivity; /* 16.16 fixed point */ + + char *name; }; #define ET8EK8_REG_8BIT 1