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 <[email protected]> Thanks! Quentin
