From: plagnioj@jcrosoft.com (Jean-Christophe PLAGNIOL-VILLARD)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm/dt: Add SoC detection macros
Date: Sat, 17 Sep 2011 20:19:07 +0200 [thread overview]
Message-ID: <20110917181907.GA16141@game.jcrosoft.org> (raw)
In-Reply-To: <20110917112321.GE16381@n2100.arm.linux.org.uk>
On 12:23 Sat 17 Sep , Russell King - ARM Linux wrote:
> On Sat, Sep 17, 2011 at 12:34:57PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 11:28 Sat 17 Sep , Russell King - ARM Linux wrote:
> > > One last point to raise here is - and it's quite a fundamental one - do we
> > > really want this? If we're making decisions based on the SoC type, that
> > > suggests to me that the hardware description in DT is incomplete, and
> > > we're hiding stuff in the kernel behind the SoC type. That doesn't sound
> > > particularly appealing given the point of DT is to encode the hardware
> > > specific information outside the kernel code.
> >
> > except if a machine can run on 2 soc so detect it will avoid to have 2 Device
> > Tree
>
> This code is structured to match the SoC based upon an entry in the DT,
> so for tegra2 vs tegra3 it's already having to have two different DTs
> to distinguish between them.
>
> However, I still go back to my original point: the point of DT is to
> provide a description of the hardware which the kernel is running on -
> not only for current hardware but possibly future hardware as well. Eg,
> if Tegra4 comes along with more peripherals than Tegra3 but has basic
> hardware which the kernel already supports, just wired up differently,
> then Tegra4 should just work with a new DT file and no code changes.
>
> What I'm saying is that in that scenario it should not be necessary to
> edit the kernel to invent new SoC types, and then teach it that Tegra4
> is mostly the same as Tegra3. That information should all be encoded
> into the DT rather than the C code in the kernel.
>
> So, I think adding this SoC type stuff is the wrong approach to the
> problem.
I agree about it I just mean that if you have the same board with 2 SoC which
are pin to pin compatible detect the soc type will be usefull as the soc
resource may not be the same
Best Regards,
J.
next prev parent reply other threads:[~2011-09-17 18:19 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-09 8:02 [PATCH] arm/dt: Add SoC detection macros Allen Martin
2011-09-09 16:45 ` Olof Johansson
2011-09-17 10:28 ` Russell King - ARM Linux
2011-09-17 10:34 ` Jean-Christophe PLAGNIOL-VILLARD
2011-09-17 11:23 ` Russell King - ARM Linux
2011-09-17 18:19 ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2011-09-17 20:56 ` Arnd Bergmann
2011-09-18 0:46 ` Jean-Christophe PLAGNIOL-VILLARD
2011-09-18 9:28 ` Arnd Bergmann
2011-09-19 17:26 ` Allen Martin
2011-09-19 20:08 ` Olof Johansson
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=20110917181907.GA16141@game.jcrosoft.org \
--to=plagnioj@jcrosoft.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).