Re: [PATCHv4 6/6] byteorder: add copy_{endian} helpers

[Geert Uytterhoeven <geert@linux-m68k.org>]

On Wed, 4 Jun 2008, Harvey Harrison wrote:
> On Mon, 2008-06-02 at 21:15 +0200, Geert Uytterhoeven wrote:
> > On Mon, 2 Jun 2008, Harvey Harrison wrote:
> > > Add helpers for the idiom:
> > > *(__le16 *)ptr = cpu_to_le16(val);
> > > 
> > > Can now be written as:
> > > copy_le16(ptr, val);
> > 
> > Copy a little endian 16-bit value?
> > 
> > It doesn't just copy. It does a byteswap during the copy.
> 
> True, I had originally called this put_le16.  akpm suggested copy_le16.
> What do people think of copy_as_le16?

Sounds still a bit strange to me...

As `get' and `put' have connotations of reference counts, what about
`load' and `store', e.g. `load_le32()' and `store_be16()'?

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds