Hi Chris, On Tue, Sep 20, 2016 at 5:46 PM, Chris Brandt <chris.brandt@xxxxxxxxxxx> wrote: > Since the MMC and SDHI1 on the RSK share the same socket connector (CN1), > you cannot enable MMC and SDHI1 at the same time. Therefore the status > has been set to disabled because SDHI is more popular with this board. > However, keeping this code in here serves as a good way to document how > the MMC on the RZ/A1 has been known to work for someone that does want > to use MMC instead of SDHI1. > > A fixed 3.3 regulator is included because it is required by the mmc > driver. I have mixed feelings about adding these "disabled" nodes, purely for documentation. What do other people think? > --- a/arch/arm/boot/dts/r7s72100-rskrza1.dts > +++ b/arch/arm/boot/dts/r7s72100-rskrza1.dts > @@ -33,6 +33,15 @@ > #address-cells = <1>; > #size-cells = <1>; > }; > + > + d3_3v: regulator-d3-3v { > + compatible = "regulator-fixed"; > + regulator-name = "D3.3V"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + regulator-always-on; 'status = "disabled";', too? > + }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html