Эх сурвалжийг харах

asus-laptop: remove redundant initializers

Initializing rv to AE_OK is pointless because later function results are
assigned to them and only then the variable is used

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@gmail.com>
Acked-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
Giedrius Statkevičius 9 жил өмнө
parent
commit
2ce6d9932d

+ 3 - 3
drivers/platform/x86/asus-laptop.c

@@ -771,7 +771,7 @@ static int asus_read_brightness(struct backlight_device *bd)
 {
 {
 	struct asus_laptop *asus = bl_get_data(bd);
 	struct asus_laptop *asus = bl_get_data(bd);
 	unsigned long long value;
 	unsigned long long value;
-	acpi_status rv = AE_OK;
+	acpi_status rv;
 
 
 	rv = acpi_evaluate_integer(asus->handle, METHOD_BRIGHTNESS_GET,
 	rv = acpi_evaluate_integer(asus->handle, METHOD_BRIGHTNESS_GET,
 				   NULL, &value);
 				   NULL, &value);
@@ -867,7 +867,7 @@ static ssize_t infos_show(struct device *dev, struct device_attribute *attr,
 	int len = 0;
 	int len = 0;
 	unsigned long long temp;
 	unsigned long long temp;
 	char buf[16];		/* enough for all info */
 	char buf[16];		/* enough for all info */
-	acpi_status rv = AE_OK;
+	acpi_status rv;
 
 
 	/*
 	/*
 	 * We use the easy way, we don't care of off and count,
 	 * We use the easy way, we don't care of off and count,
@@ -1267,7 +1267,7 @@ static DEVICE_ATTR_RO(ls_value);
 static int asus_gps_status(struct asus_laptop *asus)
 static int asus_gps_status(struct asus_laptop *asus)
 {
 {
 	unsigned long long status;
 	unsigned long long status;
-	acpi_status rv = AE_OK;
+	acpi_status rv;
 
 
 	rv = acpi_evaluate_integer(asus->handle, METHOD_GPS_STATUS,
 	rv = acpi_evaluate_integer(asus->handle, METHOD_GPS_STATUS,
 				   NULL, &status);
 				   NULL, &status);