[PATCH v3 3/5] arm: stm32mp: Implement support for TZC 400 controller

Alex G. mr.nuke.me at gmail.com
Mon May 31 19:16:26 CEST 2021


Hi Patrick,

On 5/28/21 4:59 AM, Patrick DELAUNAY wrote:
> Hi,
> 
> Any reason to prefer  uint16_t and uint32_t ?

I use standard C types. u32, _u32, and __u32 are non-standard, and they 
have different meanings depending on how they're used. I avoid them for 
this reason.

Alex

> See checkpatch warning
> 
> arch/arm/mach-stm32mp/include/mach/tzc.h:24: check: Prefer kernel type 
> 'u16' over 'uint16_t'
> arch/arm/mach-stm32mp/include/mach/tzc.h:25: check: Prefer kernel type 
> 'u16' over 'uint16_t'
> arch/arm/mach-stm32mp/tzc400.c:41: check: Prefer kernel type 'u16' over 
> 'uint16_t'
> arch/arm/mach-stm32mp/tzc400.c:52: check: Prefer kernel type 'u32' over 
> 'uint32_t'
> arch/arm/mach-stm32mp/tzc400.c:81: check: Prefer kernel type 'u32' over 
> 'uint32_t'
> arch/arm/mach-stm32mp/tzc400.c:82: check: Prefer kernel type 'u32' over 
> 'uint32_t'
> arch/arm/mach-stm32mp/tzc400.c:93: check: Prefer kernel type 'u32' over 
> 'uint32_t'
> arch/arm/mach-stm32mp/tzc400.c:94: check: Prefer kernel type 'u32' over 
> 'uint32_t'
> arch/arm/mach-stm32mp/tzc400.c:113: check: Prefer kernel type 'u32' over 
> 'uint32_t'
> 
> But except these remarks:
> 
> Reviewed-by: Patrick Delaunay <patrick.delaunay at foss.st.com>
> 
> Thanks
> Patrick
> 


More information about the U-Boot mailing list