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

From: Jonathan Cameron
Date: Wed Sep 13 2017 - 19:17:06 EST




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.