Subject: Re: [PATCH 7/16] powerpc: add support for ps3 platform

From: Michael Ellerman <[email-masked]>
Date: Fri, 17 Nov 2006 12:42:14 +1100

On Thu, 2006-11-16 at 17:33 -0800, Geoff Levand wrote:
> Benjamin Herrenschmidt wrote:
> >> > +
> >> > +enum page_size {
> >> > +	page_size_4k = 12U,
> >> > +	page_size_64k = 16U,
> >> > +	page_size_16m = 24U,
> >> > +};
> >> 
> >> Please use ALL_CAPS for such constants.
> > 
> > And call them PAGE_SHIFT_* while at it :-)
> 
> The HV docs call this page size, so that is what I used for
> the symbol.  I'll look at the usage again and reconsider.

No need to look at it, it's a shift not a size, the page size is not 12
bytes. Keeping it consistent with the HV docs is a concern, but so is
making it consistent with every other part of the kernel - which calls
this a shift, see page.h f.e.

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person