Re: [PATCH 4.14 REGRESSION fix] Input: axp20x-pek - Fix module not auto-loading for axp221 pek

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 16, 2017 at 03:34:22PM +0200, Hans de Goede wrote:
> Now that we have a platform_device_id table and multiple supported ids
> we should be using MODULE_DEVICE_TABLE instead of MODULE_ALIAS.
> 
> This fixes a regression on Bay and Cherry Trail devices, where the power
> button is now enumerated as an "axp221-pek" and it was impossible to
> wakeup these devices from suspend since the module did not load.
> 
> Cc: Quentin Schulz <quentin.schulz@xxxxxxxxxxxxxxxxxx>
> Fixes: c3cc94470bd3 ("Input: axp20x-pek - add support for AXP221 PEK")
> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>

Applied, thank you.

> ---
>  drivers/input/misc/axp20x-pek.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/input/misc/axp20x-pek.c b/drivers/input/misc/axp20x-pek.c
> index 6cee5adc3b5c..debeeaeb8812 100644
> --- a/drivers/input/misc/axp20x-pek.c
> +++ b/drivers/input/misc/axp20x-pek.c
> @@ -403,6 +403,7 @@ static const struct platform_device_id axp_pek_id_match[] = {
>  	},
>  	{ /* sentinel */ }
>  };
> +MODULE_DEVICE_TABLE(platform, axp_pek_id_match);
>  
>  static struct platform_driver axp20x_pek_driver = {
>  	.probe		= axp20x_pek_probe,
> @@ -417,4 +418,3 @@ module_platform_driver(axp20x_pek_driver);
>  MODULE_DESCRIPTION("axp20x Power Button");
>  MODULE_AUTHOR("Carlo Caione <carlo@xxxxxxxxxx>");
>  MODULE_LICENSE("GPL");
> -MODULE_ALIAS("platform:axp20x-pek");
> -- 
> 2.14.2
> 

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux