[PATCH 3/7] Kconfig: Escape variables to make in default strings

[Tom Rini]

On Wed, Feb 26, 2020 at 11:35:18AM +0900, Masahiro Yamada wrote:
> On Tue, Feb 25, 2020 at 3:05 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > We have some variables that need to include a variable to pass to make
> > to evaluate later, typically ARCH and BOARDDIR, to find a file to use.
> > The way we're doing this today isn't correct but works.  With an update
> > to Kconfig we will need to escape these properly, so do so.
> >
> > Cc: Masahiro Yamada <masahiroy at kernel.org>
> > Signed-off-by: Tom Rini <trini at konsulko.com>
> > ---
> >  arch/arm/mach-davinci/Kconfig | 2 +-
> >  arch/arm/mach-orion5x/Kconfig | 2 +-
> >  common/spl/Kconfig            | 4 ++--
> >  3 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/mach-davinci/Kconfig b/arch/arm/mach-davinci/Kconfig
> > index 8a81c078811d..83f749c0440a 100644
> > --- a/arch/arm/mach-davinci/Kconfig
> > +++ b/arch/arm/mach-davinci/Kconfig
> > @@ -135,6 +135,6 @@ source "board/davinci/da8xxevm/Kconfig"
> >  source "board/lego/ev3/Kconfig"
> >
> >  config SPL_LDSCRIPT
> > -       default "board/$(BOARDDIR)/u-boot-spl-da850evm.lds"
> > +       default "board/\$(BOARDDIR)/u-boot-spl-da850evm.lds"
> 
> 
> Only one file path for this.
> 
> $ find  . -name  u-boot-spl-da850evm.lds
> ./board/davinci/da8xxevm/u-boot-spl-da850evm.lds
> 
> 
> I prefer hard-coding.
> 
> default "board/davinci/da8xxevm/u-boot-spl-da850evm.lds"

Done, new patch.

> >  endif
> > diff --git a/arch/arm/mach-orion5x/Kconfig b/arch/arm/mach-orion5x/Kconfig
> > index 2984a3edda21..c4bc524eb81e 100644
> > --- a/arch/arm/mach-orion5x/Kconfig
> > +++ b/arch/arm/mach-orion5x/Kconfig
> > @@ -16,6 +16,6 @@ config SYS_SOC
> >  source "board/LaCie/edminiv2/Kconfig"
> >
> >  config SPL_LDSCRIPT
> > -       default "$(CPUDIR)/orion5x/u-boot-spl.lds" if ORION5X
> > +       default "\$(CPUDIR)/orion5x/u-boot-spl.lds" if ORION5X
> 
> Ditto.
> 
> We know the directory path of orion5x.
> 
> $ find  . -name orion5x
> ./arch/arm/cpu/arm926ejs/orion5x
> 
> You can hard-code it.

Done, new patch.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200226/08078bcb/attachment.sig>