linux-kernel - Re: [PATCH 1/2] dt-bindings: power: Add regulator-pd yaml file

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <PAXPR04MB9185D6762854207129D81CC789E7A@PAXPR04MB9185.eurprd04.prod.outlook.com>
Date:   Tue, 29 Aug 2023 13:21:40 +0000
From:   Shenwei Wang <shenwei.wang@....com>
To:     Rob Herring <robh+dt@...nel.org>
CC:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Ulf Hansson <ulf.hansson@...aro.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Liam Girdwood <lgirdwood@...il.com>,
        Mark Brown <broonie@...nel.org>,
        "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        dl-linux-imx <linux-imx@....com>
Subject: Re: [PATCH 1/2] dt-bindings: power: Add regulator-pd yaml file



> -----Original Message-----
> From: Rob Herring <robh+dt@...nel.org>
> Sent: Monday, August 28, 2023 4:14 PM
> To: Shenwei Wang <shenwei.wang@....com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>; Ulf Hansson
> <ulf.hansson@...aro.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@...aro.org>; Conor Dooley <conor+dt@...nel.org>;
> Liam Girdwood <lgirdwood@...il.com>; Mark Brown <broonie@...nel.org>;
> imx@...ts.linux.dev; devicetree@...r.kernel.org; linux-kernel@...r.kernel.org;
> dl-linux-imx <linux-imx@....com>
> Subject: [EXT] Re: [PATCH 1/2] dt-bindings: power: Add regulator-pd yaml file
> > > > The fixed-regulator is a virtual regulator driver that uses the GPIO pin.
> > >
> > > We do not talk about drivers but bindings and DTS. Why do you bring
> > > again drivers, all the time?
> > >
> > > > You claimed this
> > > > as a hardware chip.
> > >
> > > ??? Sorry, this is getting boring. The DTS-snippet is a hardware chip.
> > > If it is not, then drop it from your DTS. I insist. Srsly, third time I insist.
> > >
> > >
> > > >
> > > > The regulator-pd driver also uses the same GPIO pin.
> > >
> > > Again, what is with the drivers? Can you stop bringing it to the discussion?
> > >
> >
> > I have to admit you have a real talent for debate.
> 
> It takes 2...
> 
> You've gotten feedback from multiple people that your proposal is not going to
> be accepted. The prior attempt of the same thing had similar feedback from
> even more people. Please go re-read the responses until you understand.
> 
> For fixed-regulator, I can tell you very easily what the h/w looks like:
> 
> Vfix---|gate|---Vfix-gated
>             |
> GPIO--------|
> 
> 'gate' here may be a chip or discrete transistor. That's a very common board
> level component.
> 

The difference is in how we model the hardware. In your example, you model the GPIO 
as a simple switch to fit the fixed regulator use case. However, we could also model the 
same GPIO as a power domain if we consider the device connected to it. 
This allows for more nuanced hardware modeling based on the context and components 
involved.

Regulator-1 -+-> [Device A] 

This give you one regulator(via GPIO Pin) and one power domain (Device A).

The following are the example diagram given by the power domain overview doc:

                     Regulator-1 -+-> Regulator-2 -+-> [Consumer A]
                                  |
                                  +-> [Consumer B]

                   This gives us two regulators and two power domains:

                   - Domain 1: Regulator-2, Consumer B.
                   - Domain 2: Consumer A.

Thanks,
Shenwei

> If you want to discuss this any further, describe the h/w in terms of simplified
> schematics. Otherwise, there is nothing more to discuss.
> 
> Rob