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@xxxxxxxxxxxxx>
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?
Cheers,
Angelo
