Hi Stefan,
> > +static int
> > +mac802154_send_association_resp_locked(struct ieee802154_sub_if_data *sdata,
> > + struct ieee802154_pan_device *target,
> > + struct ieee802154_assoc_resp_pl *assoc_resp_pl)
> > +{
> > + u64 teaddr = swab64((__force u64)target->extended_addr);
> > + struct ieee802154_association_resp_frame frame = {};
> > + struct ieee802154_local *local = sdata->local;
> > + struct wpan_dev *wpan_dev = &sdata->wpan_dev;
> > + struct sk_buff *skb;
> > + int ret;
> > +
> > + frame.mhr.fc.type = IEEE802154_FC_TYPE_MAC_CMD;
> > + frame.mhr.fc.security_enabled = 0;
> > + frame.mhr.fc.frame_pending = 0;
> > + frame.mhr.fc.ack_request = 1; /* We always expect an ack here */
> > + frame.mhr.fc.intra_pan = 1;
> > + frame.mhr.fc.dest_addr_mode = IEEE802154_EXTENDED_ADDRESSING;
> > + frame.mhr.fc.version = IEEE802154_2003_STD;
> > + frame.mhr.fc.source_addr_mode = IEEE802154_EXTENDED_ADDRESSING;
> > + frame.mhr.seq = 10;
>
> Where does the 10 come from and what is the meaning?
Good point, that is a leftover from my earlier WIP versions. A few lines
below this is overwritten by:
frame.mhr.seq = atomic_inc_return(&wpan_dev->dsn) & 0xFF;
I'll of course drop the `seq = 10` line.
Thanks,
Miquèl
