reduce smatch gags in t4nex
Right now all of SMATCH is turned off for the
t4nex driver. We should reduce this. Right now I've left too in place, one of which is about unchecked function returns, which is mostly being done to deal with the general code that Chelsio shares. The second is about snprintf overflows. In practice the cases are OK due to how the versions work, but longer term we can remove that by moving those version strings into the UFM logic.