[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