[PATCH 14/35] common: Drop CONFIG_HAS_POST

[Simon Glass]

Hi Tom,

On Thu, 12 Dec 2019 at 06:59, Tom Rini <trini at konsulko.com> wrote:
>
> On Wed, Dec 11, 2019 at 05:47:33PM -0700, Simon Glass wrote:
> > This only exists to control whether the post/ directory is build. It is
> > just as easy to check this in the Makefile. Remove CONFIG_HAS_POST and use
> > an ifdef in the Makefile instead.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> >  Makefile         | 4 +++-
> >  include/common.h | 4 ----
> >  2 files changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/Makefile b/Makefile
> > index 0766f78dcb..6b7d80139e 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -762,7 +762,9 @@ libs-y += cmd/
> >  libs-y += common/
> >  libs-y += env/
> >  libs-$(CONFIG_API) += api/
> > -libs-$(CONFIG_HAS_POST) += post/
> > +ifdef CONFIG_POST
> > +libs-y += post/
> > +endif
>
> Why isn't this just libs-$(CONFIG_POST) += post/ ?

Unfortunately this is not a boolean CONFIG, and doesn't even use
Kconfig. See for example xpedite517x.h:

#define CONFIG_POST                     (CONFIG_SYS_POST_MEMORY |\
                                         CONFIG_SYS_POST_I2C)

Regards,
Simon