Hi Marc, On Fri, Oct 28, 2022 at 12:29 PM Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> wrote: > On 28.10.2022 12:06:45, Geert Uytterhoeven wrote: > > When introducing support for R-Car V3U, which has 8 instead of 2 > > channels, the ECC error bitmask was extended to take into account the > > extra channels, but rcar_canfd_global_error() was not updated to act > > upon the extra bits. > > > > Replace the RCANFD_GERFL_EEF[01] macros by a new macro that takes the > > channel number, fixing R-Car V3U while simplifying the code. > > > > Fixes: 45721c406dcf50d4 ("can: rcar_canfd: Add support for r8a779a0 SoC") > > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > Is this stable material? Upstream DTS[1] has only the first two channels enabled, so it's not critical. But it never hurts to end up in stable, helping e.g. CiP. [1] arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds