{"action":"created","issue":{"url":"https://api.github.com/repos/openwrt/openwrt/issues/1623","repository_url":"https://api.github.com/repos/openwrt/openwrt","labels_url":"https://api.github.com/repos/openwrt/openwrt/issues/1623/labels{/name}","comments_url":"https://api.github.com/repos/openwrt/openwrt/issues/1623/comments","events_url":"https://api.github.com/repos/openwrt/openwrt/issues/1623/events","html_url":"https://github.com/openwrt/openwrt/pull/1623","id":389895852,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM3NzgyNzYz","number":1623,"title":"[RFC] ath79: add support for COMFAST CF-E110N","user":{"login":"rogerpueyo","id":6009603,"node_id":"MDQ6VXNlcjYwMDk2MDM=","avatar_url":"https://avatars1.githubusercontent.com/u/6009603?v=4","gravatar_id":"","url":"https://api.github.com/users/rogerpueyo","html_url":"https://github.com/rogerpueyo","followers_url":"https://api.github.com/users/rogerpueyo/followers","following_url":"https://api.github.com/users/rogerpueyo/following{/other_user}","gists_url":"https://api.github.com/users/rogerpueyo/gists{/gist_id}","starred_url":"https://api.github.com/users/rogerpueyo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rogerpueyo/subscriptions","organizations_url":"https://api.github.com/users/rogerpueyo/orgs","repos_url":"https://api.github.com/users/rogerpueyo/repos","events_url":"https://api.github.com/users/rogerpueyo/events{/privacy}","received_events_url":"https://api.github.com/users/rogerpueyo/received_events","type":"User","site_admin":false},"labels":[{"id":927495841,"node_id":"MDU6TGFiZWw5Mjc0OTU4NDE=","url":"https://api.github.com/repos/openwrt/openwrt/labels/target/ath79","name":"target/ath79","color":"0043e4","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2018-12-11T18:29:17Z","updated_at":"2018-12-16T14:55:33Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/openwrt/openwrt/pulls/1623","html_url":"https://github.com/openwrt/openwrt/pull/1623","diff_url":"https://github.com/openwrt/openwrt/pull/1623.diff","patch_url":"https://github.com/openwrt/openwrt/pull/1623.patch"},"body":"This patch adds support for the COMFAST CF-E110N, an outdoor wireless\r\nCPE with two Ethernet ports and a 802.11bgn radio. It will supersede https://github.com/openwrt/openwrt/pull/1453 https://github.com/openwrt/openwrt/pull/1531, which were based on the deprecated ar71xx architecture.\r\n\r\nThere are a couple of issues I haven't been able to solve:\r\n - The green rssimediumhigh (GPIO14) and red rssilow (GPIO11) LEDs don't seem to work, even if they did in the ar71xx target (see https://github.com/openwrt/openwrt/pull/1531). They are lit on and    off during kernel boot, but they can't be controlled afterwards.\r\n - The green rssihigh (GPIO16) LED is also lit on and off during kernel boot but, afterwards, it blinks to eth1's link and activity.\r\n\r\nAny ideas? Thanks!"},"comment":{"url":"https://api.github.com/repos/openwrt/openwrt/issues/comments/447649389","html_url":"https://github.com/openwrt/openwrt/pull/1623#issuecomment-447649389","issue_url":"https://api.github.com/repos/openwrt/openwrt/issues/1623","id":447649389,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NzY0OTM4OQ==","user":{"login":"rogerpueyo","id":6009603,"node_id":"MDQ6VXNlcjYwMDk2MDM=","avatar_url":"https://avatars1.githubusercontent.com/u/6009603?v=4","gravatar_id":"","url":"https://api.github.com/users/rogerpueyo","html_url":"https://github.com/rogerpueyo","followers_url":"https://api.github.com/users/rogerpueyo/followers","following_url":"https://api.github.com/users/rogerpueyo/following{/other_user}","gists_url":"https://api.github.com/users/rogerpueyo/gists{/gist_id}","starred_url":"https://api.github.com/users/rogerpueyo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rogerpueyo/subscriptions","organizations_url":"https://api.github.com/users/rogerpueyo/orgs","repos_url":"https://api.github.com/users/rogerpueyo/repos","events_url":"https://api.github.com/users/rogerpueyo/events{/privacy}","received_events_url":"https://api.github.com/users/rogerpueyo/received_events","type":"User","site_admin":false},"created_at":"2018-12-16T14:55:33Z","updated_at":"2018-12-16T14:55:33Z","author_association":"CONTRIBUTOR","body":"You're right, it says \"pinctrl-1\", but I've also tried \"pinctrl-0\", putting everything together with the &jtag, moving it up and down... If I use \"pinctrl-0\":\r\n - In /sys:\r\n```\r\nroot@OpenWrt:~# ls -la /sys/firmware/devicetree/base/ahb/apb/pinmux@1804002c\r\n-r--r--r--    1 root     root             4 Dec 16 12:58 #pinctrl-cells\r\n-r--r--r--    1 root     root             4 Dec 16 12:58 #size-cells\r\ndrwxr-xr-x    6 root     root             0 Dec 16 12:58 .\r\ndrwxr-xr-x   14 root     root             0 Dec 16 12:58 ..\r\n-r--r--r--    1 root     root            15 Dec 16 12:58 compatible\r\n-r--r--r--    1 root     root             7 Dec 16 12:58 name\r\n-r--r--r--    1 root     root             0 Dec 16 12:58 pinctrl-single,bit-per-mux\r\n-r--r--r--    1 root     root             4 Dec 16 12:58 pinctrl-single,function-mask\r\n-r--r--r--    1 root     root             4 Dec 16 12:58 pinctrl-single,register-width\r\ndrwxr-xr-x    2 root     root             0 Dec 16 12:58 pinmux_jtag_disable_pins\r\ndrwxr-xr-x    2 root     root             0 Dec 16 12:58 pinmux_rssihigh_pin\r\ndrwxr-xr-x    2 root     root             0 Dec 16 12:58 pinmux_rssilow_pin\r\ndrwxr-xr-x    2 root     root             0 Dec 16 12:58 pinmux_rssimediumhigh_pin\r\n-r--r--r--    1 root     root             8 Dec 16 12:58 reg\r\n```\r\n\r\n - in dmesg:\r\n```\r\n[    0.494515] pinctrl-single 1804002c.pinmux: 576 pins at pa b804002c size 72\r\n```\r\n\r\nand that's it. :(\r\n\r\nI've also tried creating two different, non-overlapping pinmux and pinmux2 sections, one for the previous jtag_disable part, another for tweaking the GPIOs, but that didn't help either. Directly on the SoC's .dtsi file:\r\n\r\n```\r\n\t\t\tpinmux: pinmux@1804006c {\r\n\t\t\t\tcompatible = \"pinctrl-single\";\r\n\r\n\t\t\t\treg = <0x1804006c 0x8>;\r\n\r\n\t\t\t\t#size-cells = <0>;\r\n\r\n\t\t\t\tpinctrl-single,bit-per-mux;\r\n\t\t\t\tpinctrl-single,register-width = <32>;\r\n\t\t\t\tpinctrl-single,function-mask = <0x1>;\r\n\t\t\t\t#pinctrl-cells = <2>;\r\n\r\n\t\t\t\tjtag_disable_pins: pinmux_jtag_disable_pins {\r\n\t\t\t\t\tpinctrl-single,bits = <0x00 0x2 0x2>;\r\n\t\t\t\t};\r\n\t\t\t};\r\n\r\n\t\t\tpinmux2: pinmux@1804002c {\r\n\t\t\t\tcompatible = \"pinctrl-single\";\r\n\r\n\t\t\t\treg = <0x1804002c 0x40>;\r\n\r\n\t\t\t\t#size-cells = <0>;\r\n\r\n\t\t\t\tpinctrl-single,bit-per-mux;\r\n\t\t\t\tpinctrl-single,register-width = <32>;\r\n\t\t\t\tpinctrl-single,function-mask = <0x0>;\r\n\t\t\t\t#pinctrl-cells = <2>;\r\n\t\t\t};\r\n\r\n```\r\n\r\nAnd use different `pinctrl-single,function-mask` masks, etc. I couldn't make it either. :("}}
{"actor":{"display_login":"rogerpueyo"}}