The "bad status" case is impossible -- the switch value is constrained to 2

bits -- so just add the default case to the one that returns
STATUS_WIRE_FAILED.
This commit is contained in:
mycroft 2003-10-28 23:37:59 +00:00
parent 898602e6b1
commit 16413c6f77

View File

@ -1,4 +1,4 @@
/* $NetBSD: umass.c,v 1.106 2003/10/25 18:28:31 christos Exp $ */
/* $NetBSD: umass.c,v 1.107 2003/10/28 23:37:59 mycroft Exp $ */
/*
* Copyright (c) 2003 The NetBSD Foundation, Inc.
@ -131,7 +131,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: umass.c,v 1.106 2003/10/25 18:28:31 christos Exp $");
__KERNEL_RCSID(0, "$NetBSD: umass.c,v 1.107 2003/10/28 23:37:59 mycroft Exp $");
#include "atapibus.h"
#include "scsibus.h"
@ -1641,13 +1641,9 @@ umass_cbi_state(usbd_xfer_handle xfer, usbd_private_handle priv,
status = STATUS_CMD_FAILED;
break;
case IDB_VALUE_PHASE:
default: /* XXX: gcc */
status = STATUS_WIRE_FAILED;
break;
default:
printf("%s: bad status\n",
USBDEVNAME(sc->sc_dev));
status = -1;
break;
}
sc->transfer_state = TSTATE_IDLE;