[OpenWrt-Devel] [PATCH] bcm63xx: DGND3700v1: device tree improvements
mail at adrianschmutzler.de
mail at adrianschmutzler.de
Sat May 23 19:01:45 EDT 2020
Hi,
> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces at lists.openwrt.org]
> On Behalf Of Daniel González Cabanelas
> Sent: Sonntag, 24. Mai 2020 00:24
> To: openwrt-devel at lists.openwrt.org
> Cc: noltari at gmail.com
> Subject: [OpenWrt-Devel] [PATCH] bcm63xx: DGND3700v1: device tree
> improvements
>
> Improve the device tree file and related board data for the DGND3700v1/
> DGND3800B router:
> - Improve LEDs definitions, use shorter names.
> - Make the board name more readable.
> - Fix the switch LAN labels, the order is reversed.
> - Use the real name of the board for the board name instead of device
> name.
> - Use standarized names for partition nodes and leds.
This deals with several different issues at the same time. I'd prefer to have it split up (e.g. separate board name change from LED changes and switch changes).
>
> Signed-off-by: Daniel González Cabanelas <dgcbueu at gmail.com>
> ---
> .../bcm63xx/base-files/etc/board.d/01_leds | 12 ++--
> .../dts/bcm6368-netgear-dgnd3700-v1.dts | 64 +++++++++----------
> .../549-board_DGND3700v1_3800B.patch | 2 +-
> 3 files changed, 39 insertions(+), 39 deletions(-)
>
> diff --git a/target/linux/bcm63xx/base-files/etc/board.d/01_leds
> b/target/linux/bcm63xx/base-files/etc/board.d/01_leds
> index 91d67f0c0b..6b82d9e952 100755
> --- a/target/linux/bcm63xx/base-files/etc/board.d/01_leds
> +++ b/target/linux/bcm63xx/base-files/etc/board.d/01_leds
> @@ -66,12 +66,12 @@ inventel,livebox-1)
> ucidef_set_led_netdev "wlan0" "WIFI" "Livebox1:red:wifi" "wlan0"
> ;;
> netgear,dgnd3700-v1)
> - ucidef_set_led_netdev "lan" "LAN" "DGND3700v1_3800B:green:lan"
> "eth0.1"
> - ucidef_set_led_netdev "wan" "WAN"
> "DGND3700v1_3800B:green:inet" "eth0.2"
> - ucidef_set_led_netdev "wlan0" "WIFI2G"
> "DGND3700v1_3800B:green:wifi2g" "wlan0"
> - ucidef_set_led_netdev "wlan1" "WIFI5G"
> "DGND3700v1_3800B:blue:wifi5g" "wlan1"
> - ucidef_set_led_usbdev "usb1" "USB1"
> "DGND3700v1_3800B:green:usb-back" "1-1"
> - ucidef_set_led_usbdev "usb2" "USB2"
> "DGND3700v1_3800B:green:usb-front" "1-2"
> + ucidef_set_led_netdev "lan" "LAN" "$model:green:lan" "eth0.1"
> + ucidef_set_led_netdev "wan" "WAN" "$model:green:inet" "eth0.2"
> + ucidef_set_led_netdev "wlan0" "WIFI2G" "$model:green:wifi2g"
> "wlan0"
> + ucidef_set_led_netdev "wlan1" "WIFI5G" "$model:blue:wifi5g"
> "wlan1"
> + ucidef_set_led_usbdev "usb1" "USB1" "$model:green:usb-back" "1-
> 1"
> + ucidef_set_led_usbdev "usb2" "USB2" "$model:green:usb-front" "1-
> 2"
Is there any way to do sysupgrade on these devices? If yes, you will need migration of the names in /etc/config/system ...
> ;;
> netgear,dgnd3700-v2)
> ucidef_set_led_netdev "lan" "LAN" "$model:green:ethernet" "eth0"
> diff --git a/target/linux/bcm63xx/dts/bcm6368-netgear-dgnd3700-v1.dts
> b/target/linux/bcm63xx/dts/bcm6368-netgear-dgnd3700-v1.dts
> index 546b0b1f60..c17bb3a600 100644
> --- a/target/linux/bcm63xx/dts/bcm6368-netgear-dgnd3700-v1.dts
> +++ b/target/linux/bcm63xx/dts/bcm6368-netgear-dgnd3700-v1.dts
> @@ -5,12 +5,12 @@
> #include <dt-bindings/input/input.h>
>
> / {
> - model = "Netgear DGND3700v1/DGND3800B";
> + model = "Netgear DGND3700v1 / DGND3800B";
I don't think this is really necessary ...
> compatible = "netgear,dgnd3700-v1", "brcm,bcm6368";
>
> aliases {
> led-boot = &led_power_green;
> - led-failsafe = &led_power_green;
> + led-failsafe = &led_power_red;
This should be a separate commit again.
> led-running = &led_power_green;
> led-upgrade = &led_power_green;
> };
> @@ -51,49 +51,49 @@
> leds {
> compatible = "gpio-leds";
>
> - dsl_green {
> - label = "DGND3700v1_3800B:green:dsl";
> + led at 2 {
I don't know whether this is different on bcm63xx, but based on what I'm used to the old node name is preferred (dsl_green).
Best
Adrian
> + label = "dgnd3700-v1:green:dsl";
> gpios = <&pinctrl 2 1>;
> };
> - inet_red {
> - label = "DGND3700v1_3800B:red:inet";
> + led at 4 {
> + label = "dgnd3700-v1:red:inet";
> gpios = <&pinctrl 4 1>;
> };
> - inet_green {
> - label = "DGND3700v1_3800B:green:inet";
> + led at 5 {
> + label = "dgnd3700-v1:green:inet";
> gpios = <&pinctrl 5 1>;
> };
> - wps_green {
> - label = "DGND3700v1_3800B:green:wps";
> + led at 11 {
> + label = "dgnd3700-v1:green:wps";
> gpios = <&pinctrl 11 1>;
> };
> - usbfront_green {
> - label = "DGND3700v1_3800B:green:usb-front";
> + led at 13 {
> + label = "dgnd3700-v1:green:usb-front";
> gpios = <&pinctrl 13 1>;
> };
> - usbback_green {
> - label = "DGND3700v1_3800B:green:usb-back";
> + led at 14 {
> + label = "dgnd3700-v1:green:usb-back";
> gpios = <&pinctrl 14 1>;
> };
> - power_red {
> - label = "DGND3700v1_3800B:red:power";
> + led_power_red: led at 22 {
> + label = "dgnd3700-v1:red:power";
> gpios = <&pinctrl 22 1>;
> };
> - lan_green {
> - label = "DGND3700v1_3800B:green:lan";
> + led at 23 {
> + label = "dgnd3700-v1:green:lan";
> gpios = <&pinctrl 23 1>;
> };
> - led_power_green: power_green {
> - label = "DGND3700v1_3800B:green:power";
> + led_power_green: led at 24 {
> + label = "dgnd3700-v1:green:power";
> gpios = <&pinctrl 24 1>;
> default-state = "on";
> };
> - wifi2g_green {
> - label = "DGND3700v1_3800B:green:wifi2g";
> + led at 26 {
> + label = "dgnd3700-v1:green:wifi2g";
> gpios = <&pinctrl 26 1>;
> };
> - wifi5g_blue {
> - label = "DGND3700v1_3800B:blue:wifi5g";
> + led at 27 {
> + label = "dgnd3700-v1:blue:wifi5g";
> gpios = <&pinctrl 27 1>;
> };
> };
> @@ -107,25 +107,25 @@
> #address-cells = <1>;
> #size-cells = <1>;
>
> - cfe at 0 {
> + partition at 0 {
> label = "CFE";
> reg = <0x000000 0x020000>;
> read-only;
> };
>
> - linux at 20000 {
> + partition at 20000 {
> label = "linux";
> reg = <0x020000 0x1e20000>;
> compatible = "brcm,bcm963xx-imagetag";
> };
>
> - board_data at 1e40000 {
> + partition at 1e40000 {
> label = "board_data";
> reg = <0x1e40000 0x1a0000>;
> read-only;
> };
>
> - nvram at 1fe0000 {
> + partition at 1fe0000 {
> label = "nvram";
> reg = <0x1fe0000 0x20000>;
> };
> @@ -156,22 +156,22 @@
>
> lan at 1 {
> reg = <1>;
> - label = "lan1";
> + label = "lan4";
> };
>
> lan at 2 {
> reg = <2>;
> - label = "lan2";
> + label = "lan3";
> };
>
> lan at 3 {
> reg = <3>;
> - label = "lan3";
> + label = "lan2";
> };
>
> lan at 4 {
> reg = <4>;
> - label = "lan4";
> + label = "lan1";
> };
>
> cpu at 8 {
> diff --git a/target/linux/bcm63xx/patches-5.4/549-
> board_DGND3700v1_3800B.patch b/target/linux/bcm63xx/patches-5.4/549-
> board_DGND3700v1_3800B.patch
> index 936aab115b..7569e9643e 100644
> --- a/target/linux/bcm63xx/patches-5.4/549-
> board_DGND3700v1_3800B.patch
> +++ b/target/linux/bcm63xx/patches-5.4/549-
> board_DGND3700v1_3800B.patch
> @@ -5,7 +5,7 @@
> };
>
> +static struct board_info __initdata board_DGND3700v1_3800B = {
> -+ .name = "DGND3700v1_3800B",
> ++ .name = "U12L144T01",
> + .expected_cpu_id = 0x6368,
> +
> + .has_pci = 1,
> --
> 2.26.2
>
>
>
>
>
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: openpgp-digital-signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/openwrt-devel/attachments/20200524/9f74cd60/attachment.sig>
-------------- next part --------------
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list