Re: [PATCH V3] arm64: dts: r8a77990: ebisu: Add and enable SDHI device nodes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 21, 2018 at 12:33 AM Marek Vasut <marek.vasut@xxxxxxxxx> wrote:
> On 11/20/2018 11:16 AM, Geert Uytterhoeven wrote:
> > On Tue, Nov 6, 2018 at 9:46 PM Marek Vasut <marek.vasut@xxxxxxxxx> wrote:
> >> From: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx>
> >>
> >> This patch adds SDHI{0,1,3} device nodes for the r8a77990 SoC
> >> and enables SD card slot connected to SDHI0, micro SD card slot
> >> connected to SDHI1 and eMMC connected to SDHI3 on the Ebisu board
> >> using the R8A77990 SoC.
> >>
> >> Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx>
> >> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxx>
> >
> >> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> >> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> >> @@ -129,6 +129,15 @@
> >>                 };
> >>         };
> >>
> >> +       reg_1p8v: regulator0 {
> >> +               compatible = "regulator-fixed";
> >> +               regulator-name = "fixed-1.8V";
> >> +               regulator-min-microvolt = <1800000>;
> >> +               regulator-max-microvolt = <1800000>;
> >> +               regulator-boot-on;
> >> +               regulator-always-on;
> >> +       };
> >> +
> >>         reg_3p3v: regulator1 {
> >>                 compatible = "regulator-fixed";
> >>                 regulator-name = "fixed-3.3V";
> >> @@ -180,6 +189,54 @@
> >>                 #clock-cells = <0>;
> >>                 clock-frequency = <74250000>;
> >>         };
> >> +
> >> +       vcc_sdhi0: regulator-vcc-sdhi0 {
> >> +               compatible = "regulator-fixed";
> >> +
> >> +               regulator-name = "SDHI0 Vcc";
> >> +               regulator-min-microvolt = <3300000>;
> >> +               regulator-max-microvolt = <3300000>;
> >> +
> >> +               gpio = <&gpio5 17 GPIO_ACTIVE_HIGH>;
> >> +               enable-active-high;
> >> +       };
> >> +
> >> +       vccq_sdhi0: regulator-vccq-sdhi0 {
> >> +               compatible = "regulator-gpio";
> >> +
> >> +               regulator-name = "SDHI0 VccQ";
> >> +               regulator-min-microvolt = <1800000>;
> >> +               regulator-max-microvolt = <3300000>;
> >> +
> >> +               gpios = <&gpio5 18 GPIO_ACTIVE_HIGH>;
> >> +               gpios-states = <1>;
> >> +               states = <3300000 1
> >> +                         1800000 0>;
> >
> > I know this is how it's done in the example in
> > Documentation/devicetree/bindings/regulator/gpio-regulator.txt,
> > but usually brackets are used to group tuples, like:
> >
> >     states = <3300000 1>, <1800000 0>;
> >
> > Perhaps the example should be changed?
>
> Well, looking through the DTS, I don't see anyone using tuples for the
> GPIO regulator states. Then again, it could be updated.

Probably a good idea given the current work on schema validation:

    DT YAML Encoding
    ...
    dts ‘< >’ are maintained
      ○ Will need to be stricter about these
      ○ Same for dtb, not for validation:
          ■ prop = <1 2 3 4>;
          ■ prop = <1>, <2>, <3>, <4>;

https://www.elinux.org/images/6/6b/LPC2018_json-schema_for_Devicetree.pdf

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



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux