[coreboot] New patch to review for coreboot: 1de5f93 nvramtool: Remove the building warning on older gcc

Zheng Bao (zheng.bao@amd.com) gerrit at coreboot.org
Tue Sep 11 04:18:28 CEST 2012

Zheng Bao (zheng.bao at amd.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1501


commit 1de5f9391147b465bb2d40f9e62466d76b3dc036
Author: Zheng Bao <fishbaozi at gmail.com>
Date:   Tue Sep 11 12:13:39 2012 +0800

    nvramtool: Remove the building warning on older gcc
    Some older gcc requires the default entry in switch, otherwise
    build warning "enumeration value not handled in switch" will come
    Change-Id: Ic8ea9960e4aca599e0ea62ec345122c9df57e766
    Signed-off-by: Zheng Bao <zheng.bao at amd.com>
    Signed-off-by: Zheng Bao <fishbaozi at gmail.com>
 util/nvramtool/cmos_lowlevel.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/util/nvramtool/cmos_lowlevel.c b/util/nvramtool/cmos_lowlevel.c
index 55b1879..ef0c383 100644
--- a/util/nvramtool/cmos_lowlevel.c
+++ b/util/nvramtool/cmos_lowlevel.c
@@ -55,6 +55,7 @@ void select_hal(hal_t hal, void *data)
 		case HAL_MEMORY:
+		default:
 			current_access = &memory_hal;

More information about the coreboot mailing list