From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Tue, 25 Sep 2012 22:24:32 +0200
Subject: [U-Boot] [PATCH 2/3 V2] common: Implement support for
	linker-generated arrays
In-Reply-To: <20120925201612.676E2200335@gemini.denx.de>
References: <1348449718-16463-1-git-send-email-marex@denx.de>
	<1348589520-14395-3-git-send-email-marex@denx.de>
	<20120925201612.676E2200335@gemini.denx.de>
Message-ID: <201209252224.32686.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Wolfgang Denk,

> Dear Marek Vasut,
> 
> In message <1348589520-14395-3-git-send-email-marex@denx.de> you wrote:
> > This patch adds support for linker-generated array. These arrays
> > are a generalization of the U-Boot command declaration approach.
> 
> ...
> 
> >                                         ... Such array though can
> > 
> > later be accessed and used via special accessor.
> 
> Can you please translate this to me?  I don't get what it means.

That you call a function that returns the pointer to the first member of the 
array.

> 
> > The actual implementation relies on placing any variable that is to
> > represent an element of LG-array into subsection of .u_boot_list
> 
> "into subsection" ? Do you mean "into a subsection", or "into a
> specific subsection" or what?


... into a matching subsection ...

> ...
> 
> > --- /dev/null
> > +++ b/include/linker_lists.h
> 
> ...
> 
> > +/**
> > + * ll_entry_declare() - Declare linker-generated array entry
> 
> ...
> 
> Incorrect multiline comment style. Please fix globally.

You mean what ...

/**
 *

This? Kernel-doc style ...

> Best regards,
> 
> Wolfgang Denk

Best regards,
Marek Vasut