Re: [PATCH] arm64: dts: ulcb-kf: Add support for TI WL1837

[Simon Horman]

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@xxxxxxxxxxxxxx>
> > > ---
> > >  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.
> > 
>