[PATCH v2 00/14] Refactor Ralink Pinctrl and Add Documentation

Arınç ÜNAL arinc.unal at arinc9.com
Thu Apr 14 10:39:02 PDT 2022


Hey everyone.

This patch series brings complete refactoring to the Ralink pinctrl driver
and its subdrivers.

The mt7620 pinctrl subdriver supports MT7628 and MT7688 SoCs along with
MT7620. These two share the same pin layout. The code used for MT7628 and
MT7688 is renamed from MT7628/mt7628an to MT76X8.

Ralink pinctrl driver is called rt2880 which is the name of the Ralink
RT2880 SoC. A subdriver for the Ralink RT2880 SoC is called rt288x. Rename
rt2880 to ralink.

Rename code from pinmux to pinctrl for where the operation is not about the
muxing of pins.

Rename rt288x pinctrl subdriver for the RT2880 SoC to rt2880.

Variables for functions include "grp" on the Ralink MT7620 and MT7621
subdrivers. Rename them to "func" instead as they define the functions for
the pin groups. This is already the case for the other 3 subdrivers;
RT2880, RT305x, RT3883.

Fix Kconfig to call the subdrivers, well, subdrivers.

Add new compatible strings for each subdriver and update DT binding
accordingly.

Add Ralink pinctrl driver to MAINTAINERS and add me and Sergio as the
maintainers.

Finally, fix the current rt2880 documentation and add binding for all of
the subdrivers.

I have the patches here should anyone prefer to read them there:
https://github.com/arinc9/linux/commits/ralink-pinctrl-refactor

Ralink pinctrl driver and the subdrivers were compile tested.
MT7621 pinctrl subdriver was tested on a private mt7621 board.
YAML bindings checked with:
ARCH=mips CROSS_COMPILE=mips-linux-gnu- make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/pinctrl/ -j$(nproc)

v2:
- Add Sergio’s Reviewed-by: tag to where it's given.
- Properly explain the addition of the new compatible strings.
- Remove asterisk on MAINTAINERS commit.
- Properly explain the compatible string change for the rt2880 binding.
- Mention adding myself as a maintainer to rt2880 binding.
- Fix typo "indiviual" on YAML bindings. Limit lines to 80 characters.

Arınç ÜNAL (14):
  pinctrl: ralink: rename MT7628(an) functions to MT76X8
  pinctrl: ralink: rename pinctrl-rt2880 to pinctrl-ralink
  pinctrl: ralink: rename pinmux functions to pinctrl
  pinctrl: ralink: rename pinctrl-rt288x to pinctrl-rt2880
  pinctrl: ralink: rename variable names for functions on MT7620 and MT7621
  pinctrl: ralink: rename driver names to subdrivers
  pinctrl: ralink: add new compatible strings for each pinctrl subdriver
  MAINTAINERS: add Ralink pinctrl driver
  mips: dts: ralink: mt7621: use the new compatible string for MT7621 pinctrl
  dt-bindings: pinctrl: rt2880: fix binding name, pin groups and functions
  dt-bindings: pinctrl: add binding for Ralink MT7620 pinctrl
  dt-bindings: pinctrl: add binding for Ralink MT7621 pinctrl
  dt-bindings: pinctrl: add binding for Ralink RT305X pinctrl
  dt-bindings: pinctrl: add binding for Ralink RT3883 pinctrl

 Documentation/devicetree/bindings/pinctrl/ralink,mt7620-pinctrl.yaml            |  91 +++++++++
 .../bindings/pinctrl/{ralink,rt2880-pinmux.yaml => ralink,mt7621-pinctrl.yaml}  |  27 +--
 Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinctrl.yaml            |  68 +++++++
 Documentation/devicetree/bindings/pinctrl/ralink,rt305x-pinctrl.yaml            |  92 +++++++++
 Documentation/devicetree/bindings/pinctrl/ralink,rt3883-pinctrl.yaml            |  71 +++++++
 MAINTAINERS                                                                     |   7 +
 arch/mips/boot/dts/ralink/mt7621.dtsi                                           |   2 +-
 drivers/pinctrl/ralink/Kconfig                                                  |  28 ++-
 drivers/pinctrl/ralink/Makefile                                                 |   4 +-
 drivers/pinctrl/ralink/pinctrl-mt7620.c                                         | 302 +++++++++++++++---------------
 drivers/pinctrl/ralink/pinctrl-mt7621.c                                         |  76 ++++----
 drivers/pinctrl/ralink/pinctrl-ralink.c                                         | 349 ++++++++++++++++++++++++++++++++++
 drivers/pinctrl/ralink/{pinmux.h => pinctrl-ralink.h}                           |  16 +-
 drivers/pinctrl/ralink/pinctrl-rt2880.c                                         | 381 +++++---------------------------------
 drivers/pinctrl/ralink/pinctrl-rt288x.c                                         |  60 ------
 drivers/pinctrl/ralink/pinctrl-rt305x.c                                         |  66 +++----
 drivers/pinctrl/ralink/pinctrl-rt3883.c                                         |  50 ++---
 17 files changed, 1009 insertions(+), 681 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/pinctrl/ralink,mt7620-pinctrl.yaml
 rename Documentation/devicetree/bindings/pinctrl/{ralink,rt2880-pinmux.yaml => ralink,mt7621-pinctrl.yaml} (63%)
 create mode 100644 Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinctrl.yaml
 create mode 100644 Documentation/devicetree/bindings/pinctrl/ralink,rt305x-pinctrl.yaml
 create mode 100644 Documentation/devicetree/bindings/pinctrl/ralink,rt3883-pinctrl.yaml
 create mode 100644 drivers/pinctrl/ralink/pinctrl-ralink.c
 rename drivers/pinctrl/ralink/{pinmux.h => pinctrl-ralink.h} (75%)
 delete mode 100644 drivers/pinctrl/ralink/pinctrl-rt288x.c





More information about the linux-arm-kernel mailing list