Hi,

> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org]
> On Behalf Of Luochongjun
> Sent: Donnerstag, 7. Mai 2020 13:47
> To: openwrt-devel@lists.openwrt.org
> Cc: Luochongjun <luochong...@gl-inet.com>
> Subject: [OpenWrt-Devel] [PATCH] ath79: add support for gl-e750
> 
> The gl-e750 is a portable travel router that gives you safe access to the
> internet while traveling.
> 
> Specifications:
> - SoC: Qualcomm Atheros AR9531 (650MHz)
> - RAM: 128 MB DDR2
> - Flash: 16 MB SPI NOR (W25Q128FVSG) + 128 MB SPI NAND
> (GD5F1GQ4UFYIG)
> - Ethernet: 10/100: 1xLAN
> - Wireless: QCA9531 2.4GHz (bgn) + QCA9887 5GHz (ac)
> - USB: 1x USB 2.0 port
> - Switch: 1x switch
> - Button: 1x reset button
> - OLED Screen: 128*64 px

No LEDs?

Some other comments below.

> 
> Flash instruction:
> Support for sysupgrade directive upgrades, as well as luci upgrades.
> 
> OLED screen control:
> OLED controller is connected to QCA9531 through serial port, and can send
> instructions to OLED controller directly through serial port.
> Refer to the links below for a list of supported instructions:
> https://github.com/gl-inet/GL-E750-MCU-instruction
> 
> Signed-off-by: Luochongjun <luochong...@gl-inet.com>
> ---
>  target/linux/ath79/dts/qca9531_glinet_gl-e750.dts  | 141
> +++++++++++++++++++++
>  .../generic/base-files/etc/board.d/02_network      |   1 +
>  target/linux/ath79/image/nand.mk                   |  15 +++
>  3 files changed, 157 insertions(+)
>  create mode 100644 target/linux/ath79/dts/qca9531_glinet_gl-e750.dts
> 
> diff --git a/target/linux/ath79/dts/qca9531_glinet_gl-e750.dts
> b/target/linux/ath79/dts/qca9531_glinet_gl-e750.dts
> new file mode 100644
> index 0000000..645c69a
> --- /dev/null
> +++ b/target/linux/ath79/dts/qca9531_glinet_gl-e750.dts
> @@ -0,0 +1,141 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later OR MIT /dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +
> +#include "qca953x.dtsi"
> +
> +/ {
> +     compatible = "glinet,gl-e750", "qca,qca9531";
> +     model = "GL.iNet GL-E750";
> +
> +     keys {
> +             compatible = "gpio-keys";
> +
> +             pinctrl-names = "default";
> +             pinctrl-0 = <&jtag_disable_pins>;
> +
> +             reset {
> +                     label = "reset";
> +                     linux,code = <KEY_RESTART>;
> +                     gpios = <&gpio 3 GPIO_ACTIVE_LOW>;
> +             };
> +
> +             switch {
> +                     label = "switch";
> +                     linux,code = <BTN_0>;
> +                     gpios = <&gpio 1 GPIO_ACTIVE_LOW>;
> +             };
> +     };
> +
> +        gpio-export {
> +                compatible = "gpio-export";
> +                #size-cells = <0>;
> +
> +                gpio_lte_power {
> +                        gpio-export,name = "lte";

I'd consider lte-power or lte_power as name here.

> +                        gpio-export,output = <1>;
> +                        gpios = <&gpio 0 GPIO_ACTIVE_HIGH>;
> +                };
> +        };
> +
> +};
> +
> +&pcie0 {
> +     status = "okay";
> +};
> +
> +&uart {
> +     status = "okay";
> +};
> +
> +&usb0 {
> +     #address-cells = <1>;
> +     #size-cells = <0>;
> +     status = "okay";
> +
> +     hub_port: port@1 {
> +             reg = <1>;
> +             #trigger-source-cells = <0>;
> +     };
> +};
> +
> +&usb_phy {
> +     status = "okay";
> +};
> +
> +&spi {
> +     status = "okay";
> +     num-cs = <2>;
> +
> +     flash@0 {
> +             compatible = "jedec,spi-nor";
> +             reg = <0>;
> +             spi-max-frequency = <25000000>;
> +
> +             partitions {
> +                     compatible = "fixed-partitions";
> +                     #address-cells = <1>;
> +                     #size-cells = <1>;
> +
> +                     partition@0 {
> +                             label = "u-boot";
> +                             reg = <0x0 0x40000>;
> +                             read-only;
> +                     };
> +
> +                     partition@40000 {
> +                             label = "u-boot-env";
> +                             reg = <0x40000 0x10000>;
> +                     };
> +
> +                     art: partition@50000 {
> +                             label = "art";
> +                             reg = <0x50000 0x10000>;
> +                             read-only;
> +                     };
> +
> +                     partition@60000 {
> +                             label = "kernel";
> +                             reg = <0x60000 0x400000>;
> +                     };
> +
> +                     partition@460000 {
> +                             label = "reserved";
> +                             reg = <0x460000 0xba0000>;
> +                     };
> +
> +             };
> +     };
> +
> +     flash@1 {
> +             compatible = "spi-nand";
> +             reg = <1>;
> +             spi-max-frequency = <25000000>;
> +
> +             partitions {
> +                     compatible = "fixed-partitions";
> +                     #address-cells = <1>;
> +                     #size-cells = <1>;
> +
> +                     partition@0 {
> +                             label = "ubi";
> +                             reg = <0x0 0x8000000>;
> +                     };
> +             };
> +     };
> +};
> +
> +&eth0 {
> +     status = "okay";
> +
> +     mtd-mac-address = <&art 0x0>;
> +     phy-handle = <&swphy4>;
> +};
> +
> +&wmac {
> +     status = "okay";
> +     mtd-cal-data = <&art 0x1000>;
> +     mtd-mac-address = <&art 0x1002>;

mtd-mac-address can be dropped here, as it will be read from this offset by 
mtd-cal-data automatically.

> +};
> diff --git a/target/linux/ath79/generic/base-files/etc/board.d/02_network
> b/target/linux/ath79/generic/base-files/etc/board.d/02_network
> index d7f7eed..a50dfec 100755
> --- a/target/linux/ath79/generic/base-files/etc/board.d/02_network
> +++ b/target/linux/ath79/generic/base-files/etc/board.d/02_network
> @@ -23,6 +23,7 @@ ath79_setup_interfaces()
>       dlink,dir-505|\
>       engenius,ecb1750|\
>       glinet,gl-ar300m-lite|\
> +     glinet,gl-e750|\
>       netgear,ex6400|\
>       netgear,ex7300|\
>       ocedo,koala|\
> diff --git a/target/linux/ath79/image/nand.mk
> b/target/linux/ath79/image/nand.mk
> index 3ccd199..e6743e4 100644
> --- a/target/linux/ath79/image/nand.mk
> +++ b/target/linux/ath79/image/nand.mk
> @@ -130,6 +130,21 @@ define Device/glinet_gl-ar750s-nor  endef
> TARGET_DEVICES += glinet_gl-ar750s-nor
> 
> +define Device/glinet_gl-e750
> +  SOC := qca9531
> +  DEVICE_VENDOR := GL.iNet
> +  DEVICE_MODEL := GL-E750
> +  DEVICE_PACKAGES := kmod-ath10k-ct ath10k-firmware-qca9887-ct kmod-
> usb2 \
> +     kmod-usb-storage block-mount
> +  KERNEL_SIZE := 4096k
> +  IMAGE_SIZE := 131072k
> +  PAGESIZE := 2048
> +  VID_HDR_OFFSET := 2048
> +  BLOCKSIZE := 128k
> +  IMAGES += factory.img

Typically, GL.inet devices can be installed with sysupgrade, so a factory image 
is not necessary? (And it's not set up anyway)
I think this can be dropped.

If you send a newer patch, please make sure to add a v2 ...

Best

Adrian

> +endef
> +TARGET_DEVICES += glinet_gl-e750
> +
>  # fake rootfs is mandatory, pad-offset 129 equals (2 * uimage_header +
> 0xff)  define Device/netgear_ath79_nand
>    DEVICE_VENDOR := NETGEAR
> --
> 2.7.4
> 
> 
> 
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Attachment: openpgp-digital-signature.asc
Description: PGP signature

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to