[Patch] Removing legacy method from arm csb337
Vipul Nayyar
nayyar_vipul at yahoo.com
Sun Jul 7 13:34:54 UTC 2013
Hello,
Thanks for accepting this patch. I'll work on the guidelines given by you.
Before modifying the csb336 patch, and moving further with other bsps, Can I consider the work for this bsp closed ?
Regards
Vipul Nayyar
________________________________
From: Sebastian Huber <sebastian.huber at embedded-brains.de>
To: Vipul Nayyar <nayyar_vipul at yahoo.com>
Cc: Joel Sherrill <joel.sherrill at oarcorp.com>; "rtems-devel at rtems.org" <rtems-devel at rtems.org>
Sent: Sunday, 7 July 2013 6:39 PM
Subject: Re: [Patch] Removing legacy method from arm csb337
Hello,
I checked in a slightly modified version:
http://git.rtems.org/rtems/commit/?id=6886a485301330b1347b89432b2c49287dc41010
Some remarks:
1. Please don't use
type func()
in C, use
type func(void)
instead.
2. Don't introduce trailing white space. You can use
git config --global --add diff.color auto
git config --global color.ui auto
to instruct Git to highlight this in "git diff" for you.
3. Use "git format-patch" to produce patches. You can use "git send-email" to send this to the list. Example configuration:
git config --global --add sendemail.to rtems-devel at rtems.org
git config --global --add sendemail.smtpserver your.server
git config --global --add sendemail.from 'Your Name <your.name at x.y>'
git config --global --add sendemail.suppresscc all
On 06/07/13 19:06, Vipul Nayyar wrote:
> Thanks for the advice. In fact I removed most of the warnings with the help of Google only in my earlier patch.
> I've checked and removed functions at91rm9200_emac_isr_off() & at91rm9200_emac_isr_is_on() since they're defined but not used.
> I've made phywrite() & phyread() static since they're not used anywhere else.
>
> But still I'm getting a warning saying : 'phyread' defined but not used [-Wunused-function] . Although it is used a couple of times in the file, but still I'm getting this warning. Maybe it has something to do with being a static function. Please try this patch.
It was used only if csb637 was defined.
-- Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.huber at embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20130707/59e202a7/attachment-0001.html>
More information about the devel
mailing list