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@armadeus.com 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@armadeus.com
.../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