> but maybe it should be > > WARN_ON((ops->add_station && !ops->del_station) || > (!opt->add_station && ops->del_station)) > > etc... Ahh, got it! I really like your idea but I assume it's quite rare to implement the "stop/del/leave/disconnect" callbacks and at the same time forget to implement the "start/add/join/connect". You will probably find out pretty quickly if the "start" functions are missing, while it might take some time debugging why you lack the "stop" functions (reinitialization issues/ resource leaks for example). With that said, don't take my word for it, I was only following the existing pattern and I assume someone else had a good reason in the first place. Best regards, Ola -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
