Re: [PATCH v4 1/4] tools: Separate image types which depend on OpenSSL

Heinrich Schuchardt
2021-10-22T10:09:33Z

On 10/21/21 15:00, Marek Behún wrote:
> BTW, wouldn't it be enough to simply imply TOOLS_LIBCRYPTO for mvebu
> platform in Kconfig?
>

We should only use 'imply' for suggested settings and never for hard
requirements. TOOLS_LIBCRYPTO already defaults to 'Y'. So implying it
for mvebu would be redundant.

In an OS distribution we only want to ship a single version of mkimage.
So it is good to elimate symbol CONFIG_MXS.

How mkimage is built should not depend on CONFIG_TOOLS_LIBCRYPTO.

Tom wrote regarding this aspect in
https://lists.denx.de/pipermail/u-boot/2021-September/460251.html:

"if we're building a generically useful tool, we don't want another
symbol for it."

We should therefore remove the following dependencies in tools/Makefile:

CONFIG_MX23
CONFIG_MX28
CONFIG_ARMADA_38X

Best regards

Heinrich