Re: [PATCH 1/2] platform/x86: intel_mid_thermal: Remove duplicated platform device ID

From: Andy Shevchenko
Date: Tue Jan 03 2017 - 15:46:34 EST


On Mon, Jan 2, 2017 at 5:20 PM, Javier Martinez Canillas
<javier@xxxxxxxxxxxxxxx> wrote:
> Commit 3fca3d3d5075 ("platform-x86: intel_mid_thermal: add msic_thermal
> alias") added a "msic_thermal" entry to the driver's platform device ID
> table since that was the platform dev name registered in some platforms
> and the only dev in the platform table was "msic_sensor" (DRIVER_NAME).
>
> But then commit 634830704d80 ("x86/mid/thermal: Add msic_thermal alias")
> changed DRIVER_NAME from "msic_sensor" to "msic_thermal", and so there's
> now duplicated entries in the platform device ID table.
>

Both patches pushed to testing.
Thanks!

> Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
> ---
>
> drivers/platform/x86/intel_mid_thermal.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/platform/x86/intel_mid_thermal.c b/drivers/platform/x86/intel_mid_thermal.c
> index 0df3c9d37509..05ac7203806d 100644
> --- a/drivers/platform/x86/intel_mid_thermal.c
> +++ b/drivers/platform/x86/intel_mid_thermal.c
> @@ -549,7 +549,6 @@ static int mid_thermal_remove(struct platform_device *pdev)
>
> static const struct platform_device_id therm_id_table[] = {
> { DRIVER_NAME, 1 },
> - { "msic_thermal", 1 },
> { }
> };
>
> --
> 2.7.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html



--
With Best Regards,
Andy Shevchenko