]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
regulator: pca9450: Constify pointers to 'regulator_desc' wrap struct
authorKrzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Wed, 26 Nov 2025 16:58:50 +0000 (17:58 +0100)
committerMark Brown <broonie@kernel.org>
Wed, 26 Nov 2025 18:44:31 +0000 (18:44 +0000)
Pointer to 'struct regulator_desc' is a pointer to const and the
wrapping structure (container) is not being modified, thus entire syntax
can be replaced to preferred and safer container_of_const().

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Link: https://patch.msgid.link/20251126-container-of-const-regulator-v1-7-eeec378144d4@oss.qualcomm.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/pca9450-regulator.c

index 4be270f4d6c35a37bb26ccf16918ff897146e80f..ec6fcf59a4c7caeb23e50e516fb2db3b2ce73be2 100644 (file)
@@ -249,7 +249,7 @@ static int buck_set_dvs(const struct regulator_desc *desc,
        }
 
        if (ret == 0) {
-               struct pca9450_regulator_desc *regulator = container_of(desc,
+               const struct pca9450_regulator_desc *regulator = container_of_const(desc,
                                        struct pca9450_regulator_desc, desc);
 
                /* Enable DVS control through PMIC_STBY_REQ for this BUCK */
@@ -263,7 +263,7 @@ static int pca9450_set_dvs_levels(struct device_node *np,
                            const struct regulator_desc *desc,
                            struct regulator_config *cfg)
 {
-       struct pca9450_regulator_desc *data = container_of(desc,
+       const struct pca9450_regulator_desc *data = container_of_const(desc,
                                        struct pca9450_regulator_desc, desc);
        const struct pc9450_dvs_config *dvs = &data->dvs;
        unsigned int reg, mask;
@@ -308,7 +308,7 @@ static inline unsigned int pca9450_map_mode(unsigned int mode)
 
 static int pca9450_buck_set_mode(struct regulator_dev *rdev, unsigned int mode)
 {
-       struct pca9450_regulator_desc *desc = container_of(rdev->desc,
+       const struct pca9450_regulator_desc *desc = container_of_const(rdev->desc,
                                        struct pca9450_regulator_desc, desc);
        const struct pc9450_dvs_config *dvs = &desc->dvs;
        int val;
@@ -333,7 +333,7 @@ static int pca9450_buck_set_mode(struct regulator_dev *rdev, unsigned int mode)
 
 static unsigned int pca9450_buck_get_mode(struct regulator_dev *rdev)
 {
-       struct pca9450_regulator_desc *desc = container_of(rdev->desc,
+       const struct pca9450_regulator_desc *desc = container_of_const(rdev->desc,
                                        struct pca9450_regulator_desc, desc);
        const struct pc9450_dvs_config *dvs = &desc->dvs;
        int ret = 0, regval;