[PATCH v2] ARM: asm: rename logical shift macros push pull into lspush lspull

Nicolas Pitre nicolas.pitre at linaro.org
Tue Feb 11 12:09:35 EST 2014


On Tue, 11 Feb 2014, Dave Martin wrote:

> On Mon, Feb 10, 2014 at 04:30:01PM -0500, Nicolas Pitre wrote:
> > On Mon, 10 Feb 2014, Victor Kamensky wrote:
> > 
> > > Renames logical shift macros, 'push' and 'pull', defined in
> > > arch/arm/include/asm/assembler.h, into 'lspush' and 'lspull'.
> > 
> > I don't have any fundamental objection to the idea, except maybe for the 
> > actual names.  I just can't come up with anything better though.  
> 
> For consistency with the get_byte_ stuff, how about:
> 
> 	push -> towards_byte_0
> 	pull -> from_byte_0
> 
> That may make the purpose a little clearer, too.

I don't know if

	mov	r0, r1, from_byte_0 #8

is that much clearer though.

> (Assuming I've got them the right way around...)

As you later noticed you got it wrong.  :-)
Most likely because "full from" and "push towards" are common english 
constructs.


Nicolas



More information about the linux-arm-kernel mailing list