[OpenWrt-Devel] [PATCH][RFC] kernel: swconfig: add API for setting port link speed
zajec5 at gmail.com
Wed Dec 16 07:10:12 EST 2015
On 13 December 2015 at 13:40, Stefan Rompf <stefan at loplof.de> wrote:
>> int (*get_port_link)(struct switch_dev *dev, int port,
>> struct switch_port_link *link);
>> + int (*set_port_link)(struct switch_dev *dev, int port,
>> + enum switch_port_speed speed);
> this creates an assymetric API. I think the prototype of the set function
> should be
> int (*set_port_link)(struct switch_dev *dev, int port,
> struct switch_port_link *link);
> to allow setting other parameters that are part of struct switch_port_link,
> like duplex or flow control. I'd be rather happy to disable crappy ethernet
> flow control on my router...
Thanks, I'll work on this.
For the future: it's preferred to reply to the latest patch version.
Your comment isn't visible for the V1 in patchwork:
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
More information about the openwrt-devel