phy: drop probe registration printks

jhovold · vinodkoul · commit 95463cbb4fe6 · 2025-06-16T22:54:21.000+05:30
Drivers should generally be quiet on successful probe, but this is not followed by some PHY drivers, for example: snps-eusb2-hsphy 88e1000.phy: Registered Snps-eUSB2 phy qcom-eusb2-repeater c432000.spmi:pmic@7:phy@fd00: Registered Qcom-eUSB2 repeater qcom-eusb2-repeater c432000.spmi:pmic@a:phy@fd00: Registered Qcom-eUSB2 repeater qcom-eusb2-repeater c432000.spmi:pmic@b:phy@fd00: Registered Qcom-eUSB2 repeater snps-eusb2-hsphy fd3000.phy: Registered Snps-eUSB2 phy snps-eusb2-hsphy fd9000.phy: Registered Snps-eUSB2 phy snps-eusb2-hsphy fde000.phy: Registered Snps-eUSB2 phy snps-eusb2-hsphy 88e0000.phy: Registered Snps-eUSB2 phy snps-eusb2-hsphy 88e2000.phy: Registered Snps-eUSB2 phy Drop (or demote to debug level) unnecessary registration info messages to make boot logs a little less noisy. Signed-off-by: Johan Hovold <johan+linaro@kernel.org> Link: https://lore.kernel.org/r/20250523085112.11287-1-johan+linaro@kernel.org Signed-off-by: Vinod Koul <vkoul@kernel.org>
diff --git a/drivers/phy/broadcom/phy-bcm-ns2-pcie.c b/drivers/phy/broadcom/phy-bcm-ns2-pcie.c
@@ -61,8 +61,6 @@ static int ns2_pci_phy_probe(struct mdio_device *mdiodev)
 		return PTR_ERR(provider);
 	}
 
-	dev_info(dev, "%s PHY registered\n", dev_name(dev));
-
 	return 0;
 }
 
diff --git a/drivers/phy/broadcom/phy-bcm-ns2-usbdrd.c b/drivers/phy/broadcom/phy-bcm-ns2-usbdrd.c
@@ -395,7 +395,6 @@ static int ns2_drd_phy_probe(struct platform_device *pdev)
 
 	platform_set_drvdata(pdev, driver);
 
-	dev_info(dev, "Registered NS2 DRD Phy device\n");
 	queue_delayed_work(system_power_efficient_wq, &driver->wq_extcon,
 			   driver->debounce_jiffies);
 
diff --git a/drivers/phy/broadcom/phy-bcm-sr-pcie.c b/drivers/phy/broadcom/phy-bcm-sr-pcie.c
@@ -277,8 +277,6 @@ static int sr_pcie_phy_probe(struct platform_device *pdev)
 		return PTR_ERR(provider);
 	}
 
-	dev_info(dev, "Stingray PCIe PHY driver initialized\n");
-
 	return 0;
 }
 
diff --git a/drivers/phy/broadcom/phy-brcm-sata.c b/drivers/phy/broadcom/phy-brcm-sata.c
@@ -832,7 +832,7 @@ static int brcm_sata_phy_probe(struct platform_device *pdev)
 		return PTR_ERR(provider);
 	}
 
-	dev_info(dev, "registered %d port(s)\n", count);
+	dev_dbg(dev, "registered %d port(s)\n", count);
 
 	return 0;
 }
diff --git a/drivers/phy/marvell/phy-pxa-usb.c b/drivers/phy/marvell/phy-pxa-usb.c
@@ -325,7 +325,6 @@ static int pxa_usb_phy_probe(struct platform_device *pdev)
 		phy_create_lookup(pxa_usb_phy->phy, "usb", "mv-otg");
 	}
 
-	dev_info(dev, "Marvell PXA USB PHY");
 	return 0;
 }
 
diff --git a/drivers/phy/phy-snps-eusb2.c b/drivers/phy/phy-snps-eusb2.c
@@ -599,8 +599,6 @@ static int snps_eusb2_hsphy_probe(struct platform_device *pdev)
 	if (IS_ERR(phy_provider))
 		return PTR_ERR(phy_provider);
 
-	dev_info(dev, "Registered Snps-eUSB2 phy\n");
-
 	return 0;
 }
 
diff --git a/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c b/drivers/phy/qualcomm/phy-qcom-eusb2-repeater.c
@@ -264,8 +264,6 @@ static int eusb2_repeater_probe(struct platform_device *pdev)
 	if (IS_ERR(phy_provider))
 		return PTR_ERR(phy_provider);
 
-	dev_info(dev, "Registered Qcom-eUSB2 repeater\n");
-
 	return 0;
 }
 
diff --git a/drivers/phy/qualcomm/phy-qcom-m31.c b/drivers/phy/qualcomm/phy-qcom-m31.c
@@ -305,8 +305,6 @@ static int m31usb_phy_probe(struct platform_device *pdev)
 	phy_set_drvdata(qphy->phy, qphy);
 
 	phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
-	if (!IS_ERR(phy_provider))
-		dev_info(dev, "Registered M31 USB phy\n");
 
 	return PTR_ERR_OR_ZERO(phy_provider);
 }
diff --git a/drivers/phy/qualcomm/phy-qcom-qusb2.c b/drivers/phy/qualcomm/phy-qcom-qusb2.c
@@ -1114,9 +1114,7 @@ static int qusb2_phy_probe(struct platform_device *pdev)
 	phy_set_drvdata(generic_phy, qphy);
 
 	phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
-	if (!IS_ERR(phy_provider))
-		dev_info(dev, "Registered Qcom-QUSB2 phy\n");
-	else
+	if (IS_ERR(phy_provider))
 		pm_runtime_disable(dev);
 
 	return PTR_ERR_OR_ZERO(phy_provider);
diff --git a/drivers/phy/st/phy-stih407-usb.c b/drivers/phy/st/phy-stih407-usb.c
@@ -139,8 +139,6 @@ static int stih407_usb2_picophy_probe(struct platform_device *pdev)
 	if (IS_ERR(phy_provider))
 		return PTR_ERR(phy_provider);
 
-	dev_info(dev, "STiH407 USB Generic picoPHY driver probed!");
-
 	return 0;
 }
 
diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
@@ -757,8 +757,8 @@ static int stm32_usbphyc_probe(struct platform_device *pdev)
 	}
 
 	version = readl_relaxed(usbphyc->base + STM32_USBPHYC_VERSION);
-	dev_info(dev, "registered rev:%lu.%lu\n",
-		 FIELD_GET(MAJREV, version), FIELD_GET(MINREV, version));
+	dev_dbg(dev, "registered rev: %lu.%lu\n",
+		FIELD_GET(MAJREV, version), FIELD_GET(MINREV, version));
 
 	return 0;
 
diff --git a/drivers/phy/ti/phy-twl4030-usb.c b/drivers/phy/ti/phy-twl4030-usb.c
@@ -784,7 +784,6 @@ static int twl4030_usb_probe(struct platform_device *pdev)
 	pm_runtime_mark_last_busy(&pdev->dev);
 	pm_runtime_put_autosuspend(twl->dev);
 
-	dev_info(&pdev->dev, "Initialized TWL4030 USB module\n");
 	return 0;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -61,8 +61,6 @@ static int ns2_pci_phy_probe(struct mdio_device *mdiodev)`
`61`	`61`	`return PTR_ERR(provider);`
`62`	`62`	`}`
`63`	`63`
`64`		`- dev_info(dev, "%s PHY registered\n", dev_name(dev));`
`65`		`-`
`66`	`64`	`return 0;`
`67`	`65`	`}`
`68`	`66`
Original file line number	Diff line number	Diff line change
`@@ -277,8 +277,6 @@ static int sr_pcie_phy_probe(struct platform_device *pdev)`
`277`	`277`	`return PTR_ERR(provider);`
`278`	`278`	`}`
`279`	`279`
`280`		`- dev_info(dev, "Stingray PCIe PHY driver initialized\n");`
`281`		`-`
`282`	`280`	`return 0;`
`283`	`281`	`}`
`284`	`282`
Original file line number	Diff line number	Diff line change
`@@ -832,7 +832,7 @@ static int brcm_sata_phy_probe(struct platform_device *pdev)`
`832`	`832`	`return PTR_ERR(provider);`
`833`	`833`	`}`
`834`	`834`
`835`		`- dev_info(dev, "registered %d port(s)\n", count);`
	`835`	`+ dev_dbg(dev, "registered %d port(s)\n", count);`
`836`	`836`
`837`	`837`	`return 0;`
`838`	`838`	`}`
Original file line number	Diff line number	Diff line change
`@@ -325,7 +325,6 @@ static int pxa_usb_phy_probe(struct platform_device *pdev)`
`325`	`325`	`phy_create_lookup(pxa_usb_phy->phy, "usb", "mv-otg");`
`326`	`326`	`}`
`327`	`327`
`328`		`- dev_info(dev, "Marvell PXA USB PHY");`
`329`	`328`	`return 0;`
`330`	`329`	`}`
`331`	`330`
Original file line number	Diff line number	Diff line change
`@@ -599,8 +599,6 @@ static int snps_eusb2_hsphy_probe(struct platform_device *pdev)`
`599`	`599`	`if (IS_ERR(phy_provider))`
`600`	`600`	`return PTR_ERR(phy_provider);`
`601`	`601`
`602`		`- dev_info(dev, "Registered Snps-eUSB2 phy\n");`
`603`		`-`
`604`	`602`	`return 0;`
`605`	`603`	`}`
`606`	`604`
Original file line number	Diff line number	Diff line change
`@@ -264,8 +264,6 @@ static int eusb2_repeater_probe(struct platform_device *pdev)`
`264`	`264`	`if (IS_ERR(phy_provider))`
`265`	`265`	`return PTR_ERR(phy_provider);`
`266`	`266`
`267`		`- dev_info(dev, "Registered Qcom-eUSB2 repeater\n");`
`268`		`-`
`269`	`267`	`return 0;`
`270`	`268`	`}`
`271`	`269`
Original file line number	Diff line number	Diff line change
`@@ -305,8 +305,6 @@ static int m31usb_phy_probe(struct platform_device *pdev)`
`305`	`305`	`phy_set_drvdata(qphy->phy, qphy);`
`306`	`306`
`307`	`307`	`phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);`
`308`		`- if (!IS_ERR(phy_provider))`
`309`		`- dev_info(dev, "Registered M31 USB phy\n");`
`310`	`308`
`311`	`309`	`return PTR_ERR_OR_ZERO(phy_provider);`
`312`	`310`	`}`
Original file line number	Diff line number	Diff line change
`@@ -139,8 +139,6 @@ static int stih407_usb2_picophy_probe(struct platform_device *pdev)`
`139`	`139`	`if (IS_ERR(phy_provider))`
`140`	`140`	`return PTR_ERR(phy_provider);`
`141`	`141`
`142`		`- dev_info(dev, "STiH407 USB Generic picoPHY driver probed!");`
`143`		`-`
`144`	`142`	`return 0;`
`145`	`143`	`}`
`146`	`144`