On Wed, Sep 08, 2010 at 10:11:13PM +0100, Nikitas Angelinas wrote: > Replace sizeof(pfnnid_map) / sizeof(pfnnid_map[0]) with > ARRAY_SIZE(pfnnid_map) in mmzone.h > Thanks. Let me think about this, I don't really like the tangled web of include dependencies this results in... regards, Kyle > Signed-off-by: Nikitas Angelinas <nikitasangelinas@xxxxxxxxx> > --- > arch/parisc/include/asm/mmzone.h | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/arch/parisc/include/asm/mmzone.h b/arch/parisc/include/asm/mmzone.h > index 9608d2c..85b23b6 100644 > --- a/arch/parisc/include/asm/mmzone.h > +++ b/arch/parisc/include/asm/mmzone.h > @@ -3,6 +3,8 @@ > > #ifdef CONFIG_DISCONTIGMEM > > +#include <linux/kernel.h> > + > #define MAX_PHYSMEM_RANGES 8 /* Fix the size for now (current known max is 3) */ > extern int npmem_ranges; > > @@ -51,7 +53,7 @@ static inline int pfn_to_nid(unsigned long pfn) > return 0; > > i = pfn >> PFNNID_SHIFT; > - BUG_ON(i >= sizeof(pfnnid_map) / sizeof(pfnnid_map[0])); > + BUG_ON(i >= ARRAY_SIZE(pfnnid_map)); > r = pfnnid_map[i]; > BUG_ON(r == 0xff); > > -- > 1.7.2.3 > > -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
