Address review comments from the initial SR-IOV support patch.

This change includes the following updates:

• Rename the P7 VF PCI ID macro from NETXTREME_E_P7_VF to E_P7_VF, since the P7/Thor2 product strings no longer use the NetXtreme name. Other VF device IDs are unchanged.

• Make bnxt_vf_parse_schema() return a boolean value and use that return value in bnxt_iov_vf_add() to decide whether bnxt_set_vf_admin_mac() should be called. This also removes the separate has_admin_mac field.

• Fix indentation in bnxt_free_vf_resources() after dma_free_coherent(), so the NULL assignment is clearly separate from the function call.

• Update bnxt_hwrm_func_vf_resource_free() to use first_vf_id and last_vf_id while looping through HWRM_FUNC_VF_RESC_FREE.

This patch addresses the code review comments provided for:
https://reviews.freebsd.org/D56197