All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felipe Balbi <me@felipebalbi.com>
To: "Woodruff, Richard" <r-woodruff2@ti.com>
Cc: andrzej zaborowski <balrogg@gmail.com>,
	"me@felipebalbi.com" <me@felipebalbi.com>,
	Carlos Aguiar <carlos.aguiar@indt.org.br>,
	Tony Lindgren <tony@atomide.com>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: [PATCH 0/4] OMAP: MENELAUS: Improvements on menelaus
Date: Tue, 29 Jul 2008 22:18:00 +0300	[thread overview]
Message-ID: <20080729191759.GE7275@frodo> (raw)
In-Reply-To: <13B9B4C6EF24D648824FF11BE89671620361D432D8@dlee02.ent.ti.com>

On Tue, Jul 29, 2008 at 02:07:51PM -0500, Woodruff, Richard wrote:
> > How about renaming it to twl92230c before submitting upstream?  As far
> > as I can tell the name menelaus appears only in linux and makes it
> > hard to associate with the real hardware.  Does someone know why was
> > it renamed that way?

> A number does seem more understandable.  Though some of these chips live under a few different numbers.

it does seem more understandable, but now that it has been used for so
long I guess it would be nasty to change right now. Almost every one
here knows that menelaus is the companion chip for omap2 hw. Personally,
I never recall the correct number designation for menelaus.

I mean, I don't really have any problems in renaming the whole driver
from menelaus to twl92230c.c but I can't help asking myself why. If
everybody thinks it's a better choice, then why not ;-)

-- 
balbi

  reply	other threads:[~2008-07-29 19:18 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-29 16:44 [PATCH 0/4] OMAP: MENELAUS: Improvements on menelaus Carlos Aguiar
2008-07-29 18:37 ` Felipe Balbi
2008-07-29 18:57   ` andrzej zaborowski
2008-07-29 19:07     ` Woodruff, Richard
2008-07-29 19:18       ` Felipe Balbi [this message]
2008-07-29 19:28         ` andrzej zaborowski
2008-07-29 19:37           ` Gadiyar, Anand
2008-07-29 21:51             ` Felipe Balbi
2008-08-05  8:57             ` Tony Lindgren
2008-08-05  9:35               ` Felipe Balbi
2008-08-05  9:42                 ` Tony Lindgren
2008-08-05 13:38                   ` Carlos Aguiar
2008-08-05 13:45                     ` Tony Lindgren
2008-08-05 12:42                 ` Trilok Soni

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=20080729191759.GE7275@frodo \
    --to=me@felipebalbi.com \
    --cc=balrogg@gmail.com \
    --cc=carlos.aguiar@indt.org.br \
    --cc=linux-omap@vger.kernel.org \
    --cc=r-woodruff2@ti.com \
    --cc=tony@atomide.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.