From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 09/16] pwm: tegra: Add device tree support
Date: Mon, 02 Apr 2012 09:42:12 -0600 [thread overview]
Message-ID: <4F79C8D4.2000306@wwwdotorg.org> (raw)
In-Reply-To: <20120402083749.GA6576@avionic-0098.adnet.avionic-design.de>
On 04/02/2012 02:37 AM, Thierry Reding wrote:
> * Stephen Warren wrote:
>> On 03/28/2012 08:33 AM, Thierry Reding wrote:
>>> Add auxdata to instantiate the PWFM controller from a device tree,
>>> include the corresponding nodes in the dtsi files for Tegra 20 and
>>> Tegra 30 and add binding documentation.
>>>
>>> Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
>>> Acked-by: Stephen Warren <swarren@wwwdotorg.org>
>>
>>> diff --git a/drivers/pwm/pwm-tegra.c b/drivers/pwm/pwm-tegra.c
>> ...
>>> +#ifdef CONFIG_OF
>>> +static struct of_device_id tegra_pwm_of_match[] = {
>>> + { .compatible = "nvidia,tegra20-pwm" },
>>> + { .compatible = "nvidia,tegra30-pwm" },
>>
>> Could you swap those two lines, so that tegra30-pwm matches first. It
>> makes no difference at present, but might in the future if the driver
>> actually has to differentiate the two SoCs.
>
> I thought the matching order was determined by the compatible property in the
> device tree, not the OF match table of the driver.
At least logically, yes. However, of_match_device() appears to iterate
over each match table entry, checking whether it matches any string in
the compatible flag. Perhaps this could be considered a bug?
next prev parent reply other threads:[~2012-04-02 15:42 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-28 14:33 [PATCH v5 00/16] Add PWM framework and device tree support Thierry Reding
2012-03-28 14:33 ` [PATCH v5 01/16] pwm: Add PWM framework support Thierry Reding
2012-03-29 21:41 ` Mark Brown
2012-04-04 6:36 ` Shawn Guo
2012-04-04 6:39 ` Thierry Reding
2012-03-28 14:33 ` [PATCH v5 02/16] pwm: Allow chips to support multiple PWMs Thierry Reding
2012-03-29 21:50 ` Mark Brown
2012-04-04 6:44 ` Shawn Guo
2012-03-28 14:33 ` [PATCH v5 03/16] pwm: Add debugfs interface Thierry Reding
2012-03-29 21:56 ` Mark Brown
2012-04-04 6:47 ` Shawn Guo
2012-03-28 14:33 ` [PATCH v5 04/16] pwm: Add table-based lookup for static mappings Thierry Reding
2012-03-29 22:03 ` Mark Brown
2012-03-30 5:06 ` Thierry Reding
2012-03-30 10:18 ` Mark Brown
2012-03-30 10:38 ` Thierry Reding
2012-03-31 14:30 ` Thierry Reding
2012-04-01 15:20 ` Shawn Guo
2012-04-02 0:47 ` Shawn Guo
2012-04-02 4:50 ` Thierry Reding
2012-03-28 14:33 ` [PATCH v5 05/16] pwm: Add device tree support Thierry Reding
2012-03-28 14:53 ` Arnd Bergmann
2012-03-29 21:47 ` Mark Brown
2012-03-30 6:24 ` Thierry Reding
2012-03-28 14:33 ` [PATCH v5 06/16] ARM: tegra: Fix PWM clock programming Thierry Reding
2012-03-28 14:33 ` [PATCH v5 07/16] ARM: tegra: Provide clock for only one PWM controller Thierry Reding
2012-03-28 14:33 ` [PATCH v5 08/16] pwm: Add NVIDIA Tegra SoC support Thierry Reding
2012-03-30 18:57 ` Stephen Warren
2012-04-04 6:54 ` Shawn Guo
2012-03-28 14:33 ` [PATCH v5 09/16] pwm: tegra: Add device tree support Thierry Reding
2012-03-30 19:00 ` Stephen Warren
2012-04-02 8:37 ` Thierry Reding
2012-04-02 15:42 ` Stephen Warren [this message]
2012-04-03 17:55 ` Thierry Reding
2012-04-03 19:43 ` Rob Herring
2012-04-03 23:42 ` Grant Likely
2012-04-04 5:00 ` Thierry Reding
2012-04-04 18:32 ` Stephen Warren
2012-04-07 1:44 ` Grant Likely
2012-03-28 14:33 ` [PATCH v5 10/16] pwm: Move Blackfin PWM driver to PWM framework Thierry Reding
2012-03-28 14:33 ` [PATCH v5 11/16] pwm: Move PXA " Thierry Reding
2012-03-28 14:33 ` [PATCH v5 12/16] ARM i.MX: Move i.MX pwm driver to pwm framework Thierry Reding
2012-03-28 14:33 ` [PATCH v5 13/16] ARM Samsung: Move s3c " Thierry Reding
2012-03-28 14:33 ` [PATCH v5 14/16] ARM vt8500: Move vt8500 " Thierry Reding
2012-03-28 14:33 ` [PATCH v5 15/16] pwm-backlight: Add rudimentary device tree support Thierry Reding
2012-03-30 19:04 ` Stephen Warren
2012-04-04 18:11 ` Thierry Reding
2012-03-28 14:33 ` [PATCH v5 16/16] pwm: Take over maintainership of the PWM subsystem 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=4F79C8D4.2000306@wwwdotorg.org \
--to=swarren@wwwdotorg.org \
--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).