All of lore.kernel.org
 help / color / mirror / Atom feed
From: Biju Das <biju.das.jz@bp.renesas.com>
To: Geert Uytterhoeven <geert+renesas@glider.be>,
	Wolfram Sang <wsa+renesas@sang-engineering.com>,
	Rob Herring <robh+dt@kernel.org>,
	Chris Brandt <Chris.Brandt@renesas.com>
Cc: "linux-renesas-soc@vger.kernel.org" 
	<linux-renesas-soc@vger.kernel.org>,
	"linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: RE: [PATCH/RFC 3/4] arm64: dts: renesas: r9a07g044: Add I2C interrupt-names
Date: Wed, 14 Jul 2021 17:21:12 +0000	[thread overview]
Message-ID: <OS0PR01MB5922BCB0FA0E95A2C3A83C4386139@OS0PR01MB5922.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <d35ba11bd705e3c728f94ff0414ac6ae1156244f.1626267422.git.geert+renesas@glider.be>

Hi Geert,

Thanks for the patch.

> Subject: [PATCH/RFC 3/4] arm64: dts: renesas: r9a07g044: Add I2C
> interrupt-names
> 
> Add "interrupt-names" properties to the I2C device nodes, to make it
> easier to review the interrupt mappings.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com>

> ---
>  arch/arm64/boot/dts/renesas/r9a07g044.dtsi | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> index 9a7489dc70d10882..0dd181739fdfc0f6 100644
> --- a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> @@ -102,6 +102,8 @@ i2c0: i2c@10058000 {
>  				     <GIC_SPI 351 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 354 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 355 IRQ_TYPE_LEVEL_HIGH>;
> +			interrupt-names = "tei", "ri", "ti", "spi", "sti",
> +					  "naki", "ali", "tmoi";
>  			clocks = <&cpg CPG_MOD R9A07G044_I2C0_PCLK>;
>  			clock-frequency = <100000>;
>  			resets = <&cpg R9A07G044_I2C0_MRST>; @@ -122,6 +124,8 @@
> i2c1: i2c@10058400 {
>  				     <GIC_SPI 359 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 362 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 363 IRQ_TYPE_LEVEL_HIGH>;
> +			interrupt-names = "tei", "ri", "ti", "spi", "sti",
> +					  "naki", "ali", "tmoi";
>  			clocks = <&cpg CPG_MOD R9A07G044_I2C1_PCLK>;
>  			clock-frequency = <100000>;
>  			resets = <&cpg R9A07G044_I2C1_MRST>; @@ -142,6 +146,8 @@
> i2c2: i2c@10058800 {
>  				     <GIC_SPI 367 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 370 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 371 IRQ_TYPE_LEVEL_HIGH>;
> +			interrupt-names = "tei", "ri", "ti", "spi", "sti",
> +					  "naki", "ali", "tmoi";
>  			clocks = <&cpg CPG_MOD R9A07G044_I2C2_PCLK>;
>  			clock-frequency = <100000>;
>  			resets = <&cpg R9A07G044_I2C2_MRST>; @@ -162,6 +168,8 @@
> i2c3: i2c@10058c00 {
>  				     <GIC_SPI 375 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 378 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 379 IRQ_TYPE_LEVEL_HIGH>;
> +			interrupt-names = "tei", "ri", "ti", "spi", "sti",
> +					  "naki", "ali", "tmoi";
>  			clocks = <&cpg CPG_MOD R9A07G044_I2C3_PCLK>;
>  			clock-frequency = <100000>;
>  			resets = <&cpg R9A07G044_I2C3_MRST>;
> --
> 2.25.1


  reply	other threads:[~2021-07-14 17:21 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-14 13:19 [PATCH/RFC 0/4] dt-bindings: i2c: renesas,riic: Add interrupt-names Geert Uytterhoeven
2021-07-14 13:20 ` [PATCH/RFC 1/4] " Geert Uytterhoeven
2021-07-16 19:19   ` Rob Herring
2021-07-14 13:20 ` [PATCH/RFC 2/4] ARM: dts: rza: Add I2C interrupt-names Geert Uytterhoeven
2021-07-14 13:20 ` [PATCH/RFC 3/4] arm64: dts: renesas: r9a07g044: " Geert Uytterhoeven
2021-07-14 17:21   ` Biju Das [this message]
2021-07-14 13:20 ` [PATCH/RFC 4/4] dt-bindings: i2c: renesas,riic: Make interrupt-names required Geert Uytterhoeven
2021-07-16 19:20   ` Rob Herring
2021-08-11 14:23 ` [PATCH/RFC 0/4] dt-bindings: i2c: renesas,riic: Add interrupt-names Wolfram Sang
2021-08-11 15:29   ` Geert Uytterhoeven
2021-08-11 19:04     ` Wolfram Sang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=OS0PR01MB5922BCB0FA0E95A2C3A83C4386139@OS0PR01MB5922.jpnprd01.prod.outlook.com \
    --to=biju.das.jz@bp.renesas.com \
    --cc=Chris.Brandt@renesas.com \
    --cc=devicetree@vger.kernel.org \
    --cc=geert+renesas@glider.be \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=wsa+renesas@sang-engineering.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.