On Sat, Aug 30, 2025 at 03:01:02PM +0200, Krzysztof Kozlowski wrote:
> Samsung S3C24xx family of SoCs was removed the Linux kernel in the
> commit 61b7f8920b17 ("ARM: s3c: remove all s3c24xx support"), in January
> 2023. There are no in-kernel users of remaining S3C24xx compatibles.
>
> The driver is still being used via platform code for S3C64xx platforms.
This title and description are a bit confusing. I gather that while the
S3C24xx chipsets are not longer supported, the S3C24xx OHCI controller
is still being used in S3C64xx systems.
So what the patch does is drop support for S3C2410 _systems_ while
retaining support for S3C2410 _controllers_. Is that right? If so, can
we change $SUBJECT to say "usb: ohci: Drop DT support for S3C2410
systems"?
Alan Stern
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> ---
> drivers/usb/host/ohci-s3c2410.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/drivers/usb/host/ohci-s3c2410.c b/drivers/usb/host/ohci-s3c2410.c
> index 66d970854357..e623e24d3f8e 100644
> --- a/drivers/usb/host/ohci-s3c2410.c
> +++ b/drivers/usb/host/ohci-s3c2410.c
> @@ -448,13 +448,6 @@ static const struct dev_pm_ops ohci_hcd_s3c2410_pm_ops = {
> .resume = ohci_hcd_s3c2410_drv_resume,
> };
>
> -static const struct of_device_id ohci_hcd_s3c2410_dt_ids[] = {
> - { .compatible = "samsung,s3c2410-ohci" },
> - { /* sentinel */ }
> -};
> -
> -MODULE_DEVICE_TABLE(of, ohci_hcd_s3c2410_dt_ids);
> -
> static struct platform_driver ohci_hcd_s3c2410_driver = {
> .probe = ohci_hcd_s3c2410_probe,
> .remove = ohci_hcd_s3c2410_remove,
> @@ -462,7 +455,6 @@ static struct platform_driver ohci_hcd_s3c2410_driver = {
> .driver = {
> .name = "s3c2410-ohci",
> .pm = &ohci_hcd_s3c2410_pm_ops,
> - .of_match_table = ohci_hcd_s3c2410_dt_ids,
> },
> };
>
> --
> 2.48.1
>
