From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Alexandre Courbot
<acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH] ARM: tegra: add Tegra Note 7 device tree
Date: Thu, 01 May 2014 11:33:26 -0600 [thread overview]
Message-ID: <53628566.1070602@wwwdotorg.org> (raw)
In-Reply-To: <1398928217-32629-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
On 05/01/2014 01:10 AM, Alexandre Courbot wrote:
> Tegra Note 7 is a consumer tablet embedding a Tegra 4 SoC with 1GB RAM
> and a 720p panel.
>
> The following features are enabled by this device tree: UART, eMMC, USB
> (needs external power), PMIC, backlight, DSI panel, keys.
>
> SD card, HDMI, charger, self-powered USB, audio, wifi, bluetooth are not
> yet supported but might be by future patches (likely in that order).
> diff --git a/arch/arm/boot/dts/tegra114-tn7.dts b/arch/arm/boot/dts/tegra114-tn7.dts
> + chosen {
> + /* TN7's bootloader's arguments need to be overridden */
> + bootargs = "console=ttyS0,115200n8 console=tty1 gpt cma=128M fbcon=rotate:2";
Do we still need the cma argument at all now? The default is now 64M in
tegra_defconfig, and 128M seems quite large.
> + host1x@50000000 {
> + hdmi@54280000 {
> + status = "okay";
> +
> + hdmi-supply = <&vdd_5v0_hdmi>;
> + vdd-supply = <&vdd_3v3_hdmi>;
> + pll-supply = <&vdd_1v05_pll>;
> +
> + nvidia,ddc-i2c-bus = <&hdmi_ddc>;
> + nvidia,hpd-gpio =
> + <&gpio TEGRA_GPIO(N, 7) GPIO_ACTIVE_HIGH>;
> + };
If HDMI doesn't work yet (per commit description), should we leave it
out of the DT for now?
> + serial@70006300 {
> + status = "okay";
> + };
Is there actually an accessible UART on the board? I guess this is for
people lucky enough to have internal boards with the debug connector. If
so, this is probably fine.
WARNING: multiple messages have this Message-ID (diff)
From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: tegra: add Tegra Note 7 device tree
Date: Thu, 01 May 2014 11:33:26 -0600 [thread overview]
Message-ID: <53628566.1070602@wwwdotorg.org> (raw)
In-Reply-To: <1398928217-32629-1-git-send-email-acourbot@nvidia.com>
On 05/01/2014 01:10 AM, Alexandre Courbot wrote:
> Tegra Note 7 is a consumer tablet embedding a Tegra 4 SoC with 1GB RAM
> and a 720p panel.
>
> The following features are enabled by this device tree: UART, eMMC, USB
> (needs external power), PMIC, backlight, DSI panel, keys.
>
> SD card, HDMI, charger, self-powered USB, audio, wifi, bluetooth are not
> yet supported but might be by future patches (likely in that order).
> diff --git a/arch/arm/boot/dts/tegra114-tn7.dts b/arch/arm/boot/dts/tegra114-tn7.dts
> + chosen {
> + /* TN7's bootloader's arguments need to be overridden */
> + bootargs = "console=ttyS0,115200n8 console=tty1 gpt cma=128M fbcon=rotate:2";
Do we still need the cma argument at all now? The default is now 64M in
tegra_defconfig, and 128M seems quite large.
> + host1x at 50000000 {
> + hdmi at 54280000 {
> + status = "okay";
> +
> + hdmi-supply = <&vdd_5v0_hdmi>;
> + vdd-supply = <&vdd_3v3_hdmi>;
> + pll-supply = <&vdd_1v05_pll>;
> +
> + nvidia,ddc-i2c-bus = <&hdmi_ddc>;
> + nvidia,hpd-gpio =
> + <&gpio TEGRA_GPIO(N, 7) GPIO_ACTIVE_HIGH>;
> + };
If HDMI doesn't work yet (per commit description), should we leave it
out of the DT for now?
> + serial at 70006300 {
> + status = "okay";
> + };
Is there actually an accessible UART on the board? I guess this is for
people lucky enough to have internal boards with the debug connector. If
so, this is probably fine.
WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren@wwwdotorg.org>
To: Alexandre Courbot <acourbot@nvidia.com>,
Thierry Reding <thierry.reding@gmail.com>
Cc: linux-tegra@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Thierry Reding <treding@nvidia.com>
Subject: Re: [PATCH] ARM: tegra: add Tegra Note 7 device tree
Date: Thu, 01 May 2014 11:33:26 -0600 [thread overview]
Message-ID: <53628566.1070602@wwwdotorg.org> (raw)
In-Reply-To: <1398928217-32629-1-git-send-email-acourbot@nvidia.com>
On 05/01/2014 01:10 AM, Alexandre Courbot wrote:
> Tegra Note 7 is a consumer tablet embedding a Tegra 4 SoC with 1GB RAM
> and a 720p panel.
>
> The following features are enabled by this device tree: UART, eMMC, USB
> (needs external power), PMIC, backlight, DSI panel, keys.
>
> SD card, HDMI, charger, self-powered USB, audio, wifi, bluetooth are not
> yet supported but might be by future patches (likely in that order).
> diff --git a/arch/arm/boot/dts/tegra114-tn7.dts b/arch/arm/boot/dts/tegra114-tn7.dts
> + chosen {
> + /* TN7's bootloader's arguments need to be overridden */
> + bootargs = "console=ttyS0,115200n8 console=tty1 gpt cma=128M fbcon=rotate:2";
Do we still need the cma argument at all now? The default is now 64M in
tegra_defconfig, and 128M seems quite large.
> + host1x@50000000 {
> + hdmi@54280000 {
> + status = "okay";
> +
> + hdmi-supply = <&vdd_5v0_hdmi>;
> + vdd-supply = <&vdd_3v3_hdmi>;
> + pll-supply = <&vdd_1v05_pll>;
> +
> + nvidia,ddc-i2c-bus = <&hdmi_ddc>;
> + nvidia,hpd-gpio =
> + <&gpio TEGRA_GPIO(N, 7) GPIO_ACTIVE_HIGH>;
> + };
If HDMI doesn't work yet (per commit description), should we leave it
out of the DT for now?
> + serial@70006300 {
> + status = "okay";
> + };
Is there actually an accessible UART on the board? I guess this is for
people lucky enough to have internal boards with the debug connector. If
so, this is probably fine.
next prev parent reply other threads:[~2014-05-01 17:33 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-01 7:10 [PATCH] ARM: tegra: add Tegra Note 7 device tree Alexandre Courbot
2014-05-01 7:10 ` Alexandre Courbot
2014-05-01 7:10 ` Alexandre Courbot
[not found] ` <1398928217-32629-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-05-01 17:33 ` Stephen Warren [this message]
2014-05-01 17:33 ` Stephen Warren
2014-05-01 17:33 ` Stephen Warren
[not found] ` <53628566.1070602-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-05-02 5:37 ` Alexandre Courbot
2014-05-02 5:37 ` Alexandre Courbot
2014-05-02 5:37 ` Alexandre Courbot
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=53628566.1070602@wwwdotorg.org \
--to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
--cc=acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.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 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.