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
