Thread Links | Date Links | ||||
---|---|---|---|---|---|
Thread Prev | Thread Next | Thread Index | Date Prev | Date Next | Date Index |
Hi Manasi,
Your first comment generally makes sense. I think instead of updating the second statement, I can remove the 2nd statement altogether as it does not seem to add much value.
Regarding your second comment, I am not sure I understand what you mean by restricting development of the feature. The text I have for the subclause is aligned with CR document 437, which seems to be the latest version on DUO PDT.
Thanks,
Gaurang
From: Manasi E <emanzee@xxxxxxxxx>
Sent: Friday, July 25, 2025 5:18 AM To: Gaurang Naik <gnaik@xxxxxxxxxxxxxxxx> Cc: STDS-802-11-TGBN@xxxxxxxxxxxxxxxxx <STDS-802-11-TGBN@xxxxxxxxxxxxxxxxx> Subject: Re: [STDS-802-11-TGBN] (2) [STDS-802-11-TGBN] [STDS-802-11-TGBN] MAC PDT generic enablement WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros. Hi Gaurang Thanks for the document with updated changes.
I think the Mode Length field needs further clarification as the length depends not only on the Mode ID, but it depends on the OMP Operation in Mode Control. For example, Disable need not carry any Mode Parameters field. Have suggested modification in blue.
The Mode Length field indicates the number of octets in the Mode Parameters field. The value of the Mode Length field depends on the value of the Mode ID field and the OMP Operation in Mode Control field for that mode tuple.
For example When DPS is disabled , the below text implies that Mode Specific parameters are present. 9.4.2.X.1 Mode Specific Parameters for DPS When the value of the Mode ID field is 0, the Mode Specific Parameters field carries the parameters for DPS. The Mode Specific Parameters field for DPS is the same as the DPS Operation Parameters field defined in 9.4.1.85 (DPS Operation Parameters field).
Hence given that Mode Length clearly indicates when it will be set, do we really need to put a text that so and so mode ID does not carry Mode parameters field? This ends up restricting developing the feature in future.
Example: 9.4.2.X.3 Mode Specific Parameters for DUO When the value of the Mode ID field is 2, the Mode Specific Parameters field is not present. 9.4.2.X.4 Mode Specific Parameters for P-EDCA When the value of the Mode ID field is 3, the Mode Specific Parameters field is not present.
How about replacing like below 9.4.2.X.3 Mode Specific Parameters for DUO When the value of the Mode ID field is 2, the Mode Specific Parameters field is described in 37.11 (DUO) when present. I had shared some offline material to you to harmonize on this aspect earlier too.
Also this helps in making it uniform. Do not see why we need to describe mode parameters for one feature like NPCA and not define for others in DPS in this section of specification.
and similarly for others. Regards
Manasi Ekkundi
On Fri, 25 Jul, 2025, 12:38 Gaurang Naik, <0000305a2f8ce933-dmarc-request@xxxxxxxxxxxxxxxxx> wrote:
To unsubscribe from the STDS-802-11-TGBN list, click the following link: https://listserv.ieee.org/cgi-bin/wa?SUBED1=STDS-802-11-TGBN&A=1 |