On Mon, May 14, 2018 at 11:03 AM, Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> wrote: > On Thu, Apr 19, 2018 at 3:02 PM, Chen-Yu Tsai <wens@xxxxxxxx> wrote: >> This panel is marketed as Banana Pi 7" LCD display. On the back is >> a sticker denoting the model name S070WV20-CT16. >> >> This is a 7" 800x480 panel connected through a 24-bit RGB interface. >> However the panel only does 262k colors. >> >> Signed-off-by: Chen-Yu Tsai <wens@xxxxxxxx> >> --- >> .../display/panel/bananapi,s070wv20-ct16.txt | 7 ++++++ >> drivers/gpu/drm/panel/panel-simple.c | 25 +++++++++++++++++++ >> 2 files changed, 32 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/display/panel/bananapi,s070wv20-ct16.txt >> >> diff --git a/Documentation/devicetree/bindings/display/panel/bananapi,s070wv20-ct16.txt b/Documentation/devicetree/bindings/display/panel/bananapi,s070wv20-ct16.txt >> new file mode 100644 >> index 000000000000..2ec35ce36e9a >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/panel/bananapi,s070wv20-ct16.txt >> @@ -0,0 +1,7 @@ >> +Banana Pi 7" (S070WV20-CT16) TFT LCD Panel >> + >> +Required properties: >> +- compatible: should be "bananapi,s070wv20-ct16" >> + >> +This binding is compatible with the simple-panel binding, which is specified >> +in simple-panel.txt in this directory. >> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c >> index cbf1ab404ee7..9bc037f74d6c 100644 >> --- a/drivers/gpu/drm/panel/panel-simple.c >> +++ b/drivers/gpu/drm/panel/panel-simple.c >> @@ -745,6 +745,28 @@ static const struct panel_desc avic_tm070ddh03 = { >> }, >> }; >> >> +static const struct drm_display_mode bananapi_s070wv20_ct16_mode = { >> + .clock = 30000, >> + .hdisplay = 800, >> + .hsync_start = 800 + 40, >> + .hsync_end = 800 + 40 + 48, >> + .htotal = 800 + 40 + 48 + 40, >> + .vdisplay = 480, >> + .vsync_start = 480 + 13, >> + .vsync_end = 480 + 13 + 3, >> + .vtotal = 480 + 13 + 3 + 29, >> +}; >> + >> +static const struct panel_desc bananapi_s070wv20_ct16 = { >> + .modes = &bananapi_s070wv20_ct16_mode, >> + .num_modes = 1, >> + .bpc = 6, >> + .size = { >> + .width = 154, >> + .height = 86, >> + }, >> +}; > > I think this parallel RGB interface right? I too have same display > with DSI I'm sure these setting will not useful right? do we need to > write separate panel driver for that? AFAIK the DSI model is an RGB panel with a MIPI DSI bridge on the connector board. The model I have is dual interface. ChenYu -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html