On 05/09/14 15:48, Joel Sherrill wrote:
On 9/5/2014 1:40 AM, Sebastian Huber wrote:
>On 04/09/14 18:16, Gedare Bloom wrote:
>>On Wed, Sep 3, 2014 at 11:25 AM, Joel Sherrill
>><joel.sherr...@oarcorp.com> wrote:
>>>>---
>>>> c/src/lib/libbsp/arm/csb337/startup/bspreset.c | 1 +
>>>> c/src/lib/libbsp/arm/raspberrypi/Makefile.am | 2 +-
>>>> .../lib/libbsp/arm/raspberrypi/startup/bspreset.c | 35 ------------
>>>> testsuites/sptests/spintr_err01/init.c | 65
+++++++++++-----------
>>>> 4 files changed, 34 insertions(+), 69 deletions(-)
>>>> delete mode 100644 c/src/lib/libbsp/arm/raspberrypi/startup/bspreset.c
>>>>
>>>>diff --git a/c/src/lib/libbsp/arm/csb337/startup/bspreset.c
b/c/src/lib/libbsp/arm/csb337/startup/bspreset.c
>>>>index 1167d92..940ff7f 100644
>>>>--- a/c/src/lib/libbsp/arm/csb337/startup/bspreset.c
>>>>+++ b/c/src/lib/libbsp/arm/csb337/startup/bspreset.c
>>>>@@ -17,6 +17,7 @@ void bsp_reset(void)
>>>> {
>>>> rtems_interrupt_level level;
>>>>
>>>>+ (void) level;
>>>> rtems_interrupt_disable(level);
>>Is it ok to put (void) level; and then use level?
>>
>Logically it makes more sense to use
>
>rtems_interrupt_disable(level);
>(void) level;
>
>since rtems_interrupt_disable() sets the level, but we are only interested in
>the side-effects of rtems_interrupt_disable() and not the level itself.
>
Do you want me to swap the lines?
No, I don't think its that important.
--
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.hu...@embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel