From patchwork Sat Nov 18 12:11:34 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Bryan O'Donoghue X-Patchwork-Id: 16739 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9910:0:b0:403:3b70:6f57 with SMTP id i16csp1134367vqn; Sat, 18 Nov 2023 04:12:04 -0800 (PST) X-Google-Smtp-Source: AGHT+IFiH+iA7ncx7Zst7OKwGRdjYS8hb/Dgs/jCIWmz8a/CZP2hVdFbBGbCgHoPFCq28/3HPoNa X-Received: by 2002:a05:6808:1382:b0:3a7:35af:bbc0 with SMTP id c2-20020a056808138200b003a735afbbc0mr3264118oiw.54.1700309524527; Sat, 18 Nov 2023 04:12:04 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700309524; cv=none; d=google.com; s=arc-20160816; b=zN1IRjoJZfFQS9zBpeKY14Sqe010Qibal2j7xe8Zhmg95uvZFblPd065nrgdRrOYMl cOg0x16gq9R/zews47jkiXYjWhndhy0R/4Q9PeADos3HL/kVA6L/sEuZJnuFXi/+I3tA vOj1b5GLD1C2dSiPKQ2/i+Za9JS1DKtGdbKyiXET8yp9d42rIHMmCW5O4GE6l3KanElA w7scUULSoQTKRNSwwysXK8nyFDe0jT4FwzJhgvhMMdS7CPGjxyzX309ReESvaV0RIew3 EkwXJs0LANQzQmWI21K2y0sQkrk/PUM1qhYmbK4PjC/5i0V1Bnjou8HMDl4J5J+Z4QLJ sjww== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:content-transfer-encoding:mime-version :message-id:date:subject:from:dkim-signature; bh=1tqY3E3gRmP482a4I9SyEm7K69jCzqs5aXA0CUIn8z0=; fh=+21+21gNft/7To99MfcGJu9QkWyyZ7J1aCqrKBAZjFw=; b=P1qrbmhZ+gk7bi4KFNRuoFj68bu+9SmoWfDM4Jub1oLaYEqp+Ro1uCHqps2DJx9SK9 8ueO0YeAclt2wpT8Z2wIFFayFNz/vqaMb02EmsYKVX9rOYaKOspjJehPh5KsOp9QiqZx /bZaK5mxRs00PHAcRAKRoAdYOsLZzYi21yCFpxxSVX6PbDIyxR/IYE8MRx48SjT2fyV4 p9xcHjWqQH6Rpa8Q18HZg1MBASVshDx8Hs9hC1tzdMgthMaHm4V1ZRpIfN4jWh8l3ehk NIcUIFqTD7MYDwC23JoGp1huhsdhAPUrVt/A4qfZ+w6GulPJ6//tnCJN/+7hA2XU3TYU 9Vlg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=ANdJnKFM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from agentk.vger.email (agentk.vger.email. [2620:137:e000::3:2]) by mx.google.com with ESMTPS id f2-20020a63dc42000000b005c1cc7273bbsi4145720pgj.746.2023.11.18.04.12.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 18 Nov 2023 04:12:04 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=ANdJnKFM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id 710CA81AFE51; Sat, 18 Nov 2023 04:11:58 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229526AbjKRMLp (ORCPT + 29 others); Sat, 18 Nov 2023 07:11:45 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43834 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229469AbjKRMLo (ORCPT ); Sat, 18 Nov 2023 07:11:44 -0500 Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 35172D61 for ; Sat, 18 Nov 2023 04:11:40 -0800 (PST) Received: by mail-wm1-x344.google.com with SMTP id 5b1f17b1804b1-40a46ea95f0so3483215e9.2 for ; Sat, 18 Nov 2023 04:11:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1700309498; x=1700914298; darn=vger.kernel.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=1tqY3E3gRmP482a4I9SyEm7K69jCzqs5aXA0CUIn8z0=; b=ANdJnKFM1xVJwQR92qSWezV5UGGSfdug4s7lKnDXpUvdFTtKIMYqdDs5PrUP1so25z /EgzDWBDGO70O1Ffv164mRKpKQbSjPqe7V+EIYrA9G4PYd8xKslMse8jj0lUMgtOOi+y NjPtLU0MZKTTjT0a03ZFyUQXgYb0DqTkCn63+YZMS9Tw8c1Ctoy2xLi2GJ/VKqu24xMM OCQRb2DNBW/c5pFJwatg2yhUE6ZvtwdIKindpgUzGgD4i+bYwMl6EpX1fRj/P3CP7iPh sNRALY3a4JZF3N48ubJ6s7bfHzAwEGKR/y1xlIjP/+cLII29gcybe0V5VcE0A/B8nRUi kPsw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700309498; x=1700914298; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=1tqY3E3gRmP482a4I9SyEm7K69jCzqs5aXA0CUIn8z0=; b=k4o9c7fc3YLoPehV625KutYAI4YQBIHUbCASrvoyHsJFtqa7ZxrTx2zS/OQRe2ZEPe NJM5f+01oHaowwyfAbdjzeNGDKC0WHWu+Q3YqV1jAn12pbRpmL+lRamC9f/HaZjkXDxe O8YZw6T06k7e5rL68Q8M4Ti7/ZwxjqqttmSnafu1xIK9vC1zdJfBPaSssBBqgZTBioS+ AtsSvqRneW9pM7uenp1jmYUWChIvn1oC23tKd66Q8ChyGECXJbQh4yGkenG/ue/a7O+G aEmOk9+GVXA1Y1e2i6XkLAf+pEim8WLXXATZ61ERzPE6bHf+Zfh4YYgDiAgp8dDO3Beo nc6A== X-Gm-Message-State: AOJu0YxUhNgd6SN7KdBwTazSUEWVNjyhWFVPVYnLzsLxjgtzqoZqqWWX i2dimLAIZC84I4HYzHqYcEzhcw== X-Received: by 2002:a05:600c:4f05:b0:409:2f7:d771 with SMTP id l5-20020a05600c4f0500b0040902f7d771mr1747817wmq.4.1700309498534; Sat, 18 Nov 2023 04:11:38 -0800 (PST) Received: from [127.0.0.1] ([37.228.218.3]) by smtp.gmail.com with ESMTPSA id n18-20020a05600c3b9200b00407efbc4361sm10955775wms.9.2023.11.18.04.11.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 18 Nov 2023 04:11:37 -0800 (PST) From: Bryan O'Donoghue Subject: [PATCH v5 0/7] media: qcom: camss: Introduce support for named power-domains Date: Sat, 18 Nov 2023 12:11:34 +0000 Message-Id: <20231118-b4-camss-named-power-domains-v5-0-55eb0f35a30a@linaro.org> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAPepWGUC/4XOQW7DIBCF4atErDsWDJAoXfUeVRdjwDZSzURD6 jaKfPfidOOusnyb7/13VZPkVNXr4a4kLblmLm34l4MKE5UxQY5tK9RojbYGegeB5lqh0JwiXPg 7CUSeKZcKISAFPCF6Z1UjLpKG/PPg3z/aHoRnuE6SaIfi0XivHXbem1N7aB9yo9Jx5MLj9JXeP nMh4Y5l3NAp1yvL7ZG82I3+g4x+UrdY0ND3cfADojsi7eEtb3F7zT7RXNOspbP21p9jH/5p67r +AsnbFLdcAQAA To: hverkuil-cisco@xs4all.nl, laurent.pinchart@ideasonboard.com, Robert Foss , Todor Tomov , Bryan O'Donoghue , Andy Gross , Bjorn Andersson , Konrad Dybcio , Mauro Carvalho Chehab , matti.lehtimaki@gmail.com, quic_grosikop@quicinc.com Cc: linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.13-dev-26615 X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.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 (agentk.vger.email [0.0.0.0]); Sat, 18 Nov 2023 04:11:58 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1782903760252588801 X-GMAIL-MSGID: 1782903760252588801 V5: - Adds Konrad's RB -> b4 trailers --update - Amends comment and control flow disjunction for readability - Konrad - Link to v4: https://lore.kernel.org/r/20231103-b4-camss-named-power-domains-v4-0-33a905359dbc@linaro.org Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/b4/b4-camss-named-power-domains-v5 sm8250-testable: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/b4/b4-camss-named-power-domains-v5+sm8250 V4: - Updates camss_configure_pd() to use has_pd to determine if a VFE has a pd instead of comparing to vfe_num - Brings in is_lite fixes from Matti The determination of IS_LITE() has been a running sore in this code for some time. Named power domains remove magic index dependencies. Similarly adding an "is_lite" flag to our resources removes the last of the magic indexing sins, so this is an opportune series to add it in. Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/b4/b4-camss-named-power-domains-v4 sm8250-testable: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/b4/b4-camss-named-power-domains-v4+sm8250 Link: https://lore.kernel.org/r/20231101-b4-camss-named-power-domains-v3-0-bbdf5f22462a@linaro.org V3: - Includes bugfix reported on IRC genpd_link and genpd should be checked for NULL on the cleanup path. Matti Lehtimäki - Retains NULL check before dev_pm_domain_attach_by_name() I experimented with the suggested drop but of_property_match_string() chokes Link: https://lore.kernel.org/lkml/883ce8a7-80e1-4065-a957-424d0b4a6535@linaro.org/T/#m10e5a43d0245f13eca177ef2f65b24259c641030 Konrad - Fixes spelling caught by codespell - Konrad Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/b4/b4-camss-named-power-domains-v3 sm8250-testable: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/b4/b4-camss-named-power-domains-v3+sm8250 V2: - Incorporates Konrad's suggestion re: removing 'id' - Adds RB - Konrad - Adds in a flag to indicate if a VFE has a power domain. As I rebased this series I realised we had some magic indexing for VFE v VFE Lite, which isn't the root cause of my bug bear in this series but is the same sin - inferring functionality from indexing. Once we transition fully to named pds we won't need a 'has_pd' to flag which VFEs need power-domain attachment and which don't. That transition will require populating all upstream dtsi with pd-names and then deprecating the old way. has_pd is a far better choice than inferring from indexes so, I've added. Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/commits/aa45a2b58aa1e187a2698a65164d694251f08fa1 V1: At the moment the Qcom CAMSS driver relies on the declaration order of power-domains within the dtsi to determine which power-domain relates to a VFE and which power-domain relates to the top-level (top) CAMSS power-domain. VFE power-domains must be declared prior to the top power-domain. The top power-domain must be declared last. Early SoCs have just one top power-domain with later SoCs introducing VFE specific power-domains. Differentiating between the number of power-domains results in lots of code which is brittle and which we can mostly get rid of with named power-domains. The reliance on declaration ordering is in-effect magic number indexing. This series introduces named power-domains for CAMSS and refactors some of the code in CAMSS to support the new named power-domains. We continue to support the legacy indexing model with an intention to remove after a reasonable transition period. New SoC additions should use named power-domains from now on. Tested on x13s, rb5, db410c Link: https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/linux-next-23-10-23-camss-named-power-domains Bryan O'Donoghue (5): media: qcom: camss: Flag which VFEs require a power-domain media: qcom: camss: Convert to per-VFE pointer for power-domain linkages media: qcom: camss: Use common VFE pm_domain_on/pm_domain_off where applicable media: qcom: camss: Move VFE power-domain specifics into vfe.c media: qcom: camss: Add support for named power-domains .../media/platform/qcom/camss/camss-vfe-170.c | 36 -------- .../media/platform/qcom/camss/camss-vfe-4-1.c | 8 +- .../media/platform/qcom/camss/camss-vfe-4-7.c | 36 -------- .../media/platform/qcom/camss/camss-vfe-4-8.c | 31 ------- .../media/platform/qcom/camss/camss-vfe-480.c | 36 -------- drivers/media/platform/qcom/camss/camss-vfe.c | 77 ++++++++++++++++ drivers/media/platform/qcom/camss/camss-vfe.h | 16 ++++ drivers/media/platform/qcom/camss/camss.c | 87 ++++++++++++------- drivers/media/platform/qcom/camss/camss.h | 7 +- 9 files changed, 156 insertions(+), 178 deletions(-) --- 2.42.0 --- --- Bryan O'Donoghue (5): media: qcom: camss: Flag which VFEs require a power-domain media: qcom: camss: Convert to per-VFE pointer for power-domain linkages media: qcom: camss: Use common VFE pm_domain_on/pm_domain_off where applicable media: qcom: camss: Move VFE power-domain specifics into vfe.c media: qcom: camss: Add support for named power-domains Matti Lehtimäki (2): media: qcom: camss: Flag VFE-lites to support more VFEs media: qcom: camss: Flag CSID-lites to support more CSIDs .../media/platform/qcom/camss/camss-csid-gen2.c | 31 +++--- drivers/media/platform/qcom/camss/camss-csid.c | 5 + drivers/media/platform/qcom/camss/camss-csid.h | 7 ++ drivers/media/platform/qcom/camss/camss-vfe-170.c | 36 ------- drivers/media/platform/qcom/camss/camss-vfe-4-1.c | 8 +- drivers/media/platform/qcom/camss/camss-vfe-4-7.c | 36 ------- drivers/media/platform/qcom/camss/camss-vfe-4-8.c | 31 ------ drivers/media/platform/qcom/camss/camss-vfe-480.c | 69 +++--------- drivers/media/platform/qcom/camss/camss-vfe.c | 84 +++++++++++++++ drivers/media/platform/qcom/camss/camss-vfe.h | 26 +++++ drivers/media/platform/qcom/camss/camss.c | 116 +++++++++++++-------- drivers/media/platform/qcom/camss/camss.h | 10 +- 12 files changed, 236 insertions(+), 223 deletions(-) --- base-commit: 48016737a9af47328dd321df4dd3479ed5e2041d change-id: 20231031-b4-camss-named-power-domains-cc2ac2722543 Best regards,