All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Cc: hch@lst.de, vagabon.xyz@gmail.com, linux-mips@linux-mips.org,
	ralf@linux-mips.org
Subject: Re: [PATCH] generic clk API implementation for MIPS
Date: Thu, 28 Jun 2007 10:37:25 +0200	[thread overview]
Message-ID: <20070628083725.GA23394@lst.de> (raw)
In-Reply-To: <20070628.112223.96686654.nemoto@toshiba-tops.co.jp>

On Thu, Jun 28, 2007 at 11:22:23AM +0900, Atsushi Nemoto wrote:
> On Wed, 27 Jun 2007 17:39:32 +0200, Christoph Hellwig <hch@lst.de> wrote:
> > > This MIPS implementation is derived (and stripped) from the SH
> > > implementation.
> > 
> > Why is this not in architecture-independent code?
> 
> Yes, this is architecture independent.  If we could have consensus on
> a generic (or least common) implementation, we can put it outside arch
> directory.
> 
> But I gave up for now ;) I will leave all implementation for platform
> code.

I really dislike duplicating thing over architectures.  If you copy code
from another architecture the first though should be 'could and should
this be generic ?'.  So please try to get this lifted to common code
instead of duplicating it.

> 
> ---
> Atsushi Nemoto
---end quoted text---

  reply	other threads:[~2007-06-28  8:37 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-25 16:14 [PATCH] generic clk API implementation for MIPS Atsushi Nemoto
2007-06-26  9:37 ` Franck Bui-Huu
2007-06-26 14:33   ` Atsushi Nemoto
2007-06-26 15:20     ` Franck Bui-Huu
2007-06-26 16:33       ` Atsushi Nemoto
2007-06-26 20:02         ` Franck Bui-Huu
2007-06-27  7:51           ` Atsushi Nemoto
2007-06-27 15:39         ` Christoph Hellwig
2007-06-28  2:22           ` Atsushi Nemoto
2007-06-28  8:37             ` Christoph Hellwig [this message]
2007-06-28  9:26               ` Atsushi Nemoto
2007-06-28 20:45               ` gdbserver Ratin Rahman (mratin)
2007-06-28 20:45                 ` gdbserver Ratin Rahman (mratin)
2007-06-28 21:41                 ` gdbserver David Daney
2007-06-28 22:19                   ` gdbserver Ratin Rahman (mratin)
2007-06-28 22:19                     ` gdbserver Ratin Rahman (mratin)
2007-06-29  1:26                   ` upgrading glibc for mipsel Ratin Rahman (mratin)
2007-06-29  1:26                     ` Ratin Rahman (mratin)
2007-06-29  2:03                     ` David Daney
2007-06-29  2:26                       ` Jason Talley

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=20070628083725.GA23394@lst.de \
    --to=hch@lst.de \
    --cc=anemo@mba.ocn.ne.jp \
    --cc=linux-mips@linux-mips.org \
    --cc=ralf@linux-mips.org \
    --cc=vagabon.xyz@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.