[PATCH 1/2] De-couple MBO and WNM

Krishna Chaitanya chaitanya.mgit at gmail.com
Sun Dec 10 02:21:32 PST 2023


On Sun, Dec 10, 2023 at 2:09 PM Jouni Malinen <j at w1.fi> 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.



More information about the Hostap mailing list