[OpenWrt-Devel] [PATCH] [ar71xx] Add support for MERCURY MAC1200R

Hartmut Knaack knaack.h at gmx.de
Mon Jan 26 15:28:59 EST 2015


Roger Pueyo Centelles schrieb am 26.01.2015 um 18:10:
> Hi,
> 
> Thank you both for your comments.
> 
> @郭传鈜:
> 
>>> +       ucidef_set_led_wlan "wlan2g" "WLAN2G" "mercury:green:wlan2g" "phy1tpt"
>> I think the following line is incorrect:
>>> +       ucidef_set_led_wlan "wlan5g" "WLAN5G" "mercury:green:wlan5g" "phy0tpt"
>> There is no LED trigger called 'phy0tpt' so I think we should use netdev trigger like this:
>> ucidef_set_led_netdev "wlan5g" "WLAN5G" "mercury:green:wlan5g" "wlan0"
> 
> Ok!
> 
> 
>>> +static struct gpio_keys_button mac1200r_gpio_keys[] __initdata = {
>>> +       {
>>> +               .desc           = "WPS button",
>>> +               .type           = EV_KEY,
>>> +               .code           = KEY_WPS_BUTTON,
>>> +               .debounce_interval = MAC1200R_KEYS_DEBOUNCE_INTERVAL,
>>> +               .gpio           = MAC1200R_GPIO_BTN_WPS,
>>> +               .active_low     = 1,
>>> +       },
>>> +};
>> Although the key is called WPS/RESET , I think a reset button is more important than a WPS button.That's just my personal opinion :)
> 
> I think you are right.
> 
> 
>>> +       ap91_pci_init(art + MAC1200R_PCIE_CALDATA_OFFSET, tmpmac);
>> What? I think this function is only able to load CALDATA for ath9k devices.Maybe I'm wrong:)
> 
> The router has two radios, one ath9k and one ath10k. Isn't it needed for the ath9k calibration data? Or this radio is already covered by "ath79_register_wmac(art + MAC1200R_WMAC_CALDATA_OFFSET, tmpmac); ...?
> 
> 
>>> +$(eval $(call SingleProfile,TPLINK-LZMA,64kraw,MAC1200R,mc-mac1200r,MC-MAC1200R,ttyS0,115200,0x12000001,1,8Mlzma))
>>The factory image is broken since TP-LINK started to use a new firmware format with RSA signature in China.This should be introduced in the TOH wiki I think:)
> 
> Done!
> 
> 
> 
> @John:
> 
> Sorry. Do you mean a description for the patch? I am not aware of what SoB means.

Yes, a patch description followed by your Signed-off-by (SoB) is missing. Have a look at the submitting patches wiki page for the patch format requirements. Also, check through your patch again and make sure to mind alphabetic order wherever needed.

> 
> 
> Thanks!
> 
> Roger
> 
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
> 
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list