]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
regulator: bd96801: Constify pointers to 'regulator_desc' wrap struct
authorKrzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Wed, 26 Nov 2025 16:58:48 +0000 (17:58 +0100)
committerMark Brown <broonie@kernel.org>
Wed, 26 Nov 2025 18:44:30 +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-5-eeec378144d4@oss.qualcomm.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/bd96801-regulator.c

index 24d21172298b31b0f85537e9b1eeaab1b0f0a79b..129b20c33badcddeff04f361f4c9d677e7d3e9de 100644 (file)
@@ -337,12 +337,12 @@ static int ldo_map_notif(int irq, struct regulator_irq_data *rid,
        int i;
 
        for (i = 0; i < rid->num_states; i++) {
-               struct bd96801_regulator_data *rdata;
+               const struct bd96801_regulator_data *rdata;
                struct regulator_dev *rdev;
 
                rdev = rid->states[i].rdev;
-               rdata = container_of(rdev->desc, struct bd96801_regulator_data,
-                                    desc);
+               rdata = container_of_const(rdev->desc, struct bd96801_regulator_data,
+                                          desc);
                rid->states[i].notifs = regulator_err2notif(rdata->ldo_errs);
                rid->states[i].errors = rdata->ldo_errs;
                *dev_mask |= BIT(i);
@@ -354,9 +354,9 @@ static int bd96801_list_voltage_lr(struct regulator_dev *rdev,
                                   unsigned int selector)
 {
        int voltage;
-       struct bd96801_regulator_data *data;
+       const struct bd96801_regulator_data *data;
 
-       data = container_of(rdev->desc, struct bd96801_regulator_data, desc);
+       data = container_of_const(rdev->desc, struct bd96801_regulator_data, desc);
 
        /*
         * The BD096801 has voltage setting in two registers. One giving the