On Fri, Dec 29, 2017 at 9:46 PM, Ingo Molnar <mi...@kernel.org> wrote: > * Andy Shevchenko <andy.shevche...@gmail.com> wrote:
>> > - ireg.cx = sizeof buf; >> > + ireg.cx = sizeof(buf); >> >> sizeof is operator, not a function. >> So, what are you trying to achieve? > > That's true, but in the kernel coding style we use it as a function: Yeah, the question is do we need to change all those 177 occurrences at all? Or perhaps it might be an opportunity during some other changes to certain users... -- With Best Regards, Andy Shevchenko
