<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jul 28, 2019 at 5:11 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 28/07/2019 13:07, Vijay Kumar Banerjee wrote:<br>
> <br>
> <br>
> <br>
> On Sun, Jul 28, 2019 at 1:31 PM Christian Mauderer <<a href="mailto:list@c-mauderer.de" target="_blank">list@c-mauderer.de</a><br>
> <mailto:<a href="mailto:list@c-mauderer.de" target="_blank">list@c-mauderer.de</a>>> wrote:<br>
> <br>
>     On 27/07/2019 21:43, Vijay Kumar Banerjee wrote:<br>
>     ><br>
>     ><br>
>     ><br>
>     > On Sat, Jul 27, 2019 at 7:37 PM Christian Mauderer<br>
>     <<a href="mailto:list@c-mauderer.de" target="_blank">list@c-mauderer.de</a> <mailto:<a href="mailto:list@c-mauderer.de" target="_blank">list@c-mauderer.de</a>><br>
>     > <mailto:<a href="mailto:list@c-mauderer.de" target="_blank">list@c-mauderer.de</a> <mailto:<a href="mailto:list@c-mauderer.de" target="_blank">list@c-mauderer.de</a>>>> wrote:<br>
>     ><br>
>     >     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<br>
>     screen<br>
>     >     >> where my rectangle drawing application made a nice red<br>
>     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<br>
>     will<br>
>     >     need a<br>
>     >     > bit of time. It's a lot of stuff. Most likely the bigger<br>
>     patches won't<br>
>     >     > reach the mailing list so maybe adding a link to a branch on<br>
>     your<br>
>     >     github<br>
>     >     > repo would be good.<br>
>     >     ><br>
>     >     > Note: Please don't send a v2 right now. I'm sure there are more<br>
>     >     comments<br>
>     >     > (maybe not only from me). Collect comments for at least one<br>
>     or two<br>
>     >     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<br>
>     have been<br>
>     >     > better to reorder the patches. As long as every version<br>
>     compiles (to<br>
>     >     > allow git bisect) I would accept that order too. But as a<br>
>     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<br>
>     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<br>
>     been set<br>
>     >     up as 1440x900 instead of it's native resolution. Did you<br>
>     check whether<br>
>     >     the resolution on your screen is set correctly?<br>
>     ><br>
>     > It's not setting fb with the "preferred videomode", I can confirm that<br>
>     > the resolution<br>
>     > set by FreeBSD and RTEMS is same.<br>
> <br>
>     OK. Sooner or later it might would be good to find out how to set<br>
>     another resolution. Currently I have the following output:<br>
> <br>
>     ````<br>
>     Preferred mode: 1920x1200 @ 60Hz<br>
>     Number of extension blocks: 1<br>
>     fb0: detected videomode: 1440x900 @ 60KHz<br>
>     fbd0 on fb0<br>
>     ````<br>
> <br>
>     It's a bit odd that not the preferred mode is used. But if it is the<br>
>     same for FreeBSD it's OK for a start.<br>
> <br>
>     One other problem: I tried to create some solid regions and it seems<br>
>     that there are a few artefacts. Can you try that on your screen and tell<br>
>     me whether you get them too? For the code and an image how it looks like<br>
>     on my screen take a look at<br>
>         <a href="https://gist.github.com/c-mauderer/051689bb1199caf20a08d9844c44c113" rel="noreferrer" target="_blank">https://gist.github.com/c-mauderer/051689bb1199caf20a08d9844c44c113</a><br>
> <br>
>     Some of the lines vanished after a few seconds. I think that could be a<br>
>     memory caching issue.<br>
> <br>
> I see the same. Some lines come back after a few seconds:<br>
> <a href="https://drive.google.com/file/d/1CA61bs0nipUygRpotmyL-Hd8khG_RIbD/view?usp=sharing" rel="noreferrer" target="_blank">https://drive.google.com/file/d/1CA61bs0nipUygRpotmyL-Hd8khG_RIbD/view?usp=sharing</a> <br>
<br>
OK. Like I said: My first guess would be a caching issue. Do you know<br>
where or how the video memory is allocated? Is it a fixed area or<br>
dynamically during initialization?<br>
<br></blockquote><div>Hi,</div><div><br></div><div>Firstly, I have reordered the patches and removed unnecessary stuff from the</div><div>patches. I have also checked that each commit builds individually. The commits</div><div>are pushed in the following branch, please have a look:</div><div><a href="https://github.com/thelunatic/rtems-libbsd/tree/fb-update">https://github.com/thelunatic/rtems-libbsd/tree/fb-update</a></div><div><br></div><div>About video memory: I'm not sure. Maybe following the fbd_register from fbd.c</div><div>in a debugger will give some clue. I will have a look and report.<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
>     ><br>
>     >     Best regards<br>
>     ><br>
>     >     Christian<br>
>     ><br>
> <br>
</blockquote></div></div>