On Fri, Jul 14, 2017 at 10:37 PM, Arnd Bergmann [off-list ref] wrote:

On Fri, Jul 14, 2017 at 9:18 PM, Andy Shevchenko
[off-list ref] wrote:

quoted

On Fri, Jul 14, 2017 at 3:07 PM, Arnd Bergmann [off-list ref] wrote:

quoted

gcc points out a possible format string overflow for a large value of 'zone':

quoted

Here we need to convert

int i;

to

u8 i;

That was my first impulse, but then I decided not to change the
idiomatic 'int i' for the index variable to 'u8' as that would be
less idiomatic.

quoted

I will take it after addressing above.

P.S. You may do this change across the file.

How about changing it to 'u8 zone'?

I'm ultimately fine with that (just gentle reminder you might fix all
3 occurrences of it in that driver).

-- 
With Best Regards,
Andy Shevchenko