From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v2 04/13] ARM: tegra: Add MIPI calibration DT entries for Tegra114
Date: Tue, 15 Oct 2013 14:41:44 -0600 [thread overview]
Message-ID: <525DA888.2060800@wwwdotorg.org> (raw)
In-Reply-To: <1381850883-12722-5-git-send-email-treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
On 10/15/2013 09:27 AM, Thierry Reding wrote:
> Add a device node for the MIPI calibration block on Tegra114. There is
> no need to disable it by default because it only enables the clock while
> performing calibration and therefore shouldn't be consuming any power
> when unused.
> diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi
> + mipi: mipi {
> + compatible = "nvidia,tegra114-mipi";
> + reg = <0x700e3000 0x100>;
> + clocks = <&tegra_car TEGRA114_CLK_MIPI_CAL>;
> + #calibrate-cells = <1>;
> + };
Do you have a link to the binding documentation for this? I'm still in
the dark re: why there's a need for #calibrate-cells at all. It seems
like this should be some internal implementation detail of the Tegra
DC/..., or implicit based on the compatible value (if cell count
changes, just rev the compatible value; it's not like this is a generic
service for loosely coupled drivers).
next prev parent reply other threads:[~2013-10-15 20:41 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-15 15:27 [PATCH v2 00/13] ARM: tegra: Enable DSI, panel and 3D support Thierry Reding
[not found] ` <1381850883-12722-1-git-send-email-treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-10-15 15:27 ` [PATCH v2 01/13] ARM: tegra: Mark Tegra30 display controller compatible with Tegra20 Thierry Reding
2013-10-15 15:27 ` [PATCH v2 02/13] ARM: tegra: Enable LVDS and HDMI on Cardhu Thierry Reding
[not found] ` <1381850883-12722-3-git-send-email-treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-10-15 20:39 ` Stephen Warren
[not found] ` <525DA807.4010500-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-10-16 11:09 ` Thierry Reding
[not found] ` <20131016110911.GH21963-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2013-10-16 16:58 ` Stephen Warren
2013-10-15 15:27 ` [PATCH v2 03/13] ARM: tegra: Enable LVDS output on Harmony Thierry Reding
2013-10-15 15:27 ` [PATCH v2 04/13] ARM: tegra: Add MIPI calibration DT entries for Tegra114 Thierry Reding
[not found] ` <1381850883-12722-5-git-send-email-treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-10-15 20:41 ` Stephen Warren [this message]
[not found] ` <525DA888.2060800-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-10-16 10:57 ` Thierry Reding
[not found] ` <20131016105743.GG21963-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2013-10-16 16:53 ` Stephen Warren
2013-10-16 16:56 ` Stephen Warren
2013-10-15 15:27 ` [PATCH v2 05/13] ARM: tegra: Add host1x, DC and HDMI to Tegra114 device tree Thierry Reding
2013-10-15 15:27 ` [PATCH v2 06/13] ARM: tegra: Add Tegra114 DSI support Thierry Reding
2013-10-15 15:27 ` [PATCH v2 07/13] ARM: tegra: Add Tegra114 gr2d support Thierry Reding
2013-10-15 15:27 ` [PATCH v2 08/13] ARM: tegra: Enable DSI support on Dalmore Thierry Reding
2013-10-15 15:27 ` [PATCH v2 09/13] ARM: tegra: Enable HDMI " Thierry Reding
2013-10-15 15:28 ` [PATCH v2 10/13] ARM: tegra: Enable DRM panel support Thierry Reding
2013-10-15 15:28 ` [PATCH v2 11/13] ARM: tegra: Add Tegra114 gr3d support Thierry Reding
2013-10-15 15:28 ` [PATCH v2 12/13] ARM: tegra: Use symbolic names for gr3d clocks Thierry Reding
2013-10-15 15:28 ` [PATCH v2 13/13] WIP: ARM: tegra: Add Tegra114 powergate support Thierry Reding
[not found] ` <1381850883-12722-14-git-send-email-treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-10-15 21:50 ` Stephen Warren
[not found] ` <525DB8B2.2050203-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-10-16 2:31 ` Joseph Lo
[not found] ` <1381890671.11523.15.camel-yx3yKKdKkHfc7b1ADBJPm0n48jw8i0AO@public.gmane.org>
2013-10-16 10:51 ` Thierry Reding
2013-10-16 19:12 ` Thierry Reding
[not found] ` <20131016191225.GA660-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2013-10-17 2:39 ` Joseph Lo
2013-10-16 10:48 ` Thierry Reding
[not found] ` <20131016104848.GE21963-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2013-10-16 16:47 ` Stephen Warren
2013-10-17 2:20 ` Joseph Lo
2013-10-17 17:37 ` [PATCH v2 00/13] ARM: tegra: Enable DSI, panel and 3D support Stephen Warren
[not found] ` <52602063.30501-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-10-17 17:45 ` Stephen Warren
[not found] ` <5260224F.6020509-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-10-17 19:38 ` Thierry Reding
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=525DA888.2060800@wwwdotorg.org \
--to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox