From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gary Bisson Subject: Re: [PATCH v2 2/2] drm/panel: Add display timing for Okaya RS800480T-7X0GP Date: Wed, 10 Jun 2015 10:05:46 +0200 Message-ID: References: <1432827466-8104-1-git-send-email-gary.bisson@boundarydevices.com> <1433865599-8167-1-git-send-email-gary.bisson@boundarydevices.com> <1433865599-8167-3-git-send-email-gary.bisson@boundarydevices.com> <1433871588.5428.42.camel@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <1433871588.5428.42.camel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Philipp Zabel Cc: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, airlied-cv59FeDIM0c@public.gmane.org, Thierry Reding , Eric Nelson List-Id: devicetree@vger.kernel.org Hi Philipp, On Tue, Jun 9, 2015 at 7:39 PM, Philipp Zabel wrote: > Hi Gary, > > Am Dienstag, den 09.06.2015, 17:59 +0200 schrieb Gary Bisson: >> Add support for the Okaya RS800480T-7X0GP to the DRM simple panel >> driver. >> >> The RS800480T-7X0GP is a WVGA (800x480) panel with an 18-bit parallel >> LCD interface. It supports pixel clocks in the range of 30-40 MHz. >> >> This panel details can be found at: >> http://boundarydevices.com/product/7-800x480-display/ >> >> Signed-off-by: Gary Bisson >> --- >> .../bindings/panel/okaya,rs800480t_7x0gp.txt | 7 ++++++ >> drivers/gpu/drm/panel/panel-simple.c | 27 ++++++++++++++++++++++ >> 2 files changed, 34 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/panel/okaya,rs800480t_7x0gp.txt >> >> diff --git a/Documentation/devicetree/bindings/panel/okaya,rs800480t_7x0gp.txt b/Documentation/devicetree/bindings/panel/okaya,rs800480t_7x0gp.txt >> new file mode 100644 >> index 0000000..f7c729d >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/panel/okaya,rs800480t_7x0gp.txt >> @@ -0,0 +1,7 @@ >> +OKAYA Electric America, Inc. RS800480T-7X0GP 7" WVGA LCD panel >> + >> +Required properties: >> +- compatible: should be "okaya,rs800480t_7x0gp" >> + >> +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 97b66b8..e511a22 100644 >> --- a/drivers/gpu/drm/panel/panel-simple.c >> +++ b/drivers/gpu/drm/panel/panel-simple.c >> @@ -942,6 +942,30 @@ static const struct panel_desc lg_lp129qe = { >> }, >> }; >> >> +static const struct display_timing okaya_rs800480t_7x0gp_timing = { >> + .pixelclock = { 30000000, 30000000, 40000000 }, >> + .hactive = { 800, 800, 800 }, >> + .hfront_porch = { 40, 40, 40 }, >> + .hback_porch = { 40, 40, 40 }, >> + .hsync_len = { 1, 48, 48 }, >> + .vactive = { 480, 480, 480 }, >> + .vfront_porch = { 13, 13, 13 }, >> + .vback_porch = { 29, 29, 29 }, >> + .vsync_len = { 3, 3, 3 }, >> + .flags = DISPLAY_FLAGS_DE_HIGH, >> +}; >> + >> +static const struct panel_desc okaya_rs800480t_7x0gp = { >> + .timings = &okaya_rs800480t_7x0gp_timing, >> + .num_timings = 1, >> + .bpc = 6, >> + .size = { >> + .width = 154, >> + .height = 87, >> + }, > > Are there any poweron/poweroff sequencing delays in the datasheet that > should be met? Parallel displays often specify a few ms delay after the > voltage supply is stable and before pixel data should be driven on the > bus (.delay.prepare), as well as afterwards, before activating the > backlight (.delay.enable). You are right I forgot to add those delays. I will submit a v3. That gives prepare = unprepare = 40.5ms (rounded up to 41) and enable = disable = 50ms based on the values provided at page 6/20 of the following doc: http://boundarydevices.com/datasheets/RS800480T-7X0GP-A.pdf Regards, Gary -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html