]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
pinctrl: tegra-xusb: Return void in padctl enable/disable functions
authorKrzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Sun, 18 Jan 2026 18:09:31 +0000 (19:09 +0100)
committerLinus Walleij <linusw@kernel.org>
Mon, 19 Jan 2026 00:11:47 +0000 (01:11 +0100)
Make the padctl functions a bit simpler by returning void instead of
always '0'.  The callers - phy init/exit - still need to return 0, but
these are smaller function without if/branching.

Acked-by: Thierry Reding <treding@nvidia.com>
Reviewed-by: Jon Hunter <jonathanh@nvidia.com>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Linus Walleij <linusw@kernel.org>
drivers/pinctrl/tegra/pinctrl-tegra-xusb.c

index 49c5edeba87f829be181634d396055202297801f..34a373f7e04dd9441dba2f33477d0015b229eae5 100644 (file)
@@ -474,7 +474,7 @@ static const struct pinconf_ops tegra_xusb_padctl_pinconf_ops = {
 #endif
 };
 
-static int tegra_xusb_padctl_enable(struct tegra_xusb_padctl *padctl)
+static void tegra_xusb_padctl_enable(struct tegra_xusb_padctl *padctl)
 {
        u32 value;
 
@@ -501,10 +501,9 @@ static int tegra_xusb_padctl_enable(struct tegra_xusb_padctl *padctl)
 
 out:
        mutex_unlock(&padctl->lock);
-       return 0;
 }
 
-static int tegra_xusb_padctl_disable(struct tegra_xusb_padctl *padctl)
+static void tegra_xusb_padctl_disable(struct tegra_xusb_padctl *padctl)
 {
        u32 value;
 
@@ -534,21 +533,24 @@ static int tegra_xusb_padctl_disable(struct tegra_xusb_padctl *padctl)
 
 out:
        mutex_unlock(&padctl->lock);
-       return 0;
 }
 
 static int tegra_xusb_phy_init(struct phy *phy)
 {
        struct tegra_xusb_padctl *padctl = phy_get_drvdata(phy);
 
-       return tegra_xusb_padctl_enable(padctl);
+       tegra_xusb_padctl_enable(padctl);
+
+       return 0;
 }
 
 static int tegra_xusb_phy_exit(struct phy *phy)
 {
        struct tegra_xusb_padctl *padctl = phy_get_drvdata(phy);
 
-       return tegra_xusb_padctl_disable(padctl);
+       tegra_xusb_padctl_disable(padctl);
+
+       return 0;
 }
 
 static int pcie_phy_power_on(struct phy *phy)