<div dir="ltr">Hi all,<div><br></div><div>Ben Gras thanks for your comments and encouragement for merging with mainline as a motivation for me.</div><div><br></div><div>I worked and redeveloped code for gpio driver also tried to approach towards common api. I tried to address most of the comments. Herewith I attached my patch. Would be happy to hear comments on that.</div><div><br></div><div>You can also find out my <a href="https://gist.github.com/ketul93/d717555951174a74c8b4">gist</a> or <a href="https://github.com/ketul93/rtems/commit/e2484b80d8620cc78bf5f453e5cd904c7137a569">commit</a> on <a href="https://github.com/RTEMS/rtems">https://github.com/RTEMS/rtems</a> . </div><div>I have been updating my work on <a href="https://github.com/ketul93/RTEMS-on-BBB">https://github.com/ketul93/RTEMS-on-BBB</a>.</div><div><br></div><div>Live demo of the updated code is found on :- <a href="https://youtu.be/bXurelOA3i0">https://youtu.be/bXurelOA3i0</a> </div><div><br></div><div>Thanks.</div><div><br></div><div>Regards,</div><div>ketul</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 20 April 2015 at 19:50, Ben Gras <span dir="ltr"><<a href="mailto:beng@shrike-systems.com" target="_blank">beng@shrike-systems.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">All,<br>
<br>
Good contribution, thank you.<br>
<br>
For GSOC, this is good proof of being able to progress and get<br>
something real working based on documentation. Great!<br>
<br>
If you want this merged with mainline - which I fully encourage! -<br>
then I suggest the following should be added to/changed first:<br>
<br>
  - make the code that uses AM335X (beaglebone) specific registers,<br>
AM335X-specific :) i.e. add  #if IS_AM335X around code that should<br>
only execute there. this BSP can be compiled for 2 different SOCs.<br>
  - let it control all GPIO's - there are 4 banks of 32 each on the<br>
AM335x, but you only let the user control GPIO1. there should be a<br>
clean interface to control them all (clean means mostly: without<br>
duplicating code)<br>
  - as Gedare mentioned on IRC, copy the RPi API. this is a first<br>
(second?) step to finding a generalized GPIO API.<br>
  - don't add beagleboard.h, but add your definitions to<br>
libcpu/arm/shared/include/am335x.h<br>
  - the code should use a more consistent indenting style, and make<br>
variable names more descriptive than 'tmp'.<br>
  - don't leave the configure changes in like in acinclude.m4<br>
<br>
bonus:<br>
  - add DM3730 (beagleboard) support.<br>
<br>
Good luck!<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, Apr 17, 2015 at 7:51 PM, Ketul Shah <<a href="mailto:ketulshah1993@gmail.com">ketulshah1993@gmail.com</a>> wrote:<br>
> Hello,<br>
><br>
> I have proposed in GSoC on Beagle BSP Improvements. As starting I started<br>
> working for gpio driver development . At this stage I am able to demonstrate<br>
> USR LEDs pattern.<br>
><br>
> Videos of that can be found here on YouTube.<br>
> <a href="https://youtu.be/B3mSsfo-PAQ" target="_blank">https://youtu.be/B3mSsfo-PAQ</a> &<br>
> <a href="https://youtu.be/M1aKpOhUvv4" target="_blank">https://youtu.be/M1aKpOhUvv4</a>.<br>
><br>
> Herewith I have attached patch.txt file. Alternatively you can have<br>
> <a href="https://gist.github.com/ketul93/2e0d2c4b8b586be62e1d" target="_blank">https://gist.github.com/ketul93/2e0d2c4b8b586be62e1d</a> that includes the newly<br>
> added files code. I would be happy to hear your reviews and changes on my<br>
> work.<br>
><br>
> And also I have been updating my work on<br>
> <a href="https://github.com/ketul93/RTEMS-on-BBB" target="_blank">https://github.com/ketul93/RTEMS-on-BBB</a> repo.<br>
><br>
> It would be great to have your comments on my proposal.<br>
><br>
> Thanks and regards,<br>
> ketul<br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</div></div></blockquote></div><br></div>