mbox series

[v4,0/5] Enable env in SCSI

Message ID 20250513091710.3719292-1-quic_varada@quicinc.com
Headers show
Series Enable env in SCSI | expand

Message

Varadarajan Narayanan May 13, 2025, 9:17 a.m. UTC 
The qcs9100 based Ride platforms have UFS as their primary storage.
Hence add support to U-Boot env framework to be able to save and
retrieve the environment from UFS. The environment will be
saved/retrieved from the partition specified in the config option
CONFIG_SCSI_ENV_PART.

Also add an API to convert partition UUID string to block device
descriptor for UFS. This API will be used to get the block device
descriptor for the partition specified in CONFIG_SCSI_ENV_PART.

v4: Use 'if (CONFIG_IS_ENABLED(xxx))' instead of '#if CONFIG_IS_ENABLED(xxx)
    Add documentation about the platform

v3: s/scsi_get_blk/scsi_get_blk_by_uuid
    Fix argument name in scsi_get_blk_by_uuid

v2: Add part_get_info_by_uuid() similar to part_get_info_by_name()
    Use SCSI instead of UFS
    Use UUID for SCSI_ENV_PART instead of name

Varadarajan Narayanan (5):
  disk: part: implement generic function part_get_info_by_uuid()
  scsi: Implement get_blk() function
  env: Add support for storing env variables in SCSI devices
  doc: board/qualcomm: document iq9 building and flashing
  configs: qcs9100: Enable env in SCSI

 configs/qcs9100_defconfig  |   5 +-
 disk/part.c                |  39 +++++++++++++
 doc/board/qualcomm/iq9.rst |  62 ++++++++++++++++++++
 drivers/scsi/scsi-uclass.c |  30 ++++++++++
 env/Kconfig                |  15 ++++-
 env/Makefile               |   1 +
 env/env.c                  |   3 +
 env/scsi.c                 | 115 +++++++++++++++++++++++++++++++++++++
 include/env_internal.h     |   1 +
 include/part.h             |  20 +++++++
 include/scsi.h             |  10 ++++
 11 files changed, 299 insertions(+), 2 deletions(-)
 create mode 100644 doc/board/qualcomm/iq9.rst
 create mode 100644 env/scsi.c


base-commit: 48db49b0977cc1c9c9abf82c0fb704238fcef4fd

Comments

Casey Connolly May 29, 2025, 7:03 p.m. UTC | #1 
For all except the doc: commit

Acked-by: Casey Connolly <casey.connolly@linaro.org>

On 5/13/25 11:17, Varadarajan Narayanan wrote:
> The qcs9100 based Ride platforms have UFS as their primary storage.
> Hence add support to U-Boot env framework to be able to save and
> retrieve the environment from UFS. The environment will be
> saved/retrieved from the partition specified in the config option
> CONFIG_SCSI_ENV_PART.
> 
> Also add an API to convert partition UUID string to block device
> descriptor for UFS. This API will be used to get the block device
> descriptor for the partition specified in CONFIG_SCSI_ENV_PART.
> 
> v4: Use 'if (CONFIG_IS_ENABLED(xxx))' instead of '#if CONFIG_IS_ENABLED(xxx)
>      Add documentation about the platform
> 
> v3: s/scsi_get_blk/scsi_get_blk_by_uuid
>      Fix argument name in scsi_get_blk_by_uuid
> 
> v2: Add part_get_info_by_uuid() similar to part_get_info_by_name()
>      Use SCSI instead of UFS
>      Use UUID for SCSI_ENV_PART instead of name
> 
> Varadarajan Narayanan (5):
>    disk: part: implement generic function part_get_info_by_uuid()
>    scsi: Implement get_blk() function
>    env: Add support for storing env variables in SCSI devices
>    doc: board/qualcomm: document iq9 building and flashing
>    configs: qcs9100: Enable env in SCSI
> 
>   configs/qcs9100_defconfig  |   5 +-
>   disk/part.c                |  39 +++++++++++++
>   doc/board/qualcomm/iq9.rst |  62 ++++++++++++++++++++
>   drivers/scsi/scsi-uclass.c |  30 ++++++++++
>   env/Kconfig                |  15 ++++-
>   env/Makefile               |   1 +
>   env/env.c                  |   3 +
>   env/scsi.c                 | 115 +++++++++++++++++++++++++++++++++++++
>   include/env_internal.h     |   1 +
>   include/part.h             |  20 +++++++
>   include/scsi.h             |  10 ++++
>   11 files changed, 299 insertions(+), 2 deletions(-)
>   create mode 100644 doc/board/qualcomm/iq9.rst
>   create mode 100644 env/scsi.c
> 
> 
> base-commit: 48db49b0977cc1c9c9abf82c0fb704238fcef4fd
Tom Rini May 30, 2025, 12:38 a.m. UTC | #2 
On Tue, 13 May 2025 14:47:05 +0530, Varadarajan Narayanan wrote:

> The qcs9100 based Ride platforms have UFS as their primary storage.
> Hence add support to U-Boot env framework to be able to save and
> retrieve the environment from UFS. The environment will be
> saved/retrieved from the partition specified in the config option
> CONFIG_SCSI_ENV_PART.
> 
> Also add an API to convert partition UUID string to block device
> descriptor for UFS. This API will be used to get the block device
> descriptor for the partition specified in CONFIG_SCSI_ENV_PART.
> 
> [...]

Applied to u-boot/next, thanks!

[1/5] disk: part: implement generic function part_get_info_by_uuid()
      commit: 43fd4bcefd4ea5e586e201a0908018d1e8395c82
[2/5] scsi: Implement get_blk() function
      commit: 9a2010941f56929cb28432cc3f4b37a944f55b8e
[3/5] env: Add support for storing env variables in SCSI devices
      commit: 8bf3e967f6a999faa4782cf4a81f6aa82d31ead8
[5/5] configs: qcs9100: Enable env in SCSI
      commit: 142aab7f3e036a0cfb6efe864ddf3fe85174af44