mach-snapdragon: of_fixup: remove confusing log message
authorCaleb Connolly <caleb.connolly@linaro.org>
Fri, 11 Apr 2025 12:47:41 +0000 (14:47 +0200)
committerTom Rini <trini@konsulko.com>
Fri, 2 May 2025 14:38:02 +0000 (08:38 -0600)
The debug log here had the logic completely backwards, even though the
code is actually correct. Remove it since it's extraneous anyway.

Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
arch/arm/mach-snapdragon/of_fixup.c

index dcd09ee..a5515c0 100644 (file)
@@ -73,11 +73,12 @@ static int fixup_qcom_dwc3(struct device_node *root, struct device_node *glue_np
                return ret;
        }
 
-       if (!strncmp("usb3-phy", second_phy_name, strlen("usb3-phy"))) {
-               log_debug("Second phy isn't superspeed (is '%s') assuming first phy is SS\n",
-                         second_phy_name);
+       /*
+        * Determine which phy is the superspeed phy by checking the name of the second phy
+        * since it is typically the superspeed one.
+        */
+       if (!strncmp("usb3-phy", second_phy_name, strlen("usb3-phy")))
                hsphy_idx = 0;
-       }
 
        /* Overwrite the "phys" property to only contain the high-speed phy */
        ret = of_write_prop(dwc3, "phys", sizeof(*phandles), phandles + hsphy_idx);