svn commit: r368473 - head/sys/dev/mfi

[Justin Hibbits]

Author: jhibbits
Date: Wed Dec  9 02:07:01 2020
New Revision: 368473
URL: https://svnweb.freebsd.org/changeset/base/368473

Log:
  dev/mfi: Make a seemingly bogus conditional unconditional
  
  Summary:
  r358689 attempted to fix a clang warning/error by inferring the intent
  of the condition "(cdb[0] != 0x28 || cdb[0] != 0x2A)".  Unfortunately, it looks
  like this broke things.  Instead, fix this by making this path unconditional,
  effectively reverting to the previous state.
  
  PR:		kern/251483
  Reviewed By:	ambrisko
  MFC after:	2 days
  Differential Revision: https://reviews.freebsd.org/D27515

Modified:
  head/sys/dev/mfi/mfi_tbolt.c

Modified: head/sys/dev/mfi/mfi_tbolt.c
==============================================================================
--- head/sys/dev/mfi/mfi_tbolt.c	Wed Dec  9 02:05:14 2020	(r368472)
+++ head/sys/dev/mfi/mfi_tbolt.c	Wed Dec  9 02:07:01 2020	(r368473)
@@ -1104,16 +1104,12 @@ mfi_tbolt_send_frame(struct mfi_softc *sc, struct mfi_
 
 	if (hdr->cmd == MFI_CMD_PD_SCSI_IO) {
 		/* check for inquiry commands coming from CLI */
-		if (cdb[0] != 0x28 && cdb[0] != 0x2A) {
-			if ((req_desc = mfi_tbolt_build_mpt_cmd(sc, cm)) ==
-			    NULL) {
-				device_printf(sc->mfi_dev, "Mapping from MFI "
-				    "to MPT Failed \n");
-				return 1;
-			}
+		if ((req_desc = mfi_tbolt_build_mpt_cmd(sc, cm)) ==
+		    NULL) {
+			device_printf(sc->mfi_dev, "Mapping from MFI "
+			    "to MPT Failed \n");
+			return 1;
 		}
-		else
-			device_printf(sc->mfi_dev, "DJA NA XXX SYSPDIO\n");
 	} else if (hdr->cmd == MFI_CMD_LD_SCSI_IO ||
 	    hdr->cmd == MFI_CMD_LD_READ || hdr->cmd == MFI_CMD_LD_WRITE) {
 		cm->cm_flags |= MFI_CMD_SCSI;