Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 15:24 Tue 20 May     , Andy Fleming wrote:
>> ALIGN() returns the smallest aligned value greater than the passed
>> in address or size.  Taken from Linux.
>>
>> Signed-off-by: Andy Fleming <[EMAIL PROTECTED]>
>> ---
>>  include/common.h |    3 +++
>>  1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/common.h b/include/common.h
>> index d0f5704..68e0cbc 100644
>> --- a/include/common.h
>> +++ b/include/common.h
>> @@ -671,6 +671,9 @@ void __attribute__((weak)) show_boot_progress (int val);
>>  #define DIV_ROUND_UP(n,d) (((n) + (d) - 1) / (d))
>>  #define roundup(x, y) ((((x) + ((y) - 1)) / (y)) * (y))
>>  
>> +#define ALIGN(x,a)              __ALIGN_MASK(x,(typeof(x))(a)-1)
>> +#define __ALIGN_MASK(x,mask)    (((x)+(mask))&~(mask))
> please fix coding style and use tab instead of space for indent
> 
> #define ALIGN(x, a)           __ALIGN_MASK(x, (typeof(x)) (a) - 1)
> #define __ALIGN_MASK(x, mask) (((x) + (mask)) & ~(mask))

That's not indenting, that's alignment.  Using tabs after anything that 
is not a tab causes the code to look awful if the tab stops are 
disturbed, such as by viewing with a different tab size, or as part of a 
patch.

-Scott

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to