Cppcheck shows here duplicated break.
Code `state->speed = SPEED_1000;` will be never executed because above
it there is break statement.
Almost identical statement is placed in another realtek driver
https://github.com/openwrt/openwrt/blob/
18a53d43d646290053eff4736ec852efcf6bf510/target/linux/realtek/files-5.4/drivers/net/dsa/rtl83xx/dsa.c#L286-L294
Signed-off-by: Rafał Mikrut <mikrutrafal@protonmail.com>
case 1:
state->speed = SPEED_100;
break;
+ case 2:
state->speed = SPEED_1000;
break;
default: