<div dir="ltr"><div dir="ltr"><br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Aug 11, 2019 at 8:40 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 04/08/2019 18:41, Christian Mauderer wrote:<br>
> Hello Vijay,<br>
> <br>
> for the whole patch set:<br>
> <br>
> The order and groups are good now. I didn't find anything where I would<br>
> have big problems. All patches are compilable and libbsd still compiles<br>
> on a PowerPC target. The result is great (with some demo application I<br>
> get an image on the screen). So I'm OK with this version of the patch set.<br>
> <br>
> I'm still not entirely happy with the pinmux because it re-initializes<br>
> some pins that are already initialized by the BSP. But if no one has a<br>
> better idea that can be done in a reasonable time I would accept it like<br>
> it is. I think I would prefer it without the hack that needs an extra<br>
> overlay. If the patch set is committed in this form, we will need a<br>
> ticket for that (wait with creating it till the commit) so that a future<br>
> project can work on that ticket.<br>
> <br>
> Best regards<br>
> <br>
> Christian<br>
<br>
Hello Vijay,<br>
<br>
I pushed the patch set just now and created a ticket for the pinmux:<br>
<br>
<a href="https://devel.rtems.org/ticket/3782#ticket" rel="noreferrer" target="_blank">https://devel.rtems.org/ticket/3782#ticket</a></blockquote><div>Hi!</div><div><br></div><div>It's great to see all of it merged. Thanks for all the help and guidance.</div><div>It was an amazing experience working on this project and learning so</div><div>much from you.<br></div><div><br></div><div>Thank you,</div><div>Vijay</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br></blockquote><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>
> On 03/08/2019 22:19, Vijay Kumar Banerjee wrote:<br>
>> ---<br>
>>  .../sys/arm/ti/am335x/am335x_scm_padconf.c    | 305 ++++++++++++<br>
>>  .../sys/arm/ti/am335x/am335x_scm_padconf.h    |  47 ++<br>
>>  freebsd/sys/arm/ti/omap4/omap4_scm_padconf.h  |  83 ++++<br>
>>  freebsd/sys/arm/ti/ti_pinmux.c                | 463 ++++++++++++++++++<br>
>>  freebsd/sys/arm/ti/ti_pinmux.h                |  80 +++<br>
>>  freebsd/sys/dev/fdt/fdt_pinctrl.c             | 160 ++++++<br>
>>  6 files changed, 1138 insertions(+)<br>
>>  create mode 100644 freebsd/sys/arm/ti/am335x/am335x_scm_padconf.c<br>
>>  create mode 100644 freebsd/sys/arm/ti/am335x/am335x_scm_padconf.h<br>
>>  create mode 100644 freebsd/sys/arm/ti/omap4/omap4_scm_padconf.h<br>
>>  create mode 100644 freebsd/sys/arm/ti/ti_pinmux.c<br>
>>  create mode 100644 freebsd/sys/arm/ti/ti_pinmux.h<br>
>>  create mode 100644 freebsd/sys/dev/fdt/fdt_pinctrl.c<br>
</blockquote></div></div>