From mboxrd@z Thu Jan 1 00:00:00 1970 From: York Sun Date: Sun, 13 Dec 2015 18:10:02 -0800 Subject: [U-Boot] [PATCH 3/5][v5] include/linux: move typdef for uintptr_t In-Reply-To: <56670943.70507@nxp.com> References: <1449563070-5761-1-git-send-email-aneesh.bansal@freescale.com> <1449563070-5761-3-git-send-email-aneesh.bansal@freescale.com> <56670943.70507@nxp.com> Message-ID: <566E24FA.6020104@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Resend with my freescale id. I would like to merge this patch 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 >> --- >> 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 >> -#else >> -/* Type for `void *' pointers. */ >> -typedef unsigned long int uintptr_t; >> #endif >> >> #include >> 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 >