[PATCH v4 02/10] lib: elf: Move the generic elf loading/validating functions to lib

[Tom Rini]

On Mon, Jan 27, 2020 at 10:46:19AM +0530, Keerthy wrote:
> 
> 
> On 27/01/20 10:01 am, Keerthy wrote:
> > 
> > 
> > On 24/01/20 8:24 pm, Tom Rini wrote:
> > > On Wed, Jan 22, 2020 at 09:29:57AM +0530, Keerthy wrote:
> > > > Move the generic elf loading/validating functions to lib/
> > > > so that they can be re-used and accessed by code existing
> > > > outside cmd.
> > > > 
> > > > Signed-off-by: Keerthy <j-keerthy at ti.com>
> > > > Suggested-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> > > > Reviewed-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> > > [snip]
> > > > diff --git a/lib/Kconfig b/lib/Kconfig
> > > > index d040a87d26..5ca86cd7fb 100644
> > > > --- a/lib/Kconfig
> > > > +++ b/lib/Kconfig
> > > > @@ -601,4 +601,7 @@ config TEST_FDTDEC
> > > >   config LIB_DATE
> > > >       bool
> > > > +config LIB_ELF
> > > > +    bool "enable basic elf loading/validating functions"
> > > > +
> > > >   endmenu
> > > 
> > > This shouldn't be a user-selectable symbol, it should be select'd as
> > > needed.  So no bool text but a help text that mentions what it does
> > > provide is needed.  Thanks!
> 
> Tom,
> 
> Just for my confirmation. Do you want the symbol type to be removed or just
> the bool text? I believe symbol type is must.

I want it converted from a user visible symbol, which in Kconfig means:
config VARIABLE
    TYPE "Some text the user sees"
    help
        Description
and made a non-visible symbol which means:
config VARIABLE
    TYPE
    help
        Description

Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200127/7a07f086/attachment.sig>