Re: [PATCH 3/3] media: vgxy61: Add MODULE_ALIAS()

From: Krzysztof Kozlowski
Date: Tue Jun 11 2024 - 02:49:20 EST


On 10/06/2024 17:08, Benjamin Mugnier wrote:
> Preserve user space retro compatibility after the device rename.
>
> Signed-off-by: Benjamin Mugnier <benjamin.mugnier@xxxxxxxxxxx>
> ---
> drivers/media/i2c/vgxy61.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/i2c/vgxy61.c b/drivers/media/i2c/vgxy61.c
> index ca3b43608dad..c85f356946ca 100644
> --- a/drivers/media/i2c/vgxy61.c
> +++ b/drivers/media/i2c/vgxy61.c
> @@ -1898,3 +1898,4 @@ MODULE_AUTHOR("Mickael Guene <mickael.guene@xxxxxx>");
> MODULE_AUTHOR("Sylvain Petinot <sylvain.petinot@xxxxxxxxxxx>");
> MODULE_DESCRIPTION("VGXY61 camera subdev driver");
> MODULE_LICENSE("GPL");
> +MODULE_ALIAS("platform:st-vgxy61");

Why? Isn't this autoloated by OF alias? There was never here platform
alias so no functionality is lost.

NAK

Best regards,
Krzysztof