On Mon, Oct 23, 2023 at 09:30:21AM +0200, Hannes Reinecke wrote: > @@ -1925,10 +1925,12 @@ megaraid_abort_and_reset(adapter_t *adapter, struct scsi_cmnd *cmd, int aor) > struct list_head *pos, *next; > scb_t *scb; > > - dev_warn(&adapter->dev->dev, "%s cmd=%x <c=%d t=%d l=%d>\n", > - (aor == SCB_ABORT)? "ABORTING":"RESET", > - cmd->cmnd[0], cmd->device->channel, > - cmd->device->id, (u32)cmd->device->lun); > + if (aor == SCB_ABORT) > + dev_warn(&adapter->dev->dev, "ABORTING cmd=%x <c=%d t=%d l=%d>\n", Please avoid the overly long line here. Otherwise looks good.