On Mon, 2018-05-07 at 13:04 -0700, Peter Oh wrote: > > On 05/07/2018 12:44 PM, Johannes Berg wrote: > > On Fri, 2018-04-27 at 12:26 -0700, peter.oh@xxxxxxxxxxxxxxxxx wrote: > > > From: Peter Oh <peter.oh@xxxxxxxxxxxxxxxxx> > > > > > > There is no meaning to call ieee80211_mesh_process_chnswitch > > > without CSA IE provided, since the function will always return error. > > > > I'm not convinced, an extended channel switch element might be present > > instead? > > I expected the comment. Yes, extended channel switch element could be > present instead of channel switch element which I missed to take care > of. So you can ignore the patch for now and I'll get back with right > patch for it. Heh. Well, but then why bother at all? I mean, the inner function will do the check and fail, and nothing will happen, so why bother? johannes