the values within the patch are named
#define BW_NSS_FWCONF_MAP_ENABLE BIT(31)
#define BW_NSS_FWCONF_MAP_160MHZ_S (0)
#define BW_NSS_FWCONF_MAP_160MHZ_M (0x00000007)
#define BW_NSS_FWCONF_MAP_80_80MHZ_S (3)
#define BW_NSS_FWCONF_MAP_80_80MHZ_M (0x00000038)
#define BW_NSS_FWCONF_MAP_M (0x0000003F)
so your comment doesnt seem to belong to the patch. maybe the openwrt
guys renamed some values in the patch. my one is based on my original
sources
Am 04.07.2018 um 12:48 schrieb Sebastian Gottschall:
Am 04.07.2018 um 12:40 schrieb Sven Eckelmann:
On Mittwoch, 4. Juli 2018 11:29:53 CEST s.gottschall@xxxxxxxxxx wrote:
+/* Values defined to set 160 MHz Bandwidth NSS Mapping into FW*/
+#define BW_NSS_FWCONF_160(x) (BW_NSS_FWCONF_MAP_ENABLE | \
+ (((x - 1) <<
BW_NSS_FWCONF_MAP_160MHZ_S) \
+ & BW_NSS_FWCONF_MAP_160MHZ_M))
+#define BW_NSS_FWCONF_80_80(x) (BW_NSS_FWCONF_MAP_ENABLE | \
+ (((x - 1) <<
BW_NSS_FWCONF_MAP_80_80MHZ_S) \
+ &
BW_NSS_FWCONF_MAP_80_80MHZ_M))
Please check the shift and mask macros in core.h
#define BW_NSS_FWCONF_MAP_160MHZ_LSB 0
#define BW_NSS_FWCONF_MAP_160MHZ_MASK 0x00000007
#define BW_NSS_FWCONF_MAP_80_80MHZ_LSB 3
#define BW_NSS_FWCONF_MAP_80_80MHZ_MASK 0x00000038
#define BW_NSS_FWCONF_MAP_MASK 0x0000003F
#define GET_BW_NSS_FWCONF_160(x) (MS((x),
BW_NSS_FWCONF_MAP_160MHZ) + 1)
#define GET_BW_NSS_FWCONF_80_80(x) (MS((x),
BW_NSS_FWCONF_MAP_80_80MHZ_M) + 1)
/* Values defined to set 160 MHz Bandwidth NSS Mapping into FW*/
#define BW_NSS_FWCONF_160(x) (BW_NSS_FWCONF_MAP_ENABLE | \
(SM((x) - 1,
BW_NSS_FWCONF_MAP_160MHZ))
#define BW_NSS_FWCONF_80_80(x) (BW_NSS_FWCONF_MAP_ENABLE | \
(SM((x) - 1,
BW_NSS_FWCONF_MAP_80_80MHZ))
(these snippets were not yet tested and most likely need further
adjustments)
Kind regards,
Sven
can you explain your request? the macros are unchanged and just
reformated for maximum line length restriction. the values have been
taken from QCA's wmi header
Sebastian
