* [PATCH] mfd: qcom_rpm: write fw_version to CTRL_REG
@ 2018-11-14 22:42 Jonathan Marek
2018-11-15 8:31 ` Lee Jones
2018-11-19 19:53 ` [PATCH v2] " Jonathan Marek
0 siblings, 2 replies; 4+ messages in thread
From: Jonathan Marek @ 2018-11-14 22:42 UTC (permalink / raw)
To: bjorn.andersson; +Cc: Lee Jones, open list
downstream driver does this, and it is required on msm8060 (hp touchpad)
Signed-off-by: Jonathan Marek <jonathan@marek.ca>
---
drivers/mfd/qcom_rpm.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c
index 52fafea06067..8d420c37b2a6 100644
--- a/drivers/mfd/qcom_rpm.c
+++ b/drivers/mfd/qcom_rpm.c
@@ -638,6 +638,10 @@ static int qcom_rpm_probe(struct platform_device *pdev)
return -EFAULT;
}
+ writel(fw_version[0], RPM_CTRL_REG(rpm, 0));
+ writel(fw_version[1], RPM_CTRL_REG(rpm, 1));
+ writel(fw_version[2], RPM_CTRL_REG(rpm, 2));
+
dev_info(&pdev->dev, "RPM firmware %u.%u.%u\n", fw_version[0],
fw_version[1],
fw_version[2]);
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] mfd: qcom_rpm: write fw_version to CTRL_REG
2018-11-14 22:42 [PATCH] mfd: qcom_rpm: write fw_version to CTRL_REG Jonathan Marek
@ 2018-11-15 8:31 ` Lee Jones
2018-11-19 19:53 ` [PATCH v2] " Jonathan Marek
1 sibling, 0 replies; 4+ messages in thread
From: Lee Jones @ 2018-11-15 8:31 UTC (permalink / raw)
To: Jonathan Marek; +Cc: bjorn.andersson, open list
On Wed, 14 Nov 2018, Jonathan Marek wrote:
> downstream driver does this, and it is required on msm8060 (hp touchpad)
You can leave the bit about the downstream driver.
Please expand your commit message. Why is it required? What happens
if you don't have it? What problem does it solve? Exactly what is
happening to the H/W?
> Signed-off-by: Jonathan Marek <jonathan@marek.ca>
> ---
> drivers/mfd/qcom_rpm.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c
> index 52fafea06067..8d420c37b2a6 100644
> --- a/drivers/mfd/qcom_rpm.c
> +++ b/drivers/mfd/qcom_rpm.c
> @@ -638,6 +638,10 @@ static int qcom_rpm_probe(struct platform_device *pdev)
> return -EFAULT;
> }
>
> + writel(fw_version[0], RPM_CTRL_REG(rpm, 0));
> + writel(fw_version[1], RPM_CTRL_REG(rpm, 1));
> + writel(fw_version[2], RPM_CTRL_REG(rpm, 2));
> +
> dev_info(&pdev->dev, "RPM firmware %u.%u.%u\n", fw_version[0],
> fw_version[1],
> fw_version[2]);
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2] mfd: qcom_rpm: write fw_version to CTRL_REG
2018-11-14 22:42 [PATCH] mfd: qcom_rpm: write fw_version to CTRL_REG Jonathan Marek
2018-11-15 8:31 ` Lee Jones
@ 2018-11-19 19:53 ` Jonathan Marek
2018-11-27 13:03 ` Lee Jones
1 sibling, 1 reply; 4+ messages in thread
From: Jonathan Marek @ 2018-11-19 19:53 UTC (permalink / raw)
To: bjorn.andersson; +Cc: Lee Jones, open list
This is required as part of the initialization sequence on certain SoCs.
If these registers are not initialized, the hardware can be unresponsive.
This fixes the driver on apq8060 (HP TouchPad device).
Signed-off-by: Jonathan Marek <jonathan@marek.ca>
---
v2: updated the commit message to be more descriptive
I think its important to note that the downstream driver does this for
all versions, so it is safe to add this.
drivers/mfd/qcom_rpm.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c
index 52fafea06067..8d420c37b2a6 100644
--- a/drivers/mfd/qcom_rpm.c
+++ b/drivers/mfd/qcom_rpm.c
@@ -638,6 +638,10 @@ static int qcom_rpm_probe(struct platform_device *pdev)
return -EFAULT;
}
+ writel(fw_version[0], RPM_CTRL_REG(rpm, 0));
+ writel(fw_version[1], RPM_CTRL_REG(rpm, 1));
+ writel(fw_version[2], RPM_CTRL_REG(rpm, 2));
+
dev_info(&pdev->dev, "RPM firmware %u.%u.%u\n", fw_version[0],
fw_version[1],
fw_version[2]);
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] mfd: qcom_rpm: write fw_version to CTRL_REG
2018-11-19 19:53 ` [PATCH v2] " Jonathan Marek
@ 2018-11-27 13:03 ` Lee Jones
0 siblings, 0 replies; 4+ messages in thread
From: Lee Jones @ 2018-11-27 13:03 UTC (permalink / raw)
To: Jonathan Marek; +Cc: bjorn.andersson, open list
On Mon, 19 Nov 2018, Jonathan Marek wrote:
> This is required as part of the initialization sequence on certain SoCs.
>
> If these registers are not initialized, the hardware can be unresponsive.
> This fixes the driver on apq8060 (HP TouchPad device).
>
> Signed-off-by: Jonathan Marek <jonathan@marek.ca>
> ---
> v2: updated the commit message to be more descriptive
>
> I think its important to note that the downstream driver does this for
> all versions, so it is safe to add this.
>
> drivers/mfd/qcom_rpm.c | 4 ++++
> 1 file changed, 4 insertions(+)
Applied, thanks.
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-11-27 13:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-14 22:42 [PATCH] mfd: qcom_rpm: write fw_version to CTRL_REG Jonathan Marek
2018-11-15 8:31 ` Lee Jones
2018-11-19 19:53 ` [PATCH v2] " Jonathan Marek
2018-11-27 13:03 ` Lee Jones
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).