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

[Keerthy]

On 27/01/20 7:46 pm, Tom Rini wrote:
> 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. Clear now :-)

> 
> Thanks!
>