[PATCH] leds: upboard: fix module alias

Thomas Richard posted 1 patch 3 months, 3 weeks ago
There is a newer version of this series
drivers/leds/leds-upboard.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] leds: upboard: fix module alias
Posted by Thomas Richard 3 months, 3 weeks ago
Fix module alias for auto-loading.

Fixes: 0ef2929a0181 ("leds: Add AAEON UP board LED driver")
Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
---
 drivers/leds/leds-upboard.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/leds/leds-upboard.c b/drivers/leds/leds-upboard.c
index b350eb294280fd68535c47843417f4282f97b423..12989b2f195309cc930095ecc5f855065e88d9aa 100644
--- a/drivers/leds/leds-upboard.c
+++ b/drivers/leds/leds-upboard.c
@@ -123,4 +123,4 @@ MODULE_AUTHOR("Gary Wang <garywang@aaeon.com.tw>");
 MODULE_AUTHOR("Thomas Richard <thomas.richard@bootlin.com>");
 MODULE_DESCRIPTION("UP Board LED driver");
 MODULE_LICENSE("GPL");
-MODULE_ALIAS("platform:upboard-led");
+MODULE_ALIAS("platform:upboard-leds");

---
base-commit: 3a8660878839faadb4f1a6dd72c3179c1df56787
change-id: 20251016-leds-upboard-fix-module-alias-194508af3111

Best regards,
-- 
Thomas Richard <thomas.richard@bootlin.com>
Re: [PATCH] leds: upboard: fix module alias
Posted by Krzysztof Kozlowski 3 months, 3 weeks ago
On 16/10/2025 15:28, Thomas Richard wrote:
> Fix module alias for auto-loading.

Fix what exactly? It was a completely correct alias. Please describe
here bug (so WHY you are doing this) not what you are doing.

> 
> Fixes: 0ef2929a0181 ("leds: Add AAEON UP board LED driver")
> Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
> ---
>  drivers/leds/leds-upboard.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/leds/leds-upboard.c b/drivers/leds/leds-upboard.c
> index b350eb294280fd68535c47843417f4282f97b423..12989b2f195309cc930095ecc5f855065e88d9aa 100644
> --- a/drivers/leds/leds-upboard.c
> +++ b/drivers/leds/leds-upboard.c
> @@ -123,4 +123,4 @@ MODULE_AUTHOR("Gary Wang <garywang@aaeon.com.tw>");
>  MODULE_AUTHOR("Thomas Richard <thomas.richard@bootlin.com>");
>  MODULE_DESCRIPTION("UP Board LED driver");
>  MODULE_LICENSE("GPL");
> -MODULE_ALIAS("platform:upboard-led");
> +MODULE_ALIAS("platform:upboard-leds");
Best regards,
Krzysztof
Re: [PATCH] leds: upboard: fix module alias
Posted by Thomas Richard 3 months, 3 weeks ago
On 10/16/25 9:11 PM, Krzysztof Kozlowski wrote:
> On 16/10/2025 15:28, Thomas Richard wrote:
>> Fix module alias for auto-loading.
> 
> Fix what exactly? It was a completely correct alias. Please describe
> here bug (so WHY you are doing this) not what you are doing.

The module alias does not match the cell name defined in the MFD driver,
so if the driver is built as a module, it is not automatically loaded.

Best Regards,
Thomas
Re: [PATCH] leds: upboard: fix module alias
Posted by Krzysztof Kozlowski 3 months, 3 weeks ago
On 17/10/2025 11:00, Thomas Richard wrote:
> On 10/16/25 9:11 PM, Krzysztof Kozlowski wrote:
>> On 16/10/2025 15:28, Thomas Richard wrote:
>>> Fix module alias for auto-loading.
>>
>> Fix what exactly? It was a completely correct alias. Please describe
>> here bug (so WHY you are doing this) not what you are doing.
> 
> The module alias does not match the cell name defined in the MFD driver,


Then this should be clearly explained.

Again: your commit msg should describe the bug, the reasons why you are
doing it, not just "fix" without telling what is wrong.


> so if the driver is built as a module, it is not automatically loaded.

Best regards,
Krzysztof