On Mon, Dec 06, 2021 at 06:33:26PM +0530, Satya Priya Kakitapalli (Temp) wrote: > On 11/25/2021 8:47 PM, Mark Brown wrote: > > Usually this is a fixed property of the regulator rather than something > > that varies per board - why have a DT property? > The min-dropout value (headroom) varies with boards, that's why we have a DT > property for it. We overwrite the default value in driver with actual value > read from DT Interesting. How exactly does that end up happening - presumably other systems are going to run into it? If you do have board designs which somehow managed to introduce additional dropouts (seems pretty concerning TBH) then I think the best way to handle that is to add a generic property for it and have that either added on to or override the requirements of the regulator itself which should continue to be defined in the driver. That way only boards with issues need to do anything which will avoid bugs with the property being omitted in what should be the common case.
Attachment:
signature.asc
Description: PGP signature