[PATCH] Add CONFIG_CHROMIUM option

Dan Williams dcbw at redhat.com
Wed Dec 20 12:31:30 PST 2023


On Wed, 2023-12-20 at 20:23 +0000, Matthew Wang wrote:
> Hide ChromeOS specific code behind a flag.

I don't see any code in hostap git that uses -DCHROMIUM yet though?

Wouldn't it make sense to submit a patch with that first and include
this at that point?

Dan

> 
> Signed-off-by: Matthew Wang <matthewmwang at chromium.org>
> ---
>  wpa_supplicant/Makefile  | 4 ++++
>  wpa_supplicant/defconfig | 3 +++
>  2 files changed, 7 insertions(+)
> 
> diff --git a/wpa_supplicant/Makefile b/wpa_supplicant/Makefile
> index 46e20f44a..76a2f1f06 100644
> --- a/wpa_supplicant/Makefile
> +++ b/wpa_supplicant/Makefile
> @@ -491,6 +491,10 @@ else
>  OBJS += robust_av.o
>  endif
>  
> +ifdef CONFIG_CHROMIUM
> +CFLAGS += -DCHROMIUM
> +endif
> +
>  include ../src/drivers/drivers.mak
>  ifdef CONFIG_AP
>  OBJS_d += $(DRV_BOTH_OBJS)
> diff --git a/wpa_supplicant/defconfig b/wpa_supplicant/defconfig
> index 60d1bfc57..d2abef891 100644
> --- a/wpa_supplicant/defconfig
> +++ b/wpa_supplicant/defconfig
> @@ -293,6 +293,9 @@ CONFIG_BACKEND=file
>  # none = Empty template
>  #CONFIG_OS=unix
>  
> +# Is the operating system ChromeOS? Not defined by default.
> +#CONFIG_CHROMIUM=y
> +
>  # Select event loop implementation
>  # eloop = select() loop (default)
>  # eloop_win = Windows events and WaitForMultipleObject() loop




More information about the Hostap mailing list