<div>Hi,Sven</div><div>     I am using the QCA9886 chip for the first time. I saw the example you gave.</div><div>     I think the following code should be deleted.</div><div>         ln -sf /lib/firmware/ath10k/pre-cal-pci-0000\:00\:00.0.bin  /lib/firmware/ath10k/QCA9888/hw2.0/board.bin</div><div><includetail><div> </div><div>Thank you</div><div> </div><div style="font:Verdana normal 14px;color:#000;"><div style="FONT-SIZE: 12px;FONT-FAMILY: Arial Narrow;padding:2px 0 2px 0;">------------------ Original ------------------</div><div style="FONT-SIZE: 12px;background:#efefef;padding:8px;"><div id="menu_sender"><b>From: </b> "Sven Eckelmann"<sven.eckelmann@openmesh.com>;</div><div><b>Date: </b> Tue, Aug 28, 2018 04:04 PM</div><div><b>To: </b> "John Crispin"<john@phrozen.org>; <wbr></div><div><b>Cc: </b> "wellnw"<guilin.wang@gl-inet.com>; "openwrt-devel"<openwrt-devel@lists.openwrt.org>; <wbr></div><div><b>Subject: </b> Re: [OpenWrt-Devel] [PATCH] ar71xx: add support for GL.iNet GL-X1200</div></div><div> </div><div style="position:relative;">On Dienstag, 28. August 2018 09:38:39 CEST John Crispin wrote:<br>> > +gl-x1200)<br>> > +       ucidef_set_led_wlan "wlan2g" "WLAN2G" "$board:green:wlan2g" "phy1tpt"<br>> > +       ucidef_set_led_wlan "wlan5g" "WLAN5G" "$board:green:wlan5g" "phy0tpt"<br>> > +       ;;<br>> <br>> this section aswell as several other places in this patch is not <br>> alphabetically ordered. please fix all occurrences<br><br>And this section could also be merged together with the gl-ar750s section.<br><br>> > +       gl-x1200)<br>> > +               ath10kcal_extract "art" 20480 12064<br>> > +               ln -sf /lib/firmware/ath10k/pre-cal-pci-0000\:00\:00.0.bin \<br>> > +                       /lib/firmware/ath10k/QCA9888/hw2.0/board.bin<br>> > +               ath10kcal_patch_mac $(macaddr_add $(cat /sys/class/net/eth0/address) +1)<br>> > +               ;;<br>[...]<br>> > +  DEVICE_PACKAGES := kmod-ath10k ath10k-firmware-qca9887 kmod-usb-core \<br>> > +       kmod-usb2 kmod-usb-storage<br><br>Is the 5GHz now a QCA9888 or a QCA9887? And why are you linking the pre-cal <br>data to board.bin? QCA9888 has its BDF stored in the board-2.bin [1] (selected by <br>the bmi from the OTP and the variant string from DT/SMBIOS) and per device <br>pre-cal data is stored in /lib/firmware/ath10k/pre-cal-*.bin. The OTP is then<br>merging both and applying extra modifications on top of it. The result is then<br>automatically given to the actual QCA9888 firmware.<br><br>See for example the OpenMesh A62 for an device with an QCA9888. As you may <br>notice, it is using device tree - something which ar71xx is not using. But the <br>new target ath79 does.<br><br>Kind regards,<br>      Sven<br><br>[1] https://github.com/openwrt/openwrt/pull/713#issuecomment-369511149<br></div></div><!--<![endif]--></includetail></div>