[PATCH v3 3/5] spl: mmc: Respect sector value passed to mmc_load_image_raw_partition

[Chee, Tien Fong]

On 12/12/2025 12:34 am, Jan Kiszka wrote:
> [CAUTION: This email is from outside your organization. Unless you trust the sender, do not click on links or open attachments as it may be a fraudulent email attempting to steal your information and/or compromise your computer.]
>
> From: Jan Kiszka <jan.kiszka at siemens.com>
>
> This function and the sector parameter evolved over the time. By now,
> sector is influenced by spl_mmc_get_uboot_raw_sector which allows to
> adjust the read sector with an offset that U-Boot proper may have inside
> the partition. That used to work by chance if both
> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR and
> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION were enabled. Since
> 2a00d73d081a they are a choice, and we need to drop the condition to
> maintain this feature.
>
> Signed-off-by: Jan Kiszka <jan.kiszka at siemens.com>
> ---
>   common/spl/spl_mmc.c | 4 ----
>   1 file changed, 4 deletions(-)
>
> diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
> index 1942e417075..47cfe4aef58 100644
> --- a/common/spl/spl_mmc.c
> +++ b/common/spl/spl_mmc.c
> @@ -137,11 +137,7 @@ static int mmc_load_image_raw_partition(struct spl_image_info *spl_image,
>                  return ret;
>          }
>
> -#ifdef CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR
>          return mmc_load_image_raw_sector(spl_image, bootdev, mmc, info.start + sector);
> -#else
> -       return mmc_load_image_raw_sector(spl_image, bootdev, mmc, info.start);
> -#endif
>   }
>   #endif
>
> --
> 2.51.0


Reviewed-by: Tien Fong Chee <tien.fong.chee at altera.com>

Best regards,
Tien Fong