Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-06-02 Thread Andy Gross
On Thu, Jun 02, 2016 at 04:26:02PM -0700, Stephen Boyd wrote: > On 05/12, Andy Gross wrote: > > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > > for communication buffers. > > Yes, but why? This was done so that we can remove the secure_flush mechanism, as we are the

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-06-02 Thread Andy Gross
On Thu, Jun 02, 2016 at 04:26:02PM -0700, Stephen Boyd wrote: > On 05/12, Andy Gross wrote: > > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > > for communication buffers. > > Yes, but why? This was done so that we can remove the secure_flush mechanism, as we are the

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-06-02 Thread Stephen Boyd
On 05/12, Andy Gross wrote: > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > for communication buffers. Yes, but why? > > Signed-off-by: Andy Gross > --- > drivers/firmware/qcom_scm-32.c | 189 > +++-- >

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-06-02 Thread Stephen Boyd
On 05/12, Andy Gross wrote: > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > for communication buffers. Yes, but why? > > Signed-off-by: Andy Gross > --- > drivers/firmware/qcom_scm-32.c | 189 > +++-- >

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-25 Thread Andy Gross
On 25 May 2016 at 15:50, Kevin Hilman wrote: > Andy Gross writes: > >> On Mon, May 23, 2016 at 04:02:06PM -0500, Andy Gross wrote: >>> On 23 May 2016 at 14:26, Kevin Hilman wrote: >>> > Hi Andy, >>> > >>> > On Thu, May 12, 2016 at

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-25 Thread Andy Gross
On 25 May 2016 at 15:50, Kevin Hilman wrote: > Andy Gross writes: > >> On Mon, May 23, 2016 at 04:02:06PM -0500, Andy Gross wrote: >>> On 23 May 2016 at 14:26, Kevin Hilman wrote: >>> > Hi Andy, >>> > >>> > On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: >>> >> This patch converts the

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-25 Thread Kevin Hilman
Andy Gross writes: > On Mon, May 23, 2016 at 04:02:06PM -0500, Andy Gross wrote: >> On 23 May 2016 at 14:26, Kevin Hilman wrote: >> > Hi Andy, >> > >> > On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: >> >> This patch

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-25 Thread Kevin Hilman
Andy Gross writes: > On Mon, May 23, 2016 at 04:02:06PM -0500, Andy Gross wrote: >> On 23 May 2016 at 14:26, Kevin Hilman wrote: >> > Hi Andy, >> > >> > On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: >> >> This patch converts the Qualcomm SCM driver to use the streaming DMA APIs >> >> for

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-24 Thread Andy Gross
On Mon, May 23, 2016 at 04:02:06PM -0500, Andy Gross wrote: > On 23 May 2016 at 14:26, Kevin Hilman wrote: > > Hi Andy, > > > > On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: > >> This patch converts the Qualcomm SCM driver to use the streaming DMA

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-24 Thread Andy Gross
On Mon, May 23, 2016 at 04:02:06PM -0500, Andy Gross wrote: > On 23 May 2016 at 14:26, Kevin Hilman wrote: > > Hi Andy, > > > > On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: > >> This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > >> for communication buffers. > >> >

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-23 Thread Andy Gross
On 23 May 2016 at 14:26, Kevin Hilman wrote: > Hi Andy, > > On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: >> This patch converts the Qualcomm SCM driver to use the streaming DMA APIs >> for communication buffers. >> >> Signed-off-by: Andy Gross

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-23 Thread Andy Gross
On 23 May 2016 at 14:26, Kevin Hilman wrote: > Hi Andy, > > On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: >> This patch converts the Qualcomm SCM driver to use the streaming DMA APIs >> for communication buffers. >> >> Signed-off-by: Andy Gross > > This patch has landed in linux-next in

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-23 Thread Kevin Hilman
Hi Andy, On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > for communication buffers. > > Signed-off-by: Andy Gross This patch has landed in linux-next in the form of

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-23 Thread Kevin Hilman
Hi Andy, On Thu, May 12, 2016 at 8:46 PM, Andy Gross wrote: > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > for communication buffers. > > Signed-off-by: Andy Gross This patch has landed in linux-next in the form of commit a551c3dbd689 (firmware: qcom: scm:

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-15 Thread Andy Gross
On Fri, May 13, 2016 at 04:48:52PM -0700, Bjorn Andersson wrote: > > + cmd->len = cpu_to_le32(alloc_len); > > + cmd->buf_offset = cpu_to_le32(sizeof(*cmd)); > > + cmd->resp_hdr_offset = cpu_to_le32(sizeof(*cmd) + cmd_len); > > + > > cmd->id = cpu_to_le32((svc_id << 10) | cmd_id); > >

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-15 Thread Andy Gross
On Fri, May 13, 2016 at 04:48:52PM -0700, Bjorn Andersson wrote: > > + cmd->len = cpu_to_le32(alloc_len); > > + cmd->buf_offset = cpu_to_le32(sizeof(*cmd)); > > + cmd->resp_hdr_offset = cpu_to_le32(sizeof(*cmd) + cmd_len); > > + > > cmd->id = cpu_to_le32((svc_id << 10) | cmd_id); > >

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-13 Thread Bjorn Andersson
On Thu 12 May 20:46 PDT 2016, Andy Gross wrote: > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > for communication buffers. > > Signed-off-by: Andy Gross > --- > drivers/firmware/qcom_scm-32.c | 189 >

Re: [Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-13 Thread Bjorn Andersson
On Thu 12 May 20:46 PDT 2016, Andy Gross wrote: > This patch converts the Qualcomm SCM driver to use the streaming DMA APIs > for communication buffers. > > Signed-off-by: Andy Gross > --- > drivers/firmware/qcom_scm-32.c | 189 > +++-- >

[Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-12 Thread Andy Gross
This patch converts the Qualcomm SCM driver to use the streaming DMA APIs for communication buffers. Signed-off-by: Andy Gross --- drivers/firmware/qcom_scm-32.c | 189 +++-- drivers/firmware/qcom_scm.c| 6 +-

[Patch v5 5/8] firmware: qcom: scm: Convert to streaming DMA APIS

2016-05-12 Thread Andy Gross
This patch converts the Qualcomm SCM driver to use the streaming DMA APIs for communication buffers. Signed-off-by: Andy Gross --- drivers/firmware/qcom_scm-32.c | 189 +++-- drivers/firmware/qcom_scm.c| 6 +- drivers/firmware/qcom_scm.h| 10 ++- 3