On 5/19/23 10:41, Lukasz Majewski wrote:
Hello Lukasz,
On 5/9/23 16:32, Lukasz Majewski wrote:
This option sets the current limit for 5V source to zero, so all
the PMU outputs are primarily powered from battery source
(DCDC_BAT).
This option may be set on systems, where the 5V is NOT supposed
to be in any scenario powering the system - for example on
systems where DCDC_BAT is connected to fixed and regulated 4.2V
source (so the "battery" is not present).
Is it possible to dig this information out of DT or is the DT not
available in SPL for this platform ?
This platform uses only SPL_OF_PLATDATA to keep the SPL size < 40
KiB.
I would prefer to keep this as Kconfig as the PMU driver is not
supporting DT and in fact now ARCH_MX28 only has two boards
supported - XEA and imx28evk.
Why not use the platdata then ?
This makes it far better, as the code is actually always compiled
and doesn't start to bitrot.
As fair as I can tell - the MX28 PMU driver in SPL is not supporting
OF_PLATDATA. Or have I overlooked something?
I would like to investigate the PMU issue, and avoid extra work to
provide platdata support for two supported imx287 based boards.
Hum, considering that this platform is basically legacy and winding
down, I guess that's OK too.
Are there any other issues to be solved? Or is this patch set eligible
for pulling?
(Especially that NXP has confirmed that approach used in this patch set
is in sync with their idea of PMU usage).
I think this is just waiting for Stefano to pick and do a MR
