Date:   Thu, 23 Dec 2021 22:55:11 +0100
From:   Lorenzo Bianconi <lorenzo@kernel.org>
To:     Bo Jiao <bo.jiao@mediatek.com>
Cc:     Felix Fietkau <nbd@nbd.name>,
        linux-wireless <linux-wireless@vger.kernel.org>,
        Ryder Lee <ryder.lee@mediatek.com>,
        Xing Song <xing.song@mediatek.com>,
        Sujuan Chen <sujuan.chen@mediatek.com>,
        Shayne Chen <shayne.chen@mediatek.com>,
        Evelyn Tsai <evelyn.tsai@mediatek.com>,
        linux-mediatek <linux-mediatek@lists.infradead.org>
Subject: Re: [PATCH 1/1] mt76: add mt7916 to set wlan_idx_hi
Message-ID: <YcTwP6Hu5VoTMHEq@lore-desk>
References: <cover.1640175865.git.Bo.Jiao@mediatek.com>
 <5b82d68ee7c879591e22da51bfdb4af2e5fad228.1640175865.git.Bo.Jiao@mediatek.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="ceznv8vUbhznFNWi"
Content-Disposition: inline
In-Reply-To: <5b82d68ee7c879591e22da51bfdb4af2e5fad228.1640175865.git.Bo.Jiao@mediatek.com>
Precedence: bulk


--ceznv8vUbhznFNWi
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

> From: Bo Jiao <Bo.Jiao@mediatek.com>
>=20
> since mt7916 support up to 544 wcid entries,
> add it to set wlan_idx_hi.
>=20
> Signed-off-by: Ryder Lee <ryder.lee@mediatek.com>
> Signed-off-by: Bo Jiao <Bo.Jiao@mediatek.com>
> ---
>  drivers/net/wireless/mediatek/mt76/mt76_connac.h     | 5 +++++
>  drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h | 2 +-
>  2 files changed, 6 insertions(+), 1 deletion(-)
>=20
> diff --git a/drivers/net/wireless/mediatek/mt76/mt76_connac.h b/drivers/n=
et/wireless/mediatek/mt76/mt76_connac.h
> index e999d71..8701f04 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt76_connac.h
> +++ b/drivers/net/wireless/mediatek/mt76/mt76_connac.h
> @@ -110,6 +110,11 @@ static inline bool is_mt7915(struct mt76_dev *dev)
>  	return mt76_chip(dev) =3D=3D 0x7915;
>  }
> =20
> +static inline bool is_mt7916(struct mt76_dev *dev)
> +{
> +	return mt76_chip(dev) =3D=3D 0x7906;
> +}
> +
>  static inline u8 mt76_connac_chan_bw(struct cfg80211_chan_def *chandef)
>  {
>  	static const u8 width_to_bw[] =3D {
> diff --git a/drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h b/drive=
rs/net/wireless/mediatek/mt76/mt76_connac_mcu.h
> index edad583..fc9a084 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h
> +++ b/drivers/net/wireless/mediatek/mt76/mt76_connac_mcu.h
> @@ -1464,7 +1464,7 @@ mt76_connac_mcu_get_wlan_idx(struct mt76_dev *dev, =
struct mt76_wcid *wcid,
>  {
>  	*wlan_idx_hi =3D 0;
> =20
> -	if (is_mt7921(dev) || is_mt7915(dev)) {
> +	if (is_mt7921(dev) || is_mt7915(dev) || is_mt7916(dev)) {
>  		*wlan_idx_lo =3D wcid ? to_wcid_lo(wcid->idx) : 0;
>  		*wlan_idx_hi =3D wcid ? to_wcid_hi(wcid->idx) : 0;

I guess I have already included the fix here:
https://patchwork.kernel.org/project/linux-wireless/patch/df4d462f1ef4cfc1a=
e4f6482051a28a10c78262b.1640260901.git.lorenzo@kernel.org/

Regards,
Lorenzo

>  	} else {
> --=20
> 2.18.0
>=20

--ceznv8vUbhznFNWi
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQTquNwa3Txd3rGGn7Y6cBh0uS2trAUCYcTwPwAKCRA6cBh0uS2t
rCPcAQCr/p0C/RnsjkSJjTopA5nKQTY2CUKmdRS//yflEvUnAAD/YuuBNRLO4D0L
8Sidc+yynctBR6QF5F+dvJUo2sUTzQg=
=6Ahn
-----END PGP SIGNATURE-----

--ceznv8vUbhznFNWi--