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

[Marek Vasut]

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