On December 15, 2017 1:10:44 AM PST, Andrew Lunn <and...@lunn.ch> wrote:
>On Thu, Dec 14, 2017 at 05:48:16PM -0800, Florian Fainelli wrote:
>> Add an entry for the builtin PHYs present in the Broadcom BCM5395
>switch. This
>> allows us to retrieve the PHY statistics among other things.
>
>>  static struct phy_driver broadcom_drivers[] = {
>>  {
>>      .phy_id         = PHY_ID_BCM5411,
>> @@ -679,6 +710,16 @@ static struct phy_driver broadcom_drivers[] = {
>>      .config_init    = brcm_fet_config_init,
>>      .ack_interrupt  = brcm_fet_ack_interrupt,
>>      .config_intr    = brcm_fet_config_intr,
>> +}, {
>> +    .phy_id         = PHY_ID_BCM5395,
>> +    .phy_id_mask    = 0xfffffff0,
>> +    .name           = "Broadcom BCM5395",
>> +    .flags          = PHY_IS_INTERNAL,
>> +    .features       = PHY_GBIT_FEATURES,
>> +    .get_sset_count = bcm_phy_get_sset_count,
>> +    .get_strings    = bcm_phy_get_strings,
>> +    .get_stats      = bcm53xx_phy_get_stats,
>> +    .probe          = bcm53xx_phy_probe,
>>  } };
>
>Hi Florian
>
>Is this the only PHY supported by this driver which has statistics?

I need to cross check with the various data sheets to be sure, but it is quite 
likely that all Gigabit capable PHYs listed above would also support the same 
statistics. In any case, this would be a follow on patch to this one.

Thanks!
>
>   Andrew

Hi Andrew,
-- 
Florian

Reply via email to