>> +static const struct soc_enum charge_pump_enum = >> + SOC_ENUM_SINGLE(CS42L52_CHARGE_PUMP, 4, >> + ARRAY_SIZE(charge_pump_freq_text), charge_pump_freq_text); > > Should this be runtime configurable? > Nope, I'll add it to platform data