regulator: arizona-micsupp: Don't use a common regulator name
authorRichard Fitzgerald <rf@opensource.cirrus.com>
Wed, 9 Nov 2022 16:53:26 +0000 (16:53 +0000)
committerMark Brown <broonie@kernel.org>
Tue, 22 Nov 2022 17:05:04 +0000 (17:05 +0000)
The Arizona and Madera codecs all have a datasheet name of "MICVDD"
for the regulator output. But future codecs with a regulator that
can be controlled by this driver have different naming convention
for the output of the regulator.

Move the setting of the supply name from arizona_micsupp_common_init()
to arizona_micsupp_probe() and madera_micsupp_probe().

Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20221109165331.29332-8-rf@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/arizona-micsupp.c

index 21c36972a8e98489f724c3d3f383bf73e8bbf214..596ecd8041cdc71e1fc6a32cfa1cf28f1b830f03 100644 (file)
@@ -262,7 +262,6 @@ static int arizona_micsupp_common_init(struct platform_device *pdev,
        INIT_WORK(&micsupp->check_cp_work, arizona_micsupp_check_cp);
 
        micsupp->init_data.consumer_supplies = &micsupp->supply;
-       micsupp->supply.supply = "MICVDD";
        micsupp->supply.dev_name = dev_name(micsupp->dev);
        micsupp->desc = desc;
 
@@ -319,6 +318,8 @@ static int arizona_micsupp_probe(struct platform_device *pdev)
        micsupp->dapm = &arizona->dapm;
        micsupp->dev = arizona->dev;
 
+       micsupp->supply.supply = "MICVDD";
+
        /*
         * Since the chip usually supplies itself we provide some
         * default init_data for it.  This will be overridden with
@@ -354,6 +355,8 @@ static int madera_micsupp_probe(struct platform_device *pdev)
        micsupp->dev = madera->dev;
        micsupp->init_data = arizona_micsupp_ext_default;
 
+       micsupp->supply.supply = "MICVDD";
+
        return arizona_micsupp_common_init(pdev, micsupp, &madera_micsupp,
                                           &madera->pdata.micvdd);
 }