> Added 5000 as this is present in documentation bits but CPU can only go > up to 2.5. Should I drop it? Idea was to futureproof it since it really > seems they added these bits with the intention of having a newer switch > with more advanced ports. The other way to future proof this is to have a dev_err() and -EINVAL. Testing will then quickly find any missing places in the code. Andrew