Re: [PATCH v2 1/4] i2c: pmcmsp: Use proper printk format for resource_size_t

From: Krzysztof Kozlowski
Date: Wed Jan 15 2020 - 14:52:03 EST


On Wed, Jan 15, 2020 at 06:45:53PM +0100, Wolfram Sang wrote:
>
> > + "Unable to get memory/io address region %pap\n",
> > + &(res->start));
>
> My first thought was "parens not needed"; not because I like being picky
> but because it doesn't look more readable to me.
>
> checkpatch agrees:
>
> CHECK: Unnecessary parentheses around res->start
> #30: FILE: drivers/i2c/busses/i2c-pmcmsp.c:278:
> + &(res->start));

Indeed... let me send v3.

Best regards,
Krzysztof