| Message ID | 4ee2e3e1e0e174782bd4d2af44bb6150d24af551.1728045620.git.geert+renesas@glider.be (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | renesas: dts: Use interrupts-extended | expand |
On 2024-10-04 14:52:59 +0200, Geert Uytterhoeven wrote: > Use the more concise interrupts-extended property to fully describe the > interrupts. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > arch/arm64/boot/dts/renesas/ebisu.dtsi | 5 ++--- > .../dts/renesas/r8a77970-eagle-function-expansion.dtso | 7 +++---- > arch/arm64/boot/dts/renesas/salvator-common.dtsi | 5 ++--- > 3 files changed, 7 insertions(+), 10 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi > index b4e0347d3617a83e..ab8283656660059a 100644 > --- a/arch/arm64/boot/dts/renesas/ebisu.dtsi > +++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi > @@ -434,10 +434,9 @@ video-receiver@70 { > compatible = "adi,adv7482"; > reg = <0x70>; > > - interrupt-parent = <&gpio0>; > + interrupts-extended = <&gpio0 7 IRQ_TYPE_LEVEL_LOW>, > + <&gpio0 17 IRQ_TYPE_LEVEL_LOW>; > interrupt-names = "intrq1", "intrq2"; > - interrupts = <7 IRQ_TYPE_LEVEL_LOW>, > - <17 IRQ_TYPE_LEVEL_LOW>; > > ports { > #address-cells = <1>; > diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso > index 3aa243c5f04c8022..9450d8ac94cbe977 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso > +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso > @@ -82,8 +82,7 @@ hdmi-decoder@4c { > compatible = "adi,adv7612"; > reg = <0x4c>, <0x50>, <0x52>, <0x54>, <0x56>, <0x58>; > reg-names = "main", "afe", "rep", "edid", "hdmi", "cp"; > - interrupt-parent = <&gpio3>; > - interrupts = <2 IRQ_TYPE_LEVEL_LOW>; > + interrupts-extended = <&gpio3 2 IRQ_TYPE_LEVEL_LOW>; > default-input = <0>; > > ports { > @@ -114,8 +113,8 @@ video-receiver@70 { > 0x60 0x61 0x62 0x63 0x64 0x65>; > reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater", > "infoframe", "cbus", "cec", "sdp", "txa", "txb" ; > - interrupt-parent = <&gpio3>; > - interrupts = <03 IRQ_TYPE_LEVEL_LOW>, <04 IRQ_TYPE_LEVEL_LOW>; > + interrupts-extended = <&gpio3 3 IRQ_TYPE_LEVEL_LOW>, > + <&gpio3 4 IRQ_TYPE_LEVEL_LOW>; > interrupt-names = "intrq1", "intrq2"; > > ports { > diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > index 08f4e35c414ebbcf..269cda6fae21de70 100644 > --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi > +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > @@ -544,10 +544,9 @@ video-receiver@70 { > reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater", > "infoframe", "cbus", "cec", "sdp", "txa", "txb" ; > > - interrupt-parent = <&gpio6>; > + interrupts-extended = <&gpio6 30 IRQ_TYPE_LEVEL_LOW>, > + <&gpio6 31 IRQ_TYPE_LEVEL_LOW>; > interrupt-names = "intrq1", "intrq2"; > - interrupts = <30 IRQ_TYPE_LEVEL_LOW>, > - <31 IRQ_TYPE_LEVEL_LOW>; > > ports { > #address-cells = <1>; > -- > 2.34.1 > >
On Fri, Oct 4, 2024 at 2:27 PM Geert Uytterhoeven <geert+renesas@glider.be> wrote: > > Use the more concise interrupts-extended property to fully describe the > interrupts. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > --- > arch/arm64/boot/dts/renesas/ebisu.dtsi | 5 ++--- > .../dts/renesas/r8a77970-eagle-function-expansion.dtso | 7 +++---- > arch/arm64/boot/dts/renesas/salvator-common.dtsi | 5 ++--- > 3 files changed, 7 insertions(+), 10 deletions(-) > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Cheers, Prabhakar > diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi > index b4e0347d3617a83e..ab8283656660059a 100644 > --- a/arch/arm64/boot/dts/renesas/ebisu.dtsi > +++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi > @@ -434,10 +434,9 @@ video-receiver@70 { > compatible = "adi,adv7482"; > reg = <0x70>; > > - interrupt-parent = <&gpio0>; > + interrupts-extended = <&gpio0 7 IRQ_TYPE_LEVEL_LOW>, > + <&gpio0 17 IRQ_TYPE_LEVEL_LOW>; > interrupt-names = "intrq1", "intrq2"; > - interrupts = <7 IRQ_TYPE_LEVEL_LOW>, > - <17 IRQ_TYPE_LEVEL_LOW>; > > ports { > #address-cells = <1>; > diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso > index 3aa243c5f04c8022..9450d8ac94cbe977 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso > +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso > @@ -82,8 +82,7 @@ hdmi-decoder@4c { > compatible = "adi,adv7612"; > reg = <0x4c>, <0x50>, <0x52>, <0x54>, <0x56>, <0x58>; > reg-names = "main", "afe", "rep", "edid", "hdmi", "cp"; > - interrupt-parent = <&gpio3>; > - interrupts = <2 IRQ_TYPE_LEVEL_LOW>; > + interrupts-extended = <&gpio3 2 IRQ_TYPE_LEVEL_LOW>; > default-input = <0>; > > ports { > @@ -114,8 +113,8 @@ video-receiver@70 { > 0x60 0x61 0x62 0x63 0x64 0x65>; > reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater", > "infoframe", "cbus", "cec", "sdp", "txa", "txb" ; > - interrupt-parent = <&gpio3>; > - interrupts = <03 IRQ_TYPE_LEVEL_LOW>, <04 IRQ_TYPE_LEVEL_LOW>; > + interrupts-extended = <&gpio3 3 IRQ_TYPE_LEVEL_LOW>, > + <&gpio3 4 IRQ_TYPE_LEVEL_LOW>; > interrupt-names = "intrq1", "intrq2"; > > ports { > diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > index 08f4e35c414ebbcf..269cda6fae21de70 100644 > --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi > +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi > @@ -544,10 +544,9 @@ video-receiver@70 { > reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater", > "infoframe", "cbus", "cec", "sdp", "txa", "txb" ; > > - interrupt-parent = <&gpio6>; > + interrupts-extended = <&gpio6 30 IRQ_TYPE_LEVEL_LOW>, > + <&gpio6 31 IRQ_TYPE_LEVEL_LOW>; > interrupt-names = "intrq1", "intrq2"; > - interrupts = <30 IRQ_TYPE_LEVEL_LOW>, > - <31 IRQ_TYPE_LEVEL_LOW>; > > ports { > #address-cells = <1>; > -- > 2.34.1 > >
diff --git a/arch/arm64/boot/dts/renesas/ebisu.dtsi b/arch/arm64/boot/dts/renesas/ebisu.dtsi index b4e0347d3617a83e..ab8283656660059a 100644 --- a/arch/arm64/boot/dts/renesas/ebisu.dtsi +++ b/arch/arm64/boot/dts/renesas/ebisu.dtsi @@ -434,10 +434,9 @@ video-receiver@70 { compatible = "adi,adv7482"; reg = <0x70>; - interrupt-parent = <&gpio0>; + interrupts-extended = <&gpio0 7 IRQ_TYPE_LEVEL_LOW>, + <&gpio0 17 IRQ_TYPE_LEVEL_LOW>; interrupt-names = "intrq1", "intrq2"; - interrupts = <7 IRQ_TYPE_LEVEL_LOW>, - <17 IRQ_TYPE_LEVEL_LOW>; ports { #address-cells = <1>; diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso index 3aa243c5f04c8022..9450d8ac94cbe977 100644 --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle-function-expansion.dtso @@ -82,8 +82,7 @@ hdmi-decoder@4c { compatible = "adi,adv7612"; reg = <0x4c>, <0x50>, <0x52>, <0x54>, <0x56>, <0x58>; reg-names = "main", "afe", "rep", "edid", "hdmi", "cp"; - interrupt-parent = <&gpio3>; - interrupts = <2 IRQ_TYPE_LEVEL_LOW>; + interrupts-extended = <&gpio3 2 IRQ_TYPE_LEVEL_LOW>; default-input = <0>; ports { @@ -114,8 +113,8 @@ video-receiver@70 { 0x60 0x61 0x62 0x63 0x64 0x65>; reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater", "infoframe", "cbus", "cec", "sdp", "txa", "txb" ; - interrupt-parent = <&gpio3>; - interrupts = <03 IRQ_TYPE_LEVEL_LOW>, <04 IRQ_TYPE_LEVEL_LOW>; + interrupts-extended = <&gpio3 3 IRQ_TYPE_LEVEL_LOW>, + <&gpio3 4 IRQ_TYPE_LEVEL_LOW>; interrupt-names = "intrq1", "intrq2"; ports { diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi index 08f4e35c414ebbcf..269cda6fae21de70 100644 --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi @@ -544,10 +544,9 @@ video-receiver@70 { reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater", "infoframe", "cbus", "cec", "sdp", "txa", "txb" ; - interrupt-parent = <&gpio6>; + interrupts-extended = <&gpio6 30 IRQ_TYPE_LEVEL_LOW>, + <&gpio6 31 IRQ_TYPE_LEVEL_LOW>; interrupt-names = "intrq1", "intrq2"; - interrupts = <30 IRQ_TYPE_LEVEL_LOW>, - <31 IRQ_TYPE_LEVEL_LOW>; ports { #address-cells = <1>;
Use the more concise interrupts-extended property to fully describe the interrupts. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- arch/arm64/boot/dts/renesas/ebisu.dtsi | 5 ++--- .../dts/renesas/r8a77970-eagle-function-expansion.dtso | 7 +++---- arch/arm64/boot/dts/renesas/salvator-common.dtsi | 5 ++--- 3 files changed, 7 insertions(+), 10 deletions(-)