[coreboot-gerrit] New patch to review for coreboot: a6b2784 superio/fintek/f71869ad: Make SIO HWM conf conditional

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Sat May 24 06:45:15 CEST 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5841

-gerrit

commit a6b2784ec20cfc631f141c2e43854183e29152a9
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sat May 24 14:43:03 2014 +1000

    superio/fintek/f71869ad: Make SIO HWM conf conditional
    
    Not everyone may like to enable the Super I/O HWM on their board or for
    their purposes. Hence, make hwm configuration conditional in Kconfig.
    
    Change-Id: I8f75bda30a1df4d6f33b73f3d0e6dd2e5dc9963f
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/mainboard/jetway/nf81-t56n-lf/Kconfig |  1 +
 src/superio/fintek/Kconfig                | 20 ++++++++++++++++++++
 src/superio/fintek/f71869ad/Makefile.inc  |  4 ++--
 src/superio/fintek/f71869ad/chip.h        |  2 ++
 src/superio/fintek/f71869ad/superio.c     |  2 ++
 5 files changed, 27 insertions(+), 2 deletions(-)

diff --git a/src/mainboard/jetway/nf81-t56n-lf/Kconfig b/src/mainboard/jetway/nf81-t56n-lf/Kconfig
index d90e7a1..bcb95f6 100644
--- a/src/mainboard/jetway/nf81-t56n-lf/Kconfig
+++ b/src/mainboard/jetway/nf81-t56n-lf/Kconfig
@@ -26,6 +26,7 @@ config BOARD_SPECIFIC_OPTIONS # dummy
 	select NORTHBRIDGE_AMD_AGESA_FAMILY14
 	select SOUTHBRIDGE_AMD_CIMX_SB800
 	select SUPERIO_FINTEK_F71869AD
+	select SUPERIO_FINTEK_F71869AD_HWM
 	select HAVE_OPTION_TABLE
 	select HAVE_PIRQ_TABLE
 	select HAVE_MP_TABLE
diff --git a/src/superio/fintek/Kconfig b/src/superio/fintek/Kconfig
index f577898..43f4b4d 100644
--- a/src/superio/fintek/Kconfig
+++ b/src/superio/fintek/Kconfig
@@ -39,6 +39,26 @@ config SUPERIO_FINTEK_F71869AD
 	bool
 	select SUPERIO_FINTEK_COMMON_ROMSTAGE
 
+choice
+	prompt "Super I/O Hardware Monitor"
+	default SUPERIO_NO_HWM
+	help
+	  Select the method of hardware monitor fan control to be used.
+
+config SUPERIO_NO_HWM
+	bool "None"
+	help
+	  No Super I/O Fan control - Do not set up the HWM fan control registers.
+
+config SUPERIO_FINTEK_F71869AD_HWM
+	bool "Fintek F71869AD HWM"
+	depends on SUPERIO_FINTEK_F71869AD
+	help
+	  Configure the Fintek Super I/O hardware monitor fan control registers in
+	  devicetree.cb.
+
+endchoice
+
 config SUPERIO_FINTEK_F71872
 	bool
 	select SUPERIO_FINTEK_COMMON_ROMSTAGE
diff --git a/src/superio/fintek/f71869ad/Makefile.inc b/src/superio/fintek/f71869ad/Makefile.inc
index 1d8cc6a..4de156d 100644
--- a/src/superio/fintek/f71869ad/Makefile.inc
+++ b/src/superio/fintek/f71869ad/Makefile.inc
@@ -18,6 +18,6 @@
 ## Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
 ##
 
-ramstage-$(CONFIG_SUPERIO_FINTEK_F71869AD) += f71869ad_multifunc.c
-ramstage-$(CONFIG_SUPERIO_FINTEK_F71869AD) += f71869ad_hwm.c
+ramstage-$(CONFIG_SUPERIO_FINTEK_F71869AD_HWM) += f71869ad_multifunc.c
+ramstage-$(CONFIG_SUPERIO_FINTEK_F71869AD_HWM) += f71869ad_hwm.c
 ramstage-$(CONFIG_SUPERIO_FINTEK_F71869AD) += superio.c
diff --git a/src/superio/fintek/f71869ad/chip.h b/src/superio/fintek/f71869ad/chip.h
index e12e51d..4c6c5d7 100644
--- a/src/superio/fintek/f71869ad/chip.h
+++ b/src/superio/fintek/f71869ad/chip.h
@@ -24,6 +24,7 @@
 #include <stdint.h>
 
 struct superio_fintek_f71869ad_config {
+#ifdef CONFIG_SUPERIO_FINTEK_F71869AD_HWM
 	/* Member variables are defined in devicetree.cb. */
 	uint8_t multi_function_register_1;
 	uint8_t multi_function_register_2;
@@ -41,6 +42,7 @@ struct superio_fintek_f71869ad_config {
 	uint8_t hwm_fan1_seg2_speed_count;
 	uint8_t hwm_fan1_seg3_speed_count;
 	uint8_t hwm_fan1_temp_map_sel;
+#endif /* CONFIG_SUPERIO_FINTEK_F71869AD_HWM */
 };
 
 #endif /* SUPERIO_FINTEK_F71869AD_CHIP_H */
diff --git a/src/superio/fintek/f71869ad/superio.c b/src/superio/fintek/f71869ad/superio.c
index 2664f99..82ad9e0 100644
--- a/src/superio/fintek/f71869ad/superio.c
+++ b/src/superio/fintek/f71869ad/superio.c
@@ -40,10 +40,12 @@ static void f71869ad_init(device_t dev)
 	case F71869AD_KBC:
 		pc_keyboard_init();
 		break;
+#ifdef CONFIG_SUPERIO_FINTEK_F71869AD_HWM
 	case F71869AD_HWM:
 		f71869ad_multifunc_init(dev);
 		f71869ad_hwm_init(dev);
 		break;
+#endif /* CONFIG_SUPERIO_FINTEK_F71869AD_HWM */
 	}
 }
 



More information about the coreboot-gerrit mailing list