[PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Colin King]

From: Colin Ian King <colin.king@canonical.com>

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>
---
 drivers/platform/x86/dell-smo8800.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/platform/x86/dell-smo8800.c b/drivers/platform/x86/dell-smo8800.c
index 37e646034ef8..1d87237bc731 100644
--- a/drivers/platform/x86/dell-smo8800.c
+++ b/drivers/platform/x86/dell-smo8800.c
@@ -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)
-- 
2.14.1

Re: [PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Andy Shevchenko]

On Tue, Oct 31, 2017 at 1:03 PM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> 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

Thanks for the patch.

As I read a code it was from the beginning like this. I dunno what was
a motivation of this pieces, thus I would like to hear Pali.

Per se patch looks pretty much good to me.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Pali Rohár]

On Tuesday 31 October 2017 15:47:58 Andy Shevchenko wrote:
> On Tue, Oct 31, 2017 at 1:03 PM, Colin King <colin.king@canonical.com> wrote:
> > From: Colin Ian King <colin.king@canonical.com>
> >
> > 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
> 
> Thanks for the patch.
> 
> As I read a code it was from the beginning like this. I dunno what was
> a motivation of this pieces, thus I would like to hear Pali.

IIRC this part of code comes from the lis3lv* driver which was used for
HP EliteBook machines. As both drivers provides similar interface for
userspace.

> Per se patch looks pretty much good to me.
> 

-- 
Pali Rohár
pali.rohar@gmail.com

Re: [PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Andy Shevchenko]

On Tue, Oct 31, 2017 at 3:55 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
> On Tuesday 31 October 2017 15:47:58 Andy Shevchenko wrote:
>> On Tue, Oct 31, 2017 at 1:03 PM, Colin King <colin.king@canonical.com> wrote:
>> > From: Colin Ian King <colin.king@canonical.com>
>> >
>> > 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
>>
>> Thanks for the patch.
>>
>> As I read a code it was from the beginning like this. I dunno what was
>> a motivation of this pieces, thus I would like to hear Pali.
>
> IIRC this part of code comes from the lis3lv* driver which was used for
> HP EliteBook machines. As both drivers provides similar interface for
> userspace.

OK, though it doesn't clarify the intention of the byte_data
(useless?) assignments.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Pali Rohár]

On Tuesday 31 October 2017 16:07:25 Andy Shevchenko wrote:
> On Tue, Oct 31, 2017 at 3:55 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
> > On Tuesday 31 October 2017 15:47:58 Andy Shevchenko wrote:
> >> On Tue, Oct 31, 2017 at 1:03 PM, Colin King <colin.king@canonical.com> wrote:
> >> > From: Colin Ian King <colin.king@canonical.com>
> >> >
> >> > 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
> >>
> >> Thanks for the patch.
> >>
> >> As I read a code it was from the beginning like this. I dunno what was
> >> a motivation of this pieces, thus I would like to hear Pali.
> >
> > IIRC this part of code comes from the lis3lv* driver which was used for
> > HP EliteBook machines. As both drivers provides similar interface for
> > userspace.
> 
> OK, though it doesn't clarify the intention of the byte_data
> (useless?) assignments.

Probably similar code pattern exists in that lis3lv* driver...

-- 
Pali Rohár
pali.rohar@gmail.com

Re: [PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Andy Shevchenko]

On Tue, Oct 31, 2017 at 4:13 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
> On Tuesday 31 October 2017 16:07:25 Andy Shevchenko wrote:
>> On Tue, Oct 31, 2017 at 3:55 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
>> > On Tuesday 31 October 2017 15:47:58 Andy Shevchenko wrote:
>> >> On Tue, Oct 31, 2017 at 1:03 PM, Colin King <colin.king@canonical.com> wrote:

>> OK, though it doesn't clarify the intention of the byte_data
>> (useless?) assignments.
>
> Probably similar code pattern exists in that lis3lv* driver...

So, it seems to me OK to apply the patch. No objections?

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Pali Rohár]

On Tuesday 31 October 2017 20:08:45 Andy Shevchenko wrote:
> On Tue, Oct 31, 2017 at 4:13 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
> > On Tuesday 31 October 2017 16:07:25 Andy Shevchenko wrote:
> >> On Tue, Oct 31, 2017 at 3:55 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
> >> > On Tuesday 31 October 2017 15:47:58 Andy Shevchenko wrote:
> >> >> On Tue, Oct 31, 2017 at 1:03 PM, Colin King <colin.king@canonical.com> wrote:
> 
> >> OK, though it doesn't clarify the intention of the byte_data
> >> (useless?) assignments.
> >
> > Probably similar code pattern exists in that lis3lv* driver...
> 
> So, it seems to me OK to apply the patch. No objections?

No objections, you can add my:

Acked-by: Pali Rohár <pali.rohar@gmail.com>

-- 
Pali Rohár
pali.rohar@gmail.com

Re: [PATCH] platform: x86: dell-smo8800: remove redundant assignments to byte_data

[Andy Shevchenko]

On Tue, Oct 31, 2017 at 8:41 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
> On Tuesday 31 October 2017 20:08:45 Andy Shevchenko wrote:
>> On Tue, Oct 31, 2017 at 4:13 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
>> > On Tuesday 31 October 2017 16:07:25 Andy Shevchenko wrote:
>> >> On Tue, Oct 31, 2017 at 3:55 PM, Pali Rohár <pali.rohar@gmail.com> wrote:
>> >> > On Tuesday 31 October 2017 15:47:58 Andy Shevchenko wrote:
>> >> >> On Tue, Oct 31, 2017 at 1:03 PM, Colin King <colin.king@canonical.com> wrote:
>>
>> >> OK, though it doesn't clarify the intention of the byte_data
>> >> (useless?) assignments.
>> >
>> > Probably similar code pattern exists in that lis3lv* driver...
>>
>> So, it seems to me OK to apply the patch. No objections?
>
> No objections, you can add my:
>
> Acked-by: Pali Rohár <pali.rohar@gmail.com>

Applied to my review and testing queue, thanks!

>
> --
> Pali Rohár
> pali.rohar@gmail.com



-- 
With Best Regards,
Andy Shevchenko