On Dienstag, 28. August 2018 09:38:39 CEST John Crispin wrote:
> > +gl-x1200)
> > +       ucidef_set_led_wlan "wlan2g" "WLAN2G" "$board:green:wlan2g" 
> > "phy1tpt"
> > +       ucidef_set_led_wlan "wlan5g" "WLAN5G" "$board:green:wlan5g" 
> > "phy0tpt"
> > +       ;;
> 
> this section aswell as several other places in this patch is not 
> alphabetically ordered. please fix all occurrences

And this section could also be merged together with the gl-ar750s section.

> > +       gl-x1200)
> > +               ath10kcal_extract "art" 20480 12064
> > +               ln -sf /lib/firmware/ath10k/pre-cal-pci-0000\:00\:00.0.bin \
> > +                       /lib/firmware/ath10k/QCA9888/hw2.0/board.bin
> > +               ath10kcal_patch_mac $(macaddr_add $(cat 
> > /sys/class/net/eth0/address) +1)
> > +               ;;
[...]
> > +  DEVICE_PACKAGES := kmod-ath10k ath10k-firmware-qca9887 kmod-usb-core \
> > +       kmod-usb2 kmod-usb-storage

Is the 5GHz now a QCA9888 or a QCA9887? And why are you linking the pre-cal 
data to board.bin? QCA9888 has its BDF stored in the board-2.bin [1] (selected 
by 
the bmi from the OTP and the variant string from DT/SMBIOS) and per device 
pre-cal data is stored in /lib/firmware/ath10k/pre-cal-*.bin. The OTP is then
merging both and applying extra modifications on top of it. The result is then
automatically given to the actual QCA9888 firmware.

See for example the OpenMesh A62 for an device with an QCA9888. As you may 
notice, it is using device tree - something which ar71xx is not using. But the 
new target ath79 does.

Kind regards,
        Sven

[1] https://github.com/openwrt/openwrt/pull/713#issuecomment-369511149

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to