The early demo board was only kept by the manufacturer. They made less than 10 pieces and they are not using it now. I'll fix the style problems and send another patch these days.
2015-11-29 19:10 GMT+08:00 Piotr Dymacz <[email protected]>: > Hello, > > See my comments inline, below. > > Cheers, > Piotr > > 2015-11-29 3:52 GMT+01:00 GuoGuo <[email protected]>: >> The PBR-M1 support in current OpenWrt is for the early demo board and it >> doesn't work on the final board.This patch fixes the support for it. > > What with users with "early demo board"? > As I understand, this patch simply breaks support for their board version? > >> The LED called pbr-m1:buzzer is a beeper connected to GPIO26 so I used >> gpio-beeper instead of gpio-leds. >> But as I said in another mail,the beeper starts beeping after the ethernet >> driver is loaded and I can't control it unless I disable the ethernet driver >> in dts.I'm not sure if there is something wrong with my dts file. >> >> Signed-off-by: 郭传鈜 <[email protected]> >> --- >> target/linux/ramips/base-files/etc/board.d/01_leds | 6 +- >> target/linux/ramips/base-files/etc/diag.sh | 4 +- >> target/linux/ramips/dts/PBR-M1.dts | 65 >> +++++++++++++++++----- >> 3 files changed, 57 insertions(+), 18 deletions(-) >> >> 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 8cb0e48..40a5244 100755 >> --- a/target/linux/ramips/base-files/etc/board.d/01_leds >> +++ b/target/linux/ramips/base-files/etc/board.d/01_leds >> @@ -200,9 +200,9 @@ oy-0001) >> set_wifi_led "$board:green:wifi" >> ;; >> pbr-m1) >> - ucidef_set_led_netdev "internet" "internet" "$board:green:internet" >> "eth0.2" >> - ucidef_set_led_default "power" "power" "$board:green:power" "1" >> - ucidef_set_led_default "sys" "sys" "$board:green:sys" "1" >> + ucidef_set_led_netdev "internet" "internet" "$board:blue:internet" >> "eth0.2" >> + ucidef_set_led_default "power" "power" "$board:blue:power" "1" >> + ucidef_set_led_default "sys" "sys" "$board:blue:sys" "1" >> ;; >> px-4885) >> set_wifi_led "$board:orange:wifi" >> diff --git a/target/linux/ramips/base-files/etc/diag.sh >> b/target/linux/ramips/base-files/etc/diag.sh >> index ff5fbf2..b67e45b 100644 >> --- a/target/linux/ramips/base-files/etc/diag.sh >> +++ b/target/linux/ramips/base-files/etc/diag.sh >> @@ -135,7 +135,9 @@ get_status_led() { >> sl-r7205) >> status_led="$board:green:wifi" >> ;; >> - pbr-m1|\ >> + pbr-m1) >> + status_led="$board:blue:sys" >> + ;; > > This breaks alphabetical order. > Please, take a look how boards are ordered in this file and follow the > convention. > >> w306r-v20|\ >> zbt-wr8305rt) >> status_led="$board:green:sys" >> diff --git a/target/linux/ramips/dts/PBR-M1.dts >> b/target/linux/ramips/dts/PBR-M1.dts >> index 7702d9c..f866a4f 100644 >> --- a/target/linux/ramips/dts/PBR-M1.dts >> +++ b/target/linux/ramips/dts/PBR-M1.dts >> @@ -17,6 +17,12 @@ >> >> sdhci@10130000 { >> status = "okay"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&sdhci_pins>; >> + }; >> + >> + xhci@1E1C0000 { >> + status = "okay"; >> }; >> >> palmbus@1E000000 { >> @@ -81,7 +87,7 @@ >> mt76@0,0 { >> reg = <0x0000 0 0 0 0>; >> device_type = "pci"; >> - mediatek,mtd-eeprom = <&factory 0x0000>; >> + mediatek,mtd-eeprom = <&factory 0x8000>; >> mediatek,2ghz = <0>; >> }; >> }; >> @@ -90,7 +96,7 @@ >> mt76@1,0 { >> reg = <0x0000 0 0 0 0>; >> device_type = "pci"; >> - mediatek,mtd-eeprom = <&factory 0x8000>; >> + mediatek,mtd-eeprom = <&factory 0x0000>; >> mediatek,5ghz = <0>; >> }; >> }; >> @@ -103,24 +109,29 @@ >> gpio-leds { >> compatible = "gpio-leds"; >> >> - buzzer { >> - label = "pbr-m1:buzzer"; >> - gpios = <&gpio0 27 1>; >> - }; >> - >> power { >> - label = "pbr-m1:green:power"; >> - gpios = <&gpio0 25 1>; >> + label = "pbr-m1:blue:power"; >> + gpios = <&gpio0 31 1>; >> }; >> >> sys { >> - label = "pbr-m1:green:sys"; >> - gpios = <&gpio0 24 1>; >> + label = "pbr-m1:blue:sys"; >> + gpios = <&gpio1 0 1>; >> }; >> >> internet { >> - label = "pbr-m1:green:internet"; >> - gpios = <&gpio0 23 1>; >> + label = "pbr-m1:blue:internet"; >> + gpios = <&gpio0 29 1>; >> + }; >> + >> + 2g { > > Below you have "wlan2g". > >> + label = "pbr-m1:blue:wlan2g"; >> + gpios = <&gpio1 1 1>; >> + }; >> + >> + 5g { > > Below you have "wlan5g". > >> + label = "pbr-m1:blue:wlan5g"; >> + gpios = <&gpio0 28 1>; >> }; >> }; >> >> @@ -137,10 +148,36 @@ >> }; >> }; >> >> + gpio_export { >> + compatible = "gpio-export"; >> + #size-cells = <0>; >> + >> + power_usb2 { >> + gpio-export,name = "power_usb2"; >> + gpio-export,output = <1>; >> + gpios = <&gpio0 22 0>; >> + }; > > Empty line here, please. > >> + power_usb3 { >> + gpio-export,name = "power_usb3"; >> + gpio-export,output = <1>; >> + gpios = <&gpio0 25 0>; >> + }; > > Empty line here, please. > >> + power_sata { >> + gpio-export,name = "power_sata"; >> + gpio-export,output = <1>; >> + gpios = <&gpio0 27 0>; >> + }; >> + }; >> + >> + beeper: beeper { >> + compatible = "gpio-beeper"; >> + gpios = <&gpio0 26 1>; >> + }; >> + >> pinctrl { >> state_default: pinctrl0 { >> gpio { >> - ralink,group = "wdt", "rgmii2", "wdt rst", >> "jtag", "mdio"; >> + ralink,group = "wdt", "rgmii2", "jtag", >> "mdio"; >> ralink,function = "gpio"; >> }; >> }; >> -- >> 2.4.3 >> _______________________________________________ >> 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
