[PATCH v6 0/4] Introduce Automated Frequency Coordination (AFC) support

[Krishna Chaitanya]

On Wed, May 15, 2024 at 2:08 AM Lorenzo Bianconi <lorenzo at kernel.org> wrote:
>
> > On Tue, May 14, 2024 at 5:38 PM Lorenzo Bianconi <lorenzo at kernel.org> wrote:
> > >
> > > Introduce Automated Frequency Coordination (AFC) support for UNII-5 and UNII-7
> > > 6GHz bands.
> > > Update Transmit Power Envelope (TPE) IE according to the reply from AFC
> > > coordinator on UNII-5 or UNII-7 6GHz bands.
> > > AFCD and AFC hostapd client are tested with AFC DUT Test Harness [0].
> > >
> > > Changes since v5:
> > > -rebased on top of hostapd main branch
> > > Changes since v4:
> > > - fix curl header parsing
> > > - increase afc buffer size
> > > - fix afc-reply.txt
> > > - add some more error logging
> > > - add sanity check for some afc config file data
> > > Changes since v3:
> > > - fix compilation errors when IEEE80211_AX is not enabled
> > > - fix a leftover crash stopping the interface
> > > - make sure rx buffer is null-terminated
> > > Changes since v2:
> > > - multiple fixes testing with AFC DUT Test Harness
> > > Changes since v1:
> > > - use unix socket instead of a tcp one
> > > Changes since rfc:
> > > - move curl connection to a dedicated afc daemon
> > >
> > > Lorenzo Bianconi (4):
> > >   hostapd: afcd: add AFC daemon support
> > >   hostapd: export hostapd_is_usable_chans utility routine
> > >   hostapd: ap: add AFC client support
> > >   hostapd: update TPE IE according to AFC
> > >
> > >  afc/.gitignore        |    1 +
> > >  afc/Makefile          |   31 ++
> > >  afc/afc-reply.txt     |  219 +++++++++
> > >  afc/afcd.c            |  305 ++++++++++++
> > >  hostapd/Makefile      |    8 +
> > >  hostapd/config_file.c |  261 +++++++++++
> > >  hostapd/defconfig     |    3 +
> > >  hostapd/hostapd.conf  |   42 ++
> > >  src/ap/afc.c          | 1033 +++++++++++++++++++++++++++++++++++++++++
> > >  src/ap/ap_config.c    |   16 +
> > >  src/ap/ap_config.h    |   47 ++
> > >  src/ap/hostapd.c      |   16 +
> > >  src/ap/hostapd.h      |   54 +++
> > >  src/ap/hw_features.c  |    4 +-
> > >  src/ap/hw_features.h  |    6 +
> > >  src/ap/ieee802_11.c   |   47 +-
> > >  16 files changed, 2074 insertions(+), 19 deletions(-)
> > >  create mode 100644 afc/.gitignore
> > >  create mode 100644 afc/Makefile
> > >  create mode 100644 afc/afc-reply.txt
> > >  create mode 100644 afc/afcd.c
> > >  create mode 100644 src/ap/afc.c
> > I have done some basic tests and it is working fine, left a couple of
> > minor comments.
>
> Hi Krishna,
>
> thx for testing it. I guess we can address them with a follow-up patch if there
> are no other comments. What do you think?
>
Sure, those are only minor comments, I can submit them myself later if needed.
ACK for merging from my side.

Cheers.