RE: [PATCH] pinctrl: realtek: rtd1619b: Publish the OF module alias

[Yu-Chun Lin [林祐君]]

Hi,

> The RTD1619B pinctrl driver is a separate tristate object using
> 
> rtd1619b_pinctrl_of_match. The generic Realtek pinctrl support does not
> 
> publish an alias for this SoC module.
> 
> 
> 
> Export the existing OF table while retaining the initialization order.
> 
> 
> 
> Fixes: c7910f6aca56 ("pinctrl: realtek: Add pinctrl driver for RTD1619B")
> 
> Signed-off-by: hpp.iscas <hppiscas@xxxxxxx>
> 

Please use your real name.

> ---
> 
>  drivers/pinctrl/realtek/pinctrl-rtd1619b.c | 1 +
> 
>  1 file changed, 1 insertion(+)
> 
> 
> 
> diff --git a/drivers/pinctrl/realtek/pinctrl-rtd1619b.c
> b/drivers/pinctrl/realtek/pinctrl-rtd1619b.c
> 
> index b07e50d..e03e1fb 100644
> 
> --- a/drivers/pinctrl/realtek/pinctrl-rtd1619b.c
> 
> +++ b/drivers/pinctrl/realtek/pinctrl-rtd1619b.c
> 
> @@ -1575,6 +1575,7 @@ static const struct of_device_id
> rtd1619b_pinctrl_of_match[] = {
> 
>         { .compatible = "realtek,rtd1619b-pinctrl", },
> 
>         {},
> 
>  };
> 
> +MODULE_DEVICE_TABLE(of, rtd1619b_pinctrl_of_match);
> 
> 
> 
>  static struct platform_driver rtd1619b_pinctrl_driver = {
> 
>         .driver = {
> 

The code looks good to me, but 'git am' failed to apply the mbox. 
I noticed =0D in your patch, which might be the reason it failed.

Best Regards,
Yu-Chun