On Thu, Mar 16, 2023 at 12:20:48PM +0200, Mika Westerberg wrote:
+Cc MarioApplied to thunderbolt.git/next (along with the two commits from Mario).
On Wed, Mar 15, 2023 at 06:04:50PM -0400, Tom Rix wrote:
cppcheck reportsThanks for the patch! Yeah, this did not show up in any of the kbuild
drivers/thunderbolt/nhi.c:74:7: style: Local variable 'bit' shadows outer variable [shadowVariable]
int bit;
^
drivers/thunderbolt/nhi.c:66:6: note: Shadowed declaration
int bit = ring_interrupt_index(ring) & 31;
^
drivers/thunderbolt/nhi.c:74:7: note: Shadow variable
int bit;
^
For readablity rename the outer to interrupt_bit and the innner
to auto_clear_bit.
tests perhaps they are missing cppcheck :(
I'm thinking that I'll just move the two commits from "fixes" to "next"
and add this one on top (and drop the stable tags) as the code that we
should be sending to stable should not need additional fixes IMHO. I
know Mario is on vacation so probably cannot answer here so let's deal
with this when he is back.