From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 4/4] ARM: tegra: Add support for nyan board
Date: Tue, 25 Nov 2014 09:14:54 -0700 [thread overview]
Message-ID: <5474AAFE.3020103@wwwdotorg.org> (raw)
In-Reply-To: <CAPnjgZ0WOF_6_9UXCrhNmOXDTFmxty3JnO7QWwqQuo=ZHhsS5A@mail.gmail.com>
On 11/24/2014 04:48 PM, Simon Glass wrote:
> Hi Stephen,
>
> On 24 November 2014 at 10:28, Stephen Warren <swarren@wwwdotorg.org> wrote:
>> On 11/23/2014 09:12 AM, Simon Glass wrote:
>>>
>>> From: Allen Martin <amartin@nvidia.com>
>>>
>>> Nyan is a Tegra124 clamshell board that is very similar to venice2, but it
>>> has a different panel, the sdcard cd and wp sense are flipped, and it has
>>> a different revision of the AS3722 PMIC.
>>>
>>> This is the Acer Chromebook 13 CB5-311-T7NN (13.3-inch HD, NVIDIA
>>> Tegra K1, 2GB). The display is not currently supported, so it should
>>> boot on other nyan-based Chromebooks also, but only the device tree for
>>> nyan-big is provided here.
>>
>>
>> As Olof explained it to me, Nyan is a family of boards, of which "Nyan Big"
>> is a particular member. In other words, "Nyan Big" is the CB5. As such, I
>> believe the U-Boot board should be named "Nyan Big" not "Nyan", unless we
>> expect the U-Boot support for work for the various other Nyan family boards;
>> I believe the HP Tegra Chromebooks are also in the Nyan family. Either way
>> though (i.e. even if nyan.c gets shared between the Acer and HP Tegra
>> Chromebooks), I still think we want separate top-level U-Boot board names
>> for the two, so that it's easy for people to build the right DT into their
>> U-Boot binary for example.
>
> What change are you requesting for this patch?
Given Andrew's description of the board naming, I think this patch
(patch subject/description, filenames, file content) and anything else
for this board should call it nyan-big not nyan. The Acer Chrombook is
apparently nyan-big, whereas plain nyan means Norrin.
next prev parent reply other threads:[~2014-11-25 16:14 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-23 16:12 [U-Boot] [PATCH v3 1/4] dts: Bring in Chrome OS keyboard device tree definition Simon Glass
2014-11-23 16:12 ` [U-Boot] [PATCH v3 2/4] tegra: dts: Sync tegra124.dtsi with linux kernel Simon Glass
2014-11-24 17:09 ` Stephen Warren
2014-11-24 17:12 ` Simon Glass
2014-11-23 16:12 ` [U-Boot] [PATCH v3 3/4] tegra: config: Enable FIT and device tree for all boards Simon Glass
2014-11-24 17:11 ` Stephen Warren
2014-11-24 23:49 ` Simon Glass
2014-11-25 16:23 ` Stephen Warren
2014-11-25 17:44 ` Simon Glass
2014-12-01 18:41 ` Stephen Warren
2014-12-01 20:17 ` Simon Glass
2014-11-23 16:12 ` [U-Boot] [PATCH v3 4/4] ARM: tegra: Add support for nyan board Simon Glass
2014-11-23 17:02 ` Simon Glass
2014-11-24 17:21 ` Stephen Warren
2014-11-24 17:25 ` Stephen Warren
2014-11-24 17:28 ` Stephen Warren
2014-11-24 23:48 ` Simon Glass
2014-11-25 16:14 ` Stephen Warren [this message]
2014-11-25 16:18 ` Simon Glass
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=5474AAFE.3020103@wwwdotorg.org \
--to=swarren@wwwdotorg.org \
--cc=u-boot@lists.denx.de \
/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