mbox series

[v2,0/2] arm64: dts: imx8m: Add pinctrl config definitions

Message ID 20250218-pinctrl_defines-v2-0-c554cad0e1d2@gocontroll.com (mailing list archive)
Headers show
Series arm64: dts: imx8m: Add pinctrl config definitions | expand

Message

Maud Spierings via B4 Relay Feb. 18, 2025, 8:39 a.m. UTC
Up untill now the extra function mux (pull resistor, etc) has been
defined as a raw hex value, this is difficult to interpret. Add
definitions for these parts of the pinctrl registers to allow for more
readable pinctrl definitions in devictree files.

It should also be done for the mn/mq but I currently do not have the
reference manuals for these to verify these fields, though I can only
assume that they would be the same.

Signed-off-by: Maud Spierings <maudspierings@gocontroll.com>
---
Changes in v2:
- Changed mystery bit to SION
- Link to v1: https://lore.kernel.org/r/20250214-pinctrl_defines-v1-0-fbc86a42ef3c@gocontroll.com

---
Maud Spierings (2):
      arm64: dts: imx8mm: Add pinctrl config definitions
      arm64: dts: imx8mp: Add pinctrl config definitions

 arch/arm64/boot/dts/freescale/imx8mm-pinfunc.h | 27 ++++++++++++++++++++++++++
 arch/arm64/boot/dts/freescale/imx8mp-pinfunc.h | 27 ++++++++++++++++++++++++++
 2 files changed, 54 insertions(+)
---
base-commit: 0ae0fa3bf0b44c8611d114a9f69985bf451010c3
change-id: 20250214-pinctrl_defines-237e7db61c6a

Best regards,

Comments

Maud Spierings | GOcontroll Feb. 24, 2025, 1:50 p.m. UTC | #1
From: Maud Spierings via B4 Relay <devnull+maudspierings.gocontroll.com@kernel.org>
Sent: Tuesday, February 18, 2025 9:39 AM
 
>Up untill now the extra function mux (pull resistor, etc) has been
>defined as a raw hex value, this is difficult to interpret. Add
>definitions for these parts of the pinctrl registers to allow for more
>readable pinctrl definitions in devictree files.
>
>It should also be done for the mn/mq but I currently do not have the
>reference manuals for these to verify these fields, though I can only
>assume that they would be the same.
>
>Signed-off-by: Maud Spierings <maudspierings@gocontroll.com>
>---
>Changes in v2:
>- Changed mystery bit to SION
>- Link to v1: https://lore.kernel.org/r/20250214-pinctrl_defines-v1-0-fbc86a42ef3c@gocontroll.com
>
>---
>Maud Spierings (2):
>      arm64: dts: imx8mm: Add pinctrl config definitions
>      arm64: dts: imx8mp: Add pinctrl config definitions
>
> arch/arm64/boot/dts/freescale/imx8mm-pinfunc.h | 27 ++++++++++++++++++++++++++
> arch/arm64/boot/dts/freescale/imx8mp-pinfunc.h | 27 ++++++++++++++++++++++++++
> 2 files changed, 54 insertions(+)
>---
>base-commit: 0ae0fa3bf0b44c8611d114a9f69985bf451010c3
>change-id: 20250214-pinctrl_defines-237e7db61c6a
>
>Best regards,
>--
>Maud Spierings <maudspierings@gocontroll.com>

This series has partially been integrated in another patchset (the imx8mp
one). The other one aimed at the imx8mm will also be sent later with a
patchset that also adds users for it.

This patchset should therefore not be merged, thank you all for your
feedback so far.

Kind regards,
Maud