lkml.org 
[lkml]   [2020]   [Aug]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2] arm64: dts: qcom: sc7180-trogdor: add initial trogdor and lazor dt
Hi,

On Tue, Aug 25, 2020 at 3:27 PM Rob Clark <robdclark@gmail.com> wrote:
>
> + max98357a: audio-codec-0 {
> + compatible = "maxim,max98357a";
> + pinctrl-names = "default";
> + pinctrl-0 = <&amp_en>;
> + sdmode-gpios = <&tlmm 23 GPIO_ACTIVE_HIGH>;
> + #sound-dai-cells = <0>;
> + };

Even though Device Tree spec says "audio-codec", I agree that
"audio-codec-0" should be fine. Since this is at the top level that
could be important to disambiguate if there was more than one audio
codec.


> +&wifi {
> + status = "okay";
> + vdd-0.8-cx-mx-supply = <&vdd_cx_wlan>;
> + vdd-1.8-xo-supply = <&pp1800_l1c>;
> + vdd-1.3-rfa-supply = <&pp1300_l2c>;

Sorry, I should have been clearer. Here you should add:

vdd-3.3-ch0-supply = <&pp3300_l10c>;
vdd-3.3-ch1-supply = <&pp3300_l11c>;

...the hack that was downstream was to stick the "ch1" regulator in
the "ch0" slot since Bluetooth was already holding the "ch0" regulator
open and the "ch1" wasn't part of the bindings yet.

Once the supplies are added for WiFi feel free to add my Reviewed-by tag.

-Doug

\
 
 \ /
  Last update: 2020-08-26 00:44    [W:0.020 / U:1.976 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site