Commit fef10146 authored by Everest K.C.'s avatar Everest K.C. Committed by Greg Kroah-Hartman
Browse files

staging: gpib: Remove unused value 



The variable `complement_count` is assigned a value which is again
overwritten in the next statement.
Fix this by removing the first value assigning statement

This issue was reported by Coverity Scan.
Report:
CID 1600790: (#1 of 1): Unused value (UNUSED_VALUE)
assigned_value: Assigning value from length to complement_count here,
but that stored value is overwritten before it can be used.

Signed-off-by: default avatarEverest K.C. <everestkc@everestkc.com.np>
Reviewed-by: default avatarShuah Khan <skhan@linuxfoundation.org>
Link: https://lore.kernel.org/r/20241016075544.4125-1-everestkc@everestkc.com.np


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 214c2754

drivers/staging/gpib/ni_usb/ni_usb_gpib.c

+0 −1
Original line number Diff line number Diff line
@@ -759,7 +759,6 @@ static int ni_usb_write(gpib_board_t *board, uint8_t *buffer, size_t length, 
	if (!out_data)

		return -ENOMEM;

	out_data[i++] = 0x0d;

	complement_count = length;

	complement_count = length - 1;

	complement_count = ~complement_count;

	out_data[i++] = complement_count & 0xff;