mbox series

[V3,0/6] ARM: dts: imx6ull: Fix dtbs_check warnings

Message ID 20230414091947.7831-1-stefan.wahren@i2se.com
Headers show
Series ARM: dts: imx6ull: Fix dtbs_check warnings | expand

Message

Stefan Wahren April 14, 2023, 9:19 a.m. UTC 
This series tries to address some dtbs_check warnings on i.MX6ULL.

Changes in V3:
- add Krzysztof's Reviewed-Bys
- fix indentation in Patch 6 found by Krzysztof Kozlowski

Changes in V2:
- new patch to fix fsl-imx-uart warnings 
- fixed GPC typo found by Fabio Estevam
- keep enum in bindings as suggested by Krzysztof Kozlowski
- make imx6ul GPT compatible to imx6sx

Stefan Wahren (6):
  dt-bindings: serial: fsl-imx-uart: add missing properties
  dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible
  dt-bindings: imx-thermal: add imx6sll and imx6ul compatible
  dt-bindings: imxgpt: add imx6ul compatible
  ARM: dts: imx: Adjust dma-apbh node name
  ARM: dts: imx6ul: Add clock and PGC node to GPC

 .../devicetree/bindings/crypto/fsl-dcp.yaml   | 12 ++++++---
 .../bindings/serial/fsl-imx-uart.yaml         | 25 +++++++++++++++++++
 .../bindings/thermal/imx-thermal.yaml         | 14 ++++++++---
 .../devicetree/bindings/timer/fsl,imxgpt.yaml |  3 +++
 arch/arm/boot/dts/imx23.dtsi                  |  2 +-
 arch/arm/boot/dts/imx28.dtsi                  |  2 +-
 arch/arm/boot/dts/imx6qdl.dtsi                |  2 +-
 arch/arm/boot/dts/imx6sx.dtsi                 |  2 +-
 arch/arm/boot/dts/imx6ul.dtsi                 | 14 ++++++++++-
 arch/arm/boot/dts/imx7s.dtsi                  |  2 +-
 10 files changed, 65 insertions(+), 13 deletions(-)

Comments

Shawn Guo May 14, 2023, 2:30 a.m. UTC | #1 
On Fri, Apr 14, 2023 at 11:19:46AM +0200, Stefan Wahren wrote:
> Currently the dtbs_check generates warnings like this:
> 
> $nodename:0: 'dma-apbh@110000' does not match '^dma-controller(@.*)?$'
> 
> So fix all affected dma-apbh node names.
> 
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>

Applied, thanks!
Shawn Guo May 14, 2023, 2:32 a.m. UTC | #2 
On Fri, Apr 14, 2023 at 11:19:47AM +0200, Stefan Wahren wrote:
> According to fsl,imx-gpc.yaml the General Power Control requires
> a ipg clock and a Power Gating Control node. So add them to fix
> the dtbs_check warnings on i.MX6UL boards:
> 
> gpc@20dc000: 'clocks' is a required property
> gpc@20dc000: 'clock-names' is a required property
> gpc@20dc000: 'pgc' is a required property
> 
> Suggested-by: Jacky Bai <ping.bai@nxp.com>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>

Applied, thanks!
Stefan Wahren May 19, 2023, 8:03 a.m. UTC | #3 
Hi,

Am 14.04.23 um 11:19 schrieb Stefan Wahren:
> This series tries to address some dtbs_check warnings on i.MX6ULL.
> 
> Changes in V3:
> - add Krzysztof's Reviewed-Bys
> - fix indentation in Patch 6 found by Krzysztof Kozlowski
> 
> Changes in V2:
> - new patch to fix fsl-imx-uart warnings
> - fixed GPC typo found by Fabio Estevam
> - keep enum in bindings as suggested by Krzysztof Kozlowski
> - make imx6ul GPT compatible to imx6sx
> 
> Stefan Wahren (6):
>    dt-bindings: serial: fsl-imx-uart: add missing properties
>    dt-bindings: crypto: fsl-dcp: add imx6sl and imx6ull compatible
>    dt-bindings: imx-thermal: add imx6sll and imx6ul compatible
>    dt-bindings: imxgpt: add imx6ul compatible
>    ARM: dts: imx: Adjust dma-apbh node name
>    ARM: dts: imx6ul: Add clock and PGC node to GPC
> 

currently patch 3, 5 and 6 has been applied. Should i resend the rest of 
the series?