Also can you place the mv88e6xxx_serdes_get_lane() function as static inline in the serdes.h header? So that it's obvious that it's a wrapper and not a switch implementation.
Ho and you can skip the 'chip->info->ops->' from the commit subject line ;-)
