From patchwork Mon May 6 15:08:17 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Johan Hovold X-Patchwork-Id: 795056 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7DC4815534E; Mon, 6 May 2024 15:10:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715008215; cv=none; b=uZZ5r0nq77C7oNF0+ATPbxx/EaGkcFU3hfLM0hxvg+aeZypUhAe94UCT3Nclr+ozvWCA3tdpLsx7EzcpU+0G/MVNrLDl1Vujb0yP7jrszbJpXfIYzzeizN+jPV4GaAq7Eal75xeTZogQQgJIOhgX0FPedlvEWMVglxtYjwUE4SE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715008215; c=relaxed/simple; bh=4TqHne13Oc3HtRcJPwWDiVQUalvMRuzinffu+3SjKs4=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=pjNExhjix9cy8mRF1eIrerRCOTeaHKFCChxAV5KxPk4zkUtSSYlwk4wxoYDzqvQRtPkBGA4FS3FS8L15UjaC63vJFvFg8eHY4NAQGdj9Q+l3ACMcVH5qYkULrZZq/z4RIHSm8YzyqBx30YgOyq5WpZT4i5hqSgZN/I/wru9ihcs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=CzXeuxhZ; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="CzXeuxhZ" Received: by smtp.kernel.org (Postfix) with ESMTPSA id C64CDC4DDE1; Mon, 6 May 2024 15:10:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1715008214; bh=4TqHne13Oc3HtRcJPwWDiVQUalvMRuzinffu+3SjKs4=; h=From:To:Cc:Subject:Date:From; b=CzXeuxhZHXN0vrxfduiSayDvEdw+1lSD5icC0IQAEkfk8uM3EurAca1hcgAjFAtvc QT0qoVALHmky1vPIm/Udm3OafipWcyo2aZ7xfmWgCcB2AhlgDz+M/A1BrrrixHU5b+ yNwj6MXvs2ms3QsdIKxBpiNP9ZzH9g7HsspZhwxREijMns6bEZgzO+HXOUH88Em92X r82+mvFWb10QpXKbBOVXSxqmu9Dl+RBOLbFA70HwfhclAff755ktcqZjkKrPEMTwbe FvMx/XPrJpEr0oKNW2sspawE5s3xhcezbTk+SX57uhE89ZW8uLfOOkxB/6W05q1Urw E6Js2lfwqs83Q== Received: from johan by xi.lan with local (Exim 4.97.1) (envelope-from ) id 1s3zyp-000000006C7-0LIZ; Mon, 06 May 2024 17:10:15 +0200 From: Johan Hovold To: Lee Jones , Mark Brown , Linus Walleij , Bjorn Andersson Cc: Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Liam Girdwood , Das Srinagesh , Satya Priya , Stephen Boyd , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, Johan Hovold Subject: [PATCH 00/13] arm64: dts: qcom: sc8280xp-x13s: enable pm8008 camera pmic Date: Mon, 6 May 2024 17:08:17 +0200 Message-ID: <20240506150830.23709-1-johan+linaro@kernel.org> X-Mailer: git-send-email 2.43.2 Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 The Qualcomm PM8008 PMIC is a so called QPNP PMIC with seven LDO regulators, a temperature alarm block and two GPIO pins (which are also used for interrupt signalling and reset). Unlike previous QPNP PMICs it uses an I2C rather than SPMI interface, which has implications for how interrupts are handled. A previous attempt by Qualcomm to upstream support for PM8008 stalled two years ago at version 15 after a lot of back and forth discussion on how best to describe this device in the devicetree. [1] After reviewing the backstory on this and surveying the current SPMI PMIC bindings and implementation, I opted for a new approach that does not describe internal details like register offsets and interrupts in the devicetree. The original decision to include registers offsets and internal interrupts for SPMI PMICs has led to a number of PMIC dtsi being created to avoid copying lots of boiler plate declarations. This in turn causes trouble when the PMIC USID address is configurable as the address is included in every interrupt specifier. The current SPMI bindings still do not describe the devices fully and additional data is therefore already provided by drivers (e.g. additional register blocks, supplies, additional interrupt specifiers). The fact that PMICs which use two USIDs (addresses) are modelled as two separate devices causes trouble, for example, when there are dependencies between subfunctions. [2] Subfunctions also do not necessarily map neatly onto the 256-register block partitioning of the SPMI register space, something which has lead to unresolved inconsistencies in how functions like PWM are described. [3] In short, it's a bit of a mess. With the new style of bindings, by contrast, only essential information that actually differs between machines would be included in the devicetree. The bindings would also be mostly decoupled from the implementation, which has started to leak out into the binding (e.g. how the QPNP interrupts are handled). This also allows for extending the implementation without having to update the binding, which is especially important as Qualcomm does not publish any documentation (e.g. to later enable regulator over-current protection). Some PMICs support both I2C and SPMI interfaces (e.g. PM8010) and we want to be able to reuse the same bindings regardless of the interface. As a proof concept I have written a new pmc8280 driver for one of the SPMI PMICs in the Lenovo ThinkPad X13s that uses the new style of bindings and I've been using that one to control backlight and peripheral regulators for a while now. Specifically, the gpio and temperature-alarm blocks can be used with some minor updates to the current drivers. That work still needs a bit of polish before posting, but my working PoC means that I'm confident enough that the new model will work and that we can go ahead and merge regulator support for the PM8008. This series is specifically needed for the camera sensors in the X13s, for which camera subsystem (camss) support has now been merged for 6.10. The first seven patches are preparatory and can possibly be merged separately from the rest of the series. The next two patches drops the broken GPIO support for PM8008 which had already been upstreamed. The last four patches rework the binding and MFD driver, add support for the regulators and enable the camera PMIC on the X13s. Johan [1] https://lore.kernel.org/all/1655200111-18357-1-git-send-email-quic_c_skakit@quicinc.com [2] https://lore.kernel.org/lkml/20231003152927.15000-3-johan+linaro@kernel.org [3] https://lore.kernel.org/r/20220828132648.3624126-3-bryan.odonoghue@linaro.org Johan Hovold (12): dt-bindings: mfd: pm8008: add reset gpio mfd: pm8008: fix regmap irq chip initialisation mfd: pm8008: deassert reset on probe mfd: pm8008: mark regmap structures as const mfd: pm8008: use lower case hex notation mfd: pm8008: rename irq chip mfd: pm8008: drop unused driver data dt-bindings: pinctrl: qcom,pmic-gpio: drop pm8008 pinctrl: qcom: spmi-gpio: drop broken pm8008 support dt-bindings: mfd: pm8008: rework binding mfd: pm8008: rework driver arm64: dts: qcom: sc8280xp-x13s: enable pm8008 camera pmic Satya Priya (1): regulator: add pm8008 pmic regulator driver .../devicetree/bindings/mfd/qcom,pm8008.yaml | 158 ++++++++----- .../bindings/pinctrl/qcom,pmic-gpio.yaml | 3 - .../qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 123 ++++++++++ drivers/mfd/Kconfig | 1 + drivers/mfd/qcom-pm8008.c | 163 ++++++++----- drivers/pinctrl/qcom/pinctrl-spmi-gpio.c | 1 - drivers/regulator/Kconfig | 7 + drivers/regulator/Makefile | 1 + drivers/regulator/qcom-pm8008-regulator.c | 215 ++++++++++++++++++ include/dt-bindings/mfd/qcom-pm8008.h | 19 -- 10 files changed, 554 insertions(+), 137 deletions(-) create mode 100644 drivers/regulator/qcom-pm8008-regulator.c delete mode 100644 include/dt-bindings/mfd/qcom-pm8008.h