From: Icenowy Zheng <icenowy@aosc.io> To: Arend van Spriel <arend.vanspriel@broadcom.com>, Harald Geyer <harald@ccbib.org> Cc: Maxime Ripard <maxime.ripard@bootlin.com>, Chen-Yu Tsai <wens@csie.org>, Kalle Valo <kvalo@codeaurora.org>, Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Andre Przywara <andre.przywara@arm.com>, info@olimex.com, linux-wireless@vger.kernel.org Subject: Re: [PATCHv2 5/5] arm64: allwinner: a64: Add support for TERES-I laptop Date: Mon, 25 Jun 2018 15:47:18 +0800 [thread overview] Message-ID: <1CF519CD-D44E-4959-8951-27CE710E0984@aosc.io> (raw) In-Reply-To: <5B309D37.80207@broadcom.com> 于 2018年6月25日 GMT+08:00 下午3:43:51, Arend van Spriel <arend.vanspriel@broadcom.com> 写到: >On 6/24/2018 6:34 PM, Harald Geyer wrote: >> Icenowy Zheng writes: >>> >在 2018-03-15四的 16:25 +0000,Harald Geyer写道: >>>> > >+&mmc1 { >>>> > >+ pinctrl-names = "default"; >>>> > >+ pinctrl-0 = <&mmc1_pins>; >>>> > >+ vmmc-supply = <®_aldo2>; >>>> > >+ vqmmc-supply = <®_dldo4>; >>>> > >+ mmc-pwrseq = <&wifi_pwrseq>; >>>> > >+ bus-width = <4>; >>>> > >+ non-removable; >>>> > >+ status = "okay"; >>>> > >+ >>>> > >+ rtl8723bs: wifi@1 { >>>> > >+ reg = <1>; >>>> > >+ interrupt-parent = <&r_pio>; >>>> > >+ interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 */ >>>> > >+ interrupt-names = "host-wake"; >>>> > >+ }; > >[...] > >>> >- This device node has no binding. The "host-wake" interrupt is >part of >>> > Broadcom SDIO Wi-Fi binding, rather than a generic one. >> I think the general mmc and interrupts bindings apply. And the mmc >binding >> clearly states that for sub-nodes a compatible string is optional. >> >> However I just realized that the 'interrupt-names' property is not >part >> of the general interrupts binding, so I guess at least this property >should >> be removed. > >Indeed. If the device just used the SDIO interrupt this is not needed. >The Broadcom device can use either SDIO interrupt or a so-called >out-of-band host-wake interrupt, which is what the above represents. RTL8....S is also capable of use OOB interrupt. > >Regards, >Arend
WARNING: multiple messages have this Message-ID (diff)
From: icenowy@aosc.io (Icenowy Zheng) To: linux-arm-kernel@lists.infradead.org Subject: [PATCHv2 5/5] arm64: allwinner: a64: Add support for TERES-I laptop Date: Mon, 25 Jun 2018 15:47:18 +0800 [thread overview] Message-ID: <1CF519CD-D44E-4959-8951-27CE710E0984@aosc.io> (raw) In-Reply-To: <5B309D37.80207@broadcom.com> ? 2018?6?25? GMT+08:00 ??3:43:51, Arend van Spriel <arend.vanspriel@broadcom.com> ??: >On 6/24/2018 6:34 PM, Harald Geyer wrote: >> Icenowy Zheng writes: >>> >? 2018-03-15?? 16:25 +0000?Harald Geyer??? >>>> > >+&mmc1 { >>>> > >+ pinctrl-names = "default"; >>>> > >+ pinctrl-0 = <&mmc1_pins>; >>>> > >+ vmmc-supply = <®_aldo2>; >>>> > >+ vqmmc-supply = <®_dldo4>; >>>> > >+ mmc-pwrseq = <&wifi_pwrseq>; >>>> > >+ bus-width = <4>; >>>> > >+ non-removable; >>>> > >+ status = "okay"; >>>> > >+ >>>> > >+ rtl8723bs: wifi at 1 { >>>> > >+ reg = <1>; >>>> > >+ interrupt-parent = <&r_pio>; >>>> > >+ interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 */ >>>> > >+ interrupt-names = "host-wake"; >>>> > >+ }; > >[...] > >>> >- This device node has no binding. The "host-wake" interrupt is >part of >>> > Broadcom SDIO Wi-Fi binding, rather than a generic one. >> I think the general mmc and interrupts bindings apply. And the mmc >binding >> clearly states that for sub-nodes a compatible string is optional. >> >> However I just realized that the 'interrupt-names' property is not >part >> of the general interrupts binding, so I guess at least this property >should >> be removed. > >Indeed. If the device just used the SDIO interrupt this is not needed. >The Broadcom device can use either SDIO interrupt or a so-called >out-of-band host-wake interrupt, which is what the above represents. RTL8....S is also capable of use OOB interrupt. > >Regards, >Arend
next prev parent reply other threads:[~2018-06-25 7:52 UTC|newest] Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-03-15 16:25 [PATCHv2 0/5] arm64: allwinner: Add support for TERES-I laptop Harald Geyer 2018-03-15 16:25 ` Harald Geyer 2018-03-15 16:25 ` [PATCHv2 1/5] arm64: dts: allwinner: a64: Add i2c0 pins Harald Geyer 2018-03-15 16:25 ` Harald Geyer 2018-03-15 16:25 ` [PATCHv2 2/5] arm64: dts: allwinner: a64: Add watchdog Harald Geyer 2018-03-15 16:25 ` Harald Geyer 2018-03-18 12:52 ` Rob Herring 2018-03-18 12:52 ` Rob Herring 2018-03-18 12:52 ` Rob Herring 2018-03-15 16:25 ` [PATCHv2 3/5] arm64: dts: allwinner: a64: add simplefb for A64 SoC Harald Geyer 2018-03-15 16:25 ` Harald Geyer 2018-03-15 16:25 ` [PATCHv2 4/5] arm64: dts: allwinner: a64: Add pwm device Harald Geyer 2018-03-15 16:25 ` Harald Geyer 2018-03-18 12:52 ` Rob Herring 2018-03-18 12:52 ` Rob Herring 2018-03-18 13:51 ` André Przywara 2018-03-18 13:51 ` André Przywara 2018-03-15 16:25 ` [PATCHv2 5/5] arm64: allwinner: a64: Add support for TERES-I laptop Harald Geyer 2018-03-15 16:25 ` Harald Geyer 2018-03-16 6:37 ` afzal mohammed 2018-03-16 6:37 ` afzal mohammed 2018-03-16 17:50 ` afzal mohammed 2018-03-16 17:50 ` afzal mohammed 2018-03-18 20:22 ` Maxime Ripard 2018-03-18 20:22 ` Maxime Ripard 2018-03-19 7:17 ` afzal mohammed 2018-03-19 7:17 ` afzal mohammed 2018-03-19 21:27 ` Maxime Ripard 2018-03-19 21:27 ` Maxime Ripard 2018-03-19 21:27 ` Maxime Ripard 2018-03-19 15:27 ` Harald Geyer 2018-03-19 15:27 ` Harald Geyer 2018-03-20 14:13 ` Maxime Ripard 2018-03-20 14:13 ` Maxime Ripard 2018-03-20 17:09 ` Andre Przywara 2018-03-20 17:09 ` Andre Przywara 2018-03-22 17:59 ` Maxime Ripard 2018-03-22 17:59 ` Maxime Ripard 2018-06-22 16:27 ` Icenowy Zheng 2018-06-22 16:27 ` Icenowy Zheng 2018-06-24 16:34 ` Harald Geyer 2018-06-24 16:34 ` Harald Geyer 2018-06-25 7:43 ` Arend van Spriel 2018-06-25 7:43 ` Arend van Spriel 2018-06-25 7:47 ` Icenowy Zheng [this message] 2018-06-25 7:47 ` Icenowy Zheng 2018-06-25 8:13 ` Arend van Spriel 2018-06-25 8:13 ` Arend van Spriel 2018-06-25 10:42 ` Icenowy Zheng 2018-06-25 10:42 ` Icenowy Zheng
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=1CF519CD-D44E-4959-8951-27CE710E0984@aosc.io \ --to=icenowy@aosc.io \ --cc=andre.przywara@arm.com \ --cc=arend.vanspriel@broadcom.com \ --cc=catalin.marinas@arm.com \ --cc=devicetree@vger.kernel.org \ --cc=harald@ccbib.org \ --cc=info@olimex.com \ --cc=kvalo@codeaurora.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-wireless@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=maxime.ripard@bootlin.com \ --cc=robh+dt@kernel.org \ --cc=wens@csie.org \ --cc=will.deacon@arm.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: linkBe 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.