Re: [GIT PULL] x86/platform updates for v5.10
From: Linus Torvalds
Date: Mon Oct 12 2020 - 17:46:31 EST
On Mon, Oct 12, 2020 at 2:42 PM Mike Travis <mike.travis@xxxxxxx> wrote:
>
> It should have been an unsigned long instead of an int as Linus
> suggested. I'm not sure it's a write only variable as I think the mask
> is used to check if the interrupt occurred (I'll have to look closer).
At least "git grep" only shows two assignments to it.
Of course, that would miss any cases that play games with preprocessor
token pasting etc, so it's not entirely meaningful, but it's certainly
a hint..
And yes, I expect that the fix is to just make it "unsigned long", but
if it truly isn't actually used, maybe removal is better.
Linus