diff mbox

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

Message ID 1483370456-30786-1-git-send-email-javier@osg.samsung.com (mailing list archive)
State Accepted, archived
Delegated to: Andy Shevchenko
Headers show

Commit Message

Javier Martinez Canillas Jan. 2, 2017, 3:20 p.m. UTC
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.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 drivers/platform/x86/intel_mid_thermal.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Andy Shevchenko Jan. 3, 2017, 8:46 p.m. UTC | #1
On Mon, Jan 2, 2017 at 5:20 PM, Javier Martinez Canillas
<javier@osg.samsung.com> 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@osg.samsung.com>
> ---
>
>  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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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 },
 	{ }
 };