On 8 October 2014 20:06, Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> wrote: > On 8 October 2014 20:00, Christopher Li <sparse@xxxxxxxxxxx> wrote: >> On Mon, Oct 6, 2014 at 10:44 PM, Ard Biesheuvel >> <ard.biesheuvel@xxxxxxxxxx> wrote: >>>> we only address the case where the function pointer is the target of >>>> an assignment. >> >> I apply the patch in my private branch but haven't push out yet. >> >> I just want to confirm, you didn't find this kind of assignment in the >> kernel, right? >> > > I noticed this patch > > https://git.kernel.org/cgit/linux/kernel/git/kvmarm/kvmarm.git/commit/?h=queue&id=de56fb1923ca11f428bf557870e0faa99f38762e > > and saw that it was bogus: changing the return type of the pointed-to > function to 'const int' makes no sense at all, and yet it shuts up > sparse. > > With my sparse patch, that patch could have been dropped (although I > think it was merged after all) > Ping? -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html