On Fri, Apr 23, 2021 at 01:39:24PM +0200, Hannes Reinecke wrote: > Make ql_pcmd() a void function and set the SCSI result directly. > > Signed-off-by: Hannes Reinecke <hare@xxxxxxx> > --- > drivers/scsi/qlogicfas408.c | 75 ++++++++++++++++++++++++------------- > 1 file changed, 49 insertions(+), 26 deletions(-) Can you explain why this is useful? Because it does not really look like it cleans up anything as-is.