On Tue, Oct 25, 2022 at 12:06:23PM +0200, AngeloGioacchino Del Regno wrote:
Il 25/10/22 00:00, Nícolas F. R. A. Prado ha scritto:
The rt5682s codec can have two supplies: AVDD and MICVDD. They are already used by sc7180-trogdor-kingoftown.dtsi, so document them in the binding.
Signed-off-by: Nícolas F. R. A. Prado nfraprado@collabora.com
I also don't like these uppercase supply names... I wonder if it's worth changing the driver to get "avdd" *or* "AVDD" (so, if "avdd" fails -> backwards compat)...
...this way, we can change the devicetree to use the lowercase names without breaking abi.
Of course, this commit would need to be changed to document only the lowercase supply names.
Driver-wise, we have a rt5682s_supply_names array... we could do something like:
static const char *rt5682s_supply_names_legacy[RT5682S_NUM_SUPPLIES] = { [RT5682S_SUPPLY_AVDD] = "AVDD", [RT5682S_SUPPLY_MICVDD] = "MICVDD", };
static const char *rt5682s_supply_names[RT5682S_NUM_SUPPLIES] = { [RT5682S_SUPPLY_AVDD] = "avdd", [RT5682S_SUPPLY_MICVDD] = "micvdd", };
for (...) assign_supply_names; ret = devm_regulator_bulk_get(...);
if (ret) { for (...) assign_legacy_supply_names; ret = devm_regulator_bulk_get(...) if (ret) return ret; }
What do you think?
No one seems opposed to it, so I'll add that to the next version.
Thanks, Nícolas