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