[PATCH v2 01/17] EHT: Add definitions for STA Control fields of Basic Multi-Link element

Veerendranath Jakkam quic_vjakkam at quicinc.com
Sun Oct 16 05:29:27 PDT 2022


On 10/3/2022 5:59 PM, Peer, Ilan wrote:
> Use 'BASIC_ML_STA_CTRL0_LINK_ID_SHIFT' to be consistent with other definitions below.

sure

> +#define BASIC_ML_STA_CTRL1_PRES_DTIM_INFO		0x10
> +#define BASIC_ML_STA_CTRL1_PRES_NSTR_LINK_PAIR		0x20
> +#define BASIC_ML_STA_CTRL1_PRES_NSTR_BITMAP		0x40
> +#define BASIC_ML_STA_CTRL1_PRES_BSS_PARAM_COUNT		0x80
> +
> The definitions for CTRL1 seem to be wrong. They all should be shifted right by 4.

Thanks. I will fix it in v3 series




More information about the Hostap mailing list