common: Kconfig: Fix CMD_BMP/BMP dependency
authorSamuel Dionne-Riel <samuel@dionne-riel.com>
Tue, 18 Jul 2023 08:57:36 +0000 (14:27 +0530)
committerTom Rini <trini@konsulko.com>
Fri, 21 Jul 2023 19:32:12 +0000 (15:32 -0400)
Using `default y` will not select BMP when CMD_BMP has been enabled, if
it was already configured.

By using `select`, if `CMD_BMP` is turned on, it will force the presence
of `BMP`.

Fixes: 072b0e16c4 ("common: Kconfig: Add BMP configs")
Signed-off-by: Samuel Dionne-Riel <samuel@dionne-riel.com>
Signed-off-by: Nikhil M Jain <n-jain1@ti.com>
cmd/Kconfig
common/Kconfig

index ecfd575..2d6e5f9 100644 (file)
@@ -2005,6 +2005,7 @@ config CMD_2048
 config CMD_BMP
        bool "Enable 'bmp' command"
        depends on VIDEO
+       select BMP
        help
          This provides a way to obtain information about a BMP-format image
          and to display it. BMP (which presumably stands for BitMaP) is a
index f5ad63c..973482f 100644 (file)
@@ -1167,7 +1167,6 @@ config IO_TRACE
 
 config BMP
        bool "Enable bmp image display"
-       default y if CMD_BMP
        help
          Enable bmp functions to display bmp image and get bmp info.