LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] 3/3 intel_menlow: syncing with acpi function changes
@ 2008-11-05 10:46 Thomas, Sujith
  2008-11-07  1:30 ` Len Brown
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas, Sujith @ 2008-11-05 10:46 UTC (permalink / raw)
  To: Len Brown, Zhang, Rui, Thomas, Sujith
  Cc: Linux Kernel Mailing List, linux-acpi, Brown, Len, Andrew Morton

4th parameter of acpi_evaluate_integer changed to unsigned long long
acpi_driver_data() changed to inline function from macro

Signed-off-by: Sujith Thomas <sujith.thomas@intel.com>
---
 drivers/misc/intel_menlow.c |   20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

Index: linux-2.6.27/drivers/misc/intel_menlow.c
===================================================================
--- linux-2.6.27.orig/drivers/misc/intel_menlow.c
+++ linux-2.6.27/drivers/misc/intel_menlow.c
@@ -62,7 +62,7 @@ static int memory_get_int_max_bandwidth(
 {
 	struct acpi_device *device = cdev->devdata;
 	acpi_handle handle = device->handle;
-	unsigned long value;
+	unsigned long long value;
 	struct acpi_object_list arg_list;
 	union acpi_object arg;
 	acpi_status status = AE_OK;
@@ -98,7 +98,7 @@ static int memory_get_cur_bandwidth(stru
 {
 	struct acpi_device *device = cdev->devdata;
 	acpi_handle handle = device->handle;
-	unsigned long value;
+	unsigned long long value;
 	struct acpi_object_list arg_list;
 	union acpi_object arg;
 	acpi_status status = AE_OK;
@@ -112,7 +112,7 @@ static int memory_get_cur_bandwidth(stru
 	if (ACPI_FAILURE(status))
 		return -EFAULT;
 
-	return sprintf(buf, "%ld\n", value);
+	return sprintf(buf, "%lld\n", value);
 }
 
 static int memory_set_cur_bandwidth(struct thermal_cooling_device
*cdev,
@@ -139,7 +139,7 @@ static int memory_set_cur_bandwidth(stru
 
 	status =
 	    acpi_evaluate_integer(handle, MEMORY_SET_BANDWIDTH,
&arg_list,
-				  (unsigned long *)&temp);
+				  (unsigned long long *)&temp);
 
 	printk(KERN_INFO
 	       "Bandwidth value was %d: status is %d\n", state, status);
@@ -183,7 +183,7 @@ static int intel_menlow_memory_add(struc
 		goto end;
 	}
 
-	acpi_driver_data(device) = cdev;
+	device->driver_data = cdev;
 	result = sysfs_create_link(&device->dev.kobj,
 				&cdev->device.kobj, "thermal_cooling");
 	if (result)
@@ -268,7 +268,7 @@ static int sensor_get_auxtrip(acpi_handl
 		return -EINVAL;
 
 	status = acpi_evaluate_integer(handle, index ? GET_AUX1 :
GET_AUX0,
-				       NULL, (unsigned long *)value);
+				       NULL, (unsigned long long
*)value);
 	if (ACPI_FAILURE(status))
 		return -EIO;
 
@@ -296,7 +296,7 @@ static int sensor_set_auxtrip(acpi_handl
 		return -EINVAL;
 
 	status = acpi_evaluate_integer(handle, index ? GET_AUX0 :
GET_AUX1,
-				       NULL, (unsigned long *)&temp);
+				       NULL, (unsigned long long
*)&temp);
 	if (ACPI_FAILURE(status))
 		return -EIO;
 	if ((index && value < temp) || (!index && value > temp))
@@ -304,7 +304,7 @@ static int sensor_set_auxtrip(acpi_handl
 
 	arg.integer.value = value;
 	status = acpi_evaluate_integer(handle, index ? SET_AUX1 :
SET_AUX0,
-				       &args, (unsigned long *)&temp);
+				       &args, (unsigned long long
*)&temp);
 	if (ACPI_FAILURE(status))
 		return -EIO;
 
@@ -384,7 +384,7 @@ static ssize_t bios_enabled_show(struct 
 				 struct device_attribute *attr, char
*buf)
 {
 	acpi_status status;
-	unsigned long bios_enabled;
+	unsigned long long bios_enabled;
 
 	status = acpi_evaluate_integer(NULL, BIOS_ENABLED, NULL,
&bios_enabled);
 	if (ACPI_FAILURE(status))
@@ -500,7 +500,7 @@ static int __init intel_menlow_module_in
 {
 	int result = -ENODEV;
 	acpi_status status;
-	unsigned long enable;
+	unsigned long long enable;
 
 	if (acpi_disabled)
 		return result;

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] 3/3 intel_menlow: syncing with acpi function changes
  2008-11-05 10:46 [PATCH] 3/3 intel_menlow: syncing with acpi function changes Thomas, Sujith
@ 2008-11-07  1:30 ` Len Brown
  0 siblings, 0 replies; 2+ messages in thread
From: Len Brown @ 2008-11-07  1:30 UTC (permalink / raw)
  To: Thomas, Sujith
  Cc: Zhang, Rui, Linux Kernel Mailing List, linux-acpi, Andrew Morton

Sujith,

AFAICS, this patch is already applied.
since the e-mail is line-wrapped, it is tedious to verify,
let me know if i'm wrong.

thanks,
-Len


On Wed, 5 Nov 2008, Thomas, Sujith wrote:

> 4th parameter of acpi_evaluate_integer changed to unsigned long long
> acpi_driver_data() changed to inline function from macro
> 
> Signed-off-by: Sujith Thomas <sujith.thomas@intel.com>
> ---
>  drivers/misc/intel_menlow.c |   20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> Index: linux-2.6.27/drivers/misc/intel_menlow.c
> ===================================================================
> --- linux-2.6.27.orig/drivers/misc/intel_menlow.c
> +++ linux-2.6.27/drivers/misc/intel_menlow.c
> @@ -62,7 +62,7 @@ static int memory_get_int_max_bandwidth(
>  {
>  	struct acpi_device *device = cdev->devdata;
>  	acpi_handle handle = device->handle;
> -	unsigned long value;
> +	unsigned long long value;
>  	struct acpi_object_list arg_list;
>  	union acpi_object arg;
>  	acpi_status status = AE_OK;
> @@ -98,7 +98,7 @@ static int memory_get_cur_bandwidth(stru
>  {
>  	struct acpi_device *device = cdev->devdata;
>  	acpi_handle handle = device->handle;
> -	unsigned long value;
> +	unsigned long long value;
>  	struct acpi_object_list arg_list;
>  	union acpi_object arg;
>  	acpi_status status = AE_OK;
> @@ -112,7 +112,7 @@ static int memory_get_cur_bandwidth(stru
>  	if (ACPI_FAILURE(status))
>  		return -EFAULT;
>  
> -	return sprintf(buf, "%ld\n", value);
> +	return sprintf(buf, "%lld\n", value);
>  }
>  
>  static int memory_set_cur_bandwidth(struct thermal_cooling_device
> *cdev,
> @@ -139,7 +139,7 @@ static int memory_set_cur_bandwidth(stru
>  
>  	status =
>  	    acpi_evaluate_integer(handle, MEMORY_SET_BANDWIDTH,
> &arg_list,
> -				  (unsigned long *)&temp);
> +				  (unsigned long long *)&temp);
>  
>  	printk(KERN_INFO
>  	       "Bandwidth value was %d: status is %d\n", state, status);
> @@ -183,7 +183,7 @@ static int intel_menlow_memory_add(struc
>  		goto end;
>  	}
>  
> -	acpi_driver_data(device) = cdev;
> +	device->driver_data = cdev;
>  	result = sysfs_create_link(&device->dev.kobj,
>  				&cdev->device.kobj, "thermal_cooling");
>  	if (result)
> @@ -268,7 +268,7 @@ static int sensor_get_auxtrip(acpi_handl
>  		return -EINVAL;
>  
>  	status = acpi_evaluate_integer(handle, index ? GET_AUX1 :
> GET_AUX0,
> -				       NULL, (unsigned long *)value);
> +				       NULL, (unsigned long long
> *)value);
>  	if (ACPI_FAILURE(status))
>  		return -EIO;
>  
> @@ -296,7 +296,7 @@ static int sensor_set_auxtrip(acpi_handl
>  		return -EINVAL;
>  
>  	status = acpi_evaluate_integer(handle, index ? GET_AUX0 :
> GET_AUX1,
> -				       NULL, (unsigned long *)&temp);
> +				       NULL, (unsigned long long
> *)&temp);
>  	if (ACPI_FAILURE(status))
>  		return -EIO;
>  	if ((index && value < temp) || (!index && value > temp))
> @@ -304,7 +304,7 @@ static int sensor_set_auxtrip(acpi_handl
>  
>  	arg.integer.value = value;
>  	status = acpi_evaluate_integer(handle, index ? SET_AUX1 :
> SET_AUX0,
> -				       &args, (unsigned long *)&temp);
> +				       &args, (unsigned long long
> *)&temp);
>  	if (ACPI_FAILURE(status))
>  		return -EIO;
>  
> @@ -384,7 +384,7 @@ static ssize_t bios_enabled_show(struct 
>  				 struct device_attribute *attr, char
> *buf)
>  {
>  	acpi_status status;
> -	unsigned long bios_enabled;
> +	unsigned long long bios_enabled;
>  
>  	status = acpi_evaluate_integer(NULL, BIOS_ENABLED, NULL,
> &bios_enabled);
>  	if (ACPI_FAILURE(status))
> @@ -500,7 +500,7 @@ static int __init intel_menlow_module_in
>  {
>  	int result = -ENODEV;
>  	acpi_status status;
> -	unsigned long enable;
> +	unsigned long long enable;
>  
>  	if (acpi_disabled)
>  		return result;
> --
> 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
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-11-07  1:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-11-05 10:46 [PATCH] 3/3 intel_menlow: syncing with acpi function changes Thomas, Sujith
2008-11-07  1:30 ` Len Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).