<div dir="ltr"><div dir="ltr"><br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jul 27, 2019 at 7:37 PM Christian Mauderer <<a href="mailto:list@c-mauderer.de">list@c-mauderer.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:41, Christian Mauderer wrote:<br>
> 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>
> 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>
>> [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>
>> [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>
> Best regards<br>
> <br>
> Christian<br>
> <br>
<br>
One general question: When trying the patches, my monitor has been set<br>
up as 1440x900 instead of it's native resolution. Did you check whether<br>
the resolution on your screen is set correctly?<br>
<br></blockquote><div>It's not setting fb with the "preferred videomode", I can confirm that the resolution</div><div>set by FreeBSD and RTEMS is same. </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>
</blockquote></div></div>