On Thu, Apr 25, 2019 at 10:42:01PM +0800, Ryder Lee wrote: > Fix many warnings with incorrect endian assumptions. > > Reported-by: kbuild test robot <lkp@xxxxxxxxx> > Signed-off-by: Ryder Lee <ryder.lee@xxxxxxxxxxxx> > --- > drivers/net/wireless/mediatek/mt76/mt7603/mac.c | 2 +- > drivers/net/wireless/mediatek/mt76/mt7615/mac.c | 12 ++++++------ > 2 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7603/mac.c b/drivers/net/wireless/mediatek/mt76/mt7603/mac.c > index 2f2961ee0a92..af5769d05e6f 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7603/mac.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7603/mac.c > @@ -627,7 +627,7 @@ mt7603_mac_fill_rx(struct mt7603_dev *dev, struct sk_buff *skb) > status->aggr = unicast && > !ieee80211_is_qos_nullfunc(hdr->frame_control); > status->tid = *ieee80211_get_qos_ctl(hdr) & IEEE80211_QOS_CTL_TID_MASK; > - status->seqno = IEEE80211_SEQ_TO_SN(hdr->seq_ctrl); > + status->seqno = IEEE80211_SEQ_TO_SN(__le16_to_cpu(hdr->seq_ctrl)); > > return 0; > } > diff --git a/drivers/net/wireless/mediatek/mt76/mt7615/mac.c b/drivers/net/wireless/mediatek/mt76/mt7615/mac.c > index 1bf3e7b5f6a7..4b934b0f5a39 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7615/mac.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7615/mac.c > @@ -235,7 +235,7 @@ int mt7615_mac_fill_rx(struct mt7615_dev *dev, struct sk_buff *skb) > status->aggr = unicast && > !ieee80211_is_qos_nullfunc(hdr->frame_control); > status->tid = *ieee80211_get_qos_ctl(hdr) & IEEE80211_QOS_CTL_TID_MASK; > - status->seqno = IEEE80211_SEQ_TO_SN(hdr->seq_ctrl); > + status->seqno = IEEE80211_SEQ_TO_SN(__le16_to_cpu(hdr->seq_ctrl)); Is there any reason to use underscored version instead of standard le16_to_cpu() ? Stanislaw
