Re: [PATCH v1 1/2] extcon: add Realtek DHC RTD SoC Type-C driver

From: Krzysztof Kozlowski
Date: Tue Aug 22 2023 - 11:35:21 EST


On 22/08/2023 12:28, Stanley Chang wrote:
> This patch adds the extcon driver for Realtek DHC (digital home center)
> RTD SoCs type-c module. This can be used to detect whether the port is
> configured as a downstream or upstream facing port. And notify the status
> of extcon to listeners.
>
> Signed-off-by: Stanley Chang <stanley_chang@xxxxxxxxxxx>
> ---
> drivers/extcon/Kconfig | 9 +
> drivers/extcon/Makefile | 1 +
> drivers/extcon/extcon-rtk-type-c.c | 1799 ++++++++++++++++++++++++++++
> 3 files changed, 1809 insertions(+)
> create mode 100644 drivers/extcon/extcon-rtk-type-c.c
>


> +static struct platform_driver extcon_rtk_type_c_driver = {
> + .probe = extcon_rtk_type_c_probe,
> + .remove_new = extcon_rtk_type_c_remove,
> + .driver = {
> + .name = "extcon-rtk-type_c",
> + .of_match_table = extcon_rtk_type_c_match,
> + .pm = DEV_PM_OPS,
> + },
> +};
> +
> +module_platform_driver(extcon_rtk_type_c_driver);
> +
> +MODULE_DESCRIPTION("Realtek Extcon Type C driver");
> +MODULE_ALIAS("platform:extcon-rtk-type-c");

Drop alias. You have something really missing here if you need it.

> +MODULE_AUTHOR("Stanley Chang <stanley_chang@xxxxxxxxxxx>");
> +MODULE_LICENSE("GPL");

Best regards,
Krzysztof