Hello Simon,

> From: devicetree-ow...@vger.kernel.org [mailto:devicetree-
> ow...@vger.kernel.org] On Behalf Of Simon Horman
> Sent: 15 September 2017 09:12
> To: Chris Paterson <chris.paters...@renesas.com>
> 
> On Wed, Sep 13, 2017 at 06:05:36PM +0100, Chris Paterson wrote:
> > From: Fabrizio Castro <fabrizio.cas...@bp.renesas.com>
> >
> > Enable the SDHI1 controller on iWave RZ/G1E SoM.
> >
> > Signed-off-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com>
> > Signed-off-by: Chris Paterson <chris.paters...@renesas.com>
> > ---
> > This patch is based on renesas-devel-20170913-v4.13.
> >
> >
> >  arch/arm/boot/dts/r8a7745-iwg22m.dtsi | 17 +++++++++++++++++
> >  1 file changed, 17 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/r8a7745-iwg22m.dtsi
> > b/arch/arm/boot/dts/r8a7745-iwg22m.dtsi
> > index e306e7c..f7f9cef 100644
> > --- a/arch/arm/boot/dts/r8a7745-iwg22m.dtsi
> > +++ b/arch/arm/boot/dts/r8a7745-iwg22m.dtsi
> > @@ -9,6 +9,7 @@
> >   */
> >
> >  #include "r8a7745.dtsi"
> > +#include <dt-bindings/gpio/gpio.h>
> >
> >  / {
> >     compatible = "iwave,g22m", "renesas,r8a7745"; @@ -38,6 +39,12
> @@
> >             function = "mmc";
> >     };
> >
> > +   sdhi1_pins: sd1 {
> > +           groups = "sdhi1_data4", "sdhi1_ctrl";
> > +           function = "sdhi1";
> > +           power-source = <3300>;
> > +   };
> > +
> >     i2c3_pins: i2c3 {
> >             groups = "i2c3_b";
> >             function = "i2c3";
> > @@ -54,6 +61,16 @@
> >     status = "okay";
> >  };
> >
> > +&sdhi1 {
> > +   pinctrl-0 = <&sdhi1_pins>;
> > +   pinctrl-names = "default";
> > +
> > +   vmmc-supply = <&reg_3p3v>;
> > +   vqmmc-supply = <&reg_3p3v>;
> > +   cd-gpios = <&gpio3 31 GPIO_ACTIVE_LOW>;
> 
> No wp-gpios property means this is a µSD slot, right?

Correct

> 
> Do you have any plans to enable sdr-50 and sdr-104?
> Is it not supported for some reason?

Nope (unless the hardware changes down the line).
Both vmmc and vqmmc are fixed to 3V3 on the iWave-E SOM module.

Kind regards, Chris


> 
> I don't mind if its not enabled in this patch but I would like to know if it 
> can be
> enabled or not and reflect that information in the wiki.
> 
> http://elinux.org/index.php?title=Renesas-MMC-Enabled-Speeds
> 
> > +   status = "okay";
> > +};
> > +
> >  &i2c3 {
> >     pinctrl-0 = <&i2c3_pins>;
> >     pinctrl-names = "default";
> > --
> > 1.9.1
> >
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in the
> body of a message to majord...@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html

Reply via email to