]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
pinctrl: amd: Constify pointers to 'pinctrl_desc'
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Wed, 11 Jun 2025 06:13:35 +0000 (08:13 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 18 Jun 2025 11:26:35 +0000 (13:26 +0200)
Internal functions obtaining pointers to 'struct pinctrl_desc' do not
modify the contents so they can be made pointers to const.  This makes
code safer, explicit and later allows constifying 'pinctrl_desc' in
pinctrl core code.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/20250611-pinctrl-const-desc-v2-3-b11c1d650384@linaro.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-amd.c

index 5cf3db6d78b79af9ac5e7226de239a0d2f9aaacc..85617cfed2299e131c110e9d4084370790106404 100644 (file)
@@ -872,7 +872,7 @@ static const struct pinconf_ops amd_pinconf_ops = {
 
 static void amd_gpio_irq_init(struct amd_gpio *gpio_dev)
 {
-       struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
+       const struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
        unsigned long flags;
        u32 pin_reg, mask;
        int i;
@@ -901,7 +901,7 @@ static void amd_gpio_irq_init(struct amd_gpio *gpio_dev)
 static void amd_gpio_check_pending(void)
 {
        struct amd_gpio *gpio_dev = pinctrl_dev;
-       struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
+       const struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
        int i;
 
        if (!pm_debug_messages_on)
@@ -957,7 +957,7 @@ static bool amd_gpio_should_save(struct amd_gpio *gpio_dev, unsigned int pin)
 static int amd_gpio_suspend_hibernate_common(struct device *dev, bool is_suspend)
 {
        struct amd_gpio *gpio_dev = dev_get_drvdata(dev);
-       struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
+       const struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
        unsigned long flags;
        int i;
        u32 wake_mask = is_suspend ? WAKE_SOURCE_SUSPEND : WAKE_SOURCE_HIBERNATE;
@@ -1001,7 +1001,7 @@ static int amd_gpio_hibernate(struct device *dev)
 static int amd_gpio_resume(struct device *dev)
 {
        struct amd_gpio *gpio_dev = dev_get_drvdata(dev);
-       struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
+       const struct pinctrl_desc *desc = gpio_dev->pctrl->desc;
        unsigned long flags;
        int i;