From: Simon Horman <horms@verge.net.au>
To: Eugeniu Rosca <erosca@de.adit-jv.com>
Cc: Spyridon Papageorgiou <spapageorgiou@de.adit-jv.com>,
magnus.damm@gmail.com, robh+dt@kernel.org, mark.rutland@arm.com,
linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, tfranzen@de.adit-jv.com,
Eugeniu Rosca <roscaeugeniu@gmail.com>
Subject: Re: [PATCH] arm64: dts: ulcb-kf: Add support for TI WL1837
Date: Mon, 29 Apr 2019 10:17:48 +0200 [thread overview]
Message-ID: <20190429081747.irznig7yrvztuc5x@verge.net.au> (raw)
In-Reply-To: <20190426095012.xzyzevvmom4fzdcd@verge.net.au>
Hi again,
I have been able to solicit a limited private review of this patch and
have gone ahead and applied it for inclusion in v5.2.
On Fri, Apr 26, 2019 at 11:50:12AM +0200, Simon Horman wrote:
> Hi,
>
> from my point of view what is required is a review.
> I will try to find someone to do so.
> I apologise for not doing so earlier.
>
> On Thu, Apr 25, 2019 at 01:12:45PM +0200, Eugeniu Rosca wrote:
> > Hi Simon,
> >
> > Do we have any chance getting this upstream? If so, would you kindly
> > list the acceptance criteria we have to conform to?
> >
> > Many thanks.
> >
> > On Thu, Apr 11, 2019 at 02:41:03PM +0200, Spyridon Papageorgiou wrote:
> > > This patch adds description of TI WL1837 and links interfaces
> > > to communicate with the IC, namely the SDIO interface to WLAN.
> > >
> > > Signed-off-by: Spyridon Papageorgiou <spapageorgiou@de.adit-jv.com>
> > > ---
> > > arch/arm64/boot/dts/renesas/ulcb-kf.dtsi | 49 ++++++++++++++++++++++++++++++++
> > > 1 file changed, 49 insertions(+)
> > >
> > > diff --git a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> > > index 7a09576..27851a7 100644
> > > --- a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> > > +++ b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> > > @@ -38,6 +38,18 @@
> > > regulator-min-microvolt = <5000000>;
> > > regulator-max-microvolt = <5000000>;
> > > };
> > > +
> > > + wlan_en: regulator-wlan_en {
> > > + compatible = "regulator-fixed";
> > > + regulator-name = "wlan-en-regulator";
> > > +
> > > + regulator-min-microvolt = <3300000>;
> > > + regulator-max-microvolt = <3300000>;
> > > +
> > > + gpio = <&gpio_exp_74 4 GPIO_ACTIVE_HIGH>;
> > > + startup-delay-us = <70000>;
> > > + enable-active-high;
> > > + };
> > > };
> > >
> > > &can0 {
> > > @@ -88,6 +100,13 @@
> > > line-name = "Audio_Out_OFF";
> > > };
> > >
> > > + sd-wifi-mux {
> > > + gpio-hog;
> > > + gpios = <5 GPIO_ACTIVE_HIGH>;
> > > + output-low; /* Connect WL1837 */
> > > + line-name = "SD WiFi mux";
> > > + };
> > > +
> > > hub_pwen {
> > > gpio-hog;
> > > gpios = <6 GPIO_ACTIVE_HIGH>;
> > > @@ -254,6 +273,12 @@
> > > function = "scif1";
> > > };
> > >
> > > + sdhi3_pins: sdhi3 {
> > > + groups = "sdhi3_data4", "sdhi3_ctrl";
> > > + function = "sdhi3";
> > > + power-source = <3300>;
> > > + };
> > > +
> > > usb0_pins: usb0 {
> > > groups = "usb0";
> > > function = "usb0";
> > > @@ -273,6 +298,30 @@
> > > status = "okay";
> > > };
> > >
> > > +&sdhi3 {
> > > + pinctrl-0 = <&sdhi3_pins>;
> > > + pinctrl-names = "default";
> > > +
> > > + vmmc-supply = <&wlan_en>;
> > > + vqmmc-supply = <&wlan_en>;
> > > + bus-width = <4>;
> > > + no-1-8-v;
> > > + non-removable;
> > > + cap-power-off-card;
> > > + keep-power-in-suspend;
> > > + max-frequency = <26000000>;
> > > + status = "okay";
> > > +
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > + wlcore: wlcore@2 {
> > > + compatible = "ti,wl1837";
> > > + reg = <2>;
> > > + interrupt-parent = <&gpio1>;
> > > + interrupts = <25 IRQ_TYPE_EDGE_FALLING>;
> > > + };
> > > +};
> > > +
> > > &usb2_phy0 {
> > > pinctrl-0 = <&usb0_pins>;
> > > pinctrl-names = "default";
> > > --
> > > 2.7.4
> > >
> >
> > --
> > Best regards,
> > Eugeniu.
> >
>
next prev parent reply other threads:[~2019-04-29 8:17 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-11 12:41 [PATCH] arm64: dts: ulcb-kf: Add support for TI WL1837 Spyridon Papageorgiou
2019-04-25 11:12 ` Eugeniu Rosca
2019-04-26 9:50 ` Simon Horman
2019-04-29 8:17 ` Simon Horman [this message]
2019-04-29 9:25 ` Eugeniu Rosca
2019-05-28 9:19 ` Geert Uytterhoeven
2019-05-31 9:37 ` Simon Horman
2019-05-31 13:49 ` Eugeniu Rosca
2019-06-03 11:01 ` Simon Horman
2019-05-31 15:25 ` Eugeniu Rosca
2019-06-03 7:26 ` Geert Uytterhoeven
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=20190429081747.irznig7yrvztuc5x@verge.net.au \
--to=horms@verge.net.au \
--cc=devicetree@vger.kernel.org \
--cc=erosca@de.adit-jv.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
--cc=roscaeugeniu@gmail.com \
--cc=spapageorgiou@de.adit-jv.com \
--cc=tfranzen@de.adit-jv.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).