From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv2 5/5] ARM: dts: Add dt binding documentation for exynos rotator
Date: Fri, 09 Aug 2013 10:38:30 -0600 [thread overview]
Message-ID: <52051B06.7000209@wwwdotorg.org> (raw)
In-Reply-To: <229348511.e5ckfeMu0d@amdc1227>
On 08/09/2013 07:15 AM, Tomasz Figa wrote:
> Hi Chanho,
>
> On Friday 09 of August 2013 16:40:53 Chanho Park wrote:
>> This patch describes each nodes of rotator and specifies a example how to
>> bind it.
>> diff --git a/Documentation/devicetree/bindings/gpu/samsung-rotator.txt
>> +* Samsung Image Rotator
>> +
>> +Required properties:
>> + - compatible : value should be following:
>> + (a) "samsung,exynos4210-rotator" for Rotator IP in Exynos4210
>> + (b) "samsung,exynos4212-rotator" for Rotator IP in Exynos4212/4412
>> + (c) "samsung,exynos5250-rotator" for Rotator IP in Exynos5250
Is "rotator" the name that the HW documentation uses to refer to this
block? If so, those compatible values seem fine. If not, they seem
slightly too generic for me; perhaps better to use the raw HW block name?
next prev parent reply other threads:[~2013-08-09 16:38 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-09 7:40 [PATCHv2 0/5] device tree support for exynos rotator Chanho Park
2013-08-09 7:40 ` [PATCHv2 1/5] drm/exynos: add device tree support for rotator Chanho Park
2013-08-09 9:48 ` Sachin Kamat
2013-08-09 12:50 ` Tomasz Figa
2013-08-09 7:40 ` [PATCHv2 2/5] ARM: dts: Add rotator node for exynos4210 Chanho Park
2013-08-09 12:56 ` Tomasz Figa
2013-08-09 7:40 ` [PATCHv2 3/5] ARM: dts: Add rotator node for exynos4x12 Chanho Park
2013-08-09 7:40 ` [PATCHv2 4/5] ARM: dts: Add rotator node for exynos5250 Chanho Park
2013-08-09 7:40 ` [PATCHv2 5/5] ARM: dts: Add dt binding documentation for exynos rotator Chanho Park
2013-08-09 9:40 ` Sachin Kamat
2013-08-09 13:15 ` Tomasz Figa
2013-08-09 16:38 ` Stephen Warren [this message]
2013-08-09 17:05 ` Tomasz Figa
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=52051B06.7000209@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