usr/src/sys/arm/xscale/ixp425/ixp425_npe.c:695]: (style) Unsigned variable 'cNum' can't be negative so it is unnecessary to test it. Source code is if (!(0 <= cNum && cNum < IX_NPEDL_CTXT_NUM_MAX)) { but uint32_t cNum = (addrInfo & IX_NPEDL_MASK_STATE_ADDR_CTXT_NUM) >> IX_NPEDL_OFFSET_STATE_ADDR_CTXT_NUM;
Hi, Unfortunately, it seems this was never looked at. However, this driver is no longer present in the tree. It was removed with big-endian arm support in: https://cgit.freebsd.org/src/commit/?id=ff9452772d2fbb1268af844ff4c0980942046546 For what it's worth, your analysis seems correct. Closing now as there is nothing more to be done.