[OpenWrt-Devel] [PATCH][RFC] kernel: swconfig: add API for setting port link speed

[John Crispin]

On 12/12/2015 09:04, Rafał Miłecki wrote:
> On 12 December 2015 at 07:54, John Crispin <blogic at openwrt.org> wrote:
>> Hi Rafał,
>>
>> in addition to the comment i made last night, i found another possible
>> issue. see below.
>>
>> On 11/12/2015 14:48, Rafał Miłecki wrote:
>>> Some switches can force link speed for a port. Let's add API that will
>>> allow driver to export this feature.
>>>
>>> Signed-off-by: Rafał Miłecki <zajec5 at gmail.com>
>>> ---
>>>  .../linux/generic/files/drivers/net/phy/swconfig.c | 29 ++++++++++++++++++++++
>>>  target/linux/generic/files/include/linux/switch.h  |  2 ++
>>>  2 files changed, 31 insertions(+)
>>>
>>> diff --git a/target/linux/generic/files/drivers/net/phy/swconfig.c b/target/linux/generic/files/drivers/net/phy/swconfig.c
>>> index 6bb3be1..58d6cf5 100644
>>> --- a/target/linux/generic/files/drivers/net/phy/swconfig.c
>>> +++ b/target/linux/generic/files/drivers/net/phy/swconfig.c
>>> @@ -187,6 +187,34 @@ swconfig_get_link(struct switch_dev *dev, const struct switch_attr *attr,
>>>  }
>>>
>>>  static int
>>> +swconfig_set_link(struct switch_dev *dev, const struct switch_attr *attr,
>>> +                     struct switch_val *val)
>>> +{
>>> +     enum switch_port_speed speed;
>>> +     const char *s = val->value.s;
>>> +     int len;
>>> +
>>> +     if (val->port_vlan >= dev->ports)
>>> +             return -EINVAL;
>>> +
>>> +     if (!dev->ops->set_port_link)
>>> +             return -EOPNOTSUPP;
>>> +
>>> +     len = strchrnul(s, ' ') - s;
>>> +     if (!strncmp(s, "10", len)) {
>>> +             speed = SWITCH_PORT_SPEED_10;
>>> +     } else if (!strncmp(s, "100", len)) {
>>> +             speed = SWITCH_PORT_SPEED_100;
>>> +     } else if (!strncmp(s, "1000", len)) {
>>> +             speed = SWITCH_PORT_SPEED_1000;
>>> +     } else {
>>> +             speed = SWITCH_PORT_SPEED_UNKNOWN;
>>> +     }
>>> +
>>> +     return dev->ops->set_port_link(dev, val->port_vlan, speed);
>>
>> this makes the assumption, that all drivers always add the
>> set_port_link() callback and uses it unconditionally. i am impartial on
>> how this is solved in detail as long as some kind of check is added
> 
> You had to miss
> if (!dev->ops->set_port_link)
> part :)
> 

i did indeed ...
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel