mbox series

[v3,00/10] misc: fastrpc: Add audiopd support

Message ID 20220909133938.3518520-1-abel.vesa@linaro.org (mailing list archive)
Headers show
Series misc: fastrpc: Add audiopd support | expand

Message

Abel Vesa Sept. 9, 2022, 1:39 p.m. UTC
This patchset adds audiopd support to fastrpc.

The v2 of this patchset is here:
https://lore.kernel.org/all/20220902154900.3404524-1-abel.vesa@linaro.org/

Abel Vesa (10):
  misc: fastrpc: Rename audio protection domain to root
  misc: fastrpc: Add reserved mem support
  dt-bindings: misc: fastrpc: Document memory-region property
  misc: fastrpc: Add fastrpc_remote_heap_alloc
  misc: fastrpc: Use fastrpc_map_put in fastrpc_map_create on fail
  misc: fastrpc: Rework fastrpc_req_munmap
  misc: fastrpc: Add support for audiopd
  misc: fastrpc: Safekeep mmaps on interrupted invoke
  misc: fastrpc: Add mmap request assigning for static PD pool
  misc: fastrpc: Add dma_mask to fastrpc_channel_ctx

 .../bindings/misc/qcom,fastrpc.yaml           |   5 +
 drivers/misc/fastrpc.c                        | 260 +++++++++++++++---
 include/uapi/misc/fastrpc.h                   |   7 +
 3 files changed, 240 insertions(+), 32 deletions(-)

Comments

Rob Herring (Arm) Sept. 12, 2022, 5:08 p.m. UTC | #1
On Fri, Sep 09, 2022 at 04:39:28PM +0300, Abel Vesa wrote:
> This patchset adds audiopd support to fastrpc.
> 
> The v2 of this patchset is here:
> https://lore.kernel.org/all/20220902154900.3404524-1-abel.vesa@linaro.org/
> 
> Abel Vesa (10):
>   misc: fastrpc: Rename audio protection domain to root
>   misc: fastrpc: Add reserved mem support
>   dt-bindings: misc: fastrpc: Document memory-region property
>   misc: fastrpc: Add fastrpc_remote_heap_alloc
>   misc: fastrpc: Use fastrpc_map_put in fastrpc_map_create on fail
>   misc: fastrpc: Rework fastrpc_req_munmap
>   misc: fastrpc: Add support for audiopd
>   misc: fastrpc: Safekeep mmaps on interrupted invoke
>   misc: fastrpc: Add mmap request assigning for static PD pool
>   misc: fastrpc: Add dma_mask to fastrpc_channel_ctx

How do you expect this series to be merged? Please spell it out to the 
maintainers. If you don't, Greg is just going to tell you the series 
doesn't apply (because qcom,fastrpc.yaml only exists in my tree), drop 
it, and move on to the next thing in his queue.

Anyways, I've applied the binding patch. Who knew fastrpc was going to 
be so popular this cycle...

Rob
Abel Vesa Sept. 13, 2022, 8:53 a.m. UTC | #2
On 22-09-12 12:08:27, Rob Herring wrote:
> On Fri, Sep 09, 2022 at 04:39:28PM +0300, Abel Vesa wrote:
> > This patchset adds audiopd support to fastrpc.
> > 
> > The v2 of this patchset is here:
> > https://lore.kernel.org/all/20220902154900.3404524-1-abel.vesa@linaro.org/
> > 
> > Abel Vesa (10):
> >   misc: fastrpc: Rename audio protection domain to root
> >   misc: fastrpc: Add reserved mem support
> >   dt-bindings: misc: fastrpc: Document memory-region property
> >   misc: fastrpc: Add fastrpc_remote_heap_alloc
> >   misc: fastrpc: Use fastrpc_map_put in fastrpc_map_create on fail
> >   misc: fastrpc: Rework fastrpc_req_munmap
> >   misc: fastrpc: Add support for audiopd
> >   misc: fastrpc: Safekeep mmaps on interrupted invoke
> >   misc: fastrpc: Add mmap request assigning for static PD pool
> >   misc: fastrpc: Add dma_mask to fastrpc_channel_ctx
> 
> How do you expect this series to be merged? Please spell it out to the 
> maintainers. If you don't, Greg is just going to tell you the series 
> doesn't apply (because qcom,fastrpc.yaml only exists in my tree), drop 
> it, and move on to the next thing in his queue.

Yeah, you are right. I should've mentioned that dt-bindings patch goes through
your tree while all the others go through Greg's tree. Sorry about that.

Greg, if you want I can resend this patchset without the dt-bindings, now
that it was applied by Rob to his tree.

> 
> Anyways, I've applied the binding patch. Who knew fastrpc was going to 
> be so popular this cycle...

Thanks for applying the binding patch.

> 
> Rob
Srinivas Kandagatla Sept. 16, 2022, 12:56 p.m. UTC | #3
On 09/09/2022 14:39, Abel Vesa wrote:
> This patchset adds audiopd support to fastrpc.
> 
> The v2 of this patchset is here:
> https://lore.kernel.org/all/20220902154900.3404524-1-abel.vesa@linaro.org/
> 
> Abel Vesa (10):
>    misc: fastrpc: Rename audio protection domain to root
>    misc: fastrpc: Add reserved mem support
>    dt-bindings: misc: fastrpc: Document memory-region property
>    misc: fastrpc: Add fastrpc_remote_heap_alloc
>    misc: fastrpc: Use fastrpc_map_put in fastrpc_map_create on fail
>    misc: fastrpc: Rework fastrpc_req_munmap
>    misc: fastrpc: Add support for audiopd
>    misc: fastrpc: Safekeep mmaps on interrupted invoke
>    misc: fastrpc: Add mmap request assigning for static PD pool
>    misc: fastrpc: Add dma_mask to fastrpc_channel_ctx
> 

thanks Abel for picking this up...

Tested this series on SM8450 with ECNS.

Tested-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>


--srini
>   .../bindings/misc/qcom,fastrpc.yaml           |   5 +
>   drivers/misc/fastrpc.c                        | 260 +++++++++++++++---
>   include/uapi/misc/fastrpc.h                   |   7 +
>   3 files changed, 240 insertions(+), 32 deletions(-)
>
Srinivas Kandagatla Nov. 24, 2022, 11:07 a.m. UTC | #4
Hi Greg,

On 09/09/2022 14:39, Abel Vesa wrote:
> This patchset adds audiopd support to fastrpc.
> 
> The v2 of this patchset is here:
> https://lore.kernel.org/all/20220902154900.3404524-1-abel.vesa@linaro.org/
> 
> Abel Vesa (10):
>    misc: fastrpc: Rename audio protection domain to root
>    misc: fastrpc: Add reserved mem support
>    dt-bindings: misc: fastrpc: Document memory-region property
>    misc: fastrpc: Add fastrpc_remote_heap_alloc
>    misc: fastrpc: Use fastrpc_map_put in fastrpc_map_create on fail
>    misc: fastrpc: Rework fastrpc_req_munmap
>    misc: fastrpc: Add support for audiopd
>    misc: fastrpc: Safekeep mmaps on interrupted invoke
>    misc: fastrpc: Add mmap request assigning for static PD pool
>    misc: fastrpc: Add dma_mask to fastrpc_channel_ctx
> 

Do you think you could pick these fastrpc patches as it is or should I 
collect these patches with other fastrpc patches on the list and send them?

As there are few more of fastrpc patches that could go in 6.2.

thanks,
Srini



>   .../bindings/misc/qcom,fastrpc.yaml           |   5 +
>   drivers/misc/fastrpc.c                        | 260 +++++++++++++++---
>   include/uapi/misc/fastrpc.h                   |   7 +
>   3 files changed, 240 insertions(+), 32 deletions(-)
>
Greg Kroah-Hartman Nov. 24, 2022, 4:46 p.m. UTC | #5
On Thu, Nov 24, 2022 at 11:07:55AM +0000, Srinivas Kandagatla wrote:
> Hi Greg,
> 
> On 09/09/2022 14:39, Abel Vesa wrote:
> > This patchset adds audiopd support to fastrpc.
> > 
> > The v2 of this patchset is here:
> > https://lore.kernel.org/all/20220902154900.3404524-1-abel.vesa@linaro.org/
> > 
> > Abel Vesa (10):
> >    misc: fastrpc: Rename audio protection domain to root
> >    misc: fastrpc: Add reserved mem support
> >    dt-bindings: misc: fastrpc: Document memory-region property
> >    misc: fastrpc: Add fastrpc_remote_heap_alloc
> >    misc: fastrpc: Use fastrpc_map_put in fastrpc_map_create on fail
> >    misc: fastrpc: Rework fastrpc_req_munmap
> >    misc: fastrpc: Add support for audiopd
> >    misc: fastrpc: Safekeep mmaps on interrupted invoke
> >    misc: fastrpc: Add mmap request assigning for static PD pool
> >    misc: fastrpc: Add dma_mask to fastrpc_channel_ctx
> > 
> 
> Do you think you could pick these fastrpc patches as it is or should I
> collect these patches with other fastrpc patches on the list and send them?
> 
> As there are few more of fastrpc patches that could go in 6.2.

Pleasae collect them all up and send them on to me.

thanks,

greg k-h