Hi, On 14 November 2014 at 18:18, Simon Glass <s...@chromium.org> wrote: > We have an ALIGN() macro - add a comment as to what this does. Also add a > new ALIGN_DOWN() macro, which aligns a value to the next smallest multiple. > > Signed-off-by: Simon Glass <s...@chromium.org> > --- > > Changes in v2: None > > include/common.h | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/include/common.h b/include/common.h > index ecf7fca..28fba79 100644 > --- a/include/common.h > +++ b/include/common.h > @@ -970,9 +970,24 @@ static inline phys_addr_t map_to_sysmem(const void *ptr) > } \ > ) > > +/** > + * ALIGN() - return an aligned value not less than the given value > + * > + * @x: Value to align > + * @a: Alignment to use, e.g. 4 to align to multiples of 4 > + */ > #define ALIGN(x,a) __ALIGN_MASK((x),(typeof(x))(a)-1) > + > #define __ALIGN_MASK(x,mask) (((x)+(mask))&~(mask)) > > +/** > + * ALIGN_DOWN() - return an aligned value not greater than the given value > + * > + * @x: Value to align > + * @a: Alignment to use, e.g. 4 to align to multiples of 4 > + */ > +#define ALIGN_DOWN(x, a) ((x) & ~((typeof(x))(a) - 1UL)) > + > /* > * ARCH_DMA_MINALIGN is defined in asm/cache.h for each architecture. It > * is used to align DMA buffers.
I'm going to drop this patch, since Masahiro's clean-up has been applied. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
