From: Sam Protsenko <semen.protsenko@linaro.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Tomasz Figa <tomasz.figa@gmail.com>,
Sylwester Nawrocki <s.nawrocki@samsung.com>,
Linus Walleij <linus.walleij@linaro.org>,
Rob Herring <robh+dt@kernel.org>,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-gpio@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Marek Szyprowski <m.szyprowski@samsung.com>,
Chanho Park <chanho61.park@samsung.com>,
Alim Akhtar <alim.akhtar@gmail.com>
Subject: Re: [PATCH v2 00/28] pinctrl: dt-bindings: samsung: convert to dtschema
Date: Fri, 14 Jan 2022 20:31:52 +0200 [thread overview]
Message-ID: <CAPLW+4k3VNuQGfi_mnAWYUSXYaPmoFj1D55pHH0ByUsNu9kSyQ@mail.gmail.com> (raw)
In-Reply-To: <20220111201426.326777-1-krzysztof.kozlowski@canonical.com>
On Tue, 11 Jan 2022 at 22:15, Krzysztof Kozlowski
<krzysztof.kozlowski@canonical.com> wrote:
>
> Hi,
>
> Changes since v1
> ================
> 1. Patch #1: add missing pin assignment (Alim).
> 2. Patch #2: correct double sizeof() (Alim).
> 3. Patch #7, #8: put label-override in proper patch (Alim).
> 4. Patch #24: Extend doc, change the 'if' clause for wake-up interrupts.
> 5. New patches: #25 - #28.
> Exynos850 and ExynosAutov9 seems to be different in pin ctrl interrupt
> handling, so they need their own compatibles.
> Please kindly review and provide feedback on these as I do not have
> details.
> 6. Add review tags.
>
> Dependencies
> ============
> 1. Patch #2 ("pinctrl: samsung: accept GPIO bank nodes with a suffix") is
> necessary for DTS patches.
>
> 2. Last patches #27 and #28 depend on patch #26 adding the compatibles.
>
> Best regards,
> Krzysztof
>
> Krzysztof Kozlowski (28):
> pinctrl: samsung: drop pin banks references on error paths
> pinctrl: samsung: accept GPIO bank nodes with a suffix
> ARM: dts: exynos: drop unused pinctrl defines in Exynos3250
> ARM: dts: exynos: simplify PMIC DVS pin configuration in Odroid XU
> ARM: dts: exynos: override pins by label in Peach Pit
> ARM: dts: exynos: simplify PMIC DVS pin configuration in Peach Pit
> ARM: dts: exynos: override pins by label in Peach Pi
> ARM: dts: exynos: simplify PMIC DVS pin configuration in Peach Pi
> ARM: dts: s3c64xx: drop unneeded pinctrl wake-up interrupt mapping
> ARM: dts: exynos: align pinctrl with dtschema in Exynos3250
> ARM: dts: exynos: align pinctrl with dtschema in Exynos4210
> ARM: dts: exynos: align pinctrl with dtschema in Exynos4412
> ARM: dts: exynos: align pinctrl with dtschema in Exynos5250
> ARM: dts: exynos: align pinctrl with dtschema in Exynos5260
> ARM: dts: exynos: align pinctrl with dtschema in Exynos5410
> ARM: dts: exynos: align pinctrl with dtschema in Exynos542x/5800
> arm64: dts: exynos: align pinctrl with dtschema in Exynos5433
> arm64: dts: exynos: align pinctrl with dtschema in Exynos7
> arm64: dts: exynos: align pinctrl with dtschema in Exynos850
> arm64: dts: exynos: align pinctrl with dtschema in ExynosAutov9
> ARM: dts: s3c24xx: align pinctrl with dtschema
> ARM: dts: s3c64xx: align pinctrl with dtschema
> ARM: dts: s5pv210: align pinctrl with dtschema
> dt-bindings: pinctrl: samsung: convert to dtschema
> dt-bindings: pinctrl: samsung: describe Exynos850 and ExynosAutov9
> wake-ups
> pinctrl: samsung: add support for Exynos850 and ExynosAutov9 wake-ups
> arm64: dts: exynos: use dedicated wake-up pinctrl compatible in
> Exynos850
> arm64: dts: exynos: use dedicated wake-up pinctrl compatible in
> ExynosAutov9
>
Maybe it makes sense to include my patch [1] for gpm6/gpm7 in this series?
[1] https://lore.kernel.org/linux-arm-kernel/20220103181826.2136-1-semen.protsenko@linaro.org/T/
> .../pinctrl/samsung,pinctrl-gpio-bank.yaml | 52 +++
> .../pinctrl/samsung,pinctrl-pins-cfg.yaml | 81 ++++
> .../samsung,pinctrl-wakeup-interrupt.yaml | 106 +++++
> .../bindings/pinctrl/samsung,pinctrl.yaml | 392 ++++++++++++++++++
> .../bindings/pinctrl/samsung-pinctrl.txt | 383 -----------------
> MAINTAINERS | 2 +-
> arch/arm/boot/dts/exynos3250-artik5.dtsi | 10 +-
> arch/arm/boot/dts/exynos3250-pinctrl.dtsi | 165 ++++----
> arch/arm/boot/dts/exynos4210-i9100.dts | 30 +-
> arch/arm/boot/dts/exynos4210-origen.dts | 2 +-
> arch/arm/boot/dts/exynos4210-pinctrl.dtsi | 226 +++++-----
> arch/arm/boot/dts/exynos4210-smdkv310.dts | 4 +-
> arch/arm/boot/dts/exynos4210-trats.dts | 6 +-
> .../boot/dts/exynos4210-universal_c210.dts | 12 +-
> arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi | 4 +-
> arch/arm/boot/dts/exynos4412-itop-elite.dts | 2 +-
> .../boot/dts/exynos4412-itop-scp-core.dtsi | 2 +-
> arch/arm/boot/dts/exynos4412-midas.dtsi | 30 +-
> .../boot/dts/exynos4412-odroid-common.dtsi | 8 +-
> arch/arm/boot/dts/exynos4412-odroidx.dts | 2 +-
> arch/arm/boot/dts/exynos4412-origen.dts | 4 +-
> arch/arm/boot/dts/exynos4412-p4note.dtsi | 44 +-
> arch/arm/boot/dts/exynos4412-pinctrl.dtsi | 252 +++++------
> arch/arm/boot/dts/exynos4412-smdk4412.dts | 4 +-
> arch/arm/boot/dts/exynos5250-arndale.dts | 2 +-
> arch/arm/boot/dts/exynos5250-pinctrl.dtsi | 220 +++++-----
> arch/arm/boot/dts/exynos5250-smdk5250.dts | 2 +-
> arch/arm/boot/dts/exynos5250-snow-common.dtsi | 22 +-
> arch/arm/boot/dts/exynos5250-snow-rev5.dts | 2 +-
> arch/arm/boot/dts/exynos5250-snow.dts | 2 +-
> arch/arm/boot/dts/exynos5250-spring.dts | 20 +-
> arch/arm/boot/dts/exynos5260-pinctrl.dtsi | 148 +++----
> arch/arm/boot/dts/exynos5260-xyref5260.dts | 2 +-
> arch/arm/boot/dts/exynos5410-odroidxu.dts | 22 +-
> arch/arm/boot/dts/exynos5410-pinctrl.dtsi | 170 ++++----
> arch/arm/boot/dts/exynos5410-smdk5410.dts | 4 +-
> arch/arm/boot/dts/exynos5420-arndale-octa.dts | 2 +-
> arch/arm/boot/dts/exynos5420-peach-pit.dts | 89 ++--
> arch/arm/boot/dts/exynos5420-pinctrl.dtsi | 194 ++++-----
> arch/arm/boot/dts/exynos5420-smdk5420.dts | 6 +-
> arch/arm/boot/dts/exynos5422-odroid-core.dtsi | 2 +-
> .../boot/dts/exynos5422-odroidxu3-common.dtsi | 6 +-
> arch/arm/boot/dts/exynos5800-peach-pi.dts | 89 ++--
> arch/arm/boot/dts/s3c2416-pinctrl.dtsi | 60 +--
> arch/arm/boot/dts/s3c6410-mini6410.dts | 4 +-
> arch/arm/boot/dts/s3c64xx-pinctrl.dtsi | 210 +++++-----
> arch/arm/boot/dts/s3c64xx.dtsi | 16 +-
> arch/arm/boot/dts/s5pv210-aquila.dts | 2 +-
> arch/arm/boot/dts/s5pv210-aries.dtsi | 40 +-
> arch/arm/boot/dts/s5pv210-fascinate4g.dts | 12 +-
> arch/arm/boot/dts/s5pv210-galaxys.dts | 16 +-
> arch/arm/boot/dts/s5pv210-pinctrl.dtsi | 226 +++++-----
> .../boot/dts/exynos/exynos5433-pinctrl.dtsi | 211 +++++-----
> .../dts/exynos/exynos5433-tm2-common.dtsi | 259 ++++++------
> .../boot/dts/exynos/exynos7-espresso.dts | 6 +-
> .../boot/dts/exynos/exynos7-pinctrl.dtsi | 176 ++++----
> .../boot/dts/exynos/exynos850-pinctrl.dtsi | 52 +--
> arch/arm64/boot/dts/exynos/exynos850.dtsi | 4 +-
> .../boot/dts/exynos/exynosautov9-pinctrl.dtsi | 50 +--
> arch/arm64/boot/dts/exynos/exynosautov9.dtsi | 2 +-
> drivers/pinctrl/samsung/pinctrl-exynos.c | 4 +
> drivers/pinctrl/samsung/pinctrl-samsung.c | 87 +++-
> 62 files changed, 2261 insertions(+), 2003 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/pinctrl/samsung,pinctrl-gpio-bank.yaml
> create mode 100644 Documentation/devicetree/bindings/pinctrl/samsung,pinctrl-pins-cfg.yaml
> create mode 100644 Documentation/devicetree/bindings/pinctrl/samsung,pinctrl-wakeup-interrupt.yaml
> create mode 100644 Documentation/devicetree/bindings/pinctrl/samsung,pinctrl.yaml
> delete mode 100644 Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt
>
> --
> 2.32.0
>
next prev parent reply other threads:[~2022-01-14 18:32 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-11 20:13 [PATCH v2 00/28] pinctrl: dt-bindings: samsung: convert to dtschema Krzysztof Kozlowski
2022-01-11 20:13 ` [PATCH v2 01/28] pinctrl: samsung: drop pin banks references on error paths Krzysztof Kozlowski
2022-01-12 5:28 ` Chanho Park
2022-01-14 16:38 ` Sam Protsenko
2022-01-11 20:14 ` [PATCH v2 02/28] pinctrl: samsung: accept GPIO bank nodes with a suffix Krzysztof Kozlowski
2022-01-14 17:00 ` Sam Protsenko
2022-01-11 20:14 ` [PATCH v2 03/28] ARM: dts: exynos: drop unused pinctrl defines in Exynos3250 Krzysztof Kozlowski
2022-01-11 20:14 ` [PATCH v2 04/28] ARM: dts: exynos: simplify PMIC DVS pin configuration in Odroid XU Krzysztof Kozlowski
2022-01-11 20:14 ` [PATCH v2 05/28] ARM: dts: exynos: override pins by label in Peach Pit Krzysztof Kozlowski
2022-01-11 20:14 ` [PATCH v2 06/28] ARM: dts: exynos: simplify PMIC DVS pin configuration " Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 07/28] ARM: dts: exynos: override pins by label in Peach Pi Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 08/28] ARM: dts: exynos: simplify PMIC DVS pin configuration " Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 09/28] ARM: dts: s3c64xx: drop unneeded pinctrl wake-up interrupt mapping Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 10/28] ARM: dts: exynos: align pinctrl with dtschema in Exynos3250 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 11/28] ARM: dts: exynos: align pinctrl with dtschema in Exynos4210 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 12/28] ARM: dts: exynos: align pinctrl with dtschema in Exynos4412 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 13/28] ARM: dts: exynos: align pinctrl with dtschema in Exynos5250 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 14/28] ARM: dts: exynos: align pinctrl with dtschema in Exynos5260 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 15/28] ARM: dts: exynos: align pinctrl with dtschema in Exynos5410 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 16/28] ARM: dts: exynos: align pinctrl with dtschema in Exynos542x/5800 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 17/28] arm64: dts: exynos: align pinctrl with dtschema in Exynos5433 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 18/28] arm64: dts: exynos: align pinctrl with dtschema in Exynos7 Krzysztof Kozlowski
2022-01-16 15:34 ` Alim Akhtar
2022-01-11 20:17 ` [PATCH v2 19/28] arm64: dts: exynos: align pinctrl with dtschema in Exynos850 Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 20/28] arm64: dts: exynos: align pinctrl with dtschema in ExynosAutov9 Krzysztof Kozlowski
2022-01-12 5:21 ` Chanho Park
2022-01-14 16:40 ` Sam Protsenko
2022-01-11 20:17 ` [PATCH v2 21/28] ARM: dts: s3c24xx: align pinctrl with dtschema Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 22/28] ARM: dts: s3c64xx: " Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 23/28] ARM: dts: s5pv210: " Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 24/28] dt-bindings: pinctrl: samsung: convert to dtschema Krzysztof Kozlowski
2022-01-16 0:55 ` Linus Walleij
2022-01-16 17:10 ` Krzysztof Kozlowski
2022-01-16 21:38 ` Linus Walleij
2022-01-17 7:45 ` Krzysztof Kozlowski
2022-01-17 20:26 ` Olof Johansson
2022-01-18 7:52 ` Krzysztof Kozlowski
2022-01-11 20:17 ` [PATCH v2 25/28] dt-bindings: pinctrl: samsung: describe Exynos850 and ExynosAutov9 wake-ups Krzysztof Kozlowski
2022-01-12 2:26 ` Rob Herring
2022-01-14 19:44 ` Sam Protsenko
2022-01-11 20:17 ` [PATCH v2 26/28] pinctrl: samsung: add support for " Krzysztof Kozlowski
2022-01-12 5:21 ` Chanho Park
2022-01-14 16:39 ` Sam Protsenko
2022-01-11 20:17 ` [PATCH v2 27/28] arm64: dts: exynos: use dedicated wake-up pinctrl compatible in Exynos850 Krzysztof Kozlowski
2022-01-14 18:34 ` Sam Protsenko
2022-01-11 20:17 ` [PATCH v2 28/28] arm64: dts: exynos: use dedicated wake-up pinctrl compatible in ExynosAutov9 Krzysztof Kozlowski
2022-01-12 5:28 ` Chanho Park
2022-01-14 16:31 ` Sam Protsenko
2022-01-14 18:31 ` Sam Protsenko [this message]
2022-01-15 15:46 ` [PATCH v2 00/28] pinctrl: dt-bindings: samsung: convert to dtschema Krzysztof Kozlowski
2022-01-23 11:46 ` Krzysztof Kozlowski
2022-01-23 17:09 ` Krzysztof Kozlowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAPLW+4k3VNuQGfi_mnAWYUSXYaPmoFj1D55pHH0ByUsNu9kSyQ@mail.gmail.com \
--to=semen.protsenko@linaro.org \
--cc=alim.akhtar@gmail.com \
--cc=chanho61.park@samsung.com \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski@canonical.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=robh+dt@kernel.org \
--cc=s.nawrocki@samsung.com \
--cc=tomasz.figa@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).