<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<pre style="word-wrap:break-word; font-size:10.0pt; font-family:Tahoma; color:black">I think so. One BSP per patch. You may want to send in one BSP for review first. Then if you are moving quickly, send a batch of patches for an entire architecture. But one per BSP.
Uses outside the BSP can be similarly packaged.
Vipul Nayyar <nayyar_vipul@yahoo.com> wrote:
</pre>
<div>
<div style="color:#000; background-color:#fff; font-family:arial, helvetica, sans-serif;font-size:10pt">
<div>Just checking, that the right way to move forward for me would be identifying legacy code and submitting patches ?? like I did with removing the use of BSP_install_rtems_irq_handler from arm csb336.</div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
<br>
</div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
Regards</div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
<span style="background-color:transparent;line-height:1.22;">Vipul Nayyar </span></div>
<div style="color: rgb(0, 0, 0); font-size: 12.727272033691406px;
font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
<br>
</div>
<div><br>
</div>
<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt;">
<div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;">
<div dir="ltr">
<hr size="1">
<font size="2" face="Arial"><b><span style="font-weight:bold;">From:</span></b> Joel Sherrill <Joel.Sherrill@OARcorp.com><br>
<b><span style="font-weight: bold;">To:</span></b> Vipul Nayyar <nayyar_vipul@yahoo.com>
<br>
<b><span style="font-weight: bold;">Cc:</span></b> Sebastian Huber <sebastian.huber@embedded-brains.de>; "rtems-devel@rtems.org" <rtems-devel@rtems.org>
<br>
<b><span style="font-weight: bold;">Sent:</span></b> Thursday, 27 June 2013 2:16 PM<br>
<b><span style="font-weight: bold;">Subject:</span></b> Re: [Patch] Removing legacy method from arm csb336<br>
</font></div>
<div class="y_msg_container"><br>
<div id="yiv2055406821">
<div>
<pre style="word-wrap: break-word; font-size: 10pt; font-family: Tahoma; color: black;">The goal of this is to kill bad examples. It is legacy and we don't want people to use it anymore. These should be mechanical changes. Vipul should be able to provide instructions for users with external drivers to make the changes.
Fixing these could break here or there but if we don't remove uses of deprecated methods then users will use them in new BSPs and that is unacceptable.
Vipul Nayyar <nayyar_vipul@yahoo.com> wrote:
</pre>
<div>
<div style="color: rgb(0, 0, 0); background-color: rgb(255, 255, 255); font-family: arial, helvetica, sans-serif; font-size: 10pt;">
<div><span>Hello,</span></div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, sans-serif; background-color: transparent; font-style: normal;">
<span><br>
</span></div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, sans-serif; background-color: transparent; font-style: normal;">
<span>According to the discussions with my mentor Joel Sherill, I was suggested to kill all legacy/deprecated API uses in the tree which are defined in irq-legacy.c as part of my project. </span><span style="background-color:transparent;">I guess Dr Joel can
guide us better whether all of the deprecated methods need to be removed or only some of them. cc'ing Dr Joel for guidance on this.</span></div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
<br>
</div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
Regards</div>
<div style="color: rgb(0, 0, 0); font-size: 13px; font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
<span style="background-color:transparent;line-height:1.22;">Vipul Nayyar </span></div>
<div style="color: rgb(0, 0, 0); font-size: 12.727272033691406px; font-family: arial, helvetica, clean, sans-serif; background-color: transparent; font-style: normal;">
<br>
</div>
<div><br>
</div>
<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt;">
<div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;">
<div dir="ltr">
<hr size="1">
<font size="2" face="Arial"><b><span style="font-weight:bold;">From:</span></b> Sebastian Huber <sebastian.huber@embedded-brains.de><br>
<b><span style="font-weight:bold;">To:</span></b> rtems-devel@rtems.org <br>
<b><span style="font-weight:bold;">Sent:</span></b> Thursday, 27 June 2013 1:48 PM<br>
<b><span style="font-weight:bold;">Subject:</span></b> Re: [Patch] Removing legacy method from arm csb336<br>
</font></div>
<div class="yiv2055406821y_msg_container"><br>
On 06/26/2013 05:28 PM, Vipul Nayyar wrote:<br>
> <br>
> For my GSOC Project Unified APIs, a public google doc has been setup (<br>
> <a rel="nofollow" target="_blank" href="https://docs.google.com/document/d/1W9DMnpocXUKXiQMxHbbp0aG-ecuh94P-3N7hxEJmcgs/edit?usp=sharing">
https://docs.google.com/document/d/1W9DMnpocXUKXiQMxHbbp0aG-ecuh94P-3N7hxEJmcgs/edit?usp=sharing</a>)<br>
> for community viewing. In order to establish this unified pattern across all<br>
> BSPs, I felt that deprecated methods stored in irq-legacy.c should be dealt<br>
> with first. So, I'm attaching a very basic patch which removes the deprecated<br>
> method BSP_install_rtems_irq_handler from arm csb336 and instead makes use<br>
> of rtems_interrupt_handler_install, as suggested in irq-legacy.c. Please do<br>
> give suggestions if I'm heading the right way, and any modifications that this<br>
> patch may require.<br>
<br>
I would not touch the existing code that uses these legacy functions. The irq-legacy.c is provided to keep existing code as is.<br>
<br>
-- Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone : +49 89 189 47 41-16<br>
Fax : +49 89 189 47 41-09<br>
E-Mail : <a rel="nofollow" ymailto="mailto:sebastian.huber@embedded-brains.de" target="_blank" href="mailto:sebastian.huber@embedded-brains.de">
sebastian.huber@embedded-brains.de</a><br>
PGP : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
_______________________________________________<br>
rtems-devel mailing list<br>
<a rel="nofollow" ymailto="mailto:rtems-devel@rtems.org" target="_blank" href="mailto:rtems-devel@rtems.org">rtems-devel@rtems.org</a><br>
<a rel="nofollow" target="_blank" href="http://www.rtems.org/mailman/listinfo/rtems-devel">http://www.rtems.org/mailman/listinfo/rtems-devel</a><br>
<br>
<br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
<br>
<br>
</div>
</div>
</div>
</div>
</div>
</body>
</html>