[Patch] Removing legacy method from arm csb337
Sebastian Huber
sebastian.huber at embedded-brains.de
Sun Jul 7 13:09:45 UTC 2013
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.
More information about the devel
mailing list