Re: MPC5200 Patches

On Wed, 12 Nov 2003, Gabriel Paubert wrote:

On Wed, Nov 12, 2003 at 10:59:59AM +0100, Geert Uytterhoeven wrote:

quoted

On Wed, 12 Nov 2003, Wolfgang Denk wrote:

quoted

In message [ref] you wrote:

quoted

quoted

  We have drivers/i2c/i2c-algo-8xx.c, so maybe we should have
  i2c-algo-5xxx.c instead of i2c-algo-m5xxx.c ?

  There is include/linux/i2c-algo-8xx.h - how about i2c-algo-5xxx.h
  instead of i2c-algo-m5xxx.h ?

...

quoted

What's done in include/asm-ppc/ isn't that important for the
outside (non-PPC) world, but i2c-algo-5xxx.c may be a too
generic name...

Which name do you recommend to cover the MGT5100 and MPC5200 (and
probably other MPC5xxx) processors?

i2c-algo-m5xxx.c, like the original?

Hmmm, dont't repeated x have a tendency to trigger spam filters
(especially the dumbest ones)?

Why not 5nnn or 5xyz?

i2c-algo-m5x00.c? (or i2c-algo-m5<censored>.c :-)

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

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/