[coreboot-gerrit] New patch to review for coreboot: 35b1790 mainboard/hp/dl145_g3/get_bus_conf.c: Use ARRAY_SIZE macro

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Mon Jul 7 11:19:53 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/6194

-gerrit

commit 35b17902f8031450425fbb3d355e2edf443ad71f
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Mon Jul 7 19:17:53 2014 +1000

    mainboard/hp/dl145_g3/get_bus_conf.c: Use ARRAY_SIZE macro
    
    Change-Id: Ie3287cba45bd6f4f8823ce03cd9983707c9ad0de
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/mainboard/hp/dl145_g3/get_bus_conf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/hp/dl145_g3/get_bus_conf.c b/src/mainboard/hp/dl145_g3/get_bus_conf.c
index cef2886..ba0fc88 100644
--- a/src/mainboard/hp/dl145_g3/get_bus_conf.c
+++ b/src/mainboard/hp/dl145_g3/get_bus_conf.c
@@ -82,7 +82,7 @@ void get_bus_conf(void)
 
 	m = sysconf.mb;
 
-	sysconf.hc_possible_num = sizeof(pci1234x)/sizeof(pci1234x[0]);
+	sysconf.hc_possible_num = ARRAY_SIZE(pci1234x);
 
 	for(i=0;i<sysconf.hc_possible_num; i++) {
 		sysconf.pci1234[i] = pci1234x[i];



More information about the coreboot-gerrit mailing list