]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
spl: sata: Rework the loading case it not use IS_ENABLED(...)
authorTom Rini <trini@konsulko.com>
Tue, 10 Jan 2023 16:19:35 +0000 (11:19 -0500)
committerTom Rini <trini@konsulko.com>
Fri, 20 Jan 2023 17:27:06 +0000 (12:27 -0500)
In this case, using IS_ENABLED(...) to attempt to load the image results
in harder to read and less useful code, along with having to define a
CONFIG value that would be unused. To maintain the current albeit
slightly odd behavior, maintain that if we have both SPL_FS_FAT and
SPL_SATA_RAW_U_BOOT_USE_SECTOR enabled, we use SPL_FS_FAT for the load.

Signed-off-by: Tom Rini <trini@konsulko.com>
common/spl/spl_sata.c

index 12397f0ae17e8cae86b1d3deb28a99050cecba1d..32746ce9f3cf70de6ad201a146ff93d3c97ce704 100644 (file)
 #include <fat.h>
 #include <image.h>
 
-#ifndef CONFIG_SPL_SATA_RAW_U_BOOT_SECTOR
-/* Dummy value to make the compiler happy */
-#define CONFIG_SPL_SATA_RAW_U_BOOT_SECTOR 0x100
-#endif
-
 static int spl_sata_load_image_raw(struct spl_image_info *spl_image,
                struct spl_boot_device *bootdev,
                struct blk_desc *stor_dev, unsigned long sector)
@@ -62,7 +57,7 @@ static int spl_sata_load_image_raw(struct spl_image_info *spl_image,
 static int spl_sata_load_image(struct spl_image_info *spl_image,
                               struct spl_boot_device *bootdev)
 {
-       int err = 0;
+       int err = -ENOSYS;
        struct blk_desc *stor_dev;
 
        /* try to recognize storage devices immediately */
@@ -77,16 +72,14 @@ static int spl_sata_load_image(struct spl_image_info *spl_image,
                                  CONFIG_SYS_SATA_FAT_BOOT_PARTITION))
 #endif
        {
-               err = -ENOSYS;
-
-               if (IS_ENABLED(CONFIG_SPL_FS_FAT)) {
-                       err = spl_load_image_fat(spl_image, bootdev, stor_dev,
-                                       CONFIG_SYS_SATA_FAT_BOOT_PARTITION,
-                                       CONFIG_SPL_FS_LOAD_PAYLOAD_NAME);
-               } else if (IS_ENABLED(CONFIG_SPL_SATA_RAW_U_BOOT_USE_SECTOR)) {
-                       err = spl_sata_load_image_raw(spl_image, bootdev, stor_dev,
-                               CONFIG_SPL_SATA_RAW_U_BOOT_SECTOR);
-               }
+#ifdef CONFIG_SPL_FS_FAT
+               err = spl_load_image_fat(spl_image, bootdev, stor_dev,
+                               CONFIG_SYS_SATA_FAT_BOOT_PARTITION,
+                               CONFIG_SPL_FS_LOAD_PAYLOAD_NAME);
+#elif defined(CONFIG_SPL_SATA_RAW_U_BOOT_USE_SECTOR)
+               err = spl_sata_load_image_raw(spl_image, bootdev, stor_dev,
+                       CONFIG_SPL_SATA_RAW_U_BOOT_SECTOR);
+#endif
        }
        if (err) {
                puts("Error loading sata device\n");