From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: "HeungJun, Kim" <riverful.kim@samsung.com>
Cc: 'Kyungmin Park' <kyungmin.park@samsung.com>,
'Kukjin Kim' <kgene.kim@samsung.com>,
linux-samsung-soc@vger.kernel.org,
'arm-linux' <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] ARM: EXYNOS: Adds Samsung TRATS board support
Date: Tue, 3 Jan 2012 09:50:16 +0000 [thread overview]
Message-ID: <20120103095015.GH2914@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <000d01ccc381$59fbe660$0df3b320$%kim@samsung.com>
On Mon, Dec 26, 2011 at 12:49:21PM +0900, HeungJun, Kim wrote:
> If you don't mind, I want to let you know additional information.
>
> I also agree with Mr.Park.
>
> Generally in the robot field, the term "machine" means the total things
> including the machinery part and even the brain "board", and the machine
> is more bigger concept.
Some platforms which the kernel runs on are not a 'board' but a set of
boards: a motherboard and a separate CPU card. So to call these a
'board' is wrong too.
Welcome to the problem: there is not one single term which satisfies
everyone. The solution: a compromise.
The compromise that has been chosen for the ARM kernel is to call this
stuff 'machines' or 'platforms' - and the exynos stuff has decided to
call it 'machines'. (Some people have decided to use 'board' in their
directories, which is fine _provided_ there is consistency within that
directory.)
That's actually the _most_ important thing: consistency. So if a
maintainer has decided to use one way in their directory, that's the
way which rules for that directory. So, please, get over it - and
don't start a brand new naming scheme in a directory just because you
don't agree with the maintainers opinion.
So, please name the board mach-trats.c as requested by Kukjin (who is
the maintainer for exynos.)
WARNING: multiple messages have this Message-ID (diff)
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: EXYNOS: Adds Samsung TRATS board support
Date: Tue, 3 Jan 2012 09:50:16 +0000 [thread overview]
Message-ID: <20120103095015.GH2914@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <000d01ccc381$59fbe660$0df3b320$%kim@samsung.com>
On Mon, Dec 26, 2011 at 12:49:21PM +0900, HeungJun, Kim wrote:
> If you don't mind, I want to let you know additional information.
>
> I also agree with Mr.Park.
>
> Generally in the robot field, the term "machine" means the total things
> including the machinery part and even the brain "board", and the machine
> is more bigger concept.
Some platforms which the kernel runs on are not a 'board' but a set of
boards: a motherboard and a separate CPU card. So to call these a
'board' is wrong too.
Welcome to the problem: there is not one single term which satisfies
everyone. The solution: a compromise.
The compromise that has been chosen for the ARM kernel is to call this
stuff 'machines' or 'platforms' - and the exynos stuff has decided to
call it 'machines'. (Some people have decided to use 'board' in their
directories, which is fine _provided_ there is consistency within that
directory.)
That's actually the _most_ important thing: consistency. So if a
maintainer has decided to use one way in their directory, that's the
way which rules for that directory. So, please, get over it - and
don't start a brand new naming scheme in a directory just because you
don't agree with the maintainers opinion.
So, please name the board mach-trats.c as requested by Kukjin (who is
the maintainer for exynos.)
next prev parent reply other threads:[~2012-01-03 9:50 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-23 7:09 [PATCH] ARM: EXYNOS: Adds Samsung TRATS board support HeungJun, Kim
2011-12-24 1:46 ` Kukjin Kim
2011-12-25 23:55 ` Kyungmin Park
2011-12-25 23:55 ` Kyungmin Park
2011-12-26 3:49 ` HeungJun, Kim
2011-12-26 3:49 ` HeungJun, Kim
2011-12-27 0:37 ` Kukjin Kim
2011-12-27 0:37 ` Kukjin Kim
2012-01-03 9:50 ` Russell King - ARM Linux [this message]
2012-01-03 9:50 ` Russell King - ARM Linux
2011-12-27 0:37 ` Kukjin Kim
2011-12-27 0:37 ` Kukjin Kim
2011-12-27 0:45 ` Kyungmin Park
2011-12-27 0:45 ` Kyungmin Park
2011-12-27 1:11 ` Kukjin Kim
2011-12-27 1:11 ` Kukjin Kim
2011-12-27 1:35 ` Kyungmin Park
2011-12-27 1:35 ` Kyungmin Park
2012-01-27 8:21 ` HeungJun, Kim
2012-01-27 8:21 ` HeungJun, Kim
2012-02-09 13:11 ` Kukjin Kim
2012-02-09 13:11 ` Kukjin Kim
2012-02-09 18:58 ` Thomas Abraham
2012-02-09 18:58 ` Thomas Abraham
2012-02-10 8:01 ` Kyungmin Park
2012-02-10 8:01 ` Kyungmin Park
2012-02-10 17:54 ` Karol Lewandowski
2012-02-10 17:54 ` Karol Lewandowski
2012-02-10 19:00 ` Thomas Abraham
2012-02-10 19:00 ` Thomas Abraham
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=20120103095015.GH2914@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--cc=kgene.kim@samsung.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=riverful.kim@samsung.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.