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

[Marek Vasut <marex@denx.de>]

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@denx.de>
> > Cc: Chin Liang See <clsee@altera.com>
> > Cc: Dinh Nguyen <dinguyen@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.

Best regards,
Marek Vasut