[U-Boot] [PATCH 3/5] OMAP3: make gpmc_config as const
Paulraj, Sandeep
s-paulraj at ti.com
Sun Oct 11 14:22:06 CEST 2009
>
> Nishanth Menon wrote:
> > gpmc_config should not be a variant as it is board specific
> > hence make it a const parameter
>
> Having this in u-boot-ti/next results in
>
> - All non-SDP3430 boards have
>
> mem.c: In function 'gpmc_init':
> mem.c:250: warning: assignment discards qualifiers from pointer target
> type
>
> - Zoom2 fails to build with
>
> zoom2.c:54: error: conflicting types for 'enable_gpmc_cs_config'
> /include/asm/arch/sys_proto.h:38: error: previous declaration of
> 'enable_gpmc_cs_config' was here
>
> Reverting this patch makes both go away.
>
> Then we get
>
> sdp.c: In function 'board_init':
> sdp.c:109: warning: passing argument 1 of 'enable_gpmc_cs_config'
> discards qualifiers from pointer target type
> sdp.c:111: warning: passing argument 1 of 'enable_gpmc_cs_config'
> discards qualifiers from pointer target type
> sdp.c:113: warning: passing argument 1 of 'enable_gpmc_cs_config'
> discards qualifiers from pointer target type
> sdp.c:115: warning: passing argument 1 of 'enable_gpmc_cs_config'
> discards qualifiers from pointer target type
>
> for SDP3430 only, though.
>
> So I'd like to have this patch reverted in u-boot-ti/next and then
>
> a) Fix sdp.c only
>
> or
>
> b) test the other boards, too, before applying this patch.
>
> Best regards
>
> Dirk
>
My bad, should have compiled all other boards as well.
I have reverted.
Thanks,
Sandeep
More information about the U-Boot
mailing list