[PATCH v2 04/14] spl: ufs: add SPL_UFS_FS Kconfig and filesystem boot stub

[Balaji Selvanathan]

Hi Varada,

On 6/11/2026 4:59 PM, Varadarajan Narayanan wrote:
> On Sat, Jun 06, 2026 at 03:48:35PM +0530, Balaji Selvanathan wrote:
>> Add a new SPL_UFS_FS Kconfig option to enable filesystem-based boot
>> from UFS (FAT or EXT4). When selected, a UFS_MODE_FS case is added
>> to the boot-mode switch in spl_ufs_load() raw mode on failure.
>>
>> The filesystem path is currently a stub that
>> returns -ENOSYS, serving as a placeholder for future implementation.
> Is this really needed? Can't this be added along with the actual support.

Removed as suggested in v3: 
https://lore.kernel.org/u-boot/20260615-spl_ufs-v3-0-360bceb29596@oss.qualcomm.com/

Thanks,

Balaji

>
> -Varada
>
>> Signed-off-by: Balaji Selvanathan <balaji.selvanathan at oss.qualcomm.com>
>> ---
>> Changes in v2:
>> - Added the filesystem support in SPL UFS as a seperate patch
>> ---
>> ---
>>   common/spl/Kconfig   | 10 ++++++++++
>>   common/spl/spl_ufs.c |  9 +++++++++
>>   2 files changed, 19 insertions(+)
>>
>> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
>> index 82b757d1efa..8609f08413b 100644
>> --- a/common/spl/Kconfig
>> +++ b/common/spl/Kconfig
>> @@ -1682,6 +1682,16 @@ config SPL_UFS_RAW_U_BOOT_PARTITION_NUM
>>   	  with partition support. This is used if partition name is not
>>   	  specified or not found.
>>
>> +config SPL_UFS_FS
>> +	bool "Enable UFS filesystem boot mode"
>> +	depends on SPL_UFS
>> +	help
>> +	  Enable filesystem-based boot from UFS. This allows loading
>> +	  U-Boot from FAT or EXT4 filesystems on UFS partitions.
>> +	  This is tried as a fallback if raw mode loading fails.
>> +
>> +	  Note: Filesystem support is not yet fully implemented.
>> +
>>   config SPL_WATCHDOG
>>   	bool "Support watchdog drivers"
>>   	imply SPL_WDT if !HW_WATCHDOG
>> diff --git a/common/spl/spl_ufs.c b/common/spl/spl_ufs.c
>> index 0590b796267..8a60948a304 100644
>> --- a/common/spl/spl_ufs.c
>> +++ b/common/spl/spl_ufs.c
>> @@ -142,6 +142,15 @@ int spl_ufs_load(struct spl_image_info *spl_image,
>>   			return 0;
>>   #endif
>>   		break;
>> +#ifdef CONFIG_SPL_UFS_FS
>> +	case UFS_MODE_FS:
>> +		debug("spl: ufs: boot mode: fs\n");
>> +
>> +		/* TODO: Implement filesystem support */
>> +		printf("spl: ufs: filesystem boot not implemented\n");
>> +		ret = -ENOSYS;
>> +		break;
>> +#endif
>>   	default:
>>   		puts("spl: ufs: wrong boot mode\n");
>>   	}
>>
>> --
>> 2.34.1
>>