| Message ID | 1519814443-29147-1-git-send-email-jsarha@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Jyri Sarha <jsarha@ti.com> writes: > Add support for TVC S9700RTWV43TR-01B 800x480 panel found on TI's > AM335X-EVM. > > Signed-off-by: Jyri Sarha <jsarha@ti.com> > Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com> > --- > The only change is added "Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>". > Who is going to pick this up? > > The first version of the patch can be found here: > https://lists.freedesktop.org/archives/dri-devel/2018-February/166902.html Since panels are maintained in drm-misc now, I believe you can just merge it yourself.
On Wed, Feb 28, 2018 at 12:40:43PM +0200, Jyri Sarha wrote: > Add support for TVC S9700RTWV43TR-01B 800x480 panel found on TI's > AM335X-EVM. > > Signed-off-by: Jyri Sarha <jsarha@ti.com> > Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com> > --- > The only change is added "Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>". > Who is going to pick this up? > > The first version of the patch can be found here: > https://lists.freedesktop.org/archives/dri-devel/2018-February/166902.html > > drivers/gpu/drm/panel/panel-simple.c | 26 ++++++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > index 5591984..3b0ba9f 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -1962,6 +1962,29 @@ static const struct panel_desc tpk_f10a_0102 = { > }, > }; > > +static const struct drm_display_mode tvc_s9700rtwv43tr_01b_mode = { > + .clock = 3000, This seems to be missing a 0. Does the panel even work with 3 MHz? I'd be somewhat surprised if it did. Thierry
On 28/02/18 20:44, Thierry Reding wrote: > On Wed, Feb 28, 2018 at 12:40:43PM +0200, Jyri Sarha wrote: >> Add support for TVC S9700RTWV43TR-01B 800x480 panel found on TI's >> AM335X-EVM. >> >> Signed-off-by: Jyri Sarha <jsarha@ti.com> >> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com> >> --- >> The only change is added "Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>". >> Who is going to pick this up? >> >> The first version of the patch can be found here: >> https://lists.freedesktop.org/archives/dri-devel/2018-February/166902.html >> >> drivers/gpu/drm/panel/panel-simple.c | 26 ++++++++++++++++++++++++++ >> 1 file changed, 26 insertions(+) >> >> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c >> index 5591984..3b0ba9f 100644 >> --- a/drivers/gpu/drm/panel/panel-simple.c >> +++ b/drivers/gpu/drm/panel/panel-simple.c >> @@ -1962,6 +1962,29 @@ static const struct panel_desc tpk_f10a_0102 = { >> }, >> }; >> >> +static const struct drm_display_mode tvc_s9700rtwv43tr_01b_mode = { >> + .clock = 3000, > > This seems to be missing a 0. Does the panel even work with 3 MHz? I'd > be somewhat surprised if it did. > Thanks a lot for catching that! I am still debugging how the system could even work with that clock rate. Looks like there is something fishy in our clock framework, the clock wont go down that much but it reports that it did. In reality it produced about 12MHz clock but reported that it was 3Mhz. The 12MHz appeared to be just fine by the panel. But anyway, that is another story, I'll fix that (it was supposed to be 30MHz indeed) and send another patch for review. Best regards, Jyri
diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index 5591984..3b0ba9f 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -1962,6 +1962,29 @@ static const struct panel_desc tpk_f10a_0102 = { }, }; +static const struct drm_display_mode tvc_s9700rtwv43tr_01b_mode = { + .clock = 3000, + .hdisplay = 800, + .hsync_start = 800 + 39, + .hsync_end = 800 + 39 + 47, + .htotal = 800 + 39 + 47 + 39, + .vdisplay = 480, + .vsync_start = 480 + 13, + .vsync_end = 480 + 13 + 2, + .vtotal = 480 + 13 + 2 + 29, + .vrefresh = 62, +}; + +static const struct panel_desc tvc_s9700rtwv43tr_01b = { + .modes = &tvc_s9700rtwv43tr_01b_mode, + .num_modes = 1, + .bpc = 8, + .size = { + .width = 155, + .height = 90, + }, +}; + static const struct display_timing urt_umsh_8596md_timing = { .pixelclock = { 33260000, 33260000, 33260000 }, .hactive = { 800, 800, 800 }, @@ -2214,6 +2237,9 @@ static const struct of_device_id platform_of_match[] = { .compatible = "tpk,f10a-0102", .data = &tpk_f10a_0102, }, { + .compatible = "tvc,s9700rtwv43tr-01b", + .data = &tvc_s9700rtwv43tr_01b, + }, { .compatible = "urt,umsh-8596md-t", .data = &urt_umsh_8596md_parallel, }, {