Re: [PATCH] net: phy: bcm54811: add support for BroadR-Reach mode

[Andrew Lunn]

> @@ -258,6 +257,9 @@ static const struct phy_setting settings[] = {
>  	PHY_SETTING(    100, HALF,    100baseT_Half		),
>  	PHY_SETTING(    100, HALF,    100baseFX_Half		),
>  	PHY_SETTING(    100, FULL,    100baseFX_Full		),
> +	PHY_SETTING(    100, FULL,    4BR100			),
> +	PHY_SETTING(    100, FULL,    2BR100			),
> +	PHY_SETTING(    100, FULL,    1BR100			),

Please could you explain the name convention. IEEE puts the speed
first, then some letters to indicate the media type, and then a number
for the number of pairs. Why is this not followed here? 100BaseBR4?
100BaseBR2? 100BaseBR1? Are these names part of the BroadR-Reach
standard?

Also, is there any compatibility? Are 100BaseT1 and 1BR100 compatible?

      Andrew