[PATCH] MIPS: remove meaningless generic-(CONFIG_GENERIC_CSUM) += checksum.h

[PATCH] MIPS: remove meaningless generic-(CONFIG_GENERIC_CSUM) += checksum.h

[Masahiro Yamada]

This line is weird in multiple ways.

(CONFIG_GENERIC_CSUM) might be a typo of $(CONFIG_GENERIC_CSUM).

Even if you add '$' to it, $(CONFIG_GENERIC_CSUM) is never evaluated
to 'y' because scripts/Makefile.asm-generic does not include
include/config/auto.conf. So, the asm-generic wrapper of checksum.h
is never generated.

Even if you manage to generate it, it is never included by anyone
because MIPS has the checkin header with the same file name:

  arch/mips/include/asm/checksum.h

As you see in the top Makefile, the checkin headers are included before
generated ones.

  LINUXINCLUDE    := \
                  -I$(srctree)/arch/$(SRCARCH)/include \
                  -I$(objtree)/arch/$(SRCARCH)/include/generated \
                  ...

Commit 4e0748f5beb9 ("MIPS: Use generic checksum functions for MIPS R6")
already added the asm-generic fallback code in the checkin header:

  #ifdef CONFIG_GENERIC_CSUM
  #include <asm/generic/checksum.h>
  #else
    ...
  #endif

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 arch/mips/include/asm/Kbuild | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/mips/include/asm/Kbuild b/arch/mips/include/asm/Kbuild
index f15d5db..87b86cd 100644
--- a/arch/mips/include/asm/Kbuild
+++ b/arch/mips/include/asm/Kbuild
@@ -3,7 +3,6 @@ generated-y += syscall_table_32_o32.h
 generated-y += syscall_table_64_n32.h
 generated-y += syscall_table_64_n64.h
 generated-y += syscall_table_64_o32.h
-generic-(CONFIG_GENERIC_CSUM) += checksum.h
 generic-y += current.h
 generic-y += device.h
 generic-y += dma-contiguous.h
-- 
2.7.4

Re: [PATCH] MIPS: remove meaningless generic-(CONFIG_GENERIC_CSUM) += checksum.h

[Paul Burton]

Hi Masahiro,

On Mon, Jan 21, 2019 at 11:48:49AM +0900, Masahiro Yamada wrote:
> This line is weird in multiple ways.
> 
> (CONFIG_GENERIC_CSUM) might be a typo of $(CONFIG_GENERIC_CSUM).
> 
> Even if you add '$' to it, $(CONFIG_GENERIC_CSUM) is never evaluated
> to 'y' because scripts/Makefile.asm-generic does not include
> include/config/auto.conf. So, the asm-generic wrapper of checksum.h
> is never generated.
> 
> Even if you manage to generate it, it is never included by anyone
> because MIPS has the checkin header with the same file name:
> 
>   arch/mips/include/asm/checksum.h
> 
> As you see in the top Makefile, the checkin headers are included before
> generated ones.
> 
>   LINUXINCLUDE    := \
>                   -I$(srctree)/arch/$(SRCARCH)/include \
>                   -I$(objtree)/arch/$(SRCARCH)/include/generated \
>                   ...
> 
> Commit 4e0748f5beb9 ("MIPS: Use generic checksum functions for MIPS R6")
> already added the asm-generic fallback code in the checkin header:
> 
>   #ifdef CONFIG_GENERIC_CSUM
>   #include <asm/generic/checksum.h>
>   #else
>     ...
>   #endif
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

Good catch. Would you prefer to take this through your kbuild tree or
that I take it through the MIPS tree?

If the former:

    Acked-by: Paul Burton <paul.burton@mips.com>

Thanks,
    Paul

Re: [PATCH] MIPS: remove meaningless generic-(CONFIG_GENERIC_CSUM) += checksum.h

[Masahiro Yamada]

Hi Paul,

On Tue, Jan 22, 2019 at 2:44 AM Paul Burton <paul.burton@mips.com> wrote:
>
> Hi Masahiro,
>
> On Mon, Jan 21, 2019 at 11:48:49AM +0900, Masahiro Yamada wrote:
> > This line is weird in multiple ways.
> >
> > (CONFIG_GENERIC_CSUM) might be a typo of $(CONFIG_GENERIC_CSUM).
> >
> > Even if you add '$' to it, $(CONFIG_GENERIC_CSUM) is never evaluated
> > to 'y' because scripts/Makefile.asm-generic does not include
> > include/config/auto.conf. So, the asm-generic wrapper of checksum.h
> > is never generated.
> >
> > Even if you manage to generate it, it is never included by anyone
> > because MIPS has the checkin header with the same file name:
> >
> >   arch/mips/include/asm/checksum.h
> >
> > As you see in the top Makefile, the checkin headers are included before
> > generated ones.
> >
> >   LINUXINCLUDE    := \
> >                   -I$(srctree)/arch/$(SRCARCH)/include \
> >                   -I$(objtree)/arch/$(SRCARCH)/include/generated \
> >                   ...
> >
> > Commit 4e0748f5beb9 ("MIPS: Use generic checksum functions for MIPS R6")
> > already added the asm-generic fallback code in the checkin header:
> >
> >   #ifdef CONFIG_GENERIC_CSUM
> >   #include <asm/generic/checksum.h>
> >   #else
> >     ...
> >   #endif
> >
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>
> Good catch. Would you prefer to take this through your kbuild tree or
> that I take it through the MIPS tree?


Could you apply it to MIPS tree? Thanks.




-- 
Best Regards
Masahiro Yamada

Re: [PATCH] MIPS: remove meaningless generic-(CONFIG_GENERIC_CSUM) += checksum.h

[Paul Burton]

Hello,

Masahiro Yamada wrote:
> This line is weird in multiple ways.
> 
> (CONFIG_GENERIC_CSUM) might be a typo of $(CONFIG_GENERIC_CSUM).
> 
> Even if you add '$' to it, $(CONFIG_GENERIC_CSUM) is never evaluated
> to 'y' because scripts/Makefile.asm-generic does not include
> include/config/auto.conf. So, the asm-generic wrapper of checksum.h
> is never generated.
> 
> Even if you manage to generate it, it is never included by anyone
> because MIPS has the checkin header with the same file name:
> 
> arch/mips/include/asm/checksum.h
> 
> As you see in the top Makefile, the checkin headers are included before
> generated ones.
> 
> LINUXINCLUDE    :=                   -I$(srctree)/arch/$(SRCARCH)/include                   -I$(objtree)/arch/$(SRCARCH)/include/generated                   ...
> 
> Commit 4e0748f5beb9 ("MIPS: Use generic checksum functions for MIPS R6")
> already added the asm-generic fallback code in the checkin header:
> 
> #ifdef CONFIG_GENERIC_CSUM
> #include <asm/generic/checksum.h>
> #else
> ...
> #endif
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

Applied to mips-next.

Thanks,
    Paul

[ This message was auto-generated; if you believe anything is incorrect
  then please email paul.burton@mips.com to report it. ]