Re: [PATCH v2] MIPS: Makefile: Remove "ifdef need-compiler" for Kbuild.platforms

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Dec 11, 2021 at 1:22 AM Nathan Chancellor <nathan@xxxxxxxxxx> wrote:
>
> On Fri, Dec 10, 2021 at 12:05:05PM +0800, Tiezhu Yang wrote:
> > After commit 13ceb48bc19c ("MIPS: Loongson2ef: Remove unnecessary
> > {as,cc}-option calls"), no need to use "ifdef need-compiler" for
> > Kbuild.platforms, because the cause of the build issue mentioned
> > in commit 0706f74f719e ("MIPS: fix *-pkg builds for loongson2ef
> > platform") has been disappeared, so just remove it.
> >
> > Signed-off-by: Tiezhu Yang <yangtiezhu@xxxxxxxxxxx>
>
> Reviewed-by: Nathan Chancellor <nathan@xxxxxxxxxx>
>
> > ---
> >
> > v2: fix the typo "casue" --> "cause" in the commit message,
> >     sorry for that.
> >
> >  arch/mips/Makefile | 2 --
> >  1 file changed, 2 deletions(-)
> >
> > diff --git a/arch/mips/Makefile b/arch/mips/Makefile
> > index ace7f03..e036fc0 100644
> > --- a/arch/mips/Makefile
> > +++ b/arch/mips/Makefile
> > @@ -253,9 +253,7 @@ endif
> >  #
> >  # Board-dependent options and extra files
> >  #
> > -ifdef need-compiler
> >  include $(srctree)/arch/mips/Kbuild.platforms
> > -endif
> >
> >  ifdef CONFIG_PHYSICAL_START
> >  load-y                                       = $(CONFIG_PHYSICAL_START)
> > --
> > 2.1.0
> >


Nathan's patch cleaned up  arch/mips/loognson2ef/Platform,
but I still see similar code in arch/mips/sgi-ip22/Platform.


ifdef CONFIG_SGI_IP28
  ifeq ($(call cc-option-yn,-march=r10000 -mr10k-cache-barrier=store), n)
      $(error gcc doesn't support needed option -mr10k-cache-barrier=store)
  endif
endif


Doesn't this cause a failure of *-pkg builds for sgi-ip22 platform?






-- 
Best Regards
Masahiro Yamada



[Index of Archives]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux