[U-Boot] [PATCH] arm: imx: Fix conflicting definition types of 'get_reset_cause'

Otavio Salvador otavio at ossystems.com.br
Mon Mar 2 19:35:15 CET 2015


On Mon, Mar 2, 2015 at 3:32 PM, Stefano Babic <sbabic at denx.de> wrote:
> Hi Marek, Otavio,
>
> On 02/03/2015 19:27, Marek Vasut wrote:
>> On Monday, March 02, 2015 at 07:10:58 PM, Otavio Salvador wrote:
>>> The common i.MX definition of 'get_reset_cause' function is non-static
>>> so the overriden ones should follow it. This fixes following error:
>>>
>>> ,----
>>>
>>> |  arch/arm/imx-common/cpu.c:29:14: error: static declaration
>>> |
>>> |   of 'get_reset_cause' follows non-static declaration
>>> |   static char *get_reset_cause(void)
>>>
>>> `----
>>>
>>> The fix has been done in all i.MX based SoCs plus vf610.
>>>
>>> Fixes [YOCTO: #7384].
>>>
>>> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
>>
>> You should instead drop the prototype from sys_proto.h and repair
>> the only one non-static occurance in arch/arm/imx-common/cpu.c .
>>
>
> This is the fix I sent this morning:
>
> http://patchwork.ozlabs.org/patch/445002/

Sorry; I missed this one. This fixes the issue. :-D

Thanks!

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


More information about the U-Boot mailing list