[U-Boot] [PATCH 4/4] imx-common: consolidate macros and prototypes into sys_proto.h

Peng Fan b51431 at freescale.com
Thu Aug 13 02:26:04 CEST 2015


Hi Stefano,
On Wed, Aug 12, 2015 at 05:49:15PM +0200, Stefano Babic wrote:
>Hi Peng,
>
>On 10/08/2015 14:53, Peng Fan wrote:
>> Move most macro definitions and prototypes into
>> "arch/arm/include/asm/imx-common/sys_proto.h" to avoid duplicated
>> function prototypes and marco definitions for different i.MX SoCs.
>> 
>> This patch do not remove the sys_proto.h for different i.MX SoCs,
>> because we need to modify lots of driver code and others. This patch
>> remove duplicated macros and prototypes and incude "sys_proto.h"
>> of imx-common for each sys_proto.h of different i.MX platforms.
>> Then later we should avoid add stuff in sys_proto.h of each platform,
>> and modify driver to include common sys_proto.h.
>> 
>> Signed-off-by: Peng Fan <Peng.Fan at freescale.com>
>> Cc: Stefano Babic <sbabic at denx.de>
>> ---
>>  arch/arm/include/asm/arch-mx31/sys_proto.h  |  7 ++---
>>  arch/arm/include/asm/arch-mx35/sys_proto.h  | 12 ++++----
>>  arch/arm/include/asm/arch-mx5/sys_proto.h   | 22 +-------------
>>  arch/arm/include/asm/arch-mx6/sys_proto.h   | 45 +----------------------------
>>  arch/arm/include/asm/arch-mxs/sys_proto.h   | 15 ++--------
>>  arch/arm/include/asm/imx-common/sys_proto.h | 45 +++++++++++++++++++++++++++++
>>  6 files changed, 59 insertions(+), 87 deletions(-)
>>  create mode 100644 arch/arm/include/asm/imx-common/sys_proto.h
>> 
>
>This breaks MX35 boards:
>
>+In file included from include/asm/arch/sys_proto.h:11:0,
>+                 from arch/arm/cpu/arm1136/mx35/mx35_sdram.c:11:
>+arch/arm/include/asm/imx-common/sys_proto.h:38:23: error: unknown type
>name 'bd_t'
>
>Reason is:
>
>int fecmxc_initialize(bd_t *bis);
>
>Can you check and resubmit ? Thanks !

Oh. I'll check this. I missed to build mx35 target.

[...]

Regards,
Peng.
-- 


More information about the U-Boot mailing list