Re: [PATCH] staging: octeon: replace BUG() with WARN_ON_ONCE() in cvn_oct_xmit()

[Mark Adamenko]

> If this is impossible to ever hit, then nothing changed here, right?  So
> why is this even needed?

You're right. Would removing the default case entirely be preferred?