Re: [PATCH] wifi: rsi: Fix types to appease CFI

On Sun, 2026-08-02 at 20:40 +0200, Stefan Hansson wrote:
> Hello Johannes,
>=20
> On 02/08/2026 18:24, Johannes Berg wrote:
> > On Sun, 2026-08-02 at 14:22 +0200, Stefan Hansson wrote:
> > > Avoids errors like:
> > >=20
> > >      CFI failure at kthread+0x124/0x1cc (target: rsi_coex_scheduler_t=
hread+0x0/0x1b4 [redpine_91x]; expected type: 0x89fb613d)
> >=20
> > Yeah, well. Everyone gets _one_ warning to send patches that actually
> > build warning-free, after that I just drop them without notice...
>=20
> Could you clarify what you are referring to here? I don't get any errors=
=20
> when building after applying this patch on top of next-20260730:

Alright, so it does build, sorry. I was just looking at nipa:
https://patchwork.kernel.org/project/linux-wireless/patch/20260802-rsi-cfi-=
[email protected]/

But I see now it's just duplicating the kernel-doc issues across all the
builds ... I wasn't paying attention to _what_ it was reporting and the
kthread_complete_and_exit() use in the functions and thought it was
likely getting "control reaches end of non-void function" warnings since
you change functions to int without ever returning anything.

> If you are referring to checkpatch.pl complaining about the=20
> aforementioned error line being too long, I can wrap it if you prefer tha=
t.

Nah, that's fine, no worries.

johannes