linux-kernel - Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Tue, 8 Nov 2022 14:42:02 +0000
From:   Bryan O'Donoghue <bryan.odonoghue@...aro.org>
To:     Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        robdclark@...il.com, quic_abhinavk@...cinc.com,
        krzysztof.kozlowski+dt@...aro.org, robh+dt@...nel.org,
        quic_mkrishn@...cinc.com, linux-arm-msm@...r.kernel.org
Cc:     Sean Paul <sean@...rly.run>, David Airlie <airlied@...il.com>,
        Daniel Vetter <daniel@...ll.ch>,
        dri-devel@...ts.freedesktop.org, freedreno@...ts.freedesktop.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd*
 descriptions back in

On 08/11/2022 12:59, Dmitry Baryshkov wrote:
>>
>> Warnings about missing regulators can be resolved by updating the 
>> relevant
>> dtsi files to point to fixed always-on regulators where appropriate.
> 
> Ugh. Are they missing or are they optional/not used on these platforms?

Some platforms either don't implement them or worse possibly do 
implement but don't model them when they should.

> Can you possibly list all regulator warnings?


Downstream we have

arch/arm/boot/dts/qcom/msm8916-mdss.dtsi

mdss_dsi0: qcom,mdss_dsi@...8000 {
	vdda-supply = <&pm8916_l2>;
	vdd-supply = <&pm8916_l17>;
	vddio-supply = <&pm8916_l6>;
};

Looking at something like

arch/arm/boot/dts/qcom/msm8916-mtp.dtsi which references 
arch/arm/boot/dts/qcom/dsi-panel-jdi-1080p-video.dtsi it doesn't appear 
to delete andy of the vdd*-supply references

apq8016-sbc.dtb: dsi@...8000: 'vdd-supply' is a required property
msm8916-samsung-a3u-eur.dtb: dsi@...8000: 'vdd-supply' is a required 
property
msm8916-samsung-a5u-eur.dtb: dsi@...8000: 'vdd-supply' is a required 
property
msm8916-samsung-e5.dtb: dsi@...8000: 'vdd-supply' is a required property
msm8916-samsung-e7.dtb: dsi@...8000: 'vdd-supply' is a required property
msm8916-samsung-grandmax.dtb: dsi@...8000: 'vdd-supply' is a required 
property
msm8996-xiaomi-natrium.dtb: dsi@...000: 'panel@0', 'vcca-supply' do not 
match any of the regexes: 'pinctrl-[0-9]+'
msm8996-xiaomi-scorpio.dtb: dsi@...000: 'vdda-supply' is a required property
qrb5165-rb5.dtb: dsi@...4000: 'vdd-supply' is a required property
qrb5165-rb5.dtb: dsi@...4000: 'vddio-supply' is a required property
sc7180-idp.dtb: dsi@...4000: 'vdd-supply' is a required property
sc7180-idp.dtb: dsi@...4000: 'vddio-supply' is a required property
sc7180-trogdor-coachz-r1.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-coachz-r1.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-coachz-r1-lte.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-coachz-r1-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-coachz-r3.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-coachz-r3.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-coachz-r3-lte.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-coachz-r3-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-homestar-r2.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-homestar-r2.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-homestar-r3.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-homestar-r3.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-homestar-r4.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-homestar-r4.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-kingoftown-r0.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-kingoftown-r0.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-kingoftown-r1.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-kingoftown-r1.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-r0.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r0.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-lazor-r1.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r1.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-lazor-r1-kb.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r1-kb.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-r1-lte.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r1-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-r3.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r3.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-lazor-r3-kb.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r3-kb.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-r3-lte.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r3-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-r9.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r9.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-lazor-r9-kb.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r9-kb.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-r9-lte.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-lazor-r9-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-limozeen-r4.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-lazor-limozeen-r4.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-limozeen-r9.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-lazor-limozeen-r9.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-lazor-limozeen-nots-r4.dtb: dsi@...4000: 'vdd-supply' is 
a required property
sc7180-trogdor-lazor-limozeen-nots-r4.dtb: dsi@...4000: 'vddio-supply' 
is a required property
sc7180-trogdor-lazor-limozeen-nots-r5.dtb: dsi@...4000: 'vdd-supply' is 
a required property
sc7180-trogdor-lazor-limozeen-nots-r5.dtb: dsi@...4000: 'vddio-supply' 
is a required property
sc7180-trogdor-lazor-limozeen-nots-r9.dtb: dsi@...4000: 'vdd-supply' is 
a required property
sc7180-trogdor-lazor-limozeen-nots-r9.dtb: dsi@...4000: 'vddio-supply' 
is a required property
sc7180-trogdor-mrbland-rev0-auo.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-mrbland-rev0-auo.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-mrbland-rev0-boe.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-mrbland-rev0-boe.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-mrbland-rev1-auo.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-mrbland-rev1-auo.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-mrbland-rev1-boe.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-mrbland-rev1-boe.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-pazquel-lte-parade.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-pazquel-lte-parade.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-pazquel-lte-ti.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-pazquel-lte-ti.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-pazquel-parade.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-pazquel-parade.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-pazquel-ti.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-pazquel-ti.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-pompom-r1.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-pompom-r1.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-pompom-r1-lte.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-pompom-r1-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-pompom-r2.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-pompom-r2.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-pompom-r2-lte.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-pompom-r2-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-pompom-r3.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-pompom-r3.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sc7180-trogdor-pompom-r3-lte.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-pompom-r3-lte.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-quackingstick-r0.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-quackingstick-r0.dtb: dsi@...4000: 'vddio-supply' is a 
required property
sc7180-trogdor-quackingstick-r0-lte.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-quackingstick-r0-lte.dtb: dsi@...4000: 'vddio-supply' is 
a required property
sc7180-trogdor-wormdingler-rev0-boe.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-wormdingler-rev0-boe.dtb: dsi@...4000: 'vddio-supply' is 
a required property
sc7180-trogdor-wormdingler-rev0-inx.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-wormdingler-rev0-inx.dtb: dsi@...4000: 'vddio-supply' is 
a required property
sc7180-trogdor-wormdingler-rev1-boe.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-wormdingler-rev1-boe.dtb: dsi@...4000: 'vddio-supply' is 
a required property
sc7180-trogdor-wormdingler-rev1-inx.dtb: dsi@...4000: 'vdd-supply' is a 
required property
sc7180-trogdor-wormdingler-rev1-inx.dtb: dsi@...4000: 'vddio-supply' is 
a required property
sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb: dsi@...4000: 
'vdd-supply' is a required property
sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb: dsi@...4000: 
'vddio-supply' is a required property
sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb: dsi@...4000: 
'vdd-supply' is a required property
sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb: dsi@...4000: 
'vddio-supply' is a required property
sc7180-trogdor-r1.dtb: dsi@...4000: 'vdd-supply' is a required property
sc7180-trogdor-r1.dtb: dsi@...4000: 'vddio-supply' is a required property
sc7180-trogdor-r1-lte.dtb: dsi@...4000: 'vdd-supply' is a required property
sc7180-trogdor-r1-lte.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sda660-inforce-ifc6560.dtb: dsi@...4000: 'vdd-supply' is a required property
sda660-inforce-ifc6560.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sdm845-cheza-r1.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-cheza-r1.dtb: dsi@...4000: 'vddio-supply' is a required property
sdm845-cheza-r2.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-cheza-r2.dtb: dsi@...4000: 'vddio-supply' is a required property
sdm845-cheza-r3.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-cheza-r3.dtb: dsi@...4000: 'vddio-supply' is a required property
sdm845-db845c.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-db845c.dtb: dsi@...4000: 'vddio-supply' is a required property
sdm845-mtp.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-mtp.dtb: dsi@...4000: 'vddio-supply' is a required property
sdm845-mtp.dtb: dsi@...6000: 'vdd-supply' is a required property
sdm845-mtp.dtb: dsi@...6000: 'vddio-supply' is a required property
sdm845-oneplus-enchilada.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sdm845-oneplus-enchilada.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sdm845-oneplus-fajita.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-oneplus-fajita.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sdm845-xiaomi-beryllium.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sdm845-xiaomi-beryllium.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sdm845-xiaomi-polaris.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-xiaomi-polaris.dtb: dsi@...4000: 'vddio-supply' is a required 
property
sdm845-shift-axolotl.dtb: dsi@...4000: 'vdd-supply' is a required property
sdm845-shift-axolotl.dtb: dsi@...4000: 'vddio-supply' is a required property
sdm850-lenovo-yoga-c630.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sdm850-lenovo-yoga-c630.dtb: dsi@...4000: 'vddio-supply' is a required 
property

apq8016-sbc.dtb: dsi@...8000: 'vdd-supply' is a required property
msm8916-samsung-a5u-eur.dtb: dsi@...8000: 'vdd-supply' is a required 
property
sc7180-trogdor-homestar-r4.dtb: dsi@...4000: 'vdd-supply' is a required 
property
sc7180-trogdor-homestar-r4.dtb: dsi@...4000: 'vddio-supply' is a 
required property