Basic pinctrl support for StarFive JH7110 RISC-V SoC [LWN.net]
From: | Hal Feng <hal.feng-AT-starfivetech.com> | |
To: | <linux-riscv-AT-lists.infradead.org>, <devicetree-AT-vger.kernel.org>, <linux-gpio-AT-vger.kernel.org> | |
Subject: | [PATCH v4 0/4] Basic pinctrl support for StarFive JH7110 RISC-V SoC | |
Date: | Fri, 03 Feb 2023 22:17:57 +0800 | |
Message-ID: | <20230203141801.59083-1-hal.feng@starfivetech.com> | |
Cc: | Conor Dooley <conor-AT-kernel.org>, Palmer Dabbelt <palmer-AT-dabbelt.com>, “Rob Herring” <robh+dt-AT-kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt-AT-linaro.org>, Linus Walleij <linus.walleij-AT-linaro.org>, Andreas Schwab <schwab-AT-suse.de>, “Emil Renner Berthing” <emil.renner.berthing-AT-canonical.com>, Jianlong Huang <jianlong.huang-AT-starfivetech.com>, Hal Feng <hal.feng-AT-starfivetech.com>, <linux-kernel-AT-vger.kernel.org> | |
Archive-link: | Article |
This patch series adds basic pinctrl support for StarFive JH7110 SoC. Changes since v3: - Rebased on Linus's "devel" branch of linux-pinctrl repo, which was based on on tag v6.2-rc1. - Dropped patch 1. Patch 2 & 3: - Added a reference for '-pins$' patternProperties. - Put "additionalProperties: false" before properties section. (by Rob) - Improved the description. - Changed the node name in examples from "gpio" to "pinctrl". Patch 4: - Added some missing headers. (by Andreas) v3: https://lore.kernel.org/all/20221220005529.34744-1-hal.fe... Changes since v2: - Rebased on tag v6.1. Patch 1: - Renamed pinctrl-starfive-jh7110.h to starfive,jh7110-pinctrl.h. (by Krzysztof) - Separated the register values in the binding header and stored them in a new file arch/riscv/boot/dts/starfive/jh7110-pinfunc.h. (by Krzysztof) - Split patch 1 into sys part and aon part. Merged them into patch 2 and patch 3 respectively. Patch 2 & 3: - Dropped "reg-names" and the description of "interrupts". Dropped quotes behind "$ref" and kept consisitent quotes. (by Krzysztof) - Moved gpio properties behind interrupt properties. - Moved "required" behind "patternProperties". (by Krzysztof) - Rewrote the examples of bindings. (by Krzysztof and Emil) - Added Co-developed-by tag for Emil. - Dropped unused "clocks" property in patch 3. Patch 4 & 5: - Renamed "pinctrl-starfive.*" to "pinctrl-starfive-jh7110.*" and replaced all "starfive_" prefix with "jh7110_" in these files. (by Emil) - Dropped macro GPIO_NUM_PER_WORD. (by Emil) - Dropped unused flag member in starfive_pinctrl_soc_info structure. (by Emil) - Renamed "pinctrl-jh7110-sys.c" to "pinctrl-starfive-jh7110-sys.c". Renamed "pinctrl-jh7110-aon.c" to "pinctrl-starfive-jh7110-aon.c". (by Emil) - Added individual Kconfig options for sys and aon pinctrl drivers. (by Emil) - Made the sys and aon pinctrl drivers be modules. (by Emil) - Added "JH7110_" prefix for macro SYS_GPO_PDA_0_74_CFG, SYS_GPO_PDA_89_94_CFG and AON_GPO_PDA_0_5_CFG. (by Emil) - Dropped jh7110_sys_pinctrl_probe() and jh7110_aon_pinctrl_probe(). Got the match data in the common jh7110_pinctrl_probe() and used it to probe. (by Emil) - Dropped the of_match_ptr macro(). (by Emil) - Set the MODULE_LICENSE as "GPL" according to commit bf7fbeeae6db. v2: https://lore.kernel.org/all/20221118011108.70715-1-hal.fe... Changes since v1: - Rebased on tag v6.1-rc5. - Dropped patch 22 and 23 since they were merged in v6.1-rc1. - Removed some unused macros and register values which do not belong to bindings. Simplified pinctrl definitions in patch 24. (by Krzysztof) - Split the bindings into sys pinctrl bindings and aon pinctrl bindings, and split patch 25 into two patches. - Made the bindings follow generic pinctrl bindings. (by Krzysztof) - Fixed some wrong indentation in bindings, and checked it with `make dt_binding_check`. - Split the patch 26 into two patches which added sys and aon pinctrl driver respectively. - Restructured the pinctrl drivers so made them follow generic pinctrl bindings. Rewrote `dt_node_to_map` and extracted the public code to make it clearer. v1: https://lore.kernel.org/all/20220929143225.17907-1-hal.fe... Jianlong Huang (4): dt-bindings: pinctrl: Add StarFive JH7110 sys pinctrl dt-bindings: pinctrl: Add StarFive JH7110 aon pinctrl pinctrl: starfive: Add StarFive JH7110 sys controller driver pinctrl: starfive: Add StarFive JH7110 aon controller driver .../pinctrl/starfive,jh7110-aon-pinctrl.yaml | 123 +++ .../pinctrl/starfive,jh7110-sys-pinctrl.yaml | 141 +++ MAINTAINERS | 8 +- drivers/pinctrl/starfive/Kconfig | 33 + drivers/pinctrl/starfive/Makefile | 4 + .../starfive/pinctrl-starfive-jh7110-aon.c | 177 ++++ .../starfive/pinctrl-starfive-jh7110-sys.c | 449 ++++++++ .../starfive/pinctrl-starfive-jh7110.c | 982 ++++++++++++++++++ .../starfive/pinctrl-starfive-jh7110.h | 70 ++ .../pinctrl/starfive,jh7110-pinctrl.h | 137 +++ 10 files changed, 2121 insertions(+), 3 deletions(-) create mode 100644 Documentation/devicetree/bindings/pinctrl/starfive,jh7110-aon-pinctrl.yaml create mode 100644 Documentation/devicetree/bindings/pinctrl/starfive,jh7110-sys-pinctrl.yaml create mode 100644 drivers/pinctrl/starfive/pinctrl-starfive-jh7110-aon.c create mode 100644 drivers/pinctrl/starfive/pinctrl-starfive-jh7110-sys.c create mode 100644 drivers/pinctrl/starfive/pinctrl-starfive-jh7110.c create mode 100644 drivers/pinctrl/starfive/pinctrl-starfive-jh7110.h create mode 100644 include/dt-bindings/pinctrl/starfive,jh7110-pinctrl.h -- 2.38.1
Read more here: Source link