On 7/1/2020 9:54 PM, Jakub Kicinski wrote:
> On Wed, 1 Jul 2020 17:32:47 +0300 Ido Schimmel wrote:
>> diff --git a/include/net/devlink.h b/include/net/devlink.h
>> index 8f9db991192d..91752b79bb29 100644
>> --- a/include/net/devlink.h
>> +++ b/include/net/devlink.h
>> @@ -68,10 +68,13 @@ struct devlink_port_pci_vf_attrs {
>> * struct devlink_port_attrs - devlink port object
>> * @flavour: flavour of the port
>> * @split: indicates if this is split port
>> + * @lanes: maximum number of lanes the port supports.
>> + * 0 value is not passed to netlink and valid number is a power of 2.
> Why power of two? what about 100G R10?
Missed that, thanks.
>
>> * @switch_id: if the port is part of switch, this is buffer with ID,
>> otherwise this is NULL
>> */
>> struct devlink_port_attrs {
>> u8 split:1;
>> + u32 lanes;
>> enum devlink_port_flavour flavour;
>> struct netdev_phys_item_id switch_id;
>> union {