On 15.12.2018 20:59, Andrew Lunn wrote:
>>>> +static int tja11xx_config_init(struct phy_device *phydev)
>>>> +{
>>>> + int ret;
>>>> +
>>>> + ret = tja11xx_enable_reg_write(phydev);
>>>> + if (ret)
>>>> + return ret;
>>>> +
>>>> + phydev->irq = PHY_POLL;
>>>> + phydev->autoneg = AUTONEG_DISABLE;
>>>> + phydev->speed = SPEED_100;
>>
>> One more thing: In the data sheet there are SPEED_SELECT bits allowing
>> to set also 10MBit and 1GBit mode. Don't you want to support this?
>
> Hi Heiner
>
> Did you read footnote 2?
>
> Speed Select. 00: 10 Mbits; 01: 100Mbit/s, 10: 1000Mbit/s, 11:
> reserved. a write access value other than 01 is ignored.
>
Uh, missed that. Thanks for the hint.
> So the bits are there, but not really used. It can only do 100Mbit/s.
>
> Andrew
>
>