[U-Boot] [PATCH 4/6] arm: socfpga: Switch CONFIG_HOSTNAME to CONFIG_SYS_BOARD

Chin Liang See clsee at altera.com
Mon Dec 7 13:08:54 CET 2015


On Mon, 2015-12-07 at 13:01 +0100, Marek Vasut wrote:
> On Monday, December 07, 2015 at 11:01:24 AM, Chin Liang See wrote:
> > On Mon, 2015-12-07 at 09:36 +0100, Marek Vasut wrote:
> > > On Monday, December 07, 2015 at 07:25:31 AM, Chin Liang See
> > > wrote:
> > > > On Sat, 2015-12-05 at 21:44 +0100, Marek Vasut wrote:
> > > > > We already have the CONFIG_SYS_BOARD variable, which defines
> > > > > the
> > > > > name
> > > > > of the board. The value in CONFIG_HOSTNAME is exactly the
> > > > > same
> > > > > and is
> > > > > thus just a duplicity, so switch it to reuse CONFIG_SYS_BOARD
> > > > > .
> > > > > 
> > > > > Signed-off-by: Marek Vasut <marex at denx.de>
> > > > > Cc: Chin Liang See <clsee at altera.com>
> > > > > Cc: Dinh Nguyen <dinguyen at opensource.altera.com>
> > > > > ---
> > > > > 
> > > > >  include/configs/socfpga_arria5_socdk.h   | 2 --
> > > > >  include/configs/socfpga_common.h         | 4 ++++
> > > > >  include/configs/socfpga_cyclone5_socdk.h | 2 --
> > > > >  include/configs/socfpga_de0_nano_soc.h   | 2 --
> > > > >  include/configs/socfpga_mcvevk.h         | 2 --
> > > > >  include/configs/socfpga_sockit.h         | 2 --
> > > > >  include/configs/socfpga_socrates.h       | 2 --
> > > > >  include/configs/socfpga_sr1500.h         | 3 ---
> > > > >  8 files changed, 4 insertions(+), 15 deletions(-)
> > > > > 
> > > > > diff --git a/include/configs/socfpga_arria5_socdk.h
> > > > > b/include/configs/socfpga_arria5_socdk.h
> > > > > index a036856..3d5665d 100644
> > > > > --- a/include/configs/socfpga_arria5_socdk.h
> > > > > +++ b/include/configs/socfpga_arria5_socdk.h
> > > > > @@ -56,8 +56,6 @@
> > > > > 
> > > > >  #define CONFIG_ENV_OFFSET		512	/* just
> > > > > 
> > > > > after
> > > > > 
> > > > > the MBR */
> > > > > 
> > > > >  /* Extra Environment */
> > > > > 
> > > > > -#define CONFIG_HOSTNAME		socfpga_arria5
> > > > > -
> > > > > 
> > > > >  #define CONFIG_EXTRA_ENV_SETTINGS \
> > > > >  
> > > > >  	"verify=n\0" \
> > > > >  	"loadaddr= " __stringify(CONFIG_SYS_LOAD_ADDR) "\0"
> > > > > \
> > > > > 
> > > > > diff --git a/include/configs/socfpga_common.h
> > > > > b/include/configs/socfpga_common.h
> > > > > index 4b2d246..f74c758 100644
> > > > > --- a/include/configs/socfpga_common.h
> > > > > +++ b/include/configs/socfpga_common.h
> > > > > @@ -69,6 +69,10 @@
> > > > > 
> > > > >  #define CONFIG_CMDLINE_EDITING			/*
> > > > > Command
> > > > > 
> > > > > history etc */
> > > > > 
> > > > >  #define CONFIG_SYS_HUSH_PARSER
> > > > > 
> > > > > +#ifndef CONFIG_SYS_HOSTNAME
> > > > > +#define CONFIG_SYS_HOSTNAME	CONFIG_SYS_BOARD
> > > > > +#endif
> > > > > +
> > > > 
> > > > I saw the change of hostname for socdk would be
> > > > "socfpga_cyclone5"
> > > > to
> > > > "cyclone5-socdk" and same for other board. Wonder any impact on
> > > > this
> > > > change?
> > > 
> > > It should have no impact, unless you're using the hostname in
> > > your
> > > scripts
> > > for something. You would have no way to tell apart the cyclone5
> > > boards with
> > > the old hostname though, since socfpga_cyclone5 is too generic.
> > > In
> > > case this
> > > breaks some script of yours, setenv hostname socfpga_cyclone5
> > > would
> > > fix it.
> > 
> > I grep previously and its not used. But will take note if any side
> > effect noticed.
> 
> OK, thanks


Acked-by: Chin Liang See <clsee at altera.com>

Thanks
Chin Liang


More information about the U-Boot mailing list