Re: [PATCH 1/2] drm/panel: Add support for Armadeus ST0700 Adapt

From: SÃbastien Szymanski
Date: Wed Jul 18 2018 - 04:26:12 EST


Hi Fabio,

On 07/17/2018 02:28 PM, Fabio Estevam wrote:
> Hi SÃbastien,
>
> On Tue, Jul 17, 2018 at 4:23 AM, SÃbastien Szymanski
> <sebastien.szymanski@xxxxxxxxxxxx> wrote:
>> This patch adds support for the Armadeus ST0700 Adapt. It comes with a
>> Santek ST0700I5Y-RBSLW 7.0" WVGA (800x480) TFT and an adapter board so
>> that it can be connected on the TFT header of Armadeus Dev boards.
>>
>> Signed-off-by: SÃbastien Szymanski <sebastien.szymanski@xxxxxxxxxxxx>
>> ---
>> .../display/panel/armadeus,st0700-adapt.txt | 9 +++++++
>> drivers/gpu/drm/panel/panel-simple.c | 29 ++++++++++++++++++++++
>> 2 files changed, 38 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/display/panel/armadeus,st0700-adapt.txt
>>
>> diff --git a/Documentation/devicetree/bindings/display/panel/armadeus,st0700-adapt.txt b/Documentation/devicetree/bindings/display/panel/armadeus,st0700-adapt.txt
>> new file mode 100644
>> index 000000000000..a30d63db3c8f
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/display/panel/armadeus,st0700-adapt.txt
>> @@ -0,0 +1,9 @@
>> +Armadeus ST0700 Adapt. A Santek ST0700I5Y-RBSLW 7.0" WVGA (800x480) TFT with
>> +an adapter board.
>> +
>> +Required properties:
>> +- compatible: "armadeus,st0700-adapt"
>
> Shouldn't this be named "santek,st0700i5y" instead?
>
> Santek is the vendor of the panel and st0700i5y is the model.

You're right but Armadeus doesn't provide the panel as-is. Armadeus adds
an adapter board so it can be connected on the development boards. [1]
Even though the panel supports MEDIA_BUS_FMT_RGB888_1X24, with the
adapter board the bus format has to be MEDIA_BUS_FMT_RGB666_1X18.

[1] http://www.opossom.com/english/products-peripherals-st0700_adapt.html

Regards,

>
> Then you could add a "santek" entry in
> Documentation/devicetree/bindings/vendor-prefixes.txt.
>
> Regards,
>
> Fabio Estevam
>


--
SÃbastien Szymanski
Software engineer, Armadeus Systems
Tel: +33 (0)9 72 29 41 44
Fax: +33 (0)9 72 28 79 26