[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZSlkWZvtEFfSkqNE@finisterre.sirena.org.uk>
Date: Fri, 13 Oct 2023 16:38:01 +0100
From: Mark Brown <broonie@...nel.org>
To: Ninad Palsule <ninad@...ux.ibm.com>
Cc: joel@....id.au, eajames@...ux.ibm.com, lgirdwood@...il.com,
linux-kernel@...r.kernel.org, lakshmiy@...ibm.com
Subject: Re: [PATCH v1] regulator: core: Convert warning to debug print
On Thu, Oct 12, 2023 at 04:45:48PM -0500, Ninad Palsule wrote:
> You are right there are regulators for these supplies but they are managed
> by the
> hardware hence not added in the device tree. I checked dts/aspeed directory
> and
> non of the machine has “vcc-supply” defined and lot of them use eeprom.
Fixed voltage regulators can be used for this.
> Also, I thought that this message is only useful in the lab to indicate
> developer
> whether they missed the device tree definition or not but its not useful in
> the
> field. Hence proposed to put it under DEBUG.
The theory is that it shouldn't come up in the field since people see it
during development and fix things then, are people actually looking at
these logs? I think my understanding was that they're mostly BMCs
people interacted with via a UI of some kind.
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists