Add more comment for wpa_supplicant using soong
This CL add 2 main section:
1. Point out the scope for 2 different products, one is cuttlefish and
the other is non-cuttlefish(ex: pixel or other oems').
2. Example for adding product configuration to work using soong.
Bug: 359745733
Test: m hostapd_cli hostapd libhostapd_aidl wpa_cli wpa_supplicant \
wpa_supplicant_macsec libwpa_client libwpa_aidl libpasn
Change-Id: I62cdaa30e5b9176701a74fea2c2c8ac6eacec09f
diff --git a/hostapd/Android.bp b/hostapd/Android.bp
index 5b08d3e..533a917 100644
--- a/hostapd/Android.bp
+++ b/hostapd/Android.bp
@@ -12,6 +12,11 @@
// See the License for the specific language governing permissions and
// limitations under the License.
+// The hostapd related modules are split into 2 sections:
+// 1. For cuttlefish series products, start from `hostapd_headers` to `hostapd_cli_srcs`.
+// 2. For non-cuttlefish series products, ex: physical devices created by oems,
+// the section starts from `hostapd_driver_srcs_default` to the end of this file.
+
package {
default_applicable_licenses: [
"external_wpa_supplicant_8_license",
@@ -29,6 +34,9 @@
],
}
+// The section below is for cuttlefish series products. For non-cuttlefish
+// products please update the section starting at `hostapd_driver_srcs_default`.
+// Start of cuttlefish section.
cc_library_headers {
name: "hostapd_headers",
export_include_dirs: [
@@ -363,7 +371,29 @@
installable: false,
}
-// For converting the default to soong
+// End of cuttlefish section.
+
+// The section starting below is for non-cuttlefish products.
+// For cuttlefish series please update the section starting from `hostapd_headers`.
+
+// If you need to add a new build setting based on a product config, ex:
+// ifeq ($(BOARD_HOSTAPD_CONFIG_80211W_MFP_OPTIONAL),true)
+// L_CFLAGS += -DENABLE_HOSTAPD_CONFIG_80211W_MFP_OPTIONAL
+// endif
+
+// In order to export the Makefile variable to soong, you will need to use a `soong_config_set` method
+// under `build/core/board_config_wpa_supplicant.mk`. Ex:
+// ifeq ($(BOARD_HOSTAPD_CONFIG_80211W_MFP_OPTIONAL),true)
+// $(call soong_config_set_bool,wpa_supplicant_8,board_hostapd_config_80211w_mfp_optional,true)
+// endif
+
+// And then use the select statement in Android.bp to reflect the condition you need, ex:
+// select(soong_config_variable("wpa_supplicant_8", "board_hostapd_config_80211w_mfp_optional"), {
+// true: ["-DENABLE_HOSTAPD_CONFIG_80211W_MFP_OPTIONAL"],
+// default: [],
+// })
+
+// Start of non-cuttlefish section
cc_defaults {
name: "hostapd_driver_srcs_default",
srcs: [
@@ -711,3 +741,5 @@
"hostapd_includes_default",
],
}
+
+// End of non-cuttlefish section