* [PATCH] arm64: dts: qcom: sc7280-herobrine: Audio codec wants 1.8V, not 1.62V
@ 2022-03-25 15:11 Douglas Anderson
2022-03-25 15:42 ` Stephen Boyd
0 siblings, 1 reply; 4+ messages in thread
From: Douglas Anderson @ 2022-03-25 15:11 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Matthias Kaehlcke, Stephen Boyd, Judy Hsiao, Douglas Anderson,
Andy Gross, Rob Herring, devicetree, linux-arm-msm, linux-kernel
The L2C rail on herobrine boards is intended to go to the audio
codec. Let's override the 1.62V specified in the qcard.dtsi file to be
1.8V.
Signed-off-by: Douglas Anderson <dianders@chromium.org>
---
arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
index dc17f2079695..c2075f3e7c4b 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
@@ -339,6 +339,16 @@ keyboard_backlight: keyboard-backlight {
ts_avdd: &pp3300_left_in_mlb {};
vreg_edp_3p3: &pp3300_left_in_mlb {};
+/* Regulator overrides from Qcard */
+
+/*
+ * Herobrine boards only use l2c to power an external audio codec (like
+ * alc5682) and we want that to be at 1.8V, not at some slightly lower voltage.
+ */
+&vreg_l2c_1p8 {
+ regulator-min-microvolt = <1800000>;
+};
+
/* ADDITIONS TO NODES DEFINED IN PARENT DEVICE TREE FILES */
ap_i2c_tpm: &i2c14 {
--
2.35.1.1021.g381101b075-goog
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sc7280-herobrine: Audio codec wants 1.8V, not 1.62V
2022-03-25 15:11 [PATCH] arm64: dts: qcom: sc7280-herobrine: Audio codec wants 1.8V, not 1.62V Douglas Anderson
@ 2022-03-25 15:42 ` Stephen Boyd
2022-03-25 15:47 ` Doug Anderson
0 siblings, 1 reply; 4+ messages in thread
From: Stephen Boyd @ 2022-03-25 15:42 UTC (permalink / raw)
To: Bjorn Andersson, Douglas Anderson
Cc: Matthias Kaehlcke, Judy Hsiao, Andy Gross, Rob Herring,
devicetree, linux-arm-msm, linux-kernel
Quoting Douglas Anderson (2022-03-25 08:11:05)
> The L2C rail on herobrine boards is intended to go to the audio
> codec. Let's override the 1.62V specified in the qcard.dtsi file to be
> 1.8V.
>
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> ---
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>
> arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> index dc17f2079695..c2075f3e7c4b 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> @@ -339,6 +339,16 @@ keyboard_backlight: keyboard-backlight {
> ts_avdd: &pp3300_left_in_mlb {};
> vreg_edp_3p3: &pp3300_left_in_mlb {};
>
> +/* Regulator overrides from Qcard */
This one isn't in ALL CAPS? :)
> +
> +/*
> + * Herobrine boards only use l2c to power an external audio codec (like
> + * alc5682) and we want that to be at 1.8V, not at some slightly lower voltage.
> + */
> +&vreg_l2c_1p8 {
> + regulator-min-microvolt = <1800000>;
> +};
> +
> /* ADDITIONS TO NODES DEFINED IN PARENT DEVICE TREE FILES */
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sc7280-herobrine: Audio codec wants 1.8V, not 1.62V
2022-03-25 15:42 ` Stephen Boyd
@ 2022-03-25 15:47 ` Doug Anderson
2022-03-25 18:09 ` Stephen Boyd
0 siblings, 1 reply; 4+ messages in thread
From: Doug Anderson @ 2022-03-25 15:47 UTC (permalink / raw)
To: Stephen Boyd
Cc: Bjorn Andersson, Matthias Kaehlcke, Judy Hsiao, Andy Gross,
Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
linux-arm-msm, LKML
Hi,
On Fri, Mar 25, 2022 at 8:42 AM Stephen Boyd <swboyd@chromium.org> wrote:
>
> Quoting Douglas Anderson (2022-03-25 08:11:05)
> > The L2C rail on herobrine boards is intended to go to the audio
> > codec. Let's override the 1.62V specified in the qcard.dtsi file to be
> > 1.8V.
> >
> > Signed-off-by: Douglas Anderson <dianders@chromium.org>
> > ---
>
> Reviewed-by: Stephen Boyd <swboyd@chromium.org>
>
> >
> > arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 10 ++++++++++
> > 1 file changed, 10 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> > index dc17f2079695..c2075f3e7c4b 100644
> > --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> > @@ -339,6 +339,16 @@ keyboard_backlight: keyboard-backlight {
> > ts_avdd: &pp3300_left_in_mlb {};
> > vreg_edp_3p3: &pp3300_left_in_mlb {};
> >
> > +/* Regulator overrides from Qcard */
>
> This one isn't in ALL CAPS? :)
Yeah, I guess it sorta felt like it was a sub-part of the previous
section since it was also adjusting Qcard regulators, but you're right
that it's not actually a board-local name. What about if I change the
section heading for "BOARD-LOCAL NAMES FOR REGULATORS THAT CONNECT TO
QCARD" to:
/*
* ADJUSTMENTS TO QCARD REGULATORS
*
* Mostly this is just board-local names for regulators that come from
* Qcard, but this also has some minor regulator overrides.
*
* Names are only listed here if regulators go somewhere other than a
* testpoint.
*/
I'll wait a few days and then send a v2 if that sounds OK.
-Doug
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sc7280-herobrine: Audio codec wants 1.8V, not 1.62V
2022-03-25 15:47 ` Doug Anderson
@ 2022-03-25 18:09 ` Stephen Boyd
0 siblings, 0 replies; 4+ messages in thread
From: Stephen Boyd @ 2022-03-25 18:09 UTC (permalink / raw)
To: Doug Anderson
Cc: Bjorn Andersson, Matthias Kaehlcke, Judy Hsiao, Andy Gross,
Rob Herring, devicetree, linux-arm-msm, LKML
Quoting Doug Anderson (2022-03-25 08:47:24)
>
> Yeah, I guess it sorta felt like it was a sub-part of the previous
> section since it was also adjusting Qcard regulators, but you're right
> that it's not actually a board-local name. What about if I change the
> section heading for "BOARD-LOCAL NAMES FOR REGULATORS THAT CONNECT TO
> QCARD" to:
>
> /*
> * ADJUSTMENTS TO QCARD REGULATORS
> *
> * Mostly this is just board-local names for regulators that come from
> * Qcard, but this also has some minor regulator overrides.
> *
> * Names are only listed here if regulators go somewhere other than a
> * testpoint.
> */
>
> I'll wait a few days and then send a v2 if that sounds OK.
Sounds good to me.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-03-25 19:42 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-25 15:11 [PATCH] arm64: dts: qcom: sc7280-herobrine: Audio codec wants 1.8V, not 1.62V Douglas Anderson
2022-03-25 15:42 ` Stephen Boyd
2022-03-25 15:47 ` Doug Anderson
2022-03-25 18:09 ` Stephen Boyd
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).