[PATCH 00/14] arm64: dts: imx8mp: correct pad settings

Peng Fan (OSS) posted 14 patches 3 years, 10 months ago
arch/arm64/boot/dts/freescale/imx8mp-evk.dts  |  88 ++++++-------
.../freescale/imx8mp-icore-mx8mp-edimm2.2.dts |  40 +++---
.../freescale/imx8mp-phyboard-pollux-rdk.dts  |  48 ++++----
.../dts/freescale/imx8mp-venice-gw74xx.dts    | 116 +++++++++---------
4 files changed, 146 insertions(+), 146 deletions(-)
[PATCH 00/14] arm64: dts: imx8mp: correct pad settings
Posted by Peng Fan (OSS) 3 years, 10 months ago
From: Peng Fan <peng.fan@nxp.com>

i.MX8MP iomux pad BIT3 and BIT0 are reserved bits. Writing 1 to the
reserved bit will be ignored and reading will still return 0. Although
function not broken with reserved bits set, we should not set reserved
bits.

Peng Fan (13):
  arm64: dts: imx8mp-evk: correct mmc pad settings
  arm64: dts: imx8mp-evk: correct gpio-led pad settings
  arm64: dts: imx8mp-evk: correct vbus pad settings
  arm64: dts: imx8mp-evk: correct eqos pad settings
  arm64: dts: imx8mp-evk: correct vbus pad settings
  arm64: dts: imx8mp-evk: correct I2C5 pad settings
  arm64: dts: imx8mp-evk: correct I2C1 pad settings
  arm64: dts: imx8mp-evk: correct I2C3 pad settings
  arm64: dts: imx8mp-venice-gw74xx: correct pad settings
  arm64: dts: imx8mp-phyboard-pollux-rdk: correct uart pad settings
  arm64: dts: imx8mp-phyboard-pollux-rdk: correct eqos pad settings
  arm64: dts: imx8mp-phyboard-pollux-rdk: correct i2c2 & mmc settings
  arm64: dts: imx8mp-icore-mx8mp-edim2.2: correct pad settings

Sherry Sun (1):
  arm64: dts: imx8mp-evk: correct the uart2 pinctl value

 arch/arm64/boot/dts/freescale/imx8mp-evk.dts  |  88 ++++++-------
 .../freescale/imx8mp-icore-mx8mp-edimm2.2.dts |  40 +++---
 .../freescale/imx8mp-phyboard-pollux-rdk.dts  |  48 ++++----
 .../dts/freescale/imx8mp-venice-gw74xx.dts    | 116 +++++++++---------
 4 files changed, 146 insertions(+), 146 deletions(-)

-- 
2.25.1
Re: [PATCH 00/14] arm64: dts: imx8mp: correct pad settings
Posted by Shawn Guo 3 years, 9 months ago
On Wed, Jun 22, 2022 at 02:13:56PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> i.MX8MP iomux pad BIT3 and BIT0 are reserved bits. Writing 1 to the
> reserved bit will be ignored and reading will still return 0. Although
> function not broken with reserved bits set, we should not set reserved
> bits.
> 
> Peng Fan (13):
>   arm64: dts: imx8mp-evk: correct mmc pad settings
>   arm64: dts: imx8mp-evk: correct gpio-led pad settings
>   arm64: dts: imx8mp-evk: correct vbus pad settings
>   arm64: dts: imx8mp-evk: correct eqos pad settings
>   arm64: dts: imx8mp-evk: correct vbus pad settings
>   arm64: dts: imx8mp-evk: correct I2C5 pad settings
>   arm64: dts: imx8mp-evk: correct I2C1 pad settings
>   arm64: dts: imx8mp-evk: correct I2C3 pad settings
>   arm64: dts: imx8mp-venice-gw74xx: correct pad settings
>   arm64: dts: imx8mp-phyboard-pollux-rdk: correct uart pad settings
>   arm64: dts: imx8mp-phyboard-pollux-rdk: correct eqos pad settings
>   arm64: dts: imx8mp-phyboard-pollux-rdk: correct i2c2 & mmc settings
>   arm64: dts: imx8mp-icore-mx8mp-edim2.2: correct pad settings
> 
> Sherry Sun (1):
>   arm64: dts: imx8mp-evk: correct the uart2 pinctl value

Applied all, thanks!
Re: [PATCH 00/14] arm64: dts: imx8mp: correct pad settings
Posted by Uwe Kleine-König 3 years, 10 months ago
Hello,

On Wed, Jun 22, 2022 at 02:13:56PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> i.MX8MP iomux pad BIT3 and BIT0 are reserved bits. Writing 1 to the
> reserved bit will be ignored and reading will still return 0. Although
> function not broken with reserved bits set, we should not set reserved
> bits.

I wonder how you found these. Some time ago I wrote a tool for such
issues[1]. Currently it only supports i.MX25, i.MX6DL and i.MX6Q, but
extending it for the other SoCs should be only some industious effort.

Best regards
Uwe

[1] https://git.pengutronix.de/cgit/tools/dt-utils/tree/src/dtblint-imx-pinmux.c


-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |
RE: [PATCH 00/14] arm64: dts: imx8mp: correct pad settings
Posted by Peng Fan 3 years, 10 months ago
> Subject: Re: [PATCH 00/14] arm64: dts: imx8mp: correct pad settings
> 
> Hello,
> 
> On Wed, Jun 22, 2022 at 02:13:56PM +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > i.MX8MP iomux pad BIT3 and BIT0 are reserved bits. Writing 1 to the
> > reserved bit will be ignored and reading will still return 0. Although
> > function not broken with reserved bits set, we should not set reserved
> > bits.
> 
> I wonder how you found these. Some time ago I wrote a tool for such issues[1].
> Currently it only supports i.MX25, i.MX6DL and i.MX6Q, but extending it for
> the other SoCs should be only some industious effort.

We have customer has question about the dts pad settings in U-Boot. Since the
dts in U-Boot was synced from Linux kernel tree, and I found
the current settings in kernel dts not match with latest reference manual.

Regards,
Peng.

> 
> Best regards
> Uwe
> 
> [1]
> https://git.pengutronix.de/cgit/tools/dt-utils/tree/src/dtblint-imx-pinmux.c
> 
> 
> --
> Pengutronix e.K.                           | Uwe Kleine-König
> |
> Industrial Linux Solutions                 | https://www.pengutronix.de/
> |
Re: [PATCH 00/14] arm64: dts: imx8mp: correct pad settings
Posted by Rasmus Villemoes 3 years, 10 months ago
On 22/06/2022 08.13, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> i.MX8MP iomux pad BIT3 and BIT0 are reserved bits. Writing 1 to the
> reserved bit will be ignored and reading will still return 0. Although
> function not broken with reserved bits set, we should not set reserved
> bits.

Thank you, these have really been bugging my while trying to bring up an
imx8mp-based board and adding the right pinmux settings - not knowing
whether there was some undocumented effect from including one of those
bits has led me astray more than once.

One question: E.g. in patch 11, you change the setting from 0x49 to
0x40, which is of course exactly what the patch description says. But
when bit 8 (PE) is not set, is there any effect from either setting of
bit 6 (PUE)? Not that I suggest changing to 0x00, but I'm just curious.

For the series:

Reviewed-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Re: [PATCH 00/14] arm64: dts: imx8mp: correct pad settings
Posted by Peng Fan (OSS) 3 years, 9 months ago

在 2022/6/22 14:31, Rasmus Villemoes 写道:
> On 22/06/2022 08.13, Peng Fan (OSS) wrote:
>> From: Peng Fan <peng.fan@nxp.com>
>>
>> i.MX8MP iomux pad BIT3 and BIT0 are reserved bits. Writing 1 to the
>> reserved bit will be ignored and reading will still return 0. Although
>> function not broken with reserved bits set, we should not set reserved
>> bits.
> Thank you, these have really been bugging my while trying to bring up an
> imx8mp-based board and adding the right pinmux settings - not knowing
> whether there was some undocumented effect from including one of those
> bits has led me astray more than once.
>
> One question: E.g. in patch 11, you change the setting from 0x49 to
> 0x40, which is of course exactly what the patch description says. But
> when bit 8 (PE) is not set, is there any effect from either setting of
> bit 6 (PUE)? Not that I suggest changing to 0x00, but I'm just curious.

Set PE/PUE, it depends on board design or whether need it. Because the 
previous settings not set them, I not set them also, just
not touch the reserved bits.

THanks,
Peng.

>
> For the series:
>
> Reviewed-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>