Skip to content

Commit 661380d

Browse files
petegriffingregkh
authored andcommitted
scsi: ufs: exynos: Add gs101_ufs_drv_init() hook and enable WriteBooster
[ Upstream commit 9cc4a4a ] Factor out the common code into a new exynos_ufs_shareability() function and provide a dedicated gs101_drv_init() hook. This allows us to enable WriteBooster capability (UFSHCD_CAP_WB_EN) in a way that doesn't effect other SoCs supported in this driver. WriteBooster improves write speeds by enabling a pseudo SLC cache. Using the 'fio seqwrite' test we can achieve speeds of 945MB/s with this feature enabled (until the cache is exhausted) before dropping back to ~260MB/s (which are the speeds we see without the WriteBooster feature enabled). Assuming the UFSHCD_CAP_WB_EN capability is set by the host then WriteBooster can also be enabled and disabled via sysfs so it is possible for the system to only enable it when extra write performance is required. Signed-off-by: Peter Griffin <peter.griffin@linaro.org> Link: https://lore.kernel.org/r/20241031150033.3440894-10-peter.griffin@linaro.org Reviewed-by: Tudor Ambarus <tudor.ambarus@linaro.org> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> Stable-dep-of: 68f5ef7 ("scsi: ufs: exynos: Move UFS shareability value to drvdata") Signed-off-by: Sasha Levin <sashal@kernel.org>
1 parent 0e76176 commit 661380d

File tree

1 file changed

+17
-2
lines changed

1 file changed

+17
-2
lines changed

drivers/ufs/host/ufs-exynos.c

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ static inline void exynos_ufs_ungate_clks(struct exynos_ufs *ufs)
198198
exynos_ufs_ctrl_clkstop(ufs, false);
199199
}
200200

201-
static int exynosauto_ufs_drv_init(struct exynos_ufs *ufs)
201+
static int exynos_ufs_shareability(struct exynos_ufs *ufs)
202202
{
203203
struct exynos_ufs_uic_attr *attr = ufs->drv_data->uic_attr;
204204

@@ -214,6 +214,21 @@ static int exynosauto_ufs_drv_init(struct exynos_ufs *ufs)
214214
return 0;
215215
}
216216

217+
static int gs101_ufs_drv_init(struct exynos_ufs *ufs)
218+
{
219+
struct ufs_hba *hba = ufs->hba;
220+
221+
/* Enable WriteBooster */
222+
hba->caps |= UFSHCD_CAP_WB_EN;
223+
224+
return exynos_ufs_shareability(ufs);
225+
}
226+
227+
static int exynosauto_ufs_drv_init(struct exynos_ufs *ufs)
228+
{
229+
return exynos_ufs_shareability(ufs);
230+
}
231+
217232
static int exynosauto_ufs_post_hce_enable(struct exynos_ufs *ufs)
218233
{
219234
struct ufs_hba *hba = ufs->hba;
@@ -2128,7 +2143,7 @@ static const struct exynos_ufs_drv_data gs101_ufs_drvs = {
21282143
EXYNOS_UFS_OPT_SKIP_CONFIG_PHY_ATTR |
21292144
EXYNOS_UFS_OPT_UFSPR_SECURE |
21302145
EXYNOS_UFS_OPT_TIMER_TICK_SELECT,
2131-
.drv_init = exynosauto_ufs_drv_init,
2146+
.drv_init = gs101_ufs_drv_init,
21322147
.pre_link = gs101_ufs_pre_link,
21332148
.post_link = gs101_ufs_post_link,
21342149
.pre_pwr_change = gs101_ufs_pre_pwr_change,

0 commit comments

Comments
 (0)