On Wed, Nov 29, 2023 at 11:43:50PM +0300, Sergey Shtylyov wrote: > In init_freecom(), the results of usb_stor_control_msg() calls are stored > in the local variable and then printed out by usb_stor_dbg() (if enabled), > except for the 1st call, the result of which is completely ignored. Drop > the useless assignment. Instead, you should check the return value and handle it properly, don't just drop the checking entirely, that's not good. thanks, greg k-h
