Commit ce7ff1cf authored Oct 31, 2017 by Colin Ian King Committed by Andy Shevchenko Nov 05, 2017

platform/x86: dell-smo8800: remove redundant assignments to byte_data



Variable byte_data is being initialized and re-assigned with values that
are never read. Remove these as these redundant assignments. Cleans up
clang warning:

drivers/platform/x86/dell-smo8800.c:106:2: warning: Value stored to 'byte_data'
is never read

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Pali Rohár <pali.rohar@gmail.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

parent 9968e12a

drivers/platform/x86/dell-smo8800.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -90,7 +90,7 @@ static ssize_t smo8800_misc_read(struct file file, char __user buf,
		struct smo8800_device, miscdev);

		u32 data = 0;
		unsigned char byte_data = 0;
		unsigned char byte_data;
		ssize_t retval = 1;

		if (count < 1)
		@@ -103,7 +103,6 @@ static ssize_t smo8800_misc_read(struct file file, char __user buf,
		if (retval)
		return retval;

		byte_data = 1;
		retval = 1;

		if (data < 255)