diff mbox

[2/2] acpi:ipmi: Make IPMI user handler const

Message ID 1484055613-31286-3-git-send-email-minyard@acm.org (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Corey Minyard Jan. 10, 2017, 1:40 p.m. UTC
From: Corey Minyard <cminyard@mvista.com>

ipmi_create_user() now takes the user handlers as const, make
it const in the ACPI IPMI code.

Cc: linux-acpi@vger.kernel.org
Cc: Zhao Yakui <yakui.zhao@intel.com>
Cc: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
---
 drivers/acpi/acpi_ipmi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Rafael J. Wysocki Feb. 9, 2017, 2:02 p.m. UTC | #1
On Tuesday, January 10, 2017 07:40:13 AM minyard@acm.org wrote:
> From: Corey Minyard <cminyard@mvista.com>
> 
> ipmi_create_user() now takes the user handlers as const, make
> it const in the ACPI IPMI code.
> 
> Cc: linux-acpi@vger.kernel.org
> Cc: Zhao Yakui <yakui.zhao@intel.com>
> Cc: Lv Zheng <lv.zheng@intel.com>
> Signed-off-by: Corey Minyard <cminyard@mvista.com>

Please let me know if you want me to apply this.

Otherwise, feel free to add my ACK to it and route it whatever way you think
is appropriate.

> ---
>  drivers/acpi/acpi_ipmi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/acpi/acpi_ipmi.c b/drivers/acpi/acpi_ipmi.c
> index f77956c..747c2ba 100644
> --- a/drivers/acpi/acpi_ipmi.c
> +++ b/drivers/acpi/acpi_ipmi.c
> @@ -56,7 +56,7 @@ struct acpi_ipmi_device {
>  struct ipmi_driver_data {
>  	struct list_head ipmi_devices;
>  	struct ipmi_smi_watcher bmc_events;
> -	struct ipmi_user_hndl ipmi_hndlrs;
> +	const struct ipmi_user_hndl ipmi_hndlrs;
>  	struct mutex ipmi_lock;
>  
>  	/*
> 

Thanks,
Rafael

--
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
Corey Minyard Feb. 9, 2017, 3:07 p.m. UTC | #2
On 02/09/2017 08:02 AM, Rafael J. Wysocki wrote:
> On Tuesday, January 10, 2017 07:40:13 AM minyard@acm.org wrote:
>> From: Corey Minyard <cminyard@mvista.com>
>>
>> ipmi_create_user() now takes the user handlers as const, make
>> it const in the ACPI IPMI code.
>>
>> Cc: linux-acpi@vger.kernel.org
>> Cc: Zhao Yakui <yakui.zhao@intel.com>
>> Cc: Lv Zheng <lv.zheng@intel.com>
>> Signed-off-by: Corey Minyard <cminyard@mvista.com>
> Please let me know if you want me to apply this.
>
> Otherwise, feel free to add my ACK to it and route it whatever way you think
> is appropriate.

Thanks, I'll add your ACK and apply it through the IPMI tree.

-corey

>> ---
>>   drivers/acpi/acpi_ipmi.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/acpi/acpi_ipmi.c b/drivers/acpi/acpi_ipmi.c
>> index f77956c..747c2ba 100644
>> --- a/drivers/acpi/acpi_ipmi.c
>> +++ b/drivers/acpi/acpi_ipmi.c
>> @@ -56,7 +56,7 @@ struct acpi_ipmi_device {
>>   struct ipmi_driver_data {
>>   	struct list_head ipmi_devices;
>>   	struct ipmi_smi_watcher bmc_events;
>> -	struct ipmi_user_hndl ipmi_hndlrs;
>> +	const struct ipmi_user_hndl ipmi_hndlrs;
>>   	struct mutex ipmi_lock;
>>   
>>   	/*
>>
> Thanks,
> Rafael
>

--
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
diff mbox

Patch

diff --git a/drivers/acpi/acpi_ipmi.c b/drivers/acpi/acpi_ipmi.c
index f77956c..747c2ba 100644
--- a/drivers/acpi/acpi_ipmi.c
+++ b/drivers/acpi/acpi_ipmi.c
@@ -56,7 +56,7 @@  struct acpi_ipmi_device {
 struct ipmi_driver_data {
 	struct list_head ipmi_devices;
 	struct ipmi_smi_watcher bmc_events;
-	struct ipmi_user_hndl ipmi_hndlrs;
+	const struct ipmi_user_hndl ipmi_hndlrs;
 	struct mutex ipmi_lock;
 
 	/*