public inbox for linux-omap@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox