change log for rtems (2011-08-29)

Joel Sherrill joel.sherrill at OARcorp.com
Tue Aug 30 12:19:15 UTC 2011


On 08/30/2011 02:14 AM, Sebastian Huber wrote:
> On 08/30/2011 12:12 AM, rtems-vc at rtems.org wrote:
>> diff -u rtems/c/src/lib/libbsp/shared/bootcard.c:1.50 rtems/c/src/lib/libbsp/shared/bootcard.c:1.51
>> --- rtems/c/src/lib/libbsp/shared/bootcard.c:1.50	Wed Jul 13 13:35:04 2011
>> +++ rtems/c/src/lib/libbsp/shared/bootcard.c	Mon Aug 29 16:38:09 2011
>> @@ -178,9 +178,9 @@
>>
>>      if ( work_area_size<= Configuration.work_space_size ) {
>>        printk(
>> -      "bootcard: work space too big for work area: %p>   %p\n",
>> -      (void *) Configuration.work_space_size,
>> -      (void *) work_area_size
>> +      "bootcard: work space too big for work area: 0x%08x>   0x%08x\n",
>> +      Configuration.work_space_size,
>> +      work_area_size
>>        );
>>        bsp_cleanup(1);
>>        return 1;
> This change is wrong.  The two parameters have the type uintptr_t.
>
OK.  I spotted this while teaching a class.  The reason for the
cast was not immediately obvious.

Reverting and adding comment.

Thanks.

--joel




More information about the vc mailing list