Received: by 2002:a05:7412:d8a:b0:e2:908c:2ebd with SMTP id b10csp3965011rdg; Wed, 18 Oct 2023 10:43:35 -0700 (PDT) X-Google-Smtp-Source: AGHT+IETedlTWs3/C4m7fgCJdFaKPTQxEFaMPCpkKT7rEVDNla0BJrjqeOQvGz0vP13Ka49jUXxF X-Received: by 2002:a05:6870:7a05:b0:1e9:beae:947d with SMTP id hf5-20020a0568707a0500b001e9beae947dmr71716oab.19.1697651015142; Wed, 18 Oct 2023 10:43:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1697651015; cv=none; d=google.com; s=arc-20160816; b=Rli4+3OksFGZmznIAcinHH6su6dk/NAX9vWmGvbN3DCk5ZkM7TzN9I8HOdrJ1nwCt7 ywZHk7fo3j7GahnHq1dpGE1hsCQnzjORpNCQE/sm6A++mlFi2+EivO5wjGewS7qEtsAO m3wpFPdl/Xda6QHIJGb3r6jV3ZSvilW4b8lr5znBnL8WzKgu37XlzsnUQMkCCEyBM5g4 +IBsUHOjjs2ugCXgXAb9x0GUgEG/pkRuW+GvUJttq/AzXiVB/s9/VbY6x0SicJzaKsWx Z6f/NP//p9cPF9H6hRncEXM4yt985ha+Yuut0w2BO20przrpfm09yOhUn1vEsGPDs3bj US0A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=dKaWOPnAN5si+yKYBlPOMxbvgqGw6FhfRCUPuhXRLW4=; fh=WRghmcXF91K++LhbIQmGjTZJu6I/voCfM075PdCX8ws=; b=cOJBV0EzMF3f/FksuphcXjtIcR/ZVw6XaqLlE4VJkMEyt2C5vCk7dxVfeUuMo0IWJT 7Eg4ieUe3ipKYzI6RcjU7vGRnBLfFpqf+tgJ5UQiJRUxAG9BpIei21HG0yuDKtNnOkPb TS6SgHg+lqb1vCxvwiL4rv9NNJigdTVbEZwPQK7+F65w2KAuqUef24ZNiHGqvyyGn6b5 PZpeqYYtb6SMVojJWtpivqPC+WcLxFCQUY2Yj8kx5gs23YOoNLmimcvjw9Lf04P+QJxs MOTFDu28mynILg8Hb3GnRcz499HWZc5M9cFsPvR/FiBouILHthExIYhVK2Y/cwF35REc c4nQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=DJ2Flso8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:1 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from morse.vger.email (morse.vger.email. [2620:137:e000::3:1]) by mx.google.com with ESMTPS id j70-20020a638049000000b0057c24bae994si2629078pgd.355.2023.10.18.10.43.34 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Oct 2023 10:43:35 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:1 as permitted sender) client-ip=2620:137:e000::3:1; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=DJ2Flso8; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:1 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by morse.vger.email (Postfix) with ESMTP id 4420E822035C; Wed, 18 Oct 2023 10:43:32 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at morse.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231864AbjJRRnG (ORCPT + 99 others); Wed, 18 Oct 2023 13:43:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56224 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232475AbjJRRmo (ORCPT ); Wed, 18 Oct 2023 13:42:44 -0400 Received: from mail-pf1-x42d.google.com (mail-pf1-x42d.google.com [IPv6:2607:f8b0:4864:20::42d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EC517D56; Wed, 18 Oct 2023 10:42:08 -0700 (PDT) Received: by mail-pf1-x42d.google.com with SMTP id d2e1a72fcca58-6bd96cfb99cso3032440b3a.2; Wed, 18 Oct 2023 10:42:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1697650928; x=1698255728; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=dKaWOPnAN5si+yKYBlPOMxbvgqGw6FhfRCUPuhXRLW4=; b=DJ2Flso8Fhadvkiy9PtnH9bJCHhUqGXXhJZQR3Wp4lnxs78jHryj6CaczQkQucQJws 8PeAkB7ushmkG6WMxnPBo6xX5NA8iarpJmQUneL3PxIlaMtG+Fu0u8doIgbOWEjt39hw d89td4OuQjaWZdgevRP3Eji4L0NNqF2sXwGG5Oj/r1tTPyULiL0G4/Cl2KH+JbXPATOb p8WIcgeEYqU1+GFu4Lz6cWxVoSuObaFKpkGPqOqJTPZwPfObSTmpPkKw6PsU6pUKQQQN fB76JWOsaQHRI+3rIQPZx7d8WBNuNx2YXt8OJdIdGmrJdCmnha5Hz9FQpt4l5sFj31T0 /yrQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697650928; x=1698255728; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=dKaWOPnAN5si+yKYBlPOMxbvgqGw6FhfRCUPuhXRLW4=; b=RZ9OT9IeY64ArMVSrfwryDdzgfcTa8fC2Ymri3yDYccV1yYzsC4Lq7BWxy0NonvZgp i+11OLsw+7kqLAd9FdcU6ZGi5shKD0m9Sqlngx+298OJIqw/2lPqCI1KwIFJv8gnsEok C2uw5VLU0khsLdKI5oRxulksmMUfSPs9S3nyIR0va3YoMpCV+h7QIclAq9uHSJAa362P NEjI9wlsweas0SLgSSX3C6OGh+XGit7wBbi/THiPt44DdNEEQmgcgKfjmNMqSqDY3C/t Nv2xj/foFQGe9Hgvi41Q/7W2/lqHrc6gqdaPKwhzmRF/G/Amnt2nBREm+3mBXzciTymz B3pw== X-Gm-Message-State: AOJu0YwRcEOAEAXZ2xDYe68aNkZeb5ki4fjiPG9KJwb6Y9YKoZQBqNdU dSVCFC4sYnltlgxycCKBdIzjJabkB17bFW5RSJyfKYjs X-Received: by 2002:a05:6a21:7906:b0:16b:d137:de5a with SMTP id bg6-20020a056a21790600b0016bd137de5amr5314040pzc.28.1697650928075; Wed, 18 Oct 2023 10:42:08 -0700 (PDT) MIME-Version: 1.0 References: <20221118093931.1284465-1-paul.elder@ideasonboard.com> <20221118093931.1284465-5-paul.elder@ideasonboard.com> In-Reply-To: <20221118093931.1284465-5-paul.elder@ideasonboard.com> From: Adam Ford Date: Wed, 18 Oct 2023 12:41:56 -0500 Message-ID: Subject: Re: [PATCH v3 04/14] media: rkisp1: Add match data for i.MX8MP ISP To: Paul Elder Cc: linux-media@vger.kernel.org, Dafna Hirschfeld , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Heiko Stuebner , Helen Koike , Laurent Pinchart , linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rob Herring Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Wed, 18 Oct 2023 10:43:32 -0700 (PDT) On Fri, Nov 18, 2022 at 3:44=E2=80=AFAM Paul Elder wrote: > > Add match data to the rkisp1 driver to match the i.MX8MP ISP. > > Although the new version number isn't very precise, it ought to be fine > as the other version numbers aren't precise either, and we have separate > feature flags for important version-specific features. Despite this > version number being seemingly unimportant, it is added to distinguish > it from the ISP versions integrated in rockchip SoCs. > > Signed-off-by: Paul Elder > Reviewed-by: Rob Herring > Paul, It's been nearly a year since this commit was sent. I noticed it hasn't been applied, and I was curious to know if there is any movement here? I'm happy to test on my 8MP if necessary. Thanks! adam > --- > Changes in v3: > - Remove todo for improving the version number > - Expand the commit message to address the version number > --- > .../platform/rockchip/rkisp1/rkisp1-dev.c | 22 +++++++++++++++++++ > include/uapi/linux/rkisp1-config.h | 2 ++ > 2 files changed, 24 insertions(+) > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-dev.c b/driver= s/media/platform/rockchip/rkisp1/rkisp1-dev.c > index e348d8c86861..69464ce91d59 100644 > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-dev.c > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-dev.c > @@ -496,6 +496,24 @@ static const struct rkisp1_info rk3399_isp_info =3D = { > .features =3D RKISP1_FEATURE_MIPI_CSI2, > }; > > +static const char * const imx8mp_isp_clks[] =3D { > + "isp", > + "hclk", > + "aclk", > +}; > + > +static const struct rkisp1_isr_data imx8mp_isp_isrs[] =3D { > + { NULL, rkisp1_isr }, > +}; > + > +static const struct rkisp1_info imx8mp_isp_info =3D { > + .clks =3D imx8mp_isp_clks, > + .clk_size =3D ARRAY_SIZE(imx8mp_isp_clks), > + .isrs =3D imx8mp_isp_isrs, > + .isr_size =3D ARRAY_SIZE(imx8mp_isp_isrs), > + .isp_ver =3D IMX8MP_V10, > +}; > + > static const struct of_device_id rkisp1_of_match[] =3D { > { > .compatible =3D "rockchip,px30-cif-isp", > @@ -505,6 +523,10 @@ static const struct of_device_id rkisp1_of_match[] = =3D { > .compatible =3D "rockchip,rk3399-cif-isp", > .data =3D &rk3399_isp_info, > }, > + { > + .compatible =3D "fsl,imx8mp-isp", > + .data =3D &imx8mp_isp_info, > + }, > {}, > }; > MODULE_DEVICE_TABLE(of, rkisp1_of_match); > diff --git a/include/uapi/linux/rkisp1-config.h b/include/uapi/linux/rkis= p1-config.h > index 730673ecc63d..f602442c2018 100644 > --- a/include/uapi/linux/rkisp1-config.h > +++ b/include/uapi/linux/rkisp1-config.h > @@ -179,12 +179,14 @@ > * @RKISP1_V11: declared in the original vendor code, but not used > * @RKISP1_V12: used at least in rk3326 and px30 > * @RKISP1_V13: used at least in rk1808 > + * @IMX8MP_V10: used in at least imx8mp > */ > enum rkisp1_cif_isp_version { > RKISP1_V10 =3D 10, > RKISP1_V11, > RKISP1_V12, > RKISP1_V13, > + IMX8MP_V10, > }; > > enum rkisp1_cif_isp_histogram_mode { > -- > 2.35.1 >