Re: [PATCH 7/7] ARM: dts: Enable emmc on hp t410

2015-12-04 Thread Peter Ujfalusi
On 12/04/2015 03:25 AM, Tony Lindgren wrote:
> There's a 2GB emmc on hp t410 that's wired to the sd_2 interface.
> Note that we also need to configure the evtmux using edma_xbar for
> edma channels.
> 
> Signed-off-by: Tony Lindgren 

One question inlined for the mmc3, other than that:
Reviewed-by: Peter Ujfalusi 

> ---
>  arch/arm/boot/dts/dm8148-t410.dts | 35 +++
>  1 file changed, 35 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/dm8148-t410.dts 
> b/arch/arm/boot/dts/dm8148-t410.dts
> index 79838dd..d1927a7 100644
> --- a/arch/arm/boot/dts/dm8148-t410.dts
> +++ b/arch/arm/boot/dts/dm8148-t410.dts
> @@ -15,6 +15,13 @@
>   device_type = "memory";
>   reg = <0x8000 0x4000>;  /* 1 GB */
>   };
> +
> + vmmcsd_fixed: fixedregulator@0 {
> + compatible = "regulator-fixed";
> + regulator-name = "vmmcsd_fixed";
> + regulator-min-microvolt = <330>;
> + regulator-max-microvolt = <330>;
> + };
>  };
>  
>  _emac0 {
> @@ -26,3 +33,31 @@
>   phy_id = <_mdio>, <1>;
>   phy-mode = "rgmii";
>  };
> +
> + {
> + pinctrl-names = "default";
> + pinctrl-0 = <_pins>;
> + vmmc-supply = <_fixed>;
> + bus-width = <8>;
> + dmas = <_xbar 8 0 1/* use SDTXEVT1 instead of MCASP0TX */
> + _xbar 9 0 2>;  /* use SDRXEVT1 instead of MCASP0RX */
> + dma-names = "tx", "rx";

Do you happened to know how the audio is connected on the board?
The DM814x-EVM seams to be using McASP2 with aic3106 so it is possible that HP
copied the design from there...

> +};
> +
> + {
> + sd2_pins: pinmux_sd2_pins {
> + pinctrl-single,pins = <
> + DM814X_IOPAD(0x09c0, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[7] */
> + DM814X_IOPAD(0x09c4, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[6] */
> + DM814X_IOPAD(0x09c8, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[5] */
> + DM814X_IOPAD(0x09cc, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[4] */
> + DM814X_IOPAD(0x09d0, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[3] */
> + DM814X_IOPAD(0x09d4, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[2] */
> + DM814X_IOPAD(0x09d8, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[1] */
> + DM814X_IOPAD(0x09dc, PIN_INPUT_PULLUP | 0x1)/* 
> SD2_DAT[0] */
> + DM814X_IOPAD(0x09e0, PIN_INPUT | 0x1)   /* 
> SD2_CLK */
> + DM814X_IOPAD(0x09f4, PIN_INPUT_PULLUP | 0x2)/* 
> SD2_CMD */
> + DM814X_IOPAD(0x0920, PIN_INPUT | 40)/* SD2_SDCD */
> + >;
> + };
> +};
> 


-- 
Péter
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 7/7] ARM: dts: Enable emmc on hp t410

2015-12-04 Thread Tony Lindgren
* Peter Ujfalusi  [151204 01:21]:
> On 12/04/2015 03:25 AM, Tony Lindgren wrote:
> > There's a 2GB emmc on hp t410 that's wired to the sd_2 interface.
> > Note that we also need to configure the evtmux using edma_xbar for
> > edma channels.
> > 
> > Signed-off-by: Tony Lindgren 
> 
> One question inlined for the mmc3, other than that:
> Reviewed-by: Peter Ujfalusi 
> 
> > ---
> >  arch/arm/boot/dts/dm8148-t410.dts | 35 +++
> >  1 file changed, 35 insertions(+)
> > 
> > diff --git a/arch/arm/boot/dts/dm8148-t410.dts 
> > b/arch/arm/boot/dts/dm8148-t410.dts
> > index 79838dd..d1927a7 100644
> > --- a/arch/arm/boot/dts/dm8148-t410.dts
> > +++ b/arch/arm/boot/dts/dm8148-t410.dts
> > @@ -15,6 +15,13 @@
> > device_type = "memory";
> > reg = <0x8000 0x4000>;  /* 1 GB */
> > };
> > +
> > +   vmmcsd_fixed: fixedregulator@0 {
> > +   compatible = "regulator-fixed";
> > +   regulator-name = "vmmcsd_fixed";
> > +   regulator-min-microvolt = <330>;
> > +   regulator-max-microvolt = <330>;
> > +   };
> >  };
> >  
> >  _emac0 {
> > @@ -26,3 +33,31 @@
> > phy_id = <_mdio>, <1>;
> > phy-mode = "rgmii";
> >  };
> > +
> > + {
> > +   pinctrl-names = "default";
> > +   pinctrl-0 = <_pins>;
> > +   vmmc-supply = <_fixed>;
> > +   bus-width = <8>;
> > +   dmas = <_xbar 8 0 1/* use SDTXEVT1 instead of MCASP0TX */
> > +   _xbar 9 0 2>;  /* use SDRXEVT1 instead of MCASP0RX */
> > +   dma-names = "tx", "rx";
> 
> Do you happened to know how the audio is connected on the board?
> The DM814x-EVM seams to be using McASP2 with aic3106 so it is possible that HP
> copied the design from there...

I grepped the sources for it and these are the evtmux channels the original
code is using. So maybe they have some different audio setup, the hacks in the
original board file mention "hdmi-dai".

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 7/7] ARM: dts: Enable emmc on hp t410

2015-12-03 Thread Tony Lindgren
There's a 2GB emmc on hp t410 that's wired to the sd_2 interface.
Note that we also need to configure the evtmux using edma_xbar for
edma channels.

Signed-off-by: Tony Lindgren 
---
 arch/arm/boot/dts/dm8148-t410.dts | 35 +++
 1 file changed, 35 insertions(+)

diff --git a/arch/arm/boot/dts/dm8148-t410.dts 
b/arch/arm/boot/dts/dm8148-t410.dts
index 79838dd..d1927a7 100644
--- a/arch/arm/boot/dts/dm8148-t410.dts
+++ b/arch/arm/boot/dts/dm8148-t410.dts
@@ -15,6 +15,13 @@
device_type = "memory";
reg = <0x8000 0x4000>;  /* 1 GB */
};
+
+   vmmcsd_fixed: fixedregulator@0 {
+   compatible = "regulator-fixed";
+   regulator-name = "vmmcsd_fixed";
+   regulator-min-microvolt = <330>;
+   regulator-max-microvolt = <330>;
+   };
 };
 
 _emac0 {
@@ -26,3 +33,31 @@
phy_id = <_mdio>, <1>;
phy-mode = "rgmii";
 };
+
+ {
+   pinctrl-names = "default";
+   pinctrl-0 = <_pins>;
+   vmmc-supply = <_fixed>;
+   bus-width = <8>;
+   dmas = <_xbar 8 0 1/* use SDTXEVT1 instead of MCASP0TX */
+   _xbar 9 0 2>;  /* use SDRXEVT1 instead of MCASP0RX */
+   dma-names = "tx", "rx";
+};
+
+ {
+   sd2_pins: pinmux_sd2_pins {
+   pinctrl-single,pins = <
+   DM814X_IOPAD(0x09c0, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[7] */
+   DM814X_IOPAD(0x09c4, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[6] */
+   DM814X_IOPAD(0x09c8, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[5] */
+   DM814X_IOPAD(0x09cc, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[4] */
+   DM814X_IOPAD(0x09d0, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[3] */
+   DM814X_IOPAD(0x09d4, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[2] */
+   DM814X_IOPAD(0x09d8, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[1] */
+   DM814X_IOPAD(0x09dc, PIN_INPUT_PULLUP | 0x1)/* 
SD2_DAT[0] */
+   DM814X_IOPAD(0x09e0, PIN_INPUT | 0x1)   /* 
SD2_CLK */
+   DM814X_IOPAD(0x09f4, PIN_INPUT_PULLUP | 0x2)/* 
SD2_CMD */
+   DM814X_IOPAD(0x0920, PIN_INPUT | 40)/* SD2_SDCD */
+   >;
+   };
+};
-- 
2.6.2

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html