[U-Boot] [PATCH 3/5][v5] include/linux: move typdef for uintptr_t

York Sun
2015-12-14T02:08:50Z

I would like to merge this one if no objection.

York

On 12/08/2015 08:45 AM, York Sun wrote:
>
>
> On 12/08/2015 12:24 AM, Aneesh Bansal wrote:
>> uintptr_t which is a typdef for unsigned long is needed for creating
>> pointers (32 or 64 bit depending on Core) from 32 bit variables
>> storing the address.
>> If a 32 bit variable (u32) is typecasted to a pointer (void *),
>> compiler gives a warning in case size of pointer on the core is 64 bit.
>>
>> The typdef has been moved from include/compiler.h to include/linux/types.h
>>
>> Signed-off-by: Aneesh Bansal <aneesh.bansal@freescale.com>
>> ---
>> Changes in v5:
>> New patch instead of http://patchwork.ozlabs.org/patch/546319/
>>
>> Changes in v4: None
>>
>> Changes in v3: None
>>
>> Changes in v2: None
>>
>> include/compiler.h | 3 ---
>> include/linux/types.h | 1 +
>> 2 files changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/include/compiler.h b/include/compiler.h
>> index 47c296e..f853ed4 100644
>> --- a/include/compiler.h
>> +++ b/include/compiler.h
>> @@ -115,9 +115,6 @@ typedef unsigned int uint;
>> #ifdef CONFIG_USE_STDINT
>> /* Provided by gcc. */
>> #include <stdint.h>
>> -#else
>> -/* Type for `void *' pointers. */
>> -typedef unsigned long int uintptr_t;
>> #endif
>>
>> #include <linux/string.h>
>> diff --git a/include/linux/types.h b/include/linux/types.h
>> index 6f75be4..c7e8fdb 100644
>> --- a/include/linux/types.h
>> +++ b/include/linux/types.h
>> @@ -124,6 +124,7 @@ typedef __UINT64_TYPE__ u_int64_t;
>> typedef __INT64_TYPE__ int64_t;
>> #endif
>>
>> +typedef unsigned long uintptr_t;
>> /*
>> * Below are truly Linux-specific types that should never collide with
>> * any application/library that wants linux/types.h.
>>
>
> Tom,
>
> Please help review this change. I can compile test for power and arm but I don't
> have the setup for other platforms.
>
> York
>