<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">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:#000; background-color:#fff; 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="y_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 href="https://docs.google.com/document/d/1W9DMnpocXUKXiQMxHbbp0aG-ecuh94P-3N7hxEJmcgs/edit?usp=sharing" target="_blank">
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 ymailto="mailto:sebastian.huber@embedded-brains.de" 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 ymailto="mailto:rtems-devel@rtems.org" href="mailto:rtems-devel@rtems.org">rtems-devel@rtems.org</a><br>
<a href="http://www.rtems.org/mailman/listinfo/rtems-devel" target="_blank">http://www.rtems.org/mailman/listinfo/rtems-devel</a><br>
<br>
<br>
</div>
</div>
</div>
</div>
</div>
</body>
</html>