From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chin Liang See Date: Mon, 7 Dec 2015 20:08:54 +0800 Subject: [U-Boot] [PATCH 4/6] arm: socfpga: Switch CONFIG_HOSTNAME to CONFIG_SYS_BOARD In-Reply-To: <201512071301.17733.marex@denx.de> References: <1449348262-18488-1-git-send-email-marex@denx.de> <201512070936.53316.marex@denx.de> <1449482484.2061.4.camel@altera.com> <201512071301.17733.marex@denx.de> Message-ID: <1449490134.1803.0.camel@altera.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > > > > > Cc: Chin Liang See > > > > > Cc: Dinh Nguyen > > > > > --- > > > > > > > > > > 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 Thanks Chin Liang