Re: [PATCH v2 1/2] include: linux: sysfs: Add __ATTR_NAMED macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 13 September 2017 14:58:23 GMT-07:00, Joe Perches <joe@xxxxxxxxxxx> wrote:
>On Thu, 2017-09-14 at 00:43 +0300, Dan Carpenter wrote:
>> He was exagerating a bit to call it a "static checker" warning...
>
>Not really.
>
>False positives and false negatives exist in just about
>every static
>checker.
>
>> It's just checkpatch.pl complaining about adding spaces around the -
>> operator.
>
>checkpatch is a brain-damaged by design static checker.
>regexes can only be sensitive to patterns, not compiled code.
>
>> The sysfs file has a hyphen in the middle.
>
>Another option would be to use an underscore instead.

Userspace ABI plus it really does mean subtract.  Used in description of differential channels.
Most such IIO ABI is generated by the IIO core, this just comes up with corner cases of the ABI.

Jonathan

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux