mbox series

[v2,0/2] armada-370 irqchip DT changes

Message ID 20240624145355.8034-1-kabel@kernel.org
Headers show
Series armada-370 irqchip DT changes | expand

Message

Marek Behún June 24, 2024, 2:53 p.m. UTC
Hi Gregory, Thomas et al.

this small series coverts the marvell,mpic irqchip binding to YAML.
The conversion patch is preceded by a fix of 4 device-trees, removing
the #size-cells property from the marvell,mpic node.

v1 at:
  https://patchwork.kernel.org/project/linux-soc/list/?seriestreba%20za=864184

Changes since v1:
- #interrupt-cells property is also required

Marek Behún (2):
  ARM: dts: armada-{370-xp,375,38x,39x}: Drop #size-cells from mpic node
  dt-bindings: interrupt-controller: convert marvell,mpic binding to
    YAML

 .../marvell,armada-370-xp-mpic.txt            | 38 -----------
 .../interrupt-controller/marvell,mpic.yaml    | 63 +++++++++++++++++++
 MAINTAINERS                                   |  1 +
 arch/arm/boot/dts/marvell/armada-370-xp.dtsi  |  1 -
 arch/arm/boot/dts/marvell/armada-375.dtsi     |  1 -
 arch/arm/boot/dts/marvell/armada-38x.dtsi     |  1 -
 arch/arm/boot/dts/marvell/armada-39x.dtsi     |  1 -
 7 files changed, 64 insertions(+), 42 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,armada-370-xp-mpic.txt
 create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,mpic.yaml

Comments

Gregory CLEMENT July 5, 2024, 12:41 p.m. UTC | #1
Marek Behún <kabel@kernel.org> writes:

> The marvell,mpic interrupt controller has no children nodes. Drop the
>
> Signed-off-by: Marek Behún <kabel@kernel.org>


Applied on mvebu/dt

Thanks,

Gregory


> ---
>  arch/arm/boot/dts/marvell/armada-370-xp.dtsi | 1 -
>  arch/arm/boot/dts/marvell/armada-375.dtsi    | 1 -
>  arch/arm/boot/dts/marvell/armada-38x.dtsi    | 1 -
>  arch/arm/boot/dts/marvell/armada-39x.dtsi    | 1 -
>  4 files changed, 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/marvell/armada-370-xp.dtsi b/arch/arm/boot/dts/marvell/armada-370-xp.dtsi
> index 0b8c2a64b36f..954c891e5aee 100644
> --- a/arch/arm/boot/dts/marvell/armada-370-xp.dtsi
> +++ b/arch/arm/boot/dts/marvell/armada-370-xp.dtsi
> @@ -168,7 +168,6 @@ mbusc: mbus-controller@20000 {
>  			mpic: interrupt-controller@20a00 {
>  				compatible = "marvell,mpic";
>  				#interrupt-cells = <1>;
> -				#size-cells = <1>;
>  				interrupt-controller;
>  				msi-controller;
>  			};
> diff --git a/arch/arm/boot/dts/marvell/armada-375.dtsi b/arch/arm/boot/dts/marvell/armada-375.dtsi
> index ddc49547d786..99778b4b7e7b 100644
> --- a/arch/arm/boot/dts/marvell/armada-375.dtsi
> +++ b/arch/arm/boot/dts/marvell/armada-375.dtsi
> @@ -376,7 +376,6 @@ mpic: interrupt-controller@20a00 {
>  				compatible = "marvell,mpic";
>  				reg = <0x20a00 0x2d0>, <0x21070 0x58>;
>  				#interrupt-cells = <1>;
> -				#size-cells = <1>;
>  				interrupt-controller;
>  				msi-controller;
>  				interrupts = <GIC_PPI 15 IRQ_TYPE_LEVEL_HIGH>;
> diff --git a/arch/arm/boot/dts/marvell/armada-38x.dtsi b/arch/arm/boot/dts/marvell/armada-38x.dtsi
> index 446861b6b17b..1181b13deabc 100644
> --- a/arch/arm/boot/dts/marvell/armada-38x.dtsi
> +++ b/arch/arm/boot/dts/marvell/armada-38x.dtsi
> @@ -408,7 +408,6 @@ mpic: interrupt-controller@20a00 {
>  				compatible = "marvell,mpic";
>  				reg = <0x20a00 0x2d0>, <0x21070 0x58>;
>  				#interrupt-cells = <1>;
> -				#size-cells = <1>;
>  				interrupt-controller;
>  				msi-controller;
>  				interrupts = <GIC_PPI 15 IRQ_TYPE_LEVEL_HIGH>;
> diff --git a/arch/arm/boot/dts/marvell/armada-39x.dtsi b/arch/arm/boot/dts/marvell/armada-39x.dtsi
> index 9d1cac49c022..6d05835efb42 100644
> --- a/arch/arm/boot/dts/marvell/armada-39x.dtsi
> +++ b/arch/arm/boot/dts/marvell/armada-39x.dtsi
> @@ -268,7 +268,6 @@ mpic: interrupt-controller@20a00 {
>  				compatible = "marvell,mpic";
>  				reg = <0x20a00 0x2d0>, <0x21070 0x58>;
>  				#interrupt-cells = <1>;
> -				#size-cells = <1>;
>  				interrupt-controller;
>  				msi-controller;
>  				interrupts = <GIC_PPI 15 IRQ_TYPE_LEVEL_HIGH>;
> -- 
> 2.44.2