diff mbox

[v2,12/13] drm/panel: simple: Add simple panel device tree binding

Message ID 1446296170-3702-13-git-send-email-zyw@rock-chips.com
State New
Headers show

Commit Message

Chris Zhong Oct. 31, 2015, 12:56 p.m. UTC
This binding specifies a set of common properties for display panels. It
can be used as a basis by bindings for specific panels.
Bindings for three specific panels are provided to show how the
simple panel binding can be used.

Signed-off-by: Chris Zhong <zyw@rock-chips.com>

---

Changes in v2:
As Thierry.Reding comment, add a documentation for this panel.

 Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt | 7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt

Comments

Rob Herring Nov. 2, 2015, 3:06 p.m. UTC | #1
On Sat, Oct 31, 2015 at 7:56 AM, Chris Zhong <zyw@rock-chips.com> wrote:

Your subject should be more specific with the panel name.

> This binding specifies a set of common properties for display panels. It
> can be used as a basis by bindings for specific panels.
> Bindings for three specific panels are provided to show how the
> simple panel binding can be used.
>
> Signed-off-by: Chris Zhong <zyw@rock-chips.com>
>
> ---
>
> Changes in v2:
> As Thierry.Reding comment, add a documentation for this panel.
>
>  Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt | 7 +++++++
>  1 file changed, 7 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt

Please move to bindings/display/panel/

>
> diff --git a/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt b/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt
> new file mode 100644
> index 0000000..50be5e2
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt
> @@ -0,0 +1,7 @@
> +Boe Corporation 8.0" WUXGA TFT LCD panel
> +
> +Required properties:
> +- compatible: should be "boe,tv080wum-nl0"
> +
> +This binding is compatible with the simple-panel binding, which is specified
> +in simple-panel.txt in this directory.
> --
> 2.6.2
>
Chris Zhong Nov. 3, 2015, 12:28 a.m. UTC | #2
Hi Rob


On 11/02/2015 11:06 PM, Rob Herring wrote:
> On Sat, Oct 31, 2015 at 7:56 AM, Chris Zhong <zyw@rock-chips.com> wrote:
>
> Your subject should be more specific with the panel name.
I'll write more specific in subject next version.
>
>> This binding specifies a set of common properties for display panels. It
>> can be used as a basis by bindings for specific panels.
>> Bindings for three specific panels are provided to show how the
>> simple panel binding can be used.
>>
>> Signed-off-by: Chris Zhong <zyw@rock-chips.com>
>>
>> ---
>>
>> Changes in v2:
>> As Thierry.Reding comment, add a documentation for this panel.
>>
>>   Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt
> Please move to bindings/display/panel/
Okay.
>
>> diff --git a/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt b/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt
>> new file mode 100644
>> index 0000000..50be5e2
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt
>> @@ -0,0 +1,7 @@
>> +Boe Corporation 8.0" WUXGA TFT LCD panel
>> +
>> +Required properties:
>> +- compatible: should be "boe,tv080wum-nl0"
>> +
>> +This binding is compatible with the simple-panel binding, which is specified
>> +in simple-panel.txt in this directory.
>> --
>> 2.6.2
>>
>
>
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt b/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt
new file mode 100644
index 0000000..50be5e2
--- /dev/null
+++ b/Documentation/devicetree/bindings/panel/boe,tv080wum-nl0.txt
@@ -0,0 +1,7 @@ 
+Boe Corporation 8.0" WUXGA TFT LCD panel
+
+Required properties:
+- compatible: should be "boe,tv080wum-nl0"
+
+This binding is compatible with the simple-panel binding, which is specified
+in simple-panel.txt in this directory.