dell-laptop: use dell_smbios_find_token() instead of find_token_location()
authorMichał Kępień <kernel@kempniu.pl>
Fri, 22 Jan 2016 14:27:24 +0000 (15:27 +0100)
committerDarren Hart <dvhart@linux.intel.com>
Wed, 23 Mar 2016 17:05:42 +0000 (10:05 -0700)
Replace all uses of find_token_location() with dell_smbios_find_token()
to avoid directly accessing the da_tokens table.

Signed-off-by: Michał Kępień <kernel@kempniu.pl>
Reviewed-by: Pali Rohár <pali.rohar@gmail.com>
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
drivers/platform/x86/dell-laptop.c

index 4d1694d..76064c8 100644 (file)
@@ -874,15 +874,15 @@ static void dell_cleanup_rfkill(void)
 static int dell_send_intensity(struct backlight_device *bd)
 {
        struct calling_interface_buffer *buffer;
-       int token;
+       struct calling_interface_token *token;
        int ret;
 
-       token = find_token_location(BRIGHTNESS_TOKEN);
-       if (token == -1)
+       token = dell_smbios_find_token(BRIGHTNESS_TOKEN);
+       if (!token)
                return -ENODEV;
 
        buffer = dell_smbios_get_buffer();
-       buffer->input[0] = token;
+       buffer->input[0] = token->location;
        buffer->input[1] = bd->props.brightness;
 
        if (power_supply_is_system_supplied() > 0)
@@ -899,15 +899,15 @@ static int dell_send_intensity(struct backlight_device *bd)
 static int dell_get_intensity(struct backlight_device *bd)
 {
        struct calling_interface_buffer *buffer;
-       int token;
+       struct calling_interface_token *token;
        int ret;
 
-       token = find_token_location(BRIGHTNESS_TOKEN);
-       if (token == -1)
+       token = dell_smbios_find_token(BRIGHTNESS_TOKEN);
+       if (!token)
                return -ENODEV;
 
        buffer = dell_smbios_get_buffer();
-       buffer->input[0] = token;
+       buffer->input[0] = token->location;
 
        if (power_supply_is_system_supplied() > 0)
                dell_smbios_send_request(0, 2);
@@ -1987,8 +1987,8 @@ static void kbd_led_exit(void)
 static int __init dell_init(void)
 {
        struct calling_interface_buffer *buffer;
+       struct calling_interface_token *token;
        int max_intensity = 0;
-       int token;
        int ret;
 
        if (!dmi_check_system(dell_device_table))
@@ -2030,10 +2030,10 @@ static int __init dell_init(void)
        if (acpi_video_get_backlight_type() != acpi_backlight_vendor)
                return 0;
 
-       token = find_token_location(BRIGHTNESS_TOKEN);
-       if (token != -1) {
+       token = dell_smbios_find_token(BRIGHTNESS_TOKEN);
+       if (token) {
                buffer = dell_smbios_get_buffer();
-               buffer->input[0] = token;
+               buffer->input[0] = token->location;
                dell_smbios_send_request(0, 2);
                if (buffer->output[0] == 0)
                        max_intensity = buffer->output[3];