Shayne Chen <shayne.chen@xxxxxxxxxxxx> wrote: > --- a/drivers/net/wireless/mediatek/mt76/mt76_connac.h > +++ b/drivers/net/wireless/mediatek/mt76/mt76_connac.h > @@ -232,9 +232,14 @@ static inline bool is_mt7992(struct mt76_dev *dev) > return mt76_chip(dev) == 0x7992; > } > > +static inline bool is_mt7990(struct mt76_dev *dev) > +{ > + return mt76_chip(dev) == 0x7993; It seems like patch 01/10 want to avoid this kind of confusion. Also, there are two IDs for MT7990: #define MT7990_DEVICE_ID 0x7993 #define MT7990_DEVICE_ID_2 0x799b > +} > + > static inline bool is_mt799x(struct mt76_dev *dev) > { > - return is_mt7996(dev) || is_mt7992(dev); > + return is_mt7996(dev) || is_mt7992(dev) || is_mt7990(dev); > } >