Re: [PATCH] soc: qcom: smd-rpm: Correct size of outgoing message

2015-10-30 Thread Andy Gross 
On Thu, Oct 29, 2015 at 03:50:09PM -0700, Stephen Boyd wrote:
> On 10/29/2015 03:09 PM, Bjorn Andersson wrote:
> > With the removal of VLAIS the size was incorrectly changed to only cover
> > the headers of the packet, resulting in "empty" requests being sent to
> > the RPM. Correct this so the entire message is transfered.
> >
> > Fixes: 50e1b29b4438 ("soc: qcom: smd: Remove use of VLAIS")
> > Cc: Stephen Boyd 
> > Signed-off-by: Bjorn Andersson 
> 
> Reviewed-by: Stephen Boyd 
> 
> I thought this was already merged though.

I accidentally dropped my fixup for this during one of the rebases.

-- 
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] soc: qcom: smd-rpm: Correct size of outgoing message

2015-10-29 Thread Stephen Boyd 
On 10/29/2015 03:09 PM, Bjorn Andersson wrote:
> With the removal of VLAIS the size was incorrectly changed to only cover
> the headers of the packet, resulting in "empty" requests being sent to
> the RPM. Correct this so the entire message is transfered.
>
> Fixes: 50e1b29b4438 ("soc: qcom: smd: Remove use of VLAIS")
> Cc: Stephen Boyd 
> Signed-off-by: Bjorn Andersson 

Reviewed-by: Stephen Boyd 

I thought this was already merged though.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] soc: qcom: smd-rpm: Correct size of outgoing message

2015-10-29 Thread Bjorn Andersson 
With the removal of VLAIS the size was incorrectly changed to only cover
the headers of the packet, resulting in "empty" requests being sent to
the RPM. Correct this so the entire message is transfered.

Fixes: 50e1b29b4438 ("soc: qcom: smd: Remove use of VLAIS")
Cc: Stephen Boyd 
Signed-off-by: Bjorn Andersson 
---
 drivers/soc/qcom/smd-rpm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/soc/qcom/smd-rpm.c b/drivers/soc/qcom/smd-rpm.c
index 1ee02d2587b2..2969321e1b09 100644
--- a/drivers/soc/qcom/smd-rpm.c
+++ b/drivers/soc/qcom/smd-rpm.c
@@ -132,7 +132,7 @@ int qcom_rpm_smd_write(struct qcom_smd_rpm *rpm,
pkt->req.data_len = cpu_to_le32(count);
memcpy(pkt->payload, buf, count);
 
-   ret = qcom_smd_send(rpm->rpm_channel, pkt, sizeof(*pkt));
+   ret = qcom_smd_send(rpm->rpm_channel, pkt, size);
if (ret)
goto out;
 
-- 
2.4.2

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html