From: eric.y.miao@gmail.com (Eric Miao)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 08/25] imx/mx1: rename files defining a machine to mach-$mach.c
Date: Thu, 4 Feb 2010 04:25:03 -0800 [thread overview]
Message-ID: <f17812d71002040425o6690b9aej4bea9046457780@mail.gmail.com> (raw)
In-Reply-To: <20100204120622.GB20282@sirena.org.uk>
On Thu, Feb 4, 2010 at 4:06 AM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> On Wed, Feb 03, 2010 at 05:29:16PM -0800, Amit Kucheria wrote:
>> On 10 Jan 08, Uwe Kleine-K??nig wrote:
>
>> > ?rename arch/arm/mach-mx1/{mx1ads.c => mach-mx1ads.c} (98%)
>> > ?rename arch/arm/mach-mx1/{scb9328.c => mach-scb9328.c} (98%)
>
>> Rethinking about this (based on a similar request for my i.MX5 submission), I
>> think renaming the boards to mach-* is a bad idea.
>
>> Other SoCs (davinci, omap, msm, nomadik, etc.) use the board-* convention. We
>> already use mach-* for the SoC directory names under arch/arm/. Naming the
>> boards with mach-* will just cause more confusion.
>
> Some other socs (at least the Samsung ones) are using mach- :/
Well, is the naming scheme so important in such a sense? Yes, now
git has made renaming quite a painless hassle, however, that's really
some needless thing IMHO.
PXA on the other way, doesn't come up with any board-* or mach-*
suffix, and no one seems complaining so far.
Orion and IXP, have <board-name>-setup.c, all looks fine to me as
long as they are consistent.
OTOH, sometimes there are core modules and base board (i.e.
motherboard), do we have to go to the extend to something like,
coreboard-* and motherboard-*. I doubt that.
So in a word, as long as it's consistent all around.
next prev parent reply other threads:[~2010-02-04 12:25 UTC|newest]
Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-16 18:52 [PATCH 00/20] imx cleanup series Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 01/20] ARM: MX3: make CPU revision number detection work on all boards Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 02/20] imx/mx2: fold crm_regs.h into its only consumer Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 03/20] imx/clock-imx21: use a macro to define registers Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 04/20] imx/clock-imx27: " Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 05/20] imx/mach-mx2: use constants namespaced by the corresponding SOC (easy part) Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 06/20] imx/mach-mx3: " Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 07/20] imx/mx3: rename clock.c and iomux.c to {clock, iomux}-imx31.c Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 08/20] imx/mx1: add newline at end of Makefile Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 09/20] imx/mx1: rename files defining a machine to mach-$mach.c Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 10/20] imx: rename Kconfig symbol for "Eukrea CPUIMX27 module" Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 11/20] imx: rename Kconfig symbol for "LogicPD MX27 LITEKIT platform" Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 12/20] imx/mx2: rename files defining a machine to mach-$mach.c Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 13/20] imx/mx3: " Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 14/20] imx: provide helper macro to define IO_ADDRESS Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 15/20] imx21: define and use MX21_IO_ADDRESS Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 16/20] imx27: define and use MX27_IO_ADDRESS Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 17/20] imx31: define and use MX31_IO_ADDRESS Uwe Kleine-König
2009-12-16 18:59 ` [PATCH 18/20] imx25: redefine MX25_IO_ADDRESS using IMX_IO_ADDRESS Uwe Kleine-König
2009-12-16 19:00 ` [PATCH 19/20] imx35: define and use MX35_IO_ADDRESS Uwe Kleine-König
2009-12-16 19:00 ` [PATCH 20/20] mx3/kzm_arm11_01: define and use board specific IO_ADDRESS macro Uwe Kleine-König
2009-12-17 7:06 ` [PATCH 16/20] imx27: define and use MX27_IO_ADDRESS Lothar Waßmann
2009-12-17 8:28 ` Sascha Hauer
2009-12-17 10:05 ` Uwe Kleine-König
2009-12-17 11:49 ` Lothar Waßmann
2009-12-17 13:17 ` Uwe Kleine-König
2009-12-16 20:58 ` [PATCH 09/20] imx/mx1: rename files defining a machine to mach-$mach.c Guennadi Liakhovetski
2009-12-17 9:32 ` Uwe Kleine-König
2009-12-16 20:48 ` [PATCH 06/20] imx/mach-mx3: use constants namespaced by the corresponding SOC (easy part) Guennadi Liakhovetski
2009-12-17 7:16 ` [PATCH 04/20] imx/clock-imx27: use a macro to define registers Lothar Waßmann
2009-12-17 9:58 ` Uwe Kleine-König
2010-01-08 16:15 ` [PATCH 00/20] imx cleanup series Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 01/25] imx/mx2: fold crm_regs.h into its only consumer Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 02/25] imx/clock-imx21: use a macro to define registers Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 03/25] imx/clock-imx27: " Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 04/25] imx/mach-mx2: use constants namespaced by the corresponding SOC (easy part) Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 05/25] imx/mach-mx3: " Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 06/25] imx/mx3: rename clock.c and iomux.c to {clock, iomux}-imx31.c Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 07/25] imx/mx1: add newline at end of Makefile Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 08/25] imx/mx1: rename files defining a machine to mach-$mach.c Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 09/25] imx: rename Kconfig symbol for "Eukrea CPUIMX27 module" Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 10/25] imx: rename Kconfig symbol for "LogicPD MX27 LITEKIT platform" Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 11/25] imx/mx2: rename files defining a machine to mach-$mach.c Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 12/25] imx/mx3: " Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 13/25] imx: provide helper macro to define IO_ADDRESS Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 14/25] imx21: define and use MX21_IO_ADDRESS Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 15/25] imx27: define and use MX27_IO_ADDRESS Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 16/25] imx31: define and use MX31_IO_ADDRESS Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 17/25] imx25: redefine MX25_IO_ADDRESS using IMX_IO_ADDRESS Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 18/25] imx35: define and use MX35_IO_ADDRESS Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 19/25] mx3/kzm_arm11_01: define and use board specific IO_ADDRESS macro Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 20/25] imx: mangle addresses after adding the offset Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 21/25] imx: only define deprecated symbols conditionally Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 22/25] imx: remove mx1ads defconfig Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 23/25] imx: remove unneeded include of mach/hardware.h Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 24/25] imx: properly protect mach/mx{1, [25][157x]}.h from multiple inclusion Uwe Kleine-König
2010-01-08 16:17 ` [PATCH 25/25] imx: define functions to configure chip selects in the WEIM Uwe Kleine-König
2010-02-02 13:43 ` [PATCH 21/25] imx: only define deprecated symbols conditionally Jiri Kosina
2010-02-02 13:44 ` Jiri Kosina
2010-02-02 14:33 ` Uwe Kleine-König
2010-02-04 1:29 ` [PATCH 08/25] imx/mx1: rename files defining a machine to mach-$mach.c Amit Kucheria
2010-02-04 8:45 ` Uwe Kleine-König
2010-02-05 17:38 ` Amit Kucheria
2010-02-04 12:06 ` Mark Brown
2010-02-04 12:25 ` Eric Miao [this message]
2010-01-14 10:24 ` [PATCH 00/20] imx cleanup series Sascha Hauer
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=f17812d71002040425o6690b9aej4bea9046457780@mail.gmail.com \
--to=eric.y.miao@gmail.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).