Use a struct assignment here, which is clearer than the memcpy() and type-safe as well.
Unified · Split
| @@ -1122,8 +1122,7 @@ static int ieee80211_start_ap(struct wiphy *wiphy, struct net_device *dev, · 1121 unmodified lines | ||
| 1122 | 1122 | sdata->vif.bss_conf.enable_beacon = true; |
| 1123 | 1123 | sdata->vif.bss_conf.allow_p2p_go_ps = sdata->vif.p2p; |
| 1124 | 1124 | sdata->vif.bss_conf.twt_responder = params->twt_responder; |
| 1125 | − memcpy(&sdata->vif.bss_conf.he_obss_pd, ¶ms->he_obss_pd, | |
| 1126 | − sizeof(struct ieee80211_he_obss_pd)); | |
| 1125 | + sdata->vif.bss_conf.he_obss_pd = params->he_obss_pd; | |
| 1127 | 1126 | sdata->vif.bss_conf.he_bss_color = params->he_bss_color; |
| 1128 | 1127 | sdata->vif.bss_conf.s1g = params->chandef.chan->band == |
| 1129 | 1128 | NL80211_BAND_S1GHZ; |