All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] ARM: dts: imx7d-pico: Add Wifi support
@ 2017-08-18 11:37 Vanessa Maegima
  2017-10-31 18:11 ` Otavio Salvador
  0 siblings, 1 reply; 5+ messages in thread
From: Vanessa Maegima @ 2017-08-18 11:37 UTC (permalink / raw)
  To: linux-arm-kernel

imx7d-pico has an ap6212 wifi chip connected to usdhc2 port.

Add support for the usdhc2 port and to the WL_REG_ON regulator
so Wifi can be functional on this board.

Signed-off-by: Vanessa Maegima <vanessa.maegima@nxp.com>
---
Changes since v1:
- Fix bad identation.
- Remove unused pin MX7D_PAD_ECSPI1_SS0__GPIO4_IO19.

 arch/arm/boot/dts/imx7d-pico.dts | 39 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/arch/arm/boot/dts/imx7d-pico.dts b/arch/arm/boot/dts/imx7d-pico.dts
index e78c2c9..bddbf7c 100644
--- a/arch/arm/boot/dts/imx7d-pico.dts
+++ b/arch/arm/boot/dts/imx7d-pico.dts
@@ -52,6 +52,17 @@
 		reg = <0x80000000 0x80000000>;
 	};
 
+	reg_ap6212: regulator-ap6212 {
+		compatible = "regulator-fixed";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_reg_ap6212>;
+		regulator-name = "AP6212";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		gpio = <&gpio4 16 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
+
 	reg_2p5v: regulator-2p5v {
 		compatible = "regulator-fixed";
 		regulator-name = "2P5V";
@@ -271,6 +282,17 @@
 	status = "okay";
 };
 
+&usdhc2 { /* Wifi SDIO */
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_usdhc2>;
+	no-1-8-v;
+	non-removable;
+	keep-power-in-suspend;
+	wakeup-source;
+	vmmc-supply = <&reg_ap6212>;
+	status = "okay";
+};
+
 &usdhc3 {
 	pinctrl-names = "default", "state_100mhz", "state_200mhz";
 	pinctrl-0 = <&pinctrl_usdhc3>;
@@ -326,6 +348,12 @@
 		>;
 	};
 
+	pinctrl_reg_ap6212: regap6212grp {
+		fsl,pins = <
+			MX7D_PAD_ECSPI1_SCLK__GPIO4_IO16	0x59
+		>;
+	};
+
 	pinctrl_sai1: sai1grp {
 		fsl,pins = <
 			MX7D_PAD_ENET1_RX_CLK__SAI1_TX_BCLK	0x1f
@@ -348,6 +376,17 @@
 		>;
 	};
 
+	pinctrl_usdhc2: usdhc2grp {
+			fsl,pins = <
+				MX7D_PAD_SD2_CMD__SD2_CMD	0x59
+				MX7D_PAD_SD2_CLK__SD2_CLK	0x19
+				MX7D_PAD_SD2_DATA0__SD2_DATA0	0x59
+				MX7D_PAD_SD2_DATA1__SD2_DATA1	0x59
+				MX7D_PAD_SD2_DATA2__SD2_DATA2	0x59
+				MX7D_PAD_SD2_DATA3__SD2_DATA3	0x59
+			>;
+	};
+
 	pinctrl_usdhc3: usdhc3grp {
 		fsl,pins = <
 			MX7D_PAD_SD3_CMD__SD3_CMD		0x59
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH v2] ARM: dts: imx7d-pico: Add Wifi support
  2017-08-18 11:37 [PATCH v2] ARM: dts: imx7d-pico: Add Wifi support Vanessa Maegima
@ 2017-10-31 18:11 ` Otavio Salvador
  2017-11-01 12:38   ` Vanessa Maegima
  0 siblings, 1 reply; 5+ messages in thread
From: Otavio Salvador @ 2017-10-31 18:11 UTC (permalink / raw)
  To: linux-arm-kernel

On Fri, Aug 18, 2017 at 8:37 AM, Vanessa Maegima
<vanessa.maegima@nxp.com> wrote:
> imx7d-pico has an ap6212 wifi chip connected to usdhc2 port.
>
> Add support for the usdhc2 port and to the WL_REG_ON regulator
> so Wifi can be functional on this board.
>
> Signed-off-by: Vanessa Maegima <vanessa.maegima@nxp.com>

It seems the name of the dtb is misleading ... this dtb refers mostly
to the imx7d-pico-pi board no?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2] ARM: dts: imx7d-pico: Add Wifi support
  2017-10-31 18:11 ` Otavio Salvador
@ 2017-11-01 12:38   ` Vanessa Maegima
  2017-11-01 12:47     ` Otavio Salvador
  0 siblings, 1 reply; 5+ messages in thread
From: Vanessa Maegima @ 2017-11-01 12:38 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Otavio,

On Ter, 2017-10-31 at 16:11 -0200, Otavio Salvador wrote:
> On Fri, Aug 18, 2017 at 8:37 AM, Vanessa Maegima
> <vanessa.maegima@nxp.com> wrote:
> > 
> > imx7d-pico has an ap6212 wifi chip connected to usdhc2 port.
> > 
> > Add support for the usdhc2 port and to the WL_REG_ON regulator
> > so Wifi can be functional on this board.
> > 
> > Signed-off-by: Vanessa Maegima <vanessa.maegima@nxp.com>
> It seems the name of the dtb is misleading ... this dtb refers mostly
> to the imx7d-pico-pi board no?
> 

Yes, this refers to the imx7d-pico-pi board. At the time I added this
support, I was not aware of the board variants as I had only the pico-
pi baseboard. I can work to separate this dts into SoM and baseboard
dts if you think this would be better, but I can only support pico-pi
as I do not have access to the other baseboards.

Best,
Vanessa

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2] ARM: dts: imx7d-pico: Add Wifi support
  2017-11-01 12:38   ` Vanessa Maegima
@ 2017-11-01 12:47     ` Otavio Salvador
  2017-11-01 17:24       ` John Weber
  0 siblings, 1 reply; 5+ messages in thread
From: Otavio Salvador @ 2017-11-01 12:47 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Nov 1, 2017 at 10:38 AM, Vanessa Maegima
<vanessa.maegima@nxp.com> wrote:
> On Ter, 2017-10-31 at 16:11 -0200, Otavio Salvador wrote:
>> On Fri, Aug 18, 2017 at 8:37 AM, Vanessa Maegima
>> <vanessa.maegima@nxp.com> wrote:
>> >
>> > imx7d-pico has an ap6212 wifi chip connected to usdhc2 port.
>> >
>> > Add support for the usdhc2 port and to the WL_REG_ON regulator
>> > so Wifi can be functional on this board.
>> >
>> > Signed-off-by: Vanessa Maegima <vanessa.maegima@nxp.com>
>> It seems the name of the dtb is misleading ... this dtb refers mostly
>> to the imx7d-pico-pi board no?
>>
>
> Yes, this refers to the imx7d-pico-pi board. At the time I added this
> support, I was not aware of the board variants as I had only the pico-
> pi baseboard. I can work to separate this dts into SoM and baseboard
> dts if you think this would be better, but I can only support pico-pi
> as I do not have access to the other baseboards.

Agreed, splitting it makes sense.

I think we'd need to:

- rename imx7d-pico.dts to imx7d-pico.dtsi
- make a new imx7d-pico.dts to keep compatibility (Shawn is it need?)
- try to make a imx7d-pico-pi.dts that includes imx7d-pico.dtsi and
technexion-pico-pi.dtsi?

It allows for adding the imx6ul-pico-pi and also split hobit baseboard as well.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2] ARM: dts: imx7d-pico: Add Wifi support
  2017-11-01 12:47     ` Otavio Salvador
@ 2017-11-01 17:24       ` John Weber
  0 siblings, 0 replies; 5+ messages in thread
From: John Weber @ 2017-11-01 17:24 UTC (permalink / raw)
  To: linux-arm-kernel


On 11/1/17, 7:47 AM, "Otavio Salvador" <otavio.salvador@ossystems.com.br> wrote:

    On Wed, Nov 1, 2017 at 10:38 AM, Vanessa Maegima
    <vanessa.maegima@nxp.com> wrote:
    > On Ter, 2017-10-31 at 16:11 -0200, Otavio Salvador wrote:
    >> On Fri, Aug 18, 2017 at 8:37 AM, Vanessa Maegima
    >> <vanessa.maegima@nxp.com> wrote:
    >> >
    >> > imx7d-pico has an ap6212 wifi chip connected to usdhc2 port.
    >> >
    >> > Add support for the usdhc2 port and to the WL_REG_ON regulator
    >> > so Wifi can be functional on this board.
    >> >
    >> > Signed-off-by: Vanessa Maegima <vanessa.maegima@nxp.com>
    >> It seems the name of the dtb is misleading ... this dtb refers mostly
    >> to the imx7d-pico-pi board no?
    >>
    >
    > Yes, this refers to the imx7d-pico-pi board. At the time I added this
    > support, I was not aware of the board variants as I had only the pico-
    > pi baseboard. I can work to separate this dts into SoM and baseboard
    > dts if you think this would be better, but I can only support pico-pi
    > as I do not have access to the other baseboards.
    
    Agreed, splitting it makes sense.
    
    I think we'd need to:
    
    - rename imx7d-pico.dts to imx7d-pico.dtsi
    - make a new imx7d-pico.dts to keep compatibility (Shawn is it need?)
    - try to make a imx7d-pico-pi.dts that includes imx7d-pico.dtsi and
    technexion-pico-pi.dtsi?
    
    It allows for adding the imx6ul-pico-pi and also split hobit baseboard as well.

This is the way that it has been done in the TechNexion downstream kernel based on NXP?s 4.1.15 kernel:

For reference:
https://github.com/TechNexion/linux/tree/tn-imx_4.1.15_2.0.0_ga/arch/arm/boot/dts

We have split the baseboards out into .dtsi files with ?baseboard_pico_(pi,nymph,hobbit,dwarf).dtsi?

John

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-11-01 17:24 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-18 11:37 [PATCH v2] ARM: dts: imx7d-pico: Add Wifi support Vanessa Maegima
2017-10-31 18:11 ` Otavio Salvador
2017-11-01 12:38   ` Vanessa Maegima
2017-11-01 12:47     ` Otavio Salvador
2017-11-01 17:24       ` John Weber

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.