aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKeoseong Park <keosung.park@samsung.com>2025-04-03 18:21:40 +0900
committerPhilippe Mathieu-Daudé <philmd@linaro.org>2025-04-08 20:46:10 +0200
commit535ef195663e134d704a3c4f375bf5c488705808 (patch)
treee1846b03c14ffe882f66cfaf33b8b1ae385f5917
parent2ba700a501820f7a59c030f6f49a631a3d2c06f5 (diff)
downloadqemu-535ef195663e134d704a3c4f375bf5c488705808.zip
qemu-535ef195663e134d704a3c4f375bf5c488705808.tar.gz
qemu-535ef195663e134d704a3c4f375bf5c488705808.tar.bz2
hw/ufs: Fix incorrect comment for segment_size and allocation_unit_size
The comments for segment_size and allocation_unit_size incorrectly described them as 4KB. According to the UFS specification, segment_size is expressed in units of 512 bytes. Given segment_size = 0x2000 (8192), the actual size is 4MB. Similarly, allocation_unit_size = 1 means 1 segment = 4MB. This patch updates the comments to reflect the correct size. Signed-off-by: Keoseong Park <keosung.park@samsung.com> Reviewed-by: Jeuk Kim <jeuk20.kim@samsung.com> Message-ID: <20250403092140epcms2p355a7f039871b3e5b409754ef450b9158@epcms2p3> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
-rw-r--r--hw/ufs/ufs.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/ufs/ufs.c b/hw/ufs/ufs.c
index ee13eda..542f13b 100644
--- a/hw/ufs/ufs.c
+++ b/hw/ufs/ufs.c
@@ -1753,8 +1753,8 @@ static void ufs_init_hc(UfsHc *u)
u->geometry_desc.length = sizeof(GeometryDescriptor);
u->geometry_desc.descriptor_idn = UFS_QUERY_DESC_IDN_GEOMETRY;
u->geometry_desc.max_number_lu = (UFS_MAX_LUS == 32) ? 0x1 : 0x0;
- u->geometry_desc.segment_size = cpu_to_be32(0x2000); /* 4KB */
- u->geometry_desc.allocation_unit_size = 0x1; /* 4KB */
+ u->geometry_desc.segment_size = cpu_to_be32(0x2000); /* 4MB: 8192 * 512B */
+ u->geometry_desc.allocation_unit_size = 0x1; /* 4MB: 1 segment */
u->geometry_desc.min_addr_block_size = 0x8; /* 4KB */
u->geometry_desc.max_in_buffer_size = 0x8;
u->geometry_desc.max_out_buffer_size = 0x8;