• Rasmus Villemoes's avatar
    make env_entry::callback conditional on !CONFIG_SPL_BUILD · 080019b8
    Rasmus Villemoes authored
    The callback member of struct env_entry is always NULL for an SPL
    build. Removing it thus saves a bit of run-time memory in the
    SPL (when CONFIG_SPL_ENV_SUPPORT=y) since struct env_entry is embedded
    in struct env_entry_node - i.e. about 2KB for the normal case of
    512+change hash table entries.
    Two small fixups are needed for this, all other references to the
    callback member are already under !CONFIG_SPL_BUILD: Don't initialize
    .callback in set_flags() - hsearch_r doesn't use that value
    anyway. And make env_callback_init() initialize ->callback to NULL for
    a new entry instead of relying on an unused or deleted entry having
    NULL in ->callback.
    Signed-off-by: default avatarRasmus Villemoes <rasmus.villemoes@prevas.dk>
    Reviewed-by: Simon Glass's avatarSimon Glass <sjg@chromium.org>