[0/8] various imx6ul DT fixes

Message

Alexander Stein Jan. 25, 2022, 1:50 p.m. UTC

This patch set fixes several, but not all, dt_binding_check warnings. Some are
missing properties or their format, others address issues regarding to
compatible names. I only touched those where compatibles are stated in the yaml
bindings.
There are several imx6ul compatible names used by imx6ul.dtsi, but which are
not listed in the bindings. Namely:
* "fsl,imx6ul-gpt", "fsl,imx6sx-gpt"
* "fsl,imx6ul-tempmon", "fsl,imx6sx-tempmon"
* "fsl,imx6ul-gpc", "fsl,imx6q-gpc"
* "fsl,imx6ul-usdhc", "fsl,imx6sx-usdhc"

Despite GPC, the others are apperently compatible to imx6sx. I'm not sure how
to fix the DTB check warning. Add the imx6ul compatible to bindings and drivers?
Or change the .dtsi to use only the imx6sx compatible.

The GPC on imx6ul apparently does only support power gating for ARM CPU, so
this would require actually a new compatible which supports only 1 domain.

Alexander Stein (8):
  ARM: dts: imx6ul: add missing properties for sram
  ARM: dts: imx6ul: change operating-points to uint32-matrix
  ARM: dts: imx6ul: fix keypad compatible
  ARM: dts: imx6ul: fix adc node compatible
  ARM: dts: imx6ul: remove unsupported adc property
  ARM: dts: imx6ul: fix csi node compatible
  ARM: dts: imx6ul: fix lcdif node compatible
  ARM: dts: imx6ul: fix qspi node compatible

 arch/arm/boot/dts/imx6ul.dtsi | 36 +++++++++++++++++------------------
 1 file changed, 18 insertions(+), 18 deletions(-)

Comments

Shawn Guo Feb. 11, 2022, 3:37 a.m. UTC | #1

On Tue, Jan 25, 2022 at 02:50:06PM +0100, Alexander Stein wrote:
> This patch set fixes several, but not all, dt_binding_check warnings. Some are
> missing properties or their format, others address issues regarding to
> compatible names. I only touched those where compatibles are stated in the yaml
> bindings.
> There are several imx6ul compatible names used by imx6ul.dtsi, but which are
> not listed in the bindings. Namely:
> * "fsl,imx6ul-gpt", "fsl,imx6sx-gpt"
> * "fsl,imx6ul-tempmon", "fsl,imx6sx-tempmon"
> * "fsl,imx6ul-gpc", "fsl,imx6q-gpc"
> * "fsl,imx6ul-usdhc", "fsl,imx6sx-usdhc"
> 
> Despite GPC, the others are apperently compatible to imx6sx. I'm not sure how
> to fix the DTB check warning. Add the imx6ul compatible to bindings and drivers?

Add the compatible to bindings, I would say.

Shawn

> Or change the .dtsi to use only the imx6sx compatible.
> 
> The GPC on imx6ul apparently does only support power gating for ARM CPU, so
> this would require actually a new compatible which supports only 1 domain.
> 
> Alexander Stein (8):
>   ARM: dts: imx6ul: add missing properties for sram
>   ARM: dts: imx6ul: change operating-points to uint32-matrix
>   ARM: dts: imx6ul: fix keypad compatible
>   ARM: dts: imx6ul: fix adc node compatible
>   ARM: dts: imx6ul: remove unsupported adc property
>   ARM: dts: imx6ul: fix csi node compatible
>   ARM: dts: imx6ul: fix lcdif node compatible
>   ARM: dts: imx6ul: fix qspi node compatible
> 
>  arch/arm/boot/dts/imx6ul.dtsi | 36 +++++++++++++++++------------------
>  1 file changed, 18 insertions(+), 18 deletions(-)
> 
> -- 
> 2.25.1
>