[U-Boot] [PATCH 2/3] image: move all function comments to header file

York Sun yorksun at freescale.com
Sat Aug 16 00:10:36 CEST 2014


On 08/15/2014 03:07 PM, Bryan Wu wrote:
> On Fri, Aug 15, 2014 at 3:01 PM, Jeroen Hofstee <dasuboot at myspectrum.nl> wrote:
>> Hello Bryan,
>>
>>
>> On 15-08-14 22:55, Bryan Wu wrote:
>>>
>>> Several functions comments are C file with function definition, they
>>> should be moved to header file with function declaration.
>>>
>>> Also update genimg_get_kernel_addr() comments for CONFIG_FIT case.
>>>
>>> Signed-off-by: Bryan Wu <pengw at nvidia.com>
>>>
>>
>> Why _should_ this be done. In general I would not do it
>> to keep comment and implementation close to each other.
>> (In the hope they actually match). Doxygen and likely the
>> kernel doc thing can pick this up. The only reason I can
>> think of this being useful is for proprietary code with a public
>> api, but this is not applicable for u-boot.
>>
> 
> I was asked to do that by Simon and right now in image.c and image.h
> it's quite mix.
> Some of them are in C code with implementation others are in header
> file with declaration.
> 
> I was confused by this in u-boot, although in kernel we put comments
> in C code with implementation.
> 

I prefer to see comments near the implementations.

York




More information about the U-Boot mailing list