Re: [LEDE-DEV] [PATCH] ar71xx: uniform GL iNet products mach name

2018-01-18 Thread Piotr Dymacz

Hello Kyson,


On 19.01.2018 02:54, Kyson Lok wrote:

The mach name of GL AR150 and GL AR300 is inconsistent
with other products.


Thank you for your patch.
Please, include also vendor name (GL.iNet) in the machine name.

--
Cheers,
Piotr



Signed-off-by: Kyson Lok 
---
  target/linux/ar71xx/base-files/lib/ar71xx.sh  | 4 ++--
  target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c | 2 +-
  target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c | 2 +-
  3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/target/linux/ar71xx/base-files/lib/ar71xx.sh 
b/target/linux/ar71xx/base-files/lib/ar71xx.sh
index b664249..7d414ed 100755
--- a/target/linux/ar71xx/base-files/lib/ar71xx.sh
+++ b/target/linux/ar71xx/base-files/lib/ar71xx.sh
@@ -664,10 +664,10 @@ ar71xx_board_detect() {
*"FRITZ!WLAN Repeater 300E")
name="fritz300e"
;;
-   *"GL AR150")
+   *"GL-AR150")
name="gl-ar150"
;;
-   *"GL AR300")
+   *"GL-AR300")
name="gl-ar300"
;;
*"GL-AR300M")
diff --git a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c 
b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
index df52784..9824bb8 100644
--- a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
+++ b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
@@ -122,4 +122,4 @@ static void __init gl_ar150_setup(void)
ath79_register_wmac(art + GL_AR150_CALDATA_OFFSET, art + 
GL_AR150_WMAC_MAC_OFFSET);
  }
  
-MIPS_MACHINE(ATH79_MACH_GL_AR150, "GL-AR150", "GL AR150",gl_ar150_setup);

+MIPS_MACHINE(ATH79_MACH_GL_AR150, "GL-AR150", "GL-AR150",gl_ar150_setup);
diff --git a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c 
b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
index 6f01b9e..ae96635 100644
--- a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
+++ b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
@@ -100,4 +100,4 @@ static void __init gl_ar300_setup(void)
ath79_register_wmac(art + GL_AR300_CALDATA_OFFSET, art + 
GL_AR300_WMAC_MAC_OFFSET);
  }
  
-MIPS_MACHINE(ATH79_MACH_GL_AR300, "GL-AR300", "GL AR300",gl_ar300_setup);

+MIPS_MACHINE(ATH79_MACH_GL_AR300, "GL-AR300", "GL-AR300",gl_ar300_setup);




___
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev


[LEDE-DEV] [PATCH] ar71xx: uniform GL iNet products mach name

2018-01-18 Thread Kyson Lok
The mach name of GL AR150 and GL AR300 is inconsistent
with other products.

Signed-off-by: Kyson Lok 
---
 target/linux/ar71xx/base-files/lib/ar71xx.sh  | 4 ++--
 target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c | 2 +-
 target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/target/linux/ar71xx/base-files/lib/ar71xx.sh 
b/target/linux/ar71xx/base-files/lib/ar71xx.sh
index bf36598..78e2e5c 100755
--- a/target/linux/ar71xx/base-files/lib/ar71xx.sh
+++ b/target/linux/ar71xx/base-files/lib/ar71xx.sh
@@ -620,10 +620,10 @@ ar71xx_board_detect() {
name="gl-inet"
gl_inet_board_detect
;;
-   *"GL AR150")
+   *"GL-AR150")
name="gl-ar150"
;;
-   *"GL AR300")
+   *"GL-AR300")
name="gl-ar300"
;;
*"GL-AR300M")
diff --git a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c 
b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
index e0d1ea4..30992c8 100644
--- a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
+++ b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
@@ -122,4 +122,4 @@ static void __init gl_ar150_setup(void)
ath79_register_wmac(art + GL_AR150_CALDATA_OFFSET, art + 
GL_AR150_WMAC_MAC_OFFSET);
 }
 
-MIPS_MACHINE(ATH79_MACH_GL_AR150, "GL-AR150", "GL AR150",gl_ar150_setup);
+MIPS_MACHINE(ATH79_MACH_GL_AR150, "GL-AR150", "GL-AR150",gl_ar150_setup);
diff --git a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c 
b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
index 6f01b9e..ae96635 100644
--- a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
+++ b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
@@ -100,4 +100,4 @@ static void __init gl_ar300_setup(void)
ath79_register_wmac(art + GL_AR300_CALDATA_OFFSET, art + 
GL_AR300_WMAC_MAC_OFFSET);
 }
 
-MIPS_MACHINE(ATH79_MACH_GL_AR300, "GL-AR300", "GL AR300",gl_ar300_setup);
+MIPS_MACHINE(ATH79_MACH_GL_AR300, "GL-AR300", "GL-AR300",gl_ar300_setup);
-- 
2.7.4


___
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev


[LEDE-DEV] [PATCH] ar71xx: uniform GL iNet products mach name

2018-01-18 Thread Kyson Lok
The mach name of GL AR150 and GL AR300 is inconsistent
with other products.

Signed-off-by: Kyson Lok 
---
 target/linux/ar71xx/base-files/lib/ar71xx.sh  | 4 ++--
 target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c | 2 +-
 target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/target/linux/ar71xx/base-files/lib/ar71xx.sh 
b/target/linux/ar71xx/base-files/lib/ar71xx.sh
index b664249..7d414ed 100755
--- a/target/linux/ar71xx/base-files/lib/ar71xx.sh
+++ b/target/linux/ar71xx/base-files/lib/ar71xx.sh
@@ -664,10 +664,10 @@ ar71xx_board_detect() {
*"FRITZ!WLAN Repeater 300E")
name="fritz300e"
;;
-   *"GL AR150")
+   *"GL-AR150")
name="gl-ar150"
;;
-   *"GL AR300")
+   *"GL-AR300")
name="gl-ar300"
;;
*"GL-AR300M")
diff --git a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c 
b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
index df52784..9824bb8 100644
--- a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
+++ b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar150.c
@@ -122,4 +122,4 @@ static void __init gl_ar150_setup(void)
ath79_register_wmac(art + GL_AR150_CALDATA_OFFSET, art + 
GL_AR150_WMAC_MAC_OFFSET);
 }
 
-MIPS_MACHINE(ATH79_MACH_GL_AR150, "GL-AR150", "GL AR150",gl_ar150_setup);
+MIPS_MACHINE(ATH79_MACH_GL_AR150, "GL-AR150", "GL-AR150",gl_ar150_setup);
diff --git a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c 
b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
index 6f01b9e..ae96635 100644
--- a/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
+++ b/target/linux/ar71xx/files/arch/mips/ath79/mach-gl-ar300.c
@@ -100,4 +100,4 @@ static void __init gl_ar300_setup(void)
ath79_register_wmac(art + GL_AR300_CALDATA_OFFSET, art + 
GL_AR300_WMAC_MAC_OFFSET);
 }
 
-MIPS_MACHINE(ATH79_MACH_GL_AR300, "GL-AR300", "GL AR300",gl_ar300_setup);
+MIPS_MACHINE(ATH79_MACH_GL_AR300, "GL-AR300", "GL-AR300",gl_ar300_setup);
-- 
2.7.4


___
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev