The ARRAY_SIZE macro is convenient, yet mostly unused. Switch lots of

[coreboot.git] / src / superio / winbond / w83627hf / superio.c

diff --git a/src/superio/winbond/w83627hf/superio.c b/src/superio/winbond/w83627hf/superio.c
index 93c5f6dac2d3ac6d5c719a8a707f018a35433bf2..5f26768adfcda348885aadb861151b9b6a1522ee 100644 (file)
--- a/src/superio/winbond/w83627hf/superio.c
+++ b/src/superio/winbond/w83627hf/superio.c
@@ -13,6 +13,7 @@
 #include <uart8250.h>
 #include <pc80/keyboard.h>
 #include <pc80/mc146818rtc.h>
+#include <stdlib.h>
 #include "chip.h"
 #include "w83627hf.h"
 
@@ -83,7 +84,7 @@ static void init_hwm(unsigned long base)
                                                                             
        };
 
-       for(i = 0; i<  sizeof(hwm_reg_values)/sizeof(hwm_reg_values[0]); i+=3 ) { 
+       for(i = 0; i<  ARRAY_SIZE(hwm_reg_values); i+=3 ) { 
                reg = hwm_reg_values[i];        
                value = pnp_read_index(base, reg);              
                value &= 0xff & hwm_reg_values[i+1];
@@ -189,7 +190,7 @@ static struct pnp_info pnp_dev_info[] = {
 static void enable_dev(struct device *dev)
 {
        pnp_enable_devices(dev, &ops,
-               sizeof(pnp_dev_info)/sizeof(pnp_dev_info[0]), pnp_dev_info);
+               ARRAY_SIZE(pnp_dev_info), pnp_dev_info);
 }
 
 struct chip_operations superio_winbond_w83627hf_ops = {