Linux wireless drivers development
 help / color / mirror / Atom feed
From: Ryder Lee <Ryder.Lee@mediatek.com>
To: "robh@kernel.org" <robh@kernel.org>, "nbd@nbd.name" <nbd@nbd.name>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"linux-mediatek@lists.infradead.org"
	<linux-mediatek@lists.infradead.org>,
	"Allen Ye (葉芷勳)" <Allen.Ye@mediatek.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [PATCH v3 2/2] dt-bindings: net: wireless: mt76: clarify backoff limit format
Date: Tue, 10 Feb 2026 18:16:47 +0000	[thread overview]
Message-ID: <2d333fa847c31fc41917f07e8aafce86da7ba0af.camel@mediatek.com> (raw)
In-Reply-To: <52244d9cbd63ff6b660074d0f44724896d1e88e3.camel@mediatek.com>

On Tue, 2026-02-10 at 18:12 +0000, Ryder Lee wrote:
> On Tue, 2026-02-10 at 10:08 -0800, Ryder Lee wrote:
> > Clarify the format of path backoff limit properties in mt76
> > binding.
> > Add explicit documentation for connac2 (mt7915, mt7981, mt7986) and
> > connac3 (mt7990, mt7992, mt7996...) devices, including the
> > difference
> > in beamforming and non-beamforming entries.
> > 
> > Also reformat the description to make is more precise.
> > 
> > Signed-off-by: Allen Ye <allen.ye@mediatek.com>
> > Co-developed-by: Allen Ye <allen.ye@mediatek.com>
> > Signed-off-by: Ryder Lee <ryder.lee@mediatek.com>
> > ---
> >  .../bindings/net/wireless/mediatek,mt76.yaml  | 20
> > +++++++++++++++++++
> >  1 file changed, 20 insertions(+)
> > 
> > diff --git
> > a/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml
> > b/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml
> > index ae6b97cdc..4156e1c97 100644
> > ---
> > a/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml
> > +++
> > b/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.yaml
> > @@ -252,6 +252,16 @@ properties:
> >                        followed by 10 power limit values. The order
> > of the
> >                        channel resource unit settings is RU26,
> > RU52,
> > RU106,
> >                        RU242/SU20, RU484/SU40, RU996/SU80 and
> > RU2x996/SU160.
> > +                      - For connac2
> > +                        - Beamforming entries for BW20~BW160 and
> > OFDM do not
> > +                          include 1T1ss.
> > +                        - When 1T1ss is not used, it should be
> > filled with 0.
> > +                      - For connac3
> > +                        - Beamforming entries for BW20~BW160 and
> > RU
> > include
> > +                          1T1ss, but OFDM does not include 1T1ss.
> > +                        - 1T1ss is taken into account, so no need
> > to
> > fill with 0.
> > +                      Non-beamforming and RU entries for both
> > connac2 and
> > +                      connac3 include 1T1ss.
> >                      minItems: 1
> >                      maxItems: 7
> >                      items:
> > @@ -270,6 +280,16 @@ properties:
> >                        followed by 10 power limit values. The order
> > of the
> >                        channel resource unit settings is RU26,
> > RU52,
> > RU106,
> >                        RU242/SU20, RU484/SU40, RU996/SU80 and
> > RU2x996/SU160.
> > +                      - For connac2
> > +                        - Beamforming entries for BW20~BW160 and
> > OFDM do not
> > +                          include 1T1ss.
> > +                        - When 1T1ss is not used, it should be
> > filled with 0.
> > +                      - For connac3
> > +                        - Beamforming entries for BW20~BW160 and
> > RU
> > include
> > +                          1T1ss, but OFDM does not include 1T1ss.
> > +                        - 1T1ss is taken into account, so no need
> > to
> > fill with 0.
> > +                      Non-beamforming and RU entries for both
> > connac2 and
> > +                      connac3 include 1T1ss.
> >                      minItems: 1
> >                      maxItems: 7
> >                      items:
> 
> Oops. Please ignore this patch, I mistakenly added the wrong entries.
> 
> Ryder

Oh, never mind. This is actually correct. I must have been a bit dizzy.

Ryder


  reply	other threads:[~2026-02-10 18:16 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-10 18:08 [PATCH v3 1/2] wifi: mt76: fix backoff fields and max_power calculation Ryder Lee
2026-02-10 18:08 ` [PATCH v3 2/2] dt-bindings: net: wireless: mt76: clarify backoff limit format Ryder Lee
2026-02-10 18:12   ` Ryder Lee
2026-02-10 18:16     ` Ryder Lee [this message]
2026-02-11  6:31   ` Krzysztof Kozlowski
2026-02-11  8:13     ` Ryder Lee
2026-02-11  8:16       ` Krzysztof Kozlowski
2026-02-11  8:33         ` Ryder Lee
2026-02-11  8:41           ` Krzysztof Kozlowski
2026-02-11  8:59             ` Ryder Lee
2026-02-11  9:08               ` Krzysztof Kozlowski
2026-02-11  9:19                 ` Ryder Lee
2026-02-11  9:35                   ` Ryder Lee
2026-02-11  9:52                   ` Chen-Yu Tsai
2026-02-11 16:26                     ` Ryder Lee
2026-02-11  9:01             ` Chen-Yu Tsai
2026-02-11  9:07               ` Ryder Lee

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2d333fa847c31fc41917f07e8aafce86da7ba0af.camel@mediatek.com \
    --to=ryder.lee@mediatek.com \
    --cc=Allen.Ye@mediatek.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=nbd@nbd.name \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox