diff mbox series

[1/2] dt-bindings: display: panel-simple: Document support for Microchip AC69T88A

Message ID 20240917095330.267397-1-manikandan.m@microchip.com (mailing list archive)
State New
Headers show
Series [1/2] dt-bindings: display: panel-simple: Document support for Microchip AC69T88A | expand

Commit Message

Manikandan Muralidharan Sept. 17, 2024, 9:53 a.m. UTC
Add Microchip AC69T88A 5" LVDS interface (800x480) TFT LCD panel
compatible string

Signed-off-by: Manikandan Muralidharan <manikandan.m@microchip.com>
---
 .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 ++
 1 file changed, 2 insertions(+)

Comments

Krzysztof Kozlowski Sept. 17, 2024, 10:37 a.m. UTC | #1
On 17/09/2024 11:53, Manikandan Muralidharan wrote:
> Add Microchip AC69T88A 5" LVDS interface (800x480) TFT LCD panel
> compatible string
> 
> Signed-off-by: Manikandan Muralidharan <manikandan.m@microchip.com>
> ---
>  .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
> index b89e39790579..09911b89d140 100644
> --- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
> @@ -200,6 +200,8 @@ properties:
>        - logictechno,lttd800480070-l2rt
>          # Logic Technologies LTTD800480070-L6WH-RT 7” 800x480 TFT Resistive Touch Module
>        - logictechno,lttd800480070-l6wh-rt
> +        # Microchip AC69T88A 5" 800X480 LVDS interface TFT LCD Panel
> +      - microchip,ac69t88a-lvds-panel

Is this device some sort of multi-function? Why "lvds-panel"? What else
could it be?

Best regards,
Krzysztof
Manikandan Muralidharan Sept. 18, 2024, 3:08 a.m. UTC | #2
Hi Krzysztof,

On 17/09/24 4:07 pm, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 17/09/2024 11:53, Manikandan Muralidharan wrote:
>> Add Microchip AC69T88A 5" LVDS interface (800x480) TFT LCD panel
>> compatible string
>>
>> Signed-off-by: Manikandan Muralidharan <manikandan.m@microchip.com>
>> ---
>>   .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
>> index b89e39790579..09911b89d140 100644
>> --- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
>> +++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
>> @@ -200,6 +200,8 @@ properties:
>>         - logictechno,lttd800480070-l2rt
>>           # Logic Technologies LTTD800480070-L6WH-RT 7” 800x480 TFT Resistive Touch Module
>>         - logictechno,lttd800480070-l6wh-rt
>> +        # Microchip AC69T88A 5" 800X480 LVDS interface TFT LCD Panel
>> +      - microchip,ac69t88a-lvds-panel
> 
> Is this device some sort of multi-function? Why "lvds-panel"? What else
> could it be?
This device does not multi-function, I will rephrase and share a v2
> 
> Best regards,
> Krzysztof
>
Krzysztof Kozlowski Sept. 18, 2024, 8:17 a.m. UTC | #3
On 18/09/2024 05:08, Manikandan.M@microchip.com wrote:
> Hi Krzysztof,
> 
> On 17/09/24 4:07 pm, Krzysztof Kozlowski wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> On 17/09/2024 11:53, Manikandan Muralidharan wrote:
>>> Add Microchip AC69T88A 5" LVDS interface (800x480) TFT LCD panel
>>> compatible string
>>>
>>> Signed-off-by: Manikandan Muralidharan <manikandan.m@microchip.com>
>>> ---
>>>   .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
>>> index b89e39790579..09911b89d140 100644
>>> --- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
>>> +++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
>>> @@ -200,6 +200,8 @@ properties:
>>>         - logictechno,lttd800480070-l2rt
>>>           # Logic Technologies LTTD800480070-L6WH-RT 7” 800x480 TFT Resistive Touch Module
>>>         - logictechno,lttd800480070-l6wh-rt
>>> +        # Microchip AC69T88A 5" 800X480 LVDS interface TFT LCD Panel
>>> +      - microchip,ac69t88a-lvds-panel
>>
>> Is this device some sort of multi-function? Why "lvds-panel"? What else
>> could it be?
> This device does not multi-function, I will rephrase and share a v2

Then drop lvds-panel, please.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
index b89e39790579..09911b89d140 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
+++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
@@ -200,6 +200,8 @@  properties:
       - logictechno,lttd800480070-l2rt
         # Logic Technologies LTTD800480070-L6WH-RT 7” 800x480 TFT Resistive Touch Module
       - logictechno,lttd800480070-l6wh-rt
+        # Microchip AC69T88A 5" 800X480 LVDS interface TFT LCD Panel
+      - microchip,ac69t88a-lvds-panel
         # Mitsubishi "AA070MC01 7.0" WVGA TFT LCD panel
       - mitsubishi,aa070mc01-ca1
         # Mitsubishi AA084XE01 8.4" XGA TFT LCD panel