From 16413c6f77aae27dacb6b98bf0ceb00bbd541ebc Mon Sep 17 00:00:00 2001 From: mycroft Date: Tue, 28 Oct 2003 23:37:59 +0000 Subject: [PATCH] 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. --- sys/dev/usb/umass.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/sys/dev/usb/umass.c b/sys/dev/usb/umass.c index 36355d331c2a..9e9fa1a3b0b3 100644 --- a/sys/dev/usb/umass.c +++ b/sys/dev/usb/umass.c @@ -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 -__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;