linux-kernel - Re: [PATCH] arm64: dts: rockchip: Drop invalid regulator-init-microvolt property

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] [day] [month] [year] [list]

Date:   Mon, 10 Jul 2023 17:39:55 +0200
From:   Heiko Stuebner <heiko@...ech.de>
To:     Mark Kettenis <mark.kettenis@...all.nl>, robh+dt@...nel.org,
        conor+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org
Cc:     d3adme4t@...il.com, macromorgan@...mail.com, jbx6244@...il.com,
        f.kardame@...jaro.org, amadeus@....edu.cn, linux.amoon@...il.com,
        aurelien@...el32.net, anarsoul@...il.com, wiagn233@...look.com,
        frattaroli.nicolas@...il.com, strit@...jaro.org,
        luiz.von.dentz@...el.com, zonyitoo@...il.com, wens@...e.org,
        jensenhuang@...endlyarm.com, lasstp5011@...il.com,
        frank-w@...lic-files.de, pgwipeout@...il.com, leo@...am.net,
        andyshrk@....com, michael.riesch@...fvision.net, jonas@...boo.se,
        festevam@...x.de, tobetter@...il.com, jagan@...rulasolutions.com,
        cnsztl@...il.com, cristian.ciocaltea@...labora.com,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-rockchip@...ts.infradead.org, kernel@...labora.com
Subject: Re: [PATCH] arm64: dts: rockchip: Drop invalid regulator-init-microvolt
 property

Am Montag, 10. Juli 2023, 16:35:36 CEST schrieb Mark Kettenis:
> > From: Heiko Stuebner <heiko@...ech.de>
> > Date: Mon, 10 Jul 2023 16:16:16 +0200
> > 
> > On Fri, 7 Jul 2023 19:22:17 +0300, Cristian Ciocaltea wrote:
> > > The 'regulator-init-microvolt' property is not currently supported by
> > > any driver, it was simply carried on from downstream kernels.
> > > 
> > > The problem is also indicated by the following dtbs_check warning:
> > > 
> > >   rk3588-rock-5b.dtb: pmic@0: regulators:dcdc-reg4: Unevaluated properties are not allowed ('regulator-init-microvolt' was unexpected)
> > > 
> > > [...]
> > 
> > Applied, thanks!
> > 
> > [1/1] arm64: dts: rockchip: Drop invalid regulator-init-microvolt property
> >       commit: 4d08b19629495b29601991d09d07865694c25199
> 
> This property *is used* by the drivers in U-Boot.  Dropping this from
> the Linux DTBs will likely result in broken boards the next time the
> U-Boot DTBs are synched again from Linux.  At least that is what
> happened before...
> 
> I think the right solution is to add this property to the DT binding
> instead.

I do have this vague memory of this coming up in the past, though don't
remember what the resolution was then? Though it definitly doesn't look
like the property was added to the binding in the meantime.

Also I think that setting up initial regulator state would be (and
seemingly is) for firmware to handle, so that property should instead
be in the -uboot.dtsi.

That DT isn't a configuration-space also is a decade long mantra.


Heiko