[PATCH 01/23] raspberrypi: Use shared bspreset.c
Sebastian Huber
sebastian.huber at embedded-brains.de
Fri Sep 5 14:01:00 UTC 2014
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.sherrill at 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.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