<div dir="ltr"><div dir="ltr"><br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 26, 2019 at 5:11 PM Christian Mauderer <<a href="mailto:christian.mauderer@embedded-brains.de">christian.mauderer@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 26/07/2019 13:22, Vijay Kumar Banerjee wrote:<br>
> Hello everyone!<br>
> <br>
> I'm excited to post the following patchset. With this patchset<br>
> I have the framebuffer working in BBB and have tested it with a <br>
> BBB revC with HDMI connected Screen. <br>
> <br>
> This patchset uses mmap and hence it's necessary to apply the<br>
> patche ON TOP OF mmap patch to avoid merge conflict.<br>
> <br>
> In reply to this mail, I'll be attaching a picture of the screen<br>
> where my rectangle drawing application made a nice red rectangle<br>
> on the screen just like in FreeBSD.<br>
> <br>
> Thanks and regards,<br>
> Vijay<br>
> <br>
<br>
Hello Vijay,<br>
<br>
it's great that you now have a working driver. Well done.<br>
<br>
I hadn't had a look at most patches yet and most likely that will need a<br>
bit of time. It's a lot of stuff. Most likely the bigger patches won't<br>
reach the mailing list so maybe adding a link to a branch on your github<br>
repo would be good.<br>
<br></blockquote><div>I'll reorder them and push to remote today and add it in this thread. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Note: Please don't send a v2 right now. I'm sure there are more comments<br>
(maybe not only from me). Collect comments for at least one or two days<br>
and create an update afterwards.<br>
<br>
Some notes reading the patch overview:<br>
<br>
> [PATCH 01/10] TDA19988 : Import from FreeBSD<br>
> [PATCH 02/10] TDA19988 : port to RTEMS<br>
> [PATCH 03/10] am335x display drivers: Import from FreeBSD<br>
> [PATCH 04/10] am335x display drivers : Port to RTEMS<br>
> [PATCH 05/10] fbd : Import from FreeBSD<br>
> [PATCH 06/10] fbd : Port to RTEMS<br>
<br>
Is fbd a prerequisite of am335x_lcd? In that case it would have been<br>
better to reorder the patches. As long as every version compiles (to<br>
allow git bisect) I would accept that order too. But as a general rule<br>
prerequisites should be prior to the parts that use it.<br>
<br></blockquote><div>am335x_lcd is the prerequisite for fbd. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> [PATCH 07/10] VT : Import from FreeBSD<br>
> [PATCH 08/10] VT : Port to RTEMS<br>
<br>
Is the vt still necessary? I think it was an intermediate step to try<br>
whether it's initializing the display. Did you try without it?<br>
<br></blockquote><div>I tried without it, it's giving some noise in the screen and the the open</div><div>call to framebuffer is returning error without vt. I guess the vt initialization</div><div>is necessary. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> [PATCH 09/10] TI Pinmux : Import from FreeBSD<br>
> [PATCH 10/10] TI Pinmux : Port to RTEMS<br>
<br>
Again: prerequisite. So they should be one of the first.<br>
<br></blockquote><div>OK. I'll put it in the first. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Best regards<br>
<br>
Christian<br>
-- <br>
--------------------------------------------<br>
embedded brains GmbH<br>
Herr Christian Mauderer<br>
Dornierstr. 4<br>
D-82178 Puchheim<br>
Germany<br>
email: <a href="mailto:christian.mauderer@embedded-brains.de" target="_blank">christian.mauderer@embedded-brains.de</a><br>
Phone: +49-89-18 94 741 - 18<br>
Fax:   +49-89-18 94 741 - 08<br>
PGP: Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
</blockquote></div></div>