Re: [PATCH] LoongArch: Fix build warnings about export.h
From: Huacai Chen
Date: Thu Jun 19 2025 - 08:29:20 EST
Hi, Eric,
On Tue, Jun 17, 2025 at 2:57 AM Eric Biggers <ebiggers@xxxxxxxxxx> wrote:
>
> On Sun, Jun 08, 2025 at 10:15:09PM +0800, Huacai Chen wrote:
> > diff --git a/arch/loongarch/lib/crc32-loongarch.c b/arch/loongarch/lib/crc32-loongarch.c
> > index b37cd8537b45..db22c2ec55e2 100644
> > --- a/arch/loongarch/lib/crc32-loongarch.c
> > +++ b/arch/loongarch/lib/crc32-loongarch.c
> > @@ -11,6 +11,7 @@
> >
> > #include <asm/cpu-features.h>
> > #include <linux/crc32.h>
> > +#include <linux/export.h>
> > #include <linux/module.h>
> > #include <linux/unaligned.h>
>
> You can drop the change to crc32-loongarch.c, as it would conflict with
> https://lore.kernel.org/r/20250601224441.778374-8-ebiggers@xxxxxxxxxx/ and
> https://lore.kernel.org/r/20250612183852.114878-1-ebiggers@xxxxxxxxxx/ which are
> in crc-next for 6.17 and already handle this issue.
Thank you for your suggestion. But I plan to merge this patch for
6.16, while your commits are for 6.17, I can only keep it as is.
However, the conflicts can be easily solved when Linus merges your
code, or there is an alternative solution (if you want): update your
patches when this one gets upstream.
Huacai
>
> - Eric