Re: [PATCH 2/2] asm-generic: Don't warn that atomic_t is only 24 bit

From: Peter Fritzsche
Date: Fri Apr 30 2010 - 14:00:24 EST


Mathieu Desnoyers wrote:
> * David Howells (dhowells@xxxxxxxxxx) wrote:
> > Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxxxx> wrote:
> > > How about simply replacing arch/mn10300/include/asm/atomic.h by:
> > >
> > > arch/mn10300/include/asm/atomic.h:
> > > #include <asm-generic/atomic.h>
> > >
> > > ?
> > >
> > > This should have been done a while ago. They are almost the same,
> > > except for added cmpxchg*_local() in the asm-generic version (which
> > > would benefit to mn10300).
> >
> > Fine by me.
>
> Then I'll let Peter Fritzsche deal with that. I don't want to conflict with
> his current patches.

No, please go for your solution. It is the right thing to do and definitely
better then my "remove some comments" patch. David Howells as MN10300
maintainer already agreed to your suggestion. So I would like to recall my
patch https://patchwork.kernel.org/patch/95651/ and let you submit a patch
with your idea to the MN10300 maintainers which supersede my patch.

The state of https://patchwork.kernel.org/patch/95652/ hasn't been changed and
I would love to see it merged by Arnd Bergmann in the next merge window.

Kind regards,
Peter Fritzsche
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/