| Message ID | 1564731249-22671-12-git-send-email-fabrizio.castro@bp.renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | Add dual-LVDS panel support to EK874 | expand |
Hi Fabrizio, Thank you for the patch. On Fri, Aug 02, 2019 at 08:34:08AM +0100, Fabrizio Castro wrote: > Power rail "D12.0V" comes straight from the power barrel connector, > and it's used in both main board and sub board. > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> I don't plan to take this in my tree without patch 12/12, so if you think the rest of the series won't be ready in time for v5.4, feel free to get this patch merged through Simon or Geert already. > --- > arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts > index 46a77ee..651383c 100644 > --- a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts > +++ b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts > @@ -65,6 +65,15 @@ > reg = <0x0 0x48000000 0x0 0x78000000>; > }; > > + reg_12p0v: regulator-12p0v { > + compatible = "regulator-fixed"; > + regulator-name = "D12.0V"; > + regulator-min-microvolt = <12000000>; > + regulator-max-microvolt = <12000000>; > + regulator-boot-on; > + regulator-always-on; > + }; > + > sound: sound { > compatible = "simple-audio-card"; >
Hi Laurent, Thank you for your feedback! > From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Sent: 02 August 2019 09:30 > Subject: Re: [PATCH/RFC 11/12] arm64: dts: renesas: cat874: Add definition for 12V regulator > > Hi Fabrizio, > > Thank you for the patch. > > On Fri, Aug 02, 2019 at 08:34:08AM +0100, Fabrizio Castro wrote: > > Power rail "D12.0V" comes straight from the power barrel connector, > > and it's used in both main board and sub board. > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > I don't plan to take this in my tree without patch 12/12, so if you > think the rest of the series won't be ready in time for v5.4, feel free > to get this patch merged through Simon or Geert already. Geert, would you be happy to take this patch? Thanks, Fab > > > --- > > arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts > > index 46a77ee..651383c 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts > > +++ b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts > > @@ -65,6 +65,15 @@ > > reg = <0x0 0x48000000 0x0 0x78000000>; > > }; > > > > + reg_12p0v: regulator-12p0v { > > + compatible = "regulator-fixed"; > > + regulator-name = "D12.0V"; > > + regulator-min-microvolt = <12000000>; > > + regulator-max-microvolt = <12000000>; > > + regulator-boot-on; > > + regulator-always-on; > > + }; > > + > > sound: sound { > > compatible = "simple-audio-card"; > > > > -- > Regards, > > Laurent Pinchart
Hi Fabrizio, On Mon, Aug 5, 2019 at 11:17 AM Fabrizio Castro <fabrizio.castro@bp.renesas.com> wrote: > > From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Sent: 02 August 2019 09:30 > > Subject: Re: [PATCH/RFC 11/12] arm64: dts: renesas: cat874: Add definition for 12V regulator > > > > Hi Fabrizio, > > > > Thank you for the patch. > > > > On Fri, Aug 02, 2019 at 08:34:08AM +0100, Fabrizio Castro wrote: > > > Power rail "D12.0V" comes straight from the power barrel connector, > > > and it's used in both main board and sub board. > > > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > > I don't plan to take this in my tree without patch 12/12, so if you > > think the rest of the series won't be ready in time for v5.4, feel free > > to get this patch merged through Simon or Geert already. > > Geert, would you be happy to take this patch? Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> i.e. will queue in renesas-devel for v5.4. Gr{oetje,eeting}s, Geert
diff --git a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts index 46a77ee..651383c 100644 --- a/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts +++ b/arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts @@ -65,6 +65,15 @@ reg = <0x0 0x48000000 0x0 0x78000000>; }; + reg_12p0v: regulator-12p0v { + compatible = "regulator-fixed"; + regulator-name = "D12.0V"; + regulator-min-microvolt = <12000000>; + regulator-max-microvolt = <12000000>; + regulator-boot-on; + regulator-always-on; + }; + sound: sound { compatible = "simple-audio-card";
Power rail "D12.0V" comes straight from the power barrel connector, and it's used in both main board and sub board. Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> --- arch/arm64/boot/dts/renesas/r8a774c0-cat874.dts | 9 +++++++++ 1 file changed, 9 insertions(+)