[LEDE-DEV] [PATCH] ar71xx: add support for Comfast CF-E312A V2

[John Crispin]

Hi,

two comments inline ...


On 06/12/17 20:37, bmoffitt at bmoffitt.com wrote:
> The Comfast CF-E312A V2 is a 5 GHz radio with a built-in directional antenna,
> much like a Ubiquiti NanoStation M5.
>
> Specifiation:
> - SoC: AR9344 at 650 MHz
> - Flash: 16 MiB (W25Q128FVSG)
> - RAM: 64 MiB DDR2 (W9751G6KB)
> - Ethernet:  1 x WAN (100 Mbps) and 1 x LAN (100 Mbps)
> - Button: 1 x reset button, 1 x PoE pass-through switch
> - LED: 8 x LEDS (Power, LAN, WAN, Wireless, 4 signal LEDs)
> - UART: 1 x UART with header on PCB (GND, RX, TX, 3.3V)
>
> It is very similar to the CF-316N - similar chipset, etc. The major
> differences are:
>
> * It operates at 5 GHz, not 2.4
> * All the LEDs are white
> * It has 4 signal LEDs
>
> Of note: the eth1 port has an optional PoE pass-through, controlled with a
> slider switch.
>
> Most of the changes are in the file
> target/linux/ar71xx/files/arch/mips/ath79/mach-cf-e316n-v2.c - the code
> for the E316N was copied and modified to support the additional LEDs, the
> LED colors, and a few changes in the GPIOs.
>
> Firmware can be flashed on these units by the following method:
> 1.) Apply power to the unit
> 2.) Immediately AFTER applying power, hold down the reset button
> 3.) The WAN, LAN, and wireless lights will flash - wait three seconds
> (three flashes) and then release the button.
> 4.) After a second, the lights will "flutter" quickly and the unit will
> be visible at 192.168.1.1. A web page will be available to enable quick
> and simple uploading and flashing of firmware.
>
> During the boot process, these units also look for a tftp server at
> 192.168.1.10. If one is present, the firmware can be uploaded as a file
> called "firmware-auto.bin"
>
> Signed-off-by: bmoffitt at bmoffitt.com <bmoffitt at ayrstone.com>

we require a real name here

[...]
> diff --git a/target/linux/ar71xx/config-4.4 b/target/linux/ar71xx/config-4.4
> index 3ba3853..9682af1 100644
> --- a/target/linux/ar71xx/config-4.4
> +++ b/target/linux/ar71xx/config-4.4
> @@ -67,6 +67,7 @@ CONFIG_ATH79_MACH_C55=y
>   CONFIG_ATH79_MACH_CAP324=y
>   CONFIG_ATH79_MACH_CAP4200AG=y
>   CONFIG_ATH79_MACH_CARAMBOLA2=y
> +CONFIG_ATH79_MACH_CF_E312A=y
>   CONFIG_ATH79_MACH_CF_E316N_V2=y
>   CONFIG_ATH79_MACH_CF_E320N_V2=y
>   CONFIG_ATH79_MACH_CF_E355AC=y
> diff --git a/target/linux/ar71xx/config-4.9 b/target/linux/ar71xx/config-4.9
> index 1d47246..9c1fdf5 100644
> --- a/target/linux/ar71xx/config-4.9
> +++ b/target/linux/ar71xx/config-4.9
> @@ -65,6 +65,7 @@ CONFIG_ATH79_MACH_C55=y
>   CONFIG_ATH79_MACH_CAP324=y
>   CONFIG_ATH79_MACH_CAP4200AG=y
>   CONFIG_ATH79_MACH_CARAMBOLA2=y
> +CONFIG_ATH79_MACH_CF_E312A=y
>   CONFIG_ATH79_MACH_CF_E316N_V2=y
>   CONFIG_ATH79_MACH_CF_E320N_V2=y
>   CONFIG_ATH79_MACH_CF_E355AC=y
>

this symbol needs to be added to target/linux/ar71xx/nand/config-default 
and target/linux/ar71xx/mikrotik/config-default as "is not defined"

     John