Re: [U-Boot] [PATCH] cm_fx6: Remove CONFIG_NETMASK

2014-09-22 Thread Stefano Babic
On 21/09/2014 01:05, Fabio Estevam wrote:
 From: Fabio Estevam fabio.este...@freescale.com
 
 We should not hardcode CONFIG_NETMASK in the config file.
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] cm_fx6: Remove CONFIG_NETMASK

2014-09-21 Thread Igor Grinberg
On 09/21/14 02:05, Fabio Estevam wrote:
 From: Fabio Estevam fabio.este...@freescale.com
 
 We should not hardcode CONFIG_NETMASK in the config file.

That is correct no doubts.

 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com

Cc'd Nikita,

Acked-by: Igor Grinberg grinb...@compulab.co.il

 ---
  include/configs/cm_fx6.h | 1 -
  1 file changed, 1 deletion(-)
 
 diff --git a/include/configs/cm_fx6.h b/include/configs/cm_fx6.h
 index 10d02b4..7cf241e 100644
 --- a/include/configs/cm_fx6.h
 +++ b/include/configs/cm_fx6.h
 @@ -222,7 +222,6 @@
  #define CONFIG_MII
  #define CONFIG_ETHPRIME  FEC0
  #define CONFIG_ARP_TIMEOUT   200UL
 -#define CONFIG_NETMASK   255.255.255.0
  #define CONFIG_NET_RETRY_COUNT   5
  
  /* USB */
 

-- 
Regards,
Igor.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] cm_fx6: Remove CONFIG_NETMASK

2014-09-21 Thread Nikita Kiryanov

Hi Fabio,

On 21/09/14 02:05, Fabio Estevam wrote:

From: Fabio Estevam fabio.este...@freescale.com

We should not hardcode CONFIG_NETMASK in the config file.

Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---


Acked-by: Nikita Kiryanov nik...@compulab.co.il

--
Regards,
Nikita Kiryanov
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] cm_fx6: Remove CONFIG_NETMASK

2014-09-20 Thread Fabio Estevam
From: Fabio Estevam fabio.este...@freescale.com

We should not hardcode CONFIG_NETMASK in the config file.

Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---
 include/configs/cm_fx6.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/include/configs/cm_fx6.h b/include/configs/cm_fx6.h
index 10d02b4..7cf241e 100644
--- a/include/configs/cm_fx6.h
+++ b/include/configs/cm_fx6.h
@@ -222,7 +222,6 @@
 #define CONFIG_MII
 #define CONFIG_ETHPRIMEFEC0
 #define CONFIG_ARP_TIMEOUT 200UL
-#define CONFIG_NETMASK 255.255.255.0
 #define CONFIG_NET_RETRY_COUNT 5
 
 /* USB */
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot