On 2014-11-13 10:23, Sebastian Huber wrote:
On 13/11/14 10:09, Daniel Cederman wrote:
@@ -407,6 +410,8 @@ static rtems_interval    ticksPerSec;        /*
cached system clock rate (WHO IS ASSUMED
                                               */

  rtems_task_priority        rpciodPriority = 0;
+cpu_set_t            *rpciodCpuset = 0;
+size_t                rpciodCpusetSize = 0;

  #if (DEBUG) & DEBUG_MALLOC
  /* malloc wrappers for debugging */

Missing #ifdef RTEMS_SMP ?

I am not sure if we should really use #ifdef RTEMS_SMP for the #include
<sys/cpuset.h>.  We use also #if defined(__RTEMS_HAVE_SYS_CPUSET_H__) &&
defined(RTEMS_SMP) sometimes.  I would simply force users to update
their Newlib version.

Please fix the spacing and use the coding style of the file.


Thank you for you comments. I removed the #ifdef RTEMS_SMP around <sys/cpuset.h> and have hopefully fixed the spacing issues. I will submit new versions of the patches.

--
Daniel Cederman
Software Engineer
Aeroflex Gaisler AB
Aeroflex Microelectronic Solutions – HiRel
Kungsgatan 12
SE-411 19 Gothenburg, Sweden
Phone: +46 31 7758665
ceder...@gaisler.com
www.Aeroflex.com/Gaisler
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to