9606 qede: error: this 'for' clause does not guard
Reviewed by: Andy Fiddaman <af@citrus-it.net>
Reviewed by: Andrew Stormont <andyjstormont@gmail.com>
Reviewed by: Yuri Pankov <yuripv@yuripv.net>
Reviewed by: Ken Mays <kmays2000@gmail.com>
Approved by: Dan McDonald <danmcd@joyent.com>
diff --git a/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_dbg_fw_funcs.c b/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_dbg_fw_funcs.c
index 383deaa..62c6a7a 100644
--- a/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_dbg_fw_funcs.c
+++ b/usr/src/uts/common/io/qede/579xx/drivers/ecore/ecore_dbg_fw_funcs.c
@@ -5444,12 +5444,13 @@
if (!data_mask)
return DBG_STATUS_INVALID_ARGS;
- for (lsb = 0; lsb < 32 && !(data_mask & 1); lsb++, data_mask >>= 1);
- for (width = 0;
- width < 32 - lsb && (data_mask & 1);
- width++, data_mask >>= 1);
- if (data_mask)
- return DBG_STATUS_INVALID_ARGS;
+ for (lsb = 0; lsb < 32 && !(data_mask & 1); lsb++)
+ data_mask >>= 1;
+
+ for (width = 0; width < 32 - lsb && (data_mask & 1); width++)
+ data_mask >>= 1;
+ if (data_mask)
+ return DBG_STATUS_INVALID_ARGS;
range = (lsb << 5) | (width - 1);
}