Re: [PATCH 0/8] various imx6ul DT fixes

[Shawn Guo <shawnguo@kernel.org>]

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
>