pinctrl: samsung: Drop redundant node parameter in samsung_banks_of_node_get()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Mon, 11 Apr 2022 11:49:05 +0000 (14:49 +0300)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 14 Apr 2022 18:44:58 +0000 (21:44 +0300)
The node is taken from the device pointer, which is supplied as a parameter,
hence no need to have a separate parameter for node. Drop redundant node
parameter in samsung_banks_of_node_get().

While at it, drop "of_" part in the samsung_banks_of_node_get() and
samsung_banks_of_node_put() function names to avoid additional churn
in the next changes.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Tested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
drivers/pinctrl/samsung/pinctrl-samsung.c

index f610beab23a0757346bd2c1db1df1d91b21a71b1..2ba8b43c25bd1426bcac5069ccf9d0c0b60ef8ac 100644 (file)
@@ -1002,7 +1002,7 @@ samsung_pinctrl_get_soc_data_for_of_alias(struct platform_device *pdev)
        return &(of_data->ctrl[id]);
 }
 
-static void samsung_banks_of_node_put(struct samsung_pinctrl_drv_data *d)
+static void samsung_banks_node_put(struct samsung_pinctrl_drv_data *d)
 {
        struct samsung_pin_bank *bank;
        unsigned int i;
@@ -1016,12 +1016,11 @@ static void samsung_banks_of_node_put(struct samsung_pinctrl_drv_data *d)
  * Iterate over all driver pin banks to find one matching the name of node,
  * skipping optional "-gpio" node suffix. When found, assign node to the bank.
  */
-static void samsung_banks_of_node_get(struct device *dev,
-                                     struct samsung_pinctrl_drv_data *d,
-                                     struct device_node *node)
+static void samsung_banks_node_get(struct device *dev, struct samsung_pinctrl_drv_data *d)
 {
        const char *suffix = "-gpio-bank";
        struct samsung_pin_bank *bank;
+       struct device_node *node = dev->of_node;
        struct device_node *child;
        /* Pin bank names are up to 4 characters */
        char node_name[20];
@@ -1061,7 +1060,6 @@ static const struct samsung_pin_ctrl *
 samsung_pinctrl_get_soc_data(struct samsung_pinctrl_drv_data *d,
                             struct platform_device *pdev)
 {
-       struct device_node *node = pdev->dev.of_node;
        const struct samsung_pin_bank_data *bdata;
        const struct samsung_pin_ctrl *ctrl;
        struct samsung_pin_bank *bank;
@@ -1125,7 +1123,7 @@ samsung_pinctrl_get_soc_data(struct samsung_pinctrl_drv_data *d,
         */
        d->virt_base = virt_base[0];
 
-       samsung_banks_of_node_get(&pdev->dev, d, node);
+       samsung_banks_node_get(&pdev->dev, d);
 
        d->pin_base = pin_base;
        pin_base += d->nr_pins;
@@ -1186,7 +1184,7 @@ static int samsung_pinctrl_probe(struct platform_device *pdev)
 err_unregister:
        samsung_pinctrl_unregister(pdev, drvdata);
 err_put_banks:
-       samsung_banks_of_node_put(drvdata);
+       samsung_banks_node_put(drvdata);
        return ret;
 }