linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: swetland@google.com (Brian Swetland)
To: linux-arm-kernel@lists.infradead.org
Subject: board/device file names, and machine names
Date: Tue, 2 Mar 2010 14:56:33 -0800	[thread overview]
Message-ID: <a55d774e1003021456id57a19eud4ca5752ba91d72d@mail.gmail.com> (raw)
In-Reply-To: <1267565398.8759.77.camel@c-dwalke-linux.qualcomm.com>

On Tue, Mar 2, 2010 at 1:29 PM, Daniel Walker <dwalker@codeaurora.org> wrote:
>
> Hi,
>
> I was wondering if you (or the list) had any suggestions for proper
> names to use in these two cases. Under mach-msm we have tons of devices
> that could be added, and it can sometimes be hard to isolate a sane
> name ..
>
> I'll use the Nexus One device as an example .. HTC originally called
> this device the HTC Passion prior to release.. Google named the same
> device "mahimahi" prior to release .. Then finally it was released as
> Nexus One .
>
> So one device has at least three names (more I'm sure),
>
> ? ? ? ?Passion
> ? ? ? ?Mahimahi
> ? ? ? ?Nexus One
>
> Google has most of the code support under board files with the name
> mahimahi.
>
> To me it makes a lot more sense to use a mass marketed name like "Nexus
> One" .. Few people know what an HTC Passion is, and extremely few know
> what "mahimahi" is.

We would, of course, prefer to keep the board named mahimahi for all
the reasons that have been mentioned in various previous discussions
around trout, etc:
1. This was the name used during development for the platform.
2. This is the name the bootloader uses and the production bootloader
passes module parameters, etc under this name
3. This is the name the production userspace build for these phones
uses to identify the specific hardware and locate some features.
4. The Kconfig description provides a reasonable place to put more
expansive description of the various enduser visible product names.
5. My understanding has always been that MACH_* identifies a
particular board which may be instantiated in a number of products
6. I'm still unconvinced that a machine name like "mahimahi" is any
more or less confusing than any other common machine name for an ARM
board, which tend to be codenames, strange alphanumeric designators,
or combinations of the two.

This seems like an unforunate issue to bikeshed about, and by
insisting on renaming the board names, more hurdles are put up between
the (often competing, at least for our time) goals of "going to
mainline" and "maintaining an up to date tree that works on production
hardware without regression".

Thanks,

Brian


>
> Here is a link to mach-msm from Google's tree so you can see the naming
> they use,
>
> http://android.git.kernel.org/?p=kernel/msm.git;a=tree;f=arch/arm/mach-msm;h=f67bef98c885df15781d3122e3325f6164884118;hb=refs/heads/android-msm-2.6.29-nexusone
>
> Another example of this is,
>
> arch/arm/mach-msm/board-trout.c
>
> which in mainline we renamed to,
>
> arch/arm/mach-msm/board-dream.c
>
> Dream is actually a released name, so it seems to make more sense.
> However, there are many other names we could have used.
>
> machine_is_xx names have a similar kind of problem .. However, it's lot
> easier to mainline a machine_is_xxx name ..
>
> Any comments welcome, since this stuff is disputed and up in the air.
> I'm sure many other projects have faced similar problems also.
>
> Daniel
>

  reply	other threads:[~2010-03-02 22:56 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-02 21:29 board/device file names, and machine names Daniel Walker
2010-03-02 22:56 ` Brian Swetland [this message]
2010-03-02 23:29   ` Daniel Walker
2010-03-02 23:38     ` Brian Swetland
2010-03-03  3:42     ` Bill Gatliff
2010-03-03  3:36   ` Bill Gatliff
2010-03-03  8:00     ` Brian Swetland
2010-03-03 19:08   ` Theodore Tso
2010-03-03 19:22     ` Theodore Tso
2010-03-03 19:24     ` Russell King - ARM Linux
2010-03-02 23:51 ` Russell King - ARM Linux
2010-03-03  0:39   ` Daniel Walker
2010-03-03  0:47     ` Tim Bird
2010-03-03  0:52       ` Daniel Walker
2010-03-03  1:01         ` Brian Swetland
2010-03-03  3:52     ` Bill Gatliff
2010-03-03  8:42     ` Russell King - ARM Linux
2010-03-03  9:17       ` Pavel Machek
2010-03-03  9:47         ` Russell King - ARM Linux
2010-03-03 10:00           ` Pavel Machek
2010-03-03 10:08             ` Russell King - ARM Linux
2010-03-03 10:18               ` Pavel Machek
2010-03-03 10:19                 ` Russell King - ARM Linux
2010-03-03 14:24                   ` Pavel Machek
2010-03-03 14:38       ` Daniel Walker
2010-03-03 22:08   ` Nicolas Pitre
2010-03-03 22:27     ` Brian Swetland
2010-03-03  3:25 ` Bill Gatliff

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=a55d774e1003021456id57a19eud4ca5752ba91d72d@mail.gmail.com \
    --to=swetland@google.com \
    --cc=linux-arm-kernel@lists.infradead.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).