On Sun, Dec 10, 2023 at 2:09 PM Jouni Malinen <j@xxxxx> wrote: > > On Sun, Dec 10, 2023 at 12:52:04AM +0530, Krishna Chaitanya wrote: > > Looks like, we need to separate out BTM code to a separate file btm.c > > and include > > it for both MBO and WNM (NEED_BTM=y). This way we can get rid of rest of the > > WNM. WDYT? > > It does not have to be moved to a separate file, but it would seem > reasonable to be able to pick a subset of wnm_sta.c when CONFIG_MBO=y > without having to force CONFIG_WNM=y. For example, wnm_sta.c could be > included into the build if either CONFIG_WNM=y or CONFIG_MBO=y and then > within wnm_sta.c, some of the functions would be included based on > CONFIG_WNM and some base on either CONFIG_WNM or CONFIG_MBO. And well, > there are already couple of parts there using CONFIG_MBO on its own. Thanks. Yes, I started doing the same, except I moved all BTM functions to btm.c to make it clearer, but yeah they can be in wnm_sta.c itself. _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap