bnxt: Add support for new phy_types and speeds - Part #2

[bhargava.marreddy_broadcom.com]

Summary

1. For ifmr->ifm_active, consider new phy_types and speeds.
2. Remove all workarounds while updating ifmr->ifm_active.
3. For BNXT_IFMEDIA_ADD, added Three more speeds IFM_10G_T, IFM_2500_T & IFM_2500_KX.

Considered all review comments of https://reviews.freebsd.org/D12423 while preparing this patch.

[shurd]

This seems to be a very complex and fragile way of looking up the supported media type by speed. Why not just loop through ifm_list looking for a matching speed? Something like this:

struct ifmedia_entry *next;
uint64_t target_baudrate = bnxt_get_baudrate(softc->link_info);
int match = IFM_UNKNOWN;

LIST_FOREACH(next, &(iflib_get_media(ctx)->ifm_list), ifm_list) {
    if (ifmedia_baudrate(next->ifm_media) == target_baudrate) {
        match = next->ifm_media;
        break;
    }
}
ifmr->ifm_active |= match;

Since you already have a list of supported media types?

[bhargava.marreddy_broadcom.com]

I agree with you!! Thanks for the suggestion.

Only case needs to be taken care is, after media/phy change, bnxt_add_media_types() should get called prior to this bnxt_media_status().
Any way, I'll take care of that in other patch which is under review process currently.

[shurd]

[shurd]

committed as r325488