Re: [PATCH] arm64: dts: imx8mm-venice-gw790: Pass GSC address/size-cells

Tim Harvey
2023-09-26T15:28:17Z

On Tue, Sep 26, 2023 at 5:44 AM Fabio Estevam <festevam@gmail.com> wrote:
>
> From: Fabio Estevam <festevam@denx.de>
>
> Per gateworks-gsc.yaml, #address-cells and #size-cells are mandatory
> properties.
>
> Pass them to fix the following schema warning:
>
> imx8mm-venice-gw7903.dtb: gsc@20: '#address-cells' is a required property
> from schema $id: http://devicetree.org/schemas/mfd/gateworks-gsc.yaml#
> imx8mm-venice-gw7903.dtb: gsc@20: '#size-cells' is a required property
> from schema $id: http://devicetree.org/schemas/mfd/gateworks-gsc.yaml#
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts | 2 ++
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts | 2 ++
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts | 2 ++
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts | 2 ++
> arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts | 2 ++
> 5 files changed, 10 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> index f45c22d8fb50..fc62359c130b 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
> @@ -357,6 +357,8 @@ gsc: gsc@20 {
> interrupts = <16 IRQ_TYPE_EDGE_FALLING>;
> interrupt-controller;
> #interrupt-cells = <1>;
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> adc {
> compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
> index 6398f509efa0..06a394a41d7c 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7902.dts
> @@ -314,6 +314,8 @@ gsc: gsc@20 {
> interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
> interrupt-controller;
> #interrupt-cells = <1>;
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> adc {
> compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
> index 45470160f98f..db1737bf637d 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7903.dts
> @@ -280,6 +280,8 @@ gsc: gsc@20 {
> interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
> interrupt-controller;
> #interrupt-cells = <1>;
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> adc {
> compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
> index ef951bc9f0dd..05489a31e7fd 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7904.dts
> @@ -330,6 +330,8 @@ gsc: gsc@20 {
> interrupts = <26 IRQ_TYPE_EDGE_FALLING>;
> interrupt-controller;
> #interrupt-cells = <1>;
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> adc {
> compatible = "gw,gsc-adc";
> diff --git a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
> index 72004ab6bda5..0b1fa04f1d67 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mn-venice-gw7902.dts
> @@ -312,6 +312,8 @@ gsc: gsc@20 {
> interrupts = <6 IRQ_TYPE_EDGE_FALLING>;
> interrupt-controller;
> #interrupt-cells = <1>;
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> adc {
> compatible = "gw,gsc-adc";
> --
> 2.34.1
>

Thanks Fabio,

Acked-by: Tim Harvey <tharvey@gateworks.com>

Best Regards,

Tim