Re: [PATCH v2] acpi: move int340x_thermal.c to dptf folder

[Zhang Rui <rui.zhang@intel.com>]

On 二, 2016-07-19 at 14:57 -0700, Srinivas Pandruvada wrote:
> Since dptf has its own folder under ACPI, move this file also
> there.
> 
> Signed-off-by: Srinivas Pandruvada
> <srinivas.pandruvada@linux.intel.com>

Acked-by: Zhang Rui <rui.zhang@intel.com>

thanks,
rui
> ---
> v2:
> Resent after dptf_power patch was updated with include file path
> 
> 
>  drivers/acpi/Makefile               |  1 -
>  drivers/acpi/dptf/Makefile          |  3 +++
>  drivers/acpi/dptf/int340x_thermal.c | 53
> +++++++++++++++++++++++++++++++++++++
>  drivers/acpi/int340x_thermal.c      | 53 -------------------------
> ------------
>  4 files changed, 56 insertions(+), 54 deletions(-)
>  create mode 100644 drivers/acpi/dptf/int340x_thermal.c
>  delete mode 100644 drivers/acpi/int340x_thermal.c
> 
> diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile
> index f6d0cc0..0325a53 100644
> --- a/drivers/acpi/Makefile
> +++ b/drivers/acpi/Makefile
> @@ -44,7 +44,6 @@ acpi-y				+=
> acpi_lpss.o acpi_apd.o
>  acpi-y				+= acpi_platform.o
>  acpi-y				+= acpi_pnp.o
>  acpi-$(CONFIG_ARM_AMBA)	+= acpi_amba.o
> -acpi-y				+= int340x_thermal.o
>  acpi-y				+= power.o
>  acpi-y				+= event.o
>  acpi-$(CONFIG_ACPI_REDUCED_HARDWARE_ONLY) += evged.o
> diff --git a/drivers/acpi/dptf/Makefile b/drivers/acpi/dptf/Makefile
> index 62cbe87..06ea880 100644
> --- a/drivers/acpi/dptf/Makefile
> +++ b/drivers/acpi/dptf/Makefile
> @@ -1 +1,4 @@
> +obj-$(CONFIG_ACPI)             += int340x_thermal.o
>  obj-$(CONFIG_DPTF_POWER)	+= dptf_power.o
> +
> +ccflags-y += -Idrivers/acpi
> diff --git a/drivers/acpi/dptf/int340x_thermal.c
> b/drivers/acpi/dptf/int340x_thermal.c
> new file mode 100644
> index 0000000..33505c6
> --- /dev/null
> +++ b/drivers/acpi/dptf/int340x_thermal.c
> @@ -0,0 +1,53 @@
> +/*
> + * ACPI support for int340x thermal drivers
> + *
> + * Copyright (C) 2014, Intel Corporation
> + * Authors: Zhang Rui <rui.zhang@intel.com>
> + *
> + * This program is free software; you can redistribute it and/or
> modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/acpi.h>
> +#include <linux/module.h>
> +
> +#include "internal.h"
> +
> +#define INT3401_DEVICE 0X01
> +static const struct acpi_device_id int340x_thermal_device_ids[] = {
> +	{"INT3400"},
> +	{"INT3401", INT3401_DEVICE},
> +	{"INT3402"},
> +	{"INT3403"},
> +	{"INT3404"},
> +	{"INT3406"},
> +	{"INT3407"},
> +	{"INT3408"},
> +	{"INT3409"},
> +	{"INT340A"},
> +	{"INT340B"},
> +	{""},
> +};
> +
> +static int int340x_thermal_handler_attach(struct acpi_device *adev,
> +					const struct acpi_device_id
> *id)
> +{
> +	if (IS_ENABLED(CONFIG_INT340X_THERMAL))
> +		acpi_create_platform_device(adev);
> +	/* Intel SoC DTS thermal driver needs INT3401 to set IRQ
> descriptor */
> +	else if (IS_ENABLED(CONFIG_INTEL_SOC_DTS_THERMAL) &&
> +		 id->driver_data == INT3401_DEVICE)
> +		acpi_create_platform_device(adev);
> +	return 1;
> +}
> +
> +static struct acpi_scan_handler int340x_thermal_handler = {
> +	.ids = int340x_thermal_device_ids,
> +	.attach = int340x_thermal_handler_attach,
> +};
> +
> +void __init acpi_int340x_thermal_init(void)
> +{
> +	acpi_scan_add_handler(&int340x_thermal_handler);
> +}
> diff --git a/drivers/acpi/int340x_thermal.c
> b/drivers/acpi/int340x_thermal.c
> deleted file mode 100644
> index 33505c6..0000000
> --- a/drivers/acpi/int340x_thermal.c
> +++ /dev/null
> @@ -1,53 +0,0 @@
> -/*
> - * ACPI support for int340x thermal drivers
> - *
> - * Copyright (C) 2014, Intel Corporation
> - * Authors: Zhang Rui <rui.zhang@intel.com>
> - *
> - * This program is free software; you can redistribute it and/or
> modify
> - * it under the terms of the GNU General Public License version 2 as
> - * published by the Free Software Foundation.
> - */
> -
> -#include <linux/acpi.h>
> -#include <linux/module.h>
> -
> -#include "internal.h"
> -
> -#define INT3401_DEVICE 0X01
> -static const struct acpi_device_id int340x_thermal_device_ids[] = {
> -	{"INT3400"},
> -	{"INT3401", INT3401_DEVICE},
> -	{"INT3402"},
> -	{"INT3403"},
> -	{"INT3404"},
> -	{"INT3406"},
> -	{"INT3407"},
> -	{"INT3408"},
> -	{"INT3409"},
> -	{"INT340A"},
> -	{"INT340B"},
> -	{""},
> -};
> -
> -static int int340x_thermal_handler_attach(struct acpi_device *adev,
> -					const struct acpi_device_id
> *id)
> -{
> -	if (IS_ENABLED(CONFIG_INT340X_THERMAL))
> -		acpi_create_platform_device(adev);
> -	/* Intel SoC DTS thermal driver needs INT3401 to set IRQ
> descriptor */
> -	else if (IS_ENABLED(CONFIG_INTEL_SOC_DTS_THERMAL) &&
> -		 id->driver_data == INT3401_DEVICE)
> -		acpi_create_platform_device(adev);
> -	return 1;
> -}
> -
> -static struct acpi_scan_handler int340x_thermal_handler = {
> -	.ids = int340x_thermal_device_ids,
> -	.attach = int340x_thermal_handler_attach,
> -};
> -
> -void __init acpi_int340x_thermal_init(void)
> -{
> -	acpi_scan_add_handler(&int340x_thermal_handler);
> -}
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html