mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-12-15 15:04:27 +08:00
usb: reduce kernel log spam on driver registration
Drivers are typically supposed to be quiet unless they are actually probed, but for some reason, USB host controllers seem to be exempt from this rule, and happily broadcast their existence into the kernel log at boot even if the hardware in question is nowhere to be found. Let's fix that, and remove these pr_info() calls. Cc: Alan Stern <stern@rowland.harvard.edu> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Nicolas Ferre <nicolas.ferre@microchip.com> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com> Cc: Claudiu Beznea <claudiu.beznea@microchip.com> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Cc: Alim Akhtar <alim.akhtar@samsung.com> Cc: Avi Fishman <avifishman70@gmail.com> Cc: Tomer Maimon <tmaimon77@gmail.com> Cc: Tali Perry <tali.perry1@gmail.com> Cc: Patrick Venture <venture@google.com> Cc: Nancy Yuen <yuenn@google.com> Cc: Benjamin Fair <benjaminfair@google.com> Cc: Patrice Chotard <patrice.chotard@foss.st.com> Cc: Vladimir Zapolskiy <vz@mleia.com> Cc: linux-usb@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org Cc: linux-samsung-soc@vger.kernel.org Cc: linux-omap@vger.kernel.org Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Rob Herring <robh@kernel.org> Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com> Acked-by: Alan Stern <stern@rowland.harvard.edu> Signed-off-by: Ard Biesheuvel <ardb@kernel.org> Link: https://lore.kernel.org/r/20220825170327.674446-1-ardb@kernel.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
8f36b3b4e1
commit
10174220f5
@ -239,7 +239,6 @@ static int __init ehci_atmel_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
ehci_init_driver(&ehci_atmel_hc_driver, &ehci_atmel_drv_overrides);
|
ehci_init_driver(&ehci_atmel_hc_driver, &ehci_atmel_drv_overrides);
|
||||||
return platform_driver_register(&ehci_atmel_driver);
|
return platform_driver_register(&ehci_atmel_driver);
|
||||||
}
|
}
|
||||||
|
@ -347,7 +347,6 @@ static int __init ehci_exynos_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
ehci_init_driver(&exynos_ehci_hc_driver, &exynos_overrides);
|
ehci_init_driver(&exynos_ehci_hc_driver, &exynos_overrides);
|
||||||
return platform_driver_register(&exynos_ehci_driver);
|
return platform_driver_register(&exynos_ehci_driver);
|
||||||
}
|
}
|
||||||
|
@ -722,8 +722,6 @@ static int __init ehci_fsl_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info(DRV_NAME ": " DRIVER_DESC "\n");
|
|
||||||
|
|
||||||
ehci_init_driver(&fsl_ehci_hc_driver, &ehci_fsl_overrides);
|
ehci_init_driver(&fsl_ehci_hc_driver, &ehci_fsl_overrides);
|
||||||
|
|
||||||
fsl_ehci_hc_driver.product_desc =
|
fsl_ehci_hc_driver.product_desc =
|
||||||
|
@ -1351,7 +1351,6 @@ static int __init ehci_hcd_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
printk(KERN_INFO "%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
set_bit(USB_EHCI_LOADED, &usb_hcds_loaded);
|
set_bit(USB_EHCI_LOADED, &usb_hcds_loaded);
|
||||||
if (test_bit(USB_UHCI_LOADED, &usb_hcds_loaded) ||
|
if (test_bit(USB_UHCI_LOADED, &usb_hcds_loaded) ||
|
||||||
test_bit(USB_OHCI_LOADED, &usb_hcds_loaded))
|
test_bit(USB_OHCI_LOADED, &usb_hcds_loaded))
|
||||||
|
@ -141,8 +141,6 @@ static int __init ehci_npcm7xx_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ehci_init_driver(&ehci_npcm7xx_hc_driver, NULL);
|
ehci_init_driver(&ehci_npcm7xx_hc_driver, NULL);
|
||||||
return platform_driver_register(&npcm7xx_ehci_hcd_driver);
|
return platform_driver_register(&npcm7xx_ehci_hcd_driver);
|
||||||
}
|
}
|
||||||
|
@ -284,8 +284,6 @@ static int __init ehci_omap_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ehci_init_driver(&ehci_omap_hc_driver, &ehci_omap_overrides);
|
ehci_init_driver(&ehci_omap_hc_driver, &ehci_omap_overrides);
|
||||||
return platform_driver_register(&ehci_hcd_omap_driver);
|
return platform_driver_register(&ehci_hcd_omap_driver);
|
||||||
}
|
}
|
||||||
|
@ -361,8 +361,6 @@ static int __init ehci_orion_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ehci_init_driver(&ehci_orion_hc_driver, &orion_overrides);
|
ehci_init_driver(&ehci_orion_hc_driver, &orion_overrides);
|
||||||
return platform_driver_register(&ehci_orion_driver);
|
return platform_driver_register(&ehci_orion_driver);
|
||||||
}
|
}
|
||||||
|
@ -423,8 +423,6 @@ static int __init ehci_pci_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ehci_init_driver(&ehci_pci_hc_driver, &pci_overrides);
|
ehci_init_driver(&ehci_pci_hc_driver, &pci_overrides);
|
||||||
|
|
||||||
/* Entries for the PCI suspend/resume callbacks are special */
|
/* Entries for the PCI suspend/resume callbacks are special */
|
||||||
|
@ -529,8 +529,6 @@ static int __init ehci_platform_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ehci_init_driver(&ehci_platform_hc_driver, &platform_overrides);
|
ehci_init_driver(&ehci_platform_hc_driver, &platform_overrides);
|
||||||
return platform_driver_register(&ehci_platform_driver);
|
return platform_driver_register(&ehci_platform_driver);
|
||||||
}
|
}
|
||||||
|
@ -167,8 +167,6 @@ static int __init ehci_spear_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ehci_init_driver(&ehci_spear_hc_driver, &spear_overrides);
|
ehci_init_driver(&ehci_spear_hc_driver, &spear_overrides);
|
||||||
return platform_driver_register(&spear_ehci_hcd_driver);
|
return platform_driver_register(&spear_ehci_hcd_driver);
|
||||||
}
|
}
|
||||||
|
@ -346,8 +346,6 @@ static int __init ehci_platform_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ehci_init_driver(&ehci_platform_hc_driver, &platform_overrides);
|
ehci_init_driver(&ehci_platform_hc_driver, &platform_overrides);
|
||||||
return platform_driver_register(&ehci_platform_driver);
|
return platform_driver_register(&ehci_platform_driver);
|
||||||
}
|
}
|
||||||
|
@ -5692,7 +5692,6 @@ static int __init fotg210_hcd_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
set_bit(USB_EHCI_LOADED, &usb_hcds_loaded);
|
set_bit(USB_EHCI_LOADED, &usb_hcds_loaded);
|
||||||
if (test_bit(USB_UHCI_LOADED, &usb_hcds_loaded) ||
|
if (test_bit(USB_UHCI_LOADED, &usb_hcds_loaded) ||
|
||||||
test_bit(USB_OHCI_LOADED, &usb_hcds_loaded))
|
test_bit(USB_OHCI_LOADED, &usb_hcds_loaded))
|
||||||
|
@ -699,7 +699,6 @@ static int __init ohci_at91_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
ohci_init_driver(&ohci_at91_hc_driver, &ohci_at91_drv_overrides);
|
ohci_init_driver(&ohci_at91_hc_driver, &ohci_at91_drv_overrides);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@ -551,7 +551,6 @@ static int __init ohci_da8xx_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", DRV_NAME);
|
|
||||||
ohci_init_driver(&ohci_da8xx_hc_driver, &da8xx_overrides);
|
ohci_init_driver(&ohci_da8xx_hc_driver, &da8xx_overrides);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@ -310,7 +310,6 @@ static int __init ohci_exynos_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
ohci_init_driver(&exynos_ohci_hc_driver, &exynos_overrides);
|
ohci_init_driver(&exynos_ohci_hc_driver, &exynos_overrides);
|
||||||
return platform_driver_register(&exynos_ohci_driver);
|
return platform_driver_register(&exynos_ohci_driver);
|
||||||
}
|
}
|
||||||
|
@ -1276,7 +1276,6 @@ static int __init ohci_hcd_mod_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
printk(KERN_INFO "%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
pr_debug ("%s: block sizes: ed %zd td %zd\n", hcd_name,
|
pr_debug ("%s: block sizes: ed %zd td %zd\n", hcd_name,
|
||||||
sizeof (struct ed), sizeof (struct td));
|
sizeof (struct ed), sizeof (struct td));
|
||||||
set_bit(USB_OHCI_LOADED, &usb_hcds_loaded);
|
set_bit(USB_OHCI_LOADED, &usb_hcds_loaded);
|
||||||
|
@ -275,8 +275,6 @@ static int __init ohci_nxp_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ohci_init_driver(&ohci_nxp_hc_driver, NULL);
|
ohci_init_driver(&ohci_nxp_hc_driver, NULL);
|
||||||
return platform_driver_register(&ohci_hcd_nxp_driver);
|
return platform_driver_register(&ohci_hcd_nxp_driver);
|
||||||
}
|
}
|
||||||
|
@ -423,8 +423,6 @@ static int __init ohci_omap_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ohci_init_driver(&ohci_omap_hc_driver, &omap_overrides);
|
ohci_init_driver(&ohci_omap_hc_driver, &omap_overrides);
|
||||||
return platform_driver_register(&ohci_hcd_omap_driver);
|
return platform_driver_register(&ohci_hcd_omap_driver);
|
||||||
}
|
}
|
||||||
|
@ -306,8 +306,6 @@ static int __init ohci_pci_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ohci_init_driver(&ohci_pci_hc_driver, &pci_overrides);
|
ohci_init_driver(&ohci_pci_hc_driver, &pci_overrides);
|
||||||
|
|
||||||
#ifdef CONFIG_PM
|
#ifdef CONFIG_PM
|
||||||
|
@ -346,8 +346,6 @@ static int __init ohci_platform_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ohci_init_driver(&ohci_platform_hc_driver, &platform_overrides);
|
ohci_init_driver(&ohci_platform_hc_driver, &platform_overrides);
|
||||||
return platform_driver_register(&ohci_platform_driver);
|
return platform_driver_register(&ohci_platform_driver);
|
||||||
}
|
}
|
||||||
|
@ -608,8 +608,6 @@ static int __init ohci_pxa27x_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ohci_init_driver(&ohci_pxa27x_hc_driver, &pxa27x_overrides);
|
ohci_init_driver(&ohci_pxa27x_hc_driver, &pxa27x_overrides);
|
||||||
ohci_pxa27x_hc_driver.hub_control = pxa27x_ohci_hub_control;
|
ohci_pxa27x_hc_driver.hub_control = pxa27x_ohci_hub_control;
|
||||||
|
|
||||||
|
@ -474,7 +474,6 @@ static int __init ohci_s3c2410_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
ohci_init_driver(&ohci_s3c2410_hc_driver, NULL);
|
ohci_init_driver(&ohci_s3c2410_hc_driver, NULL);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@ -179,8 +179,6 @@ static int __init ohci_spear_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ohci_init_driver(&ohci_spear_hc_driver, &spear_overrides);
|
ohci_init_driver(&ohci_spear_hc_driver, &spear_overrides);
|
||||||
return platform_driver_register(&spear_ohci_hcd_driver);
|
return platform_driver_register(&spear_ohci_hcd_driver);
|
||||||
}
|
}
|
||||||
|
@ -324,8 +324,6 @@ static int __init ohci_platform_init(void)
|
|||||||
if (usb_disabled())
|
if (usb_disabled())
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
||||||
pr_info("%s: " DRIVER_DESC "\n", hcd_name);
|
|
||||||
|
|
||||||
ohci_init_driver(&ohci_platform_hc_driver, &platform_overrides);
|
ohci_init_driver(&ohci_platform_hc_driver, &platform_overrides);
|
||||||
return platform_driver_register(&ohci_platform_driver);
|
return platform_driver_register(&ohci_platform_driver);
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user