modules/bash: enabled by default, disabled in legacy-flash boards

- legacy-flash boards have a single purpose: to flash BIOS region through flashrom.
  - They do not need bash nor have space for it in their 4mb defined coreboot CBFS region

Test build to have legacy boards builds under osresearch#1292

Makefile

@@ -52,6 +52,9 @@ include $(CONFIG)
 # Unless otherwise specified, we are building for heads
 CONFIG_HEADS	?= y
 
+# Unless otherwise specified, we are building bash to have non-interactive shell for scripts (arrays and bashisms)
+CONFIG_BASH	?= y
+
 # Determine arch part for a host triplet
 ifeq "$(CONFIG_TARGET_ARCH)" "x86"
 MUSL_ARCH := x86_64
@@ -468,7 +471,6 @@ bin_modules-$(CONFIG_MSRTOOLS) += msrtools
 bin_modules-$(CONFIG_NKSTORECLI) += nkstorecli
 bin_modules-$(CONFIG_OPENSSL) += openssl
 bin_modules-$(CONFIG_TPM2_TOOLS) += tpm2-tools
-bin_modules-y += bash
 
 $(foreach m, $(bin_modules-y), \
 	$(call map,initrd_bin_add,$(call bins,$m)) \

boards/t430-legacy-flash/t430-legacy-flash.config

@@ -4,6 +4,7 @@ export CONFIG_COREBOOT=y
 export CONFIG_COREBOOT_VERSION=4.13
 export CONFIG_LINUX_VERSION=4.14.62
 
+CONFIG_BASH=n
 CONFIG_FLASHROM=y
 #CONFIG_GPG=y
 #CONFIG_FLASHTOOLS=y

boards/x230-legacy-flash/x230-legacy-flash.config

@@ -4,6 +4,7 @@ export CONFIG_COREBOOT=y
 export CONFIG_COREBOOT_VERSION=4.13
 export CONFIG_LINUX_VERSION=4.14.62
 
+CONFIG_BASH=n
 CONFIG_FLASHROM=y
 #CONFIG_GPG=y
 #CONFIG_FLASHTOOLS=y

modules/bash

@@ -1,5 +1,5 @@
 # GNU bash
-modules-y += bash
+modules-$(CONFIG_BASH) += bash
 
 bash_version := 5.1.16
 bash_dir := bash-$(bash_version)