From: Yuval Mintz <yuval.mi...@qlogic.com>
Date: Fri, 25 Sep 2015 05:43:06 +0000
>> > +struct qede_rx_queue {
>> > + __le16 *hw_cons_ptr;
>>
>> The __ variants of constants should be reserved for use in user visible API's
>
> Really? If so, this needs to be fixed not only here but in lots of places
> in the series [e.g., entire HW HSI uses __le variants instead of le].
> But why is it so? I.e., I understand that __le16 is defined in the uapi
> directory and
> thus accessible to users, but why the distinction?
Because it shows whether the type is something exposed to userspace or not.
If there are places where this is done incorrectly in the tree, it is not
a legitimate reason for you to do so as well.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
