<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 22, 2014 at 4:02 AM, Thierry Reding <span dir="ltr"><<a href="mailto:thierry.reding@gmail.com" target="_blank">thierry.reding@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>On Tue, May 20, 2014 at 04:36:48PM -0700, Stéphane Marchesin wrote:<br>



> This panel is used by my tegra board and supported by the simple-panel<br>
> driver.<br>
<br>
</div>One thing I like to have is some indication in the commit message about<br>
which board this panel is used on. Of course if you can't disclose that<br>
yet I'll still take the patch, but perhaps you can be a little more<br>
specific than "my Tegra board".<br>
<br>
There also seem to be different variants of this panel. I've come across<br>
datasheets for B133XTN01.0 and B133XTN01.2. The 1.0 seems to be native<br>
LVDS whereas it looks as if the 1.2 comes with an additional module to<br>
connect it to eDP. Which one is it that you use? I guess independent of<br>
the specific model the timings will still work, but device tree mandates<br>
that the compatible for the most specific model be used. Perhaps this<br>
panel should be:<br>
<br></blockquote><div><br></div><div>I'm using the B133XTN01.2. The panel specs are not exactly the same (the 1.2 has a single clock, 69.5MHz while the 1.0 does 60-80MHz), but the fixed timing that I put in the DT will work on both.</div>


<div><br></div><div>Stéphane</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">



        compatible = "auo,b133xtn01.2", "auo,b133xtn01", "simple-panel";<br>
<br>
That would allow the panel-simple driver to match on "auo,b133xtn01" and<br>
pick up the timing that presumably works for both.<br>
<br>
One other nit below:<br>
<br>
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c<br>
[...]<br>
<div>> +static const struct panel_desc auo_b133xtn01 = {<br>
> +     .modes = &auo_b133xtn01_mode,<br>
> +     .num_modes = 1,<br>
> +     .size = {<br>
> +             .width = 293,<br>
> +             .height = 164,<br>
<br>
</div>According to the datasheet I found this is 164.97 mm, in which case I<br>
think it may be better to round up to get a more accurate value of the<br>
DPI.<br>
<span><font color="#888888"><br>
Thierry<br>
</font></span><br>_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org" target="_blank">dri-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
<br></blockquote></div><br></div></div>