linux-arch.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Peter Fritzsche <peter.fritzsche@gmx.de>
To: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Cc: David Howells <dhowells@redhat.com>,
	Jamie Lokier <jamie@shareable.org>, Arnd Bergmann <arnd@arndb.de>,
	Ingo Molnar <mingo@elte.hu>,
	Andrew Morton <akpm@linux-foundation.org>,
	Heiko Carstens <heiko.carstens@de.ibm.com>,
	linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org,
	Keith M Wesolowski <wesolows@foobazco.org>,
	Martin Schwidefsky <schwidefsky@de.ibm.com>
Subject: Re: [PATCH 2/2] asm-generic: Don't warn that atomic_t is only 24 bit
Date: Fri, 30 Apr 2010 10:53:08 +0200	[thread overview]
Message-ID: <201004301053.09003.peter.fritzsche@gmx.de> (raw)
In-Reply-To: <20100429211035.GA16179@Krystal>

Mathieu Desnoyers wrote:
> * David Howells (dhowells@redhat.com) wrote:
> > Mathieu Desnoyers <mathieu.desnoyers@efficios.com> 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

  parent reply	other threads:[~2010-04-30 17:59 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1272451837-17961-1-git-send-email-peter.fritzsche@gmx.de>
2010-04-28 11:10 ` [PATCH 2/2] asm-generic: Don't warn that atomic_t is only 24 bit Peter Fritzsche
2010-04-28 11:10   ` Peter Fritzsche
2010-04-28 13:33   ` Arnd Bergmann
2010-04-28 13:54     ` Mathieu Desnoyers
2010-04-28 13:54       ` Mathieu Desnoyers
2010-04-28 15:28       ` Jamie Lokier
2010-04-28 16:27         ` Mathieu Desnoyers
2010-04-28 18:42         ` Peter Fritzsche
2010-04-28 18:42           ` Peter Fritzsche
2010-04-29 16:22         ` David Howells
2010-04-29 21:10           ` Mathieu Desnoyers
2010-04-29 21:10             ` Mathieu Desnoyers
2010-04-30  8:53             ` Peter Fritzsche [this message]
2010-04-28 16:46   ` David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201004301053.09003.peter.fritzsche@gmx.de \
    --to=peter.fritzsche@gmx.de \
    --cc=akpm@linux-foundation.org \
    --cc=arnd@arndb.de \
    --cc=dhowells@redhat.com \
    --cc=heiko.carstens@de.ibm.com \
    --cc=jamie@shareable.org \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mathieu.desnoyers@efficios.com \
    --cc=mingo@elte.hu \
    --cc=schwidefsky@de.ibm.com \
    --cc=wesolows@foobazco.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).