Re: [patch net-next 0/4] Introduce Mellanox Technologies Switch ASICs switchdev drivers

[Jiri Pirko <jiri@resnulli.us>]

Fri, Jul 24, 2015 at 02:03:20AM CEST, sfeldma@gmail.com wrote:
>On Thu, Jul 23, 2015 at 8:43 AM, Jiri Pirko <jiri@resnulli.us> wrote:
>> This patchset introduces Mellanox Technologies Switch driver infrastructure
>> and support for SwitchX-2 ASIC.
>>
>> The driver is divided into 3 logical parts:
>> 1) Bus - implements switch bus interface. Currently only PCI bus is
>>    implemented, but more buses will be added in the future. Namely I2C
>>    and SGMII.
>>    (patch #2)
>> 2) Driver - implemements of ASIC-specific functions.
>>    Currently SwitchX-2 ASIC is supported, but a plan exists to introduce
>>    support for Spectrum ASIC in the near future.
>>    (patch #4)
>> 3) Core - infrastructure that glues buses and drivers together.
>>    It implements register access logic (EMADs) and takes care of RX traps
>>    and events.
>>    (patch #1 and #3)
>>
>> Ido Schimmel (1):
>>   mlxsw: Add interface to access registers and process events
>>
>> Jiri Pirko (3):
>>   mlxsw: Introduce Mellanox switch driver core
>>   mlxsw: Add PCI bus implementation
>>   mlxsw: Introduce Mellanox SwitchX-2 ASIC support
>
>This is awesome!  Reviewing...

Thanks!


>
>checkpatch.pl shows 1 ERROR, bunch of >80 chars WARNs, and bunch of
>CHECKs on space after cast.

Those >80char WARNs are mostly macro "\" on col 81. That is afaik ok.
Maybe that should be changed in checkpatch as well.

The error is false possitive. It is again in macro, checkpatch thinks it
is a function and yells about "{" on the same line. However it is a
struct initialization so in that case it is ok.


>
>On the CHECKs on space after cast, should we modify checkpatch.pl to
>not flag those for drivers/net?

I agree.


>
>-scott