Skip to content

Commit

Permalink
MIPS: Use ALIGN(x, bytes) instead of __ALIGN_MASK(x, bytes - 1)
Browse files Browse the repository at this point in the history
ALIGN(x, bytes) expands to __ALIGN_MASK(x, bytes - 1), so use the one
that is most clear.

Signed-off-by: Matt Turner <mattst88@gmail.com>
To: linux-mips@linux-mips.org
Cc: David Daney <ddaney@caviumnetworks.com>
Patchwork: http://patchwork.linux-mips.org/patch/999/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
  • Loading branch information
Matt Turner authored and Ralf Baechle committed Feb 27, 2010
1 parent bba9076 commit 2a5d665
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions arch/mips/cavium-octeon/executive/cvmx-bootmem.c
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@ int64_t cvmx_bootmem_phy_alloc(uint64_t req_size, uint64_t address_min,
* impossible requests up front. (NOP for address_min == 0)
*/
if (alignment)
address_min = __ALIGN_MASK(address_min, (alignment - 1));
address_min = ALIGN(address_min, alignment);

/*
* Reject inconsistent args. We have adjusted these, so this
Expand Down Expand Up @@ -291,7 +291,7 @@ int64_t cvmx_bootmem_phy_alloc(uint64_t req_size, uint64_t address_min,
* satisfy request.
*/
usable_base =
__ALIGN_MASK(max(address_min, ent_addr), alignment - 1);
ALIGN(max(address_min, ent_addr), alignment);
usable_max = min(address_max, ent_addr + ent_size);
/*
* We should be able to allocate block at address
Expand Down Expand Up @@ -671,7 +671,7 @@ int64_t cvmx_bootmem_phy_named_block_alloc(uint64_t size, uint64_t min_addr,
* coallesced when they are freed. The alloc routine does the
* same rounding up on all allocations.
*/
size = __ALIGN_MASK(size, (CVMX_BOOTMEM_ALIGNMENT_SIZE - 1));
size = ALIGN(size, CVMX_BOOTMEM_ALIGNMENT_SIZE);

addr_allocated = cvmx_bootmem_phy_alloc(size, min_addr, max_addr,
alignment,
Expand Down

0 comments on commit 2a5d665

Please sign in to comment.