[PATCH 1/1] drivers/power/regulator: add SPL_DM_REGULATOR_FAN53555

[Quentin Schulz]

Hi Heinrich,

On 2/25/26 7:57 AM, Heinrich Schuchardt wrote:
> Symbol CONFIG_SPL_DM_REGULATOR_FAN53555 is selected by
> SPL_DM_PMIC_FAN53555 and used in a Makefile. But the
> symbol definition is missing.
> 

Well it is "used" in Makefile simply because I wanted to be able to NOT 
build it for xPL stages. I'm thinking we should use $(PHASE_) for most 
symbols in Makefile, even if we don't have an xPL symbol in any Kconfig 
file such that one would explicitly need to add an xPL symbol and enable 
it for the feature to be enabled in the xPL stage.

Maybe we even need a PROPER_ prefix for PHASE_ when not in xPL for 
example for things that make sense to be stage-specific. There are some 
symbols that should represent something enabled across stages (e.g. the 
CPU architecture) where the PROPER_ prefix wouldn't make sense. I'm 
often confused as to what is a symbol that must apply to all stages by 
design or simply because someone didn't bother to implement separate xPL 
switches for that (yet).

Anyway, we don't *need* an xPL symbol just because we have $(PHASE_) 
somewhere in some Makefile.

However, another symbol depends on it so this is clearly an issue.

Reviewed-by: Quentin Schulz <quentin.schulz at cherry.de>

Thanks!
Quentin