From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 36A0ACD1292 for ; Thu, 4 Apr 2024 22:44:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=GizfxkMvPV+7FHDktYU+8ORzblfvwKKV809BR28dmL8=; b=PLfdPtK34CLwzy /i9LPyM5HPjmgVBzC6dr2ZA4ImAgz7tgz84KLLQ2g0GbH/Knz4POZquQacM0WbhunDinSJXrdBu0x RMIn3XUGKi537JhF5nvU8SwDKbX3RTUZhOomfe098bWpiZtDNpPHTZbu+oedeq5gOc7xvKfwr9vo0 78H3h6ElLMevgZSTFcspJHmgWBZ9Ej5ca3kxxrGgk5z0THkHQhUaiRH0npfsX1ig0oed/D2WZ77kz dgeIklbvGYjqMF4p2WxqGdFVw8LhrHaHizXN6MyelqjITHs9u816+dkMA2qSUk2/3UfC9TQXVU5fj 37TU0asLxHk1I1e8aCLQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rsVoe-00000004Tjs-3lOY; Thu, 04 Apr 2024 22:44:16 +0000 Received: from mail-108-mta141.mxroute.com ([136.175.108.141]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rsVoc-00000004Tid-03B9 for linux-arm-kernel@lists.infradead.org; Thu, 04 Apr 2024 22:44:15 +0000 Received: from filter006.mxroute.com ([136.175.111.2] filter006.mxroute.com) (Authenticated sender: mN4UYu2MZsgR) by mail-108-mta141.mxroute.com (ZoneMTA) with ESMTPSA id 18eab48ac4f0003bea.009 for (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384); Thu, 04 Apr 2024 22:44:10 +0000 X-Zone-Loop: 946d9b1b9ded75e548010edf7dea64c19e7d618431ed X-Originating-IP: [136.175.111.2] DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=luigi311.com; s=x; h=Content-Transfer-Encoding:Content-Type:In-Reply-To: From:References:Cc:To:Subject:MIME-Version:Date:Message-ID:Sender:Reply-To: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=pKWa1rLBYgiYR8KNxTp6qkp4Py0+RJpW5btHSLGQ1SE=; b=P4xoFD9+PO2Kx4t9TlLMeOLHxU IUbGD7YtGzwoG7vXLHevCPuReRiRKx8Xfn27PklTzl1Xh20ncdtPPL6ulqRbEtXrEDq0e37fEnBHf t7KsBij2b2dpCr5Nd9+zkehEMr9QRu3M0WUnpTQ5uVxOM7dVfemQM2BsnZU2eTfONsTOPRejpdMKE Ja1FwyQnR6jmCFXJcQwNSe27eV3oVbF7s1yllaOkYOVTW3cZ0QYnPfx4rexysg969BKsEnZtL8RlZ 0d5+ga5g4E7RlK0l7yHJUqtIvWCaHf8tm/CQ1wPK5iew1gsu51PEXwMDPJz7VTEHs+EHlHtXTxWDD jqf2VwGA==; Message-ID: <998efafa-699b-4226-91d4-2ebba85d63ec@luigi311.com> Date: Thu, 4 Apr 2024 16:44:05 -0600 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 19/25] media: i2c: imx258: Change register settings for variants of the sensor To: Sakari Ailus Cc: linux-media@vger.kernel.org, dave.stevenson@raspberrypi.com, jacopo.mondi@ideasonboard.com, mchehab@kernel.org, robh@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, pavel@ucw.cz, phone-devel@vger.kernel.org References: <20240403150355.189229-1-git@luigi311.com> <20240403150355.189229-20-git@luigi311.com> Content-Language: en-US From: Luigi311 In-Reply-To: X-Authenticated-Id: git@luigi311.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240404_154414_211461_A2DBDD46 X-CRM114-Status: GOOD ( 22.30 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 4/3/24 10:18, Sakari Ailus wrote: > Hi Luis, Dave, > > On Wed, Apr 03, 2024 at 09:03:48AM -0600, git@luigi311.com wrote: >> From: Dave Stevenson >> >> Sony have advised that there are variants of the IMX258 sensor which >> require slightly different register configuration to the mainline >> imx258 driver defaults. >> >> There is no available run-time detection for the variant, so add >> configuration via the DT compatible string. >> >> The Vision Components imx258 module supports PDAF, so add the >> register differences for that variant >> >> Signed-off-by: Dave Stevenson >> Signed-off-by: Luis Garcia >> --- >> drivers/media/i2c/imx258.c | 48 ++++++++++++++++++++++++++++++++++---- >> 1 file changed, 44 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/media/i2c/imx258.c b/drivers/media/i2c/imx258.c >> index 775d957c9b87..fa48da212037 100644 >> --- a/drivers/media/i2c/imx258.c >> +++ b/drivers/media/i2c/imx258.c >> @@ -6,6 +6,7 @@ >> #include >> #include >> #include >> +#include >> #include >> #include >> #include >> @@ -321,8 +322,6 @@ static const struct imx258_reg mipi_642mbps_24mhz_4l[] = { >> >> static const struct imx258_reg mode_common_regs[] = { >> { 0x3051, 0x00 }, >> - { 0x3052, 0x00 }, >> - { 0x4E21, 0x14 }, >> { 0x6B11, 0xCF }, >> { 0x7FF0, 0x08 }, >> { 0x7FF1, 0x0F }, >> @@ -345,7 +344,6 @@ static const struct imx258_reg mode_common_regs[] = { >> { 0x7FA8, 0x03 }, >> { 0x7FA9, 0xFE }, >> { 0x7B24, 0x81 }, >> - { 0x7B25, 0x00 }, >> { 0x6564, 0x07 }, >> { 0x6B0D, 0x41 }, >> { 0x653D, 0x04 }, >> @@ -460,6 +458,33 @@ static const struct imx258_reg mode_1048_780_regs[] = { >> { 0x034F, 0x0C }, >> }; >> >> +struct imx258_variant_cfg { >> + const struct imx258_reg *regs; >> + unsigned int num_regs; >> +}; >> + >> +static const struct imx258_reg imx258_cfg_regs[] = { >> + { 0x3052, 0x00 }, >> + { 0x4E21, 0x14 }, >> + { 0x7B25, 0x00 }, >> +}; >> + >> +static const struct imx258_variant_cfg imx258_cfg = { >> + .regs = imx258_cfg_regs, >> + .num_regs = ARRAY_SIZE(imx258_cfg_regs), >> +}; >> + >> +static const struct imx258_reg imx258_pdaf_cfg_regs[] = { >> + { 0x3052, 0x01 }, >> + { 0x4E21, 0x10 }, >> + { 0x7B25, 0x01 }, >> +}; >> + >> +static const struct imx258_variant_cfg imx258_pdaf_cfg = { >> + .regs = imx258_pdaf_cfg_regs, >> + .num_regs = ARRAY_SIZE(imx258_pdaf_cfg_regs), >> +}; >> + >> static const char * const imx258_test_pattern_menu[] = { >> "Disabled", >> "Solid Colour", >> @@ -637,6 +662,8 @@ struct imx258 { >> struct v4l2_subdev sd; >> struct media_pad pad; >> >> + const struct imx258_variant_cfg *variant_cfg; >> + >> struct v4l2_ctrl_handler ctrl_handler; >> /* V4L2 Controls */ >> struct v4l2_ctrl *link_freq; >> @@ -1104,6 +1131,14 @@ static int imx258_start_streaming(struct imx258 *imx258) >> return ret; >> } >> >> + ret = imx258_write_regs(imx258, imx258->variant_cfg->regs, >> + imx258->variant_cfg->num_regs); >> + if (ret) { >> + dev_err(&client->dev, "%s failed to set variant config\n", >> + __func__); >> + return ret; >> + } >> + >> ret = imx258_write_reg(imx258, IMX258_CLK_BLANK_STOP, >> IMX258_REG_VALUE_08BIT, >> imx258->csi2_flags & V4L2_MBUS_CSI2_NONCONTINUOUS_CLOCK ? >> @@ -1492,6 +1527,10 @@ static int imx258_probe(struct i2c_client *client) >> >> imx258->csi2_flags = ep.bus.mipi_csi2.flags; >> >> + imx258->variant_cfg = of_device_get_match_data(&client->dev); > > You'll also need to keep this working for ACPI based systems. I.e. in > practice remove "of_" prefix here and add the non-PDAF variant data to the > relevant ACPI ID list. > Removing of_ is easy enough and looking at all the other commits that make this change in other drivers I dont see anything else being done besides adding in the .data section that is down below for both imx258 and pdaf versions. Is that what you are referencing or is there some other place to add variant data to ACPI ID list? >> + if (!imx258->variant_cfg) >> + imx258->variant_cfg = &imx258_cfg; >> + >> /* Initialize subdev */ >> v4l2_i2c_subdev_init(&imx258->sd, client, &imx258_subdev_ops); >> >> @@ -1579,7 +1618,8 @@ MODULE_DEVICE_TABLE(acpi, imx258_acpi_ids); >> #endif >> >> static const struct of_device_id imx258_dt_ids[] = { >> - { .compatible = "sony,imx258" }, >> + { .compatible = "sony,imx258", .data = &imx258_cfg }, >> + { .compatible = "sony,imx258-pdaf", .data = &imx258_pdaf_cfg }, >> { /* sentinel */ } >> }; >> MODULE_DEVICE_TABLE(of, imx258_dt_ids); > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel