Just a minor one in the dts...

Cheers,
Karl P

On Wed, Oct 15, 2014 at 09:11:39AM -0500, Dan Walters wrote:
> Support for the Linksys EA3500.  Here's the patch, and I entered the
> basic device info (including the serial console connections) on the
> wiki at http://wiki.openwrt.org/toh/linksys/ea3500 .  Hope it can be
> of use to someone else, I know I've been waiting a long while to run
> OpenWRT on my router!
> 
> diff --git a/target/linux/kirkwood/patches-3.14/170-ea3500.patch 
> b/target/linux/kirkwood/patches-3.14/170-ea3500.patch
> new file mode 100644
> index 0000000..155d9fc
> --- /dev/null
> +++ b/target/linux/kirkwood/patches-3.14/170-ea3500.patch
> @@ -0,0 +1,164 @@
> +diff -ruN a/arch/arm/boot/dts/kirkwood-ea3500.dts 
> b/arch/arm/boot/dts/kirkwood-ea3500.dts
> +--- a/arch/arm/boot/dts/kirkwood-ea3500.dts  1969-12-31 19:00:00.000000000 
> -0500
> ++++ b/arch/arm/boot/dts/kirkwood-ea3500.dts  2014-10-13 15:13:15.695625124 
> -0500
> +@@ -0,0 +1,149 @@
> ++/*
> ++ * kirkwood-ea3500.dts - Device Tree file for Linksys EA3500
> ++ *
> ++ * (c) 2013 Jonas Gorski <[email protected]>
> ++ * (c) 2013 Deutsche Telekom Innovation Laboratories
> ++ * (c) 2014 Luka Perkov <[email protected]>
> ++ * (c) 2014 Dan Walters <[email protected]>
> ++ *
> ++ * 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 "kirkwood.dtsi"
> ++#include "kirkwood-6282.dtsi"
> ++
> ++/ {
> ++    model = "Linksys EA3500";
> ++    compatible = "linksys,ea3500", "marvell,kirkwood-88f6282", 
> "marvell,kirkwood";
> ++
> ++    memory {
> ++            device_type = "memory";
> ++            reg = <0x00000000 0x4000000>;
> ++    };
> ++
> ++    chosen {
> ++            bootargs = "console=ttyS0,115200n8 earlyprintk";
> ++    };
> ++
> ++    mbus {
> ++            pcie-controller {
> ++                    status = "okay";
> ++
> ++                    pcie@1,0 {
> ++                            status = "okay";
> ++                    };
> ++
> ++                    pcie@2,0 {
> ++                            status = "okay";
> ++                    };
> ++            };
> ++    };
> ++
> ++    ocp@f1000000 {
> ++            pinctrl: pinctrl@10000 {
> ++                    pmx_led_green_power: pmx-led-green-power {
> ++                            marvell,pins = "mpp7";
> ++                            marvell,function = "gpo";

Shouldn't that really be gpio?

> ++                    };
> ++                    pmx_btn_wps: pmx-btn-wps {
> ++                            marvell,pins = "mpp47";
> ++                            marvell,function = "gpio";
> ++                    };
> ++                    pmx_btn_reset: pmx-btn-reset {
> ++                            marvell,pins = "mpp48";
> ++                            marvell,function = "gpio";
> ++                    };
> ++            };
> ++
> ++            rtc@10300 {
> ++                    status = "disabled";
> ++            };
> ++
> ++            serial@12000 {
> ++                    status = "okay";
> ++            };
> ++
> ++    };
> ++
> ++    gpio_keys {
> ++            compatible = "gpio-keys";
> ++            #address-cells = <1>;
> ++            #size-cells = <0>;
> ++            pinctrl-0 = < &pmx_btn_wps &pmx_btn_reset >;

Why are there spaces around the <> here? (and below) I don't see that in any of 
the
other dts files for pinctrl on
http://lxr.free-electrons.com/source/arch/arm/boot/dts/kirkwood-*** for 
instance.

> ++            pinctrl-names = "default";
> ++
> ++            button@15 {
> ++                    label = "WPS Button";
> ++                    linux,code = <KEY_WPS_BUTTON>;
> ++                    gpios = <&gpio1 15 GPIO_ACTIVE_LOW>;
> ++            };
> ++
> ++            button@16 {
> ++                    label = "Reset Button";
> ++                    linux,code = <KEY_RESTART>;
> ++                    gpios = <&gpio1 16 GPIO_ACTIVE_LOW>;
> ++            };
> ++    };
> ++
> ++    gpio-leds {
> ++            compatible = "gpio-leds";
> ++            pinctrl-0 = < &pmx_led_green_power >;

# this one here.

> ++            pinctrl-names = "default";
> ++
> ++            green-power {
> ++                    label = "ea3500:green:power";
> ++                    gpios = <&gpio0 7 GPIO_ACTIVE_HIGH>;
> ++            };
> ++    };
> ++};
> ++
> ++&nand {
> ++    status = "okay";
> ++    pinctrl-0 = <&pmx_nand>;
> ++    pinctrl-names = "default";
> ++
> ++    partition@0 {
> ++            label = "u-boot";
> ++            reg = <0x0000000 0x80000>;
> ++            read-only;
> ++    };
> ++
> ++    partition@80000 {
> ++            label = "u-boot environment";
> ++            reg = <0x80000 0x20000>;
> ++    };
> ++
> ++    partition@200000 {
> ++            label = "kernel";
> ++            reg = <0x200000 0x200000>;
> ++    };
> ++
> ++    partition@400000 {
> ++            label = "root";
> ++            reg = <0x400000 0x1200000>;
> ++    };
> ++};
> ++
> ++&mdio {
> ++    status = "okay";
> ++};
> ++
> ++&eth0 {
> ++    status = "okay";
> ++    ethernet0-port@0 {
> ++            speed = <1000>;
> ++            duplex = <1>;
> ++    };
> ++};
> ++
> ++&eth1 {
> ++    status = "okay";
> ++    ethernet1-port@0 {
> ++            speed = <1000>;
> ++            duplex = <1>;
> ++    };
> ++};
> +diff -ruN a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> +--- a/arch/arm/boot/dts/Makefile     2014-10-13 15:07:28.935284626 -0500
> ++++ b/arch/arm/boot/dts/Makefile     2014-10-13 15:07:53.676759195 -0500
> +@@ -90,6 +90,7 @@
> +     kirkwood-dns325.dtb \
> +     kirkwood-dockstar.dtb \
> +     kirkwood-dreamplug.dtb \
> ++    kirkwood-ea3500.dtb \
> +     kirkwood-ea4500.dtb \
> +     kirkwood-goflexnet.dtb \
> +     kirkwood-guruplug-server-plus.dtb \
> diff --git a/target/linux/kirkwood/profiles/115-router.mk 
> b/target/linux/kirkwood/profiles/115-router.mk
> index 000aa74..ef7f228 100644
> --- a/target/linux/kirkwood/profiles/115-router.mk
> +++ b/target/linux/kirkwood/profiles/115-router.mk
> @@ -5,6 +5,22 @@
>  # See /LICENSE for more information.
>  #
>  
> +define Profile/EA3500
> +  NAME:=Linksys EA3500
> +  PACKAGES:= \
> +     kmod-mwl8k kmod-usb2 kmod-usb-storage \
> +     uboot-envtools
> +endef
> +
> +define Profile/EA3500/Description
> + Package set compatible with Linksys EA3500 board.
> +endef
> +
> +EA3500_UBIFS_OPTS:="-m 2048 -e 126KiB -c 4096"
> +EA3500_UBI_OPTS:="-m 2048 -p 128KiB -s 512"
> +
> +$(eval $(call Profile,EA3500))
> +
>  define Profile/EA4500
>    NAME:=Linksys EA4500
>    PACKAGES:= \
> -- 
> 1.9.1
> _______________________________________________
> openwrt-devel mailing list
> [email protected]
> https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to