* [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp
@ 2023-09-06 11:33 Krzysztof Kozlowski
2023-09-07 9:03 ` AngeloGioacchino Del Regno
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-06 11:33 UTC (permalink / raw)
To: Stanley Chu, James E.J. Bottomley, Martin K. Petersen,
Matthias Brugger, AngeloGioacchino Del Regno, linux-scsi,
linux-mediatek, linux-kernel, linux-arm-kernel
Cc: Krzysztof Kozlowski
Bindings do not allow vdd-hba-max-microamp property and the driver does
not use it (does not control load of vdd-hba supply). Skip looking for
this property to avoid misleading dmesg messages:
ufshcd-qcom 1d84000.ufs: ufshcd_populate_vreg: unable to find vdd-hba-max-microamp
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
drivers/ufs/host/ufs-mediatek.c | 2 +-
drivers/ufs/host/ufshcd-pltfrm.c | 15 ++++++++++-----
drivers/ufs/host/ufshcd-pltfrm.h | 2 +-
3 files changed, 12 insertions(+), 7 deletions(-)
diff --git a/drivers/ufs/host/ufs-mediatek.c b/drivers/ufs/host/ufs-mediatek.c
index 2383ecd88f1c..941f58744d08 100644
--- a/drivers/ufs/host/ufs-mediatek.c
+++ b/drivers/ufs/host/ufs-mediatek.c
@@ -806,7 +806,7 @@ static int ufs_mtk_vreg_fix_vcc(struct ufs_hba *hba)
return 0;
}
- err = ufshcd_populate_vreg(dev, vcc_name, &info->vcc);
+ err = ufshcd_populate_vreg(dev, vcc_name, &info->vcc, false);
if (err)
return err;
diff --git a/drivers/ufs/host/ufshcd-pltfrm.c b/drivers/ufs/host/ufshcd-pltfrm.c
index 797a4dfe45d9..61cf8b957da4 100644
--- a/drivers/ufs/host/ufshcd-pltfrm.c
+++ b/drivers/ufs/host/ufshcd-pltfrm.c
@@ -121,7 +121,7 @@ static bool phandle_exists(const struct device_node *np,
#define MAX_PROP_SIZE 32
int ufshcd_populate_vreg(struct device *dev, const char *name,
- struct ufs_vreg **out_vreg)
+ struct ufs_vreg **out_vreg, bool skip_current)
{
char prop_name[MAX_PROP_SIZE];
struct ufs_vreg *vreg = NULL;
@@ -147,6 +147,11 @@ int ufshcd_populate_vreg(struct device *dev, const char *name,
if (!vreg->name)
return -ENOMEM;
+ if (skip_current) {
+ vreg->max_uA = 0;
+ goto out;
+ }
+
snprintf(prop_name, MAX_PROP_SIZE, "%s-max-microamp", name);
if (of_property_read_u32(np, prop_name, &vreg->max_uA)) {
dev_info(dev, "%s: unable to find %s\n", __func__, prop_name);
@@ -175,19 +180,19 @@ static int ufshcd_parse_regulator_info(struct ufs_hba *hba)
struct device *dev = hba->dev;
struct ufs_vreg_info *info = &hba->vreg_info;
- err = ufshcd_populate_vreg(dev, "vdd-hba", &info->vdd_hba);
+ err = ufshcd_populate_vreg(dev, "vdd-hba", &info->vdd_hba, true);
if (err)
goto out;
- err = ufshcd_populate_vreg(dev, "vcc", &info->vcc);
+ err = ufshcd_populate_vreg(dev, "vcc", &info->vcc, false);
if (err)
goto out;
- err = ufshcd_populate_vreg(dev, "vccq", &info->vccq);
+ err = ufshcd_populate_vreg(dev, "vccq", &info->vccq, false);
if (err)
goto out;
- err = ufshcd_populate_vreg(dev, "vccq2", &info->vccq2);
+ err = ufshcd_populate_vreg(dev, "vccq2", &info->vccq2, false);
out:
return err;
}
diff --git a/drivers/ufs/host/ufshcd-pltfrm.h b/drivers/ufs/host/ufshcd-pltfrm.h
index 2df108f4ac13..a86a3ada4bef 100644
--- a/drivers/ufs/host/ufshcd-pltfrm.h
+++ b/drivers/ufs/host/ufshcd-pltfrm.h
@@ -32,6 +32,6 @@ void ufshcd_init_pwr_dev_param(struct ufs_dev_params *dev_param);
int ufshcd_pltfrm_init(struct platform_device *pdev,
const struct ufs_hba_variant_ops *vops);
int ufshcd_populate_vreg(struct device *dev, const char *name,
- struct ufs_vreg **out_vreg);
+ struct ufs_vreg **out_vreg, bool skip_current);
#endif /* UFSHCD_PLTFRM_H_ */
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp
2023-09-06 11:33 [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp Krzysztof Kozlowski
@ 2023-09-07 9:03 ` AngeloGioacchino Del Regno
2023-09-14 1:05 ` Martin K. Petersen
2023-09-22 1:05 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: AngeloGioacchino Del Regno @ 2023-09-07 9:03 UTC (permalink / raw)
To: Krzysztof Kozlowski, Stanley Chu, James E.J. Bottomley,
Martin K. Petersen, Matthias Brugger, linux-scsi, linux-mediatek,
linux-kernel, linux-arm-kernel
Il 06/09/23 13:33, Krzysztof Kozlowski ha scritto:
> Bindings do not allow vdd-hba-max-microamp property and the driver does
> not use it (does not control load of vdd-hba supply). Skip looking for
> this property to avoid misleading dmesg messages:
>
> ufshcd-qcom 1d84000.ufs: ufshcd_populate_vreg: unable to find vdd-hba-max-microamp
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
For MediaTek:
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp
2023-09-06 11:33 [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp Krzysztof Kozlowski
2023-09-07 9:03 ` AngeloGioacchino Del Regno
@ 2023-09-14 1:05 ` Martin K. Petersen
2023-09-22 1:05 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Martin K. Petersen @ 2023-09-14 1:05 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Stanley Chu, James E.J. Bottomley, Martin K. Petersen,
Matthias Brugger, AngeloGioacchino Del Regno, linux-scsi,
linux-mediatek, linux-kernel, linux-arm-kernel
Krzysztof,
> Bindings do not allow vdd-hba-max-microamp property and the driver
> does not use it (does not control load of vdd-hba supply). Skip
> looking for this property to avoid misleading dmesg messages:
Applied to 6.7/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp
2023-09-06 11:33 [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp Krzysztof Kozlowski
2023-09-07 9:03 ` AngeloGioacchino Del Regno
2023-09-14 1:05 ` Martin K. Petersen
@ 2023-09-22 1:05 ` Martin K. Petersen
2 siblings, 0 replies; 4+ messages in thread
From: Martin K. Petersen @ 2023-09-22 1:05 UTC (permalink / raw)
To: Stanley Chu, James E.J. Bottomley, Matthias Brugger,
AngeloGioacchino Del Regno, linux-scsi, linux-mediatek,
linux-kernel, linux-arm-kernel, Krzysztof Kozlowski
Cc: Martin K . Petersen
On Wed, 06 Sep 2023 13:33:02 +0200, Krzysztof Kozlowski wrote:
> Bindings do not allow vdd-hba-max-microamp property and the driver does
> not use it (does not control load of vdd-hba supply). Skip looking for
> this property to avoid misleading dmesg messages:
>
> ufshcd-qcom 1d84000.ufs: ufshcd_populate_vreg: unable to find vdd-hba-max-microamp
>
>
> [...]
Applied to 6.7/scsi-queue, thanks!
[1/1] scsi: ufs: do not look for unsupported vdd-hba-max-microamp
https://git.kernel.org/mkp/scsi/c/2c99e3d7d2cb
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-09-22 1:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-06 11:33 [PATCH] scsi: ufs: do not look for unsupported vdd-hba-max-microamp Krzysztof Kozlowski
2023-09-07 9:03 ` AngeloGioacchino Del Regno
2023-09-14 1:05 ` Martin K. Petersen
2023-09-22 1:05 ` Martin K. Petersen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).