[OpenWrt-Devel] [PATCH v4] ramips: Add support for Netgear EX2700

Joseph C. Lehner joseph.c.lehner at gmail.com
Tue Feb 16 10:00:08 EST 2016


I've received a notification from patchwork that the patch status has
been changed to "Accepted" 9 days ago, but it hasn't been merged yet.

Since I have another patch depending on v4, should I submit v5, or wait
until v4 has been merged and then submit a new patch?

Regards
Joseph

On 2016-02-08 22:29, Joseph C. Lehner wrote:
> This patch adds support for the Netgear EX2700 and builds an approriate
> sysupgrade image.
> 
> What's missing is the option to build a factory image flashable via the
> router's stock web interface, but this approach is hindered by the fact
> that u-boot operforms an additional integrity check, which expects a
> uImage header in the last 64 bytes of the "kernel" partition, which
> the bootloader expects to be 960k, a size exceeded by the standard
> OpenWrt kernel.
> 
> Signed-off-by: Joseph C. Lehner <joseph.c.lehner at gmail.com>
> 
> ---
> Changes for v4:
> * Fix whitespace errors in netgear.mk
> 
> Changes for v3:
> * Fix led name in /etc/board.d/01_leds
> * Cleanup device tree
> 
> Changes for v2:
> * Fix alphabetical ordering (image/Makefile is not really sorted, so I've
>   put the EX2700 immediately below the E1700).
> 
> diff --git a/target/linux/ramips/base-files/etc/board.d/01_leds b/target/linux/ramips/base-files/etc/board.d/01_leds
> index c3a81a4..b10f5f1 100755
> --- a/target/linux/ramips/base-files/etc/board.d/01_leds
> +++ b/target/linux/ramips/base-files/etc/board.d/01_leds
> @@ -125,6 +125,10 @@ dir-810l|\
>  mzk-dp150n)
>  	ucidef_set_led_default "power" "power" "$board:green:power" "1"
>  	;;
> +ex2700)
> +	ucidef_set_led_default "power_r" "POWER (red)" "$board:red:power" "0"
> +	set_wifi_led "$board:green:router"
> +	;;
>  f5d8235-v1|\
>  f5d8235-v2)
>  	set_usb_led "$board:blue:storage"
> diff --git a/target/linux/ramips/base-files/etc/board.d/02_network b/target/linux/ramips/base-files/etc/board.d/02_network
> index dafa3e9..9853729 100755
> --- a/target/linux/ramips/base-files/etc/board.d/02_network
> +++ b/target/linux/ramips/base-files/etc/board.d/02_network
> @@ -141,7 +141,8 @@ ramips_setup_interfaces()
>  		ucidef_add_switch "switch0" \
>  			"1:lan" "2:lan" "3:lan" "4:lan" "0:wan" "9 at eth0"
>  		;;
> -	cf-wr800n)
> +	cf-wr800n|\
> +	ex2700)
>  		ucidef_add_switch "switch0" \
>  			"4:lan" "6t at eth0"
>  		;;
> diff --git a/target/linux/ramips/base-files/etc/diag.sh b/target/linux/ramips/base-files/etc/diag.sh
> index acc500b..59c5965 100644
> --- a/target/linux/ramips/base-files/etc/diag.sh
> +++ b/target/linux/ramips/base-files/etc/diag.sh
> @@ -34,6 +34,7 @@ get_status_led() {
>  	a5-v11|\
>  	d105|\
>  	dcs-930l-b1|\
> +	ex2700|\
>  	hlk-rm04|\
>  	mpr-a1|\
>  	mpr-a2)
> diff --git a/target/linux/ramips/base-files/lib/ramips.sh b/target/linux/ramips/base-files/lib/ramips.sh
> index 8025a40..aed8350 100755
> --- a/target/linux/ramips/base-files/lib/ramips.sh
> +++ b/target/linux/ramips/base-files/lib/ramips.sh
> @@ -154,6 +154,9 @@ ramips_board_detect() {
>  	*"ESR-9753")
>  		name="esr-9753"
>  		;;
> +	*"EX2700")
> +		name="ex2700";
> +		;;
>  	*"F5D8235 v1")
>  		name="f5d8235-v1"
>  		;;
> diff --git a/target/linux/ramips/base-files/lib/upgrade/platform.sh b/target/linux/ramips/base-files/lib/upgrade/platform.sh
> index 39b5f94..78edd22 100755
> --- a/target/linux/ramips/base-files/lib/upgrade/platform.sh
> +++ b/target/linux/ramips/base-files/lib/upgrade/platform.sh
> @@ -51,6 +51,7 @@ platform_check_image() {
>  	dir-810l|\
>  	e1700|\
>  	esr-9753|\
> +	ex2700|\
>  	f7c027|\
>  	firewrt|\
>  	fonera20n|\
> diff --git a/target/linux/ramips/dts/EX2700.dts b/target/linux/ramips/dts/EX2700.dts
> new file mode 100644
> index 0000000..4b5a01a
> --- /dev/null
> +++ b/target/linux/ramips/dts/EX2700.dts
> @@ -0,0 +1,148 @@
> +/*
> + * Device Tree file for the Netgear EX2700
> + *
> + * Copyright (C) 2016 Joseph C. Lehner <joseph.c.lehner at gmail.com>
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2.  This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +/dts-v1/;
> +
> +/include/ "mt7620a.dtsi"
> +
> +/ {
> +	compatible = "ralink,mt7620a-soc";
> +	model = "Netgear EX2700";
> +
> +	chosen {
> +		bootargs = "console=ttyS0,57600";
> +	};
> +
> +	palmbus at 10000000 {
> +
> +		gpio0: gpio at 600 {
> +			status = "okay";
> +		};
> +
> +		gpio1: gpio at 638 {
> +			status = "okay";
> +		};
> +
> +		spi at b00 {
> +			status = "okay";
> +
> +			m25p80 at 0 {
> +				#address-cells = <1>;
> +				#size-cells = <1>;
> +				compatible = "jedec,spi-nor";
> +				reg = <0 0>;
> +				linux,modalias = "m25p80", "mx25l3205d";
> +				spi-max-frequency = <10000000>;
> +
> +				partition at 0 {
> +					label = "u-boot";
> +					reg = <0x0 0x30000>;
> +					read-only;
> +				};
> +
> +				partition at 30000 {
> +					label = "u-boot-env";
> +					reg = <0x30000 0x10000>;
> +					read-only;
> +				};
> +
> +				partition at 40000 {
> +					label = "firmware";
> +					reg = <0x40000 0x3b0000>;
> +				};
> +
> +				art: partition at 3f0000 {
> +					label = "art";
> +					reg = <0x3f0000 0x10000>;
> +					read-only;
> +				};
> +			};
> +		};
> +	};
> +
> +	ethernet at 10100000 {
> +		mtd-mac-address = <&art 0x0>;
> +	};
> +
> +	wmac at 10180000 {
> +		ralink,mtd-eeprom = <&art 0x1000>;
> +	};
> +
> +	pinctrl {
> +		state_default: pinctrl0 {
> +			default {
> +				//  spi refclk: pins 37, 38, 39
> +				//       uartf: pins 8, 9, 10, 11, 12, 13, 14
> +				//         i2c: pins 1, 2
> +				ralink,group = "i2c", "uartf", "spi refclk";
> +				ralink,function = "gpio";
> +			};
> +		};
> +	};
> +
> +	gpio-leds {
> +		compatible = "gpio-leds";
> +
> +		power_g {
> +			label = "ex2700:green:power";
> +			gpios = <&gpio0 9 1>;
> +			default-state = "on";
> +		};
> +
> +		power_r {
> +			label = "ex2700:red:power";
> +			gpios = <&gpio0 11 1>;
> +		};
> +
> +		device_g {
> +			label = "ex2700:green:device";
> +			gpios = <&gpio0 13 1>;
> +		};
> +
> +		device_r {
> +			label = "ex2700:red:device";
> +			gpios = <&gpio0 10 1>;
> +		};
> +
> +		router_g {
> +			label = "ex2700:green:router";
> +			gpios = <&gpio0 12 1>;
> +		};
> +
> +		router_r {
> +			label = "ex2700:red:router";
> +			gpios = <&gpio0 14 1>;
> +		};
> +
> +		wps {
> +			label = "ex2700:green:wps";
> +			gpios = <&gpio1 15 1>;
> +		};
> +	};
> +
> +	gpio-keys-polled {
> +		compatible = "gpio-keys-polled";
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		poll-interval = <20>;
> +
> +		reset {
> +			label = "reset";
> +			gpios = <&gpio0 1 1>;
> +			linux,code = <0x198>;
> +		};
> +
> +		wps {
> +			label = "wps";
> +			gpios = <&gpio0 2 1>;
> +			linux,code = <0x211>;
> +		};
> +	};
> +};
> diff --git a/target/linux/ramips/image/Makefile b/target/linux/ramips/image/Makefile
> index 55b9cf4..adeea25 100644
> --- a/target/linux/ramips/image/Makefile
> +++ b/target/linux/ramips/image/Makefile
> @@ -913,6 +913,7 @@ endif
>  #
>  
>  Image/Build/Profile/E1700=$(call BuildFirmware/UMedia/$(1),$(1),e1700,E1700,0x013326)
> +Image/Build/Profile/EX2700=$(call BuildFirmware/Default4M/$(1),$(1),ex2700,EX2700)
>  Image/Build/Profile/MT7620a=$(call BuildFirmware/Default8M/$(1),$(1),mt7620a,MT7620a)
>  Image/Build/Profile/MT7620a_MT7610e=$(call BuildFirmware/Default8M/$(1),$(1),mt7620a_mt7610e,MT7620a_MT7610e)
>  Image/Build/Profile/MT7620a_MT7530=$(call BuildFirmware/Default8M/$(1),$(1),mt7620a_mt7530,MT7620a_MT7530)
> @@ -958,6 +959,7 @@ Image/Build/Profile/TINY-AC=$(call BuildFirmware/Default8M/$(1),$(1),tiny-ac,TIN
>  ifeq ($(SUBTARGET),mt7620)
>  define Image/Build/Profile/Default
>  	$(call Image/Build/Profile/E1700,$(1))
> +	$(call Image/Build/Profile/EX2700,$(1))
>  	$(call Image/Build/Profile/MT7620a,$(1))
>  	$(call Image/Build/Profile/MT7620a_MT7610e,$(1))
>  	$(call Image/Build/Profile/MT7620a_MT7530,$(1))
> diff --git a/target/linux/ramips/mt7620/profiles/netgear.mk b/target/linux/ramips/mt7620/profiles/netgear.mk
> new file mode 100644
> index 0000000..590756a
> --- /dev/null
> +++ b/target/linux/ramips/mt7620/profiles/netgear.mk
> @@ -0,0 +1,16 @@
> +#
> +# Copyright (C) 2016 OpenWrt.org
> +#
> +# This is free software, licensed under the GNU General Public License v2.
> +# See /LICENSE for more information.
> +#
> +
> +define Profile/EX2700
> +	NAME:=Netgear EX2700
> +	PACKAGES:=-kmod-usb-core -kmod-usb2 -kmod-usb-ohci -kmod-ledtrig-usbdev
> +endef
> +
> +define Profile/EX2700/Description
> +	Support for Netgear EX2700
> +endef
> +$(eval $(call Profile,EX2700))
> 
> 
> On 2016-02-08 08:44, John Crispin wrote:
>> Hi,
>>
>> the patch is whitespace broken. please fix and resend
>>
>> 	John
> 
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel



More information about the openwrt-devel mailing list