platform/x86: fujitsu-laptop: use strcpy to set ACPI device names and classes
authorMichał Kępień <kernel@kempniu.pl>
Fri, 16 Jun 2017 04:40:54 +0000 (06:40 +0200)
committerDarren Hart (VMware) <dvhart@infradead.org>
Sat, 1 Jul 2017 03:13:03 +0000 (20:13 -0700)
No formatting is needed when setting ACPI device name and class, so
switch to using strcpy() for this purpose.

Signed-off-by: Michał Kępień <kernel@kempniu.pl>
Reviewed-by: Jonathan Woithe <jwoithe@just42.net>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>
drivers/platform/x86/fujitsu-laptop.c

index 04796b27a4f01577b74a0842b58a7c05337a52d2..b4e7a93c9877535a5b7dca1c803cbd75f33455a8 100644 (file)
@@ -413,8 +413,8 @@ static int acpi_fujitsu_bl_add(struct acpi_device *device)
                return -ENOMEM;
 
        fujitsu_bl = priv;
-       sprintf(acpi_device_name(device), "%s", ACPI_FUJITSU_BL_DEVICE_NAME);
-       sprintf(acpi_device_class(device), "%s", ACPI_FUJITSU_CLASS);
+       strcpy(acpi_device_name(device), ACPI_FUJITSU_BL_DEVICE_NAME);
+       strcpy(acpi_device_class(device), ACPI_FUJITSU_CLASS);
        device->driver_data = priv;
 
        error = acpi_fujitsu_bl_input_setup(device);
@@ -802,9 +802,8 @@ static int acpi_fujitsu_laptop_add(struct acpi_device *device)
        WARN_ONCE(fext, "More than one FUJ02E3 ACPI device was found.  Driver may not work as intended.");
        fext = device;
 
-       sprintf(acpi_device_name(device), "%s",
-               ACPI_FUJITSU_LAPTOP_DEVICE_NAME);
-       sprintf(acpi_device_class(device), "%s", ACPI_FUJITSU_CLASS);
+       strcpy(acpi_device_name(device), ACPI_FUJITSU_LAPTOP_DEVICE_NAME);
+       strcpy(acpi_device_class(device), ACPI_FUJITSU_CLASS);
        device->driver_data = priv;
 
        /* kfifo */