Re: [PATCH v2 1/3] arm64: dts: qcom: sc7280: Add memory region for audiopd

[Dmitry Baryshkov]

On 27/06/2025 15:16, Srinivas Kandagatla wrote:
> On 5/22/25 7:51 PM, Dmitry Baryshkov wrote:
> > On Thu, May 22, 2025 at 08:27:40PM +0200, Konrad Dybcio wrote:
> > > On 5/22/25 2:16 PM, Dmitry Baryshkov wrote:
> > > > On Thu, 22 May 2025 at 13:44, Ling Xu <quic_lxu5@xxxxxxxxxxx> wrote:
> > > > > 在 5/20/2025 9:53 PM, Dmitry Baryshkov 写道:
> > > > > > On Tue, May 20, 2025 at 02:41:22PM +0800, Ling Xu wrote:
> > > > > > > 在 5/17/2025 5:47 AM, Dmitry Baryshkov 写道:
> > > > > > > > On Fri, May 16, 2025 at 04:30:27PM +0530, Ling Xu wrote:
> > > > > > > > > Add reserved memory region and VMIDs for audio PD dynamic loading and
> > > > > > > > > remote heap memory requirements.
> > > > > > > >
> > > > > > > > Why? Was it not working without this heap?
> > > > > > >
> > > > > > > yes, it will not working without this heap.
> > > > > > > Memory region is required for audio PD for dynamic loading and remote heap memory
> > > > > > > requirements. For more info, please refer below patches, it has provided a more
> > > > > > > detailed explanation.
> > > > > > > https://lore.kernel.org/all/bb68da04-ef52-4172-8b6e-f4027bcc2786@xxxxxxxxxxxxxxxx/
> > > > > > > https://lore.kernel.org/all/effea02f-6ffb-42e9-87df-081caafab728@xxxxxxxxxxxxxxxx/
> > > > > >
> > > > > > This triggers a bunch of questions:
> > > > > >
> > > > > > - What is audio PD?
> > > > > Audio PD is a protection domain which is running on DSP for audio
> > > > > specific processing.
> > > > > > - Is it a required component?
> > > > > yes. it is needed wherever there is some audio use cases where
> > > > > DSP audio PD dynamic loading is needed.
> > > >
> > > > Which use cases?
>
> Usecase is for audio compress offload, ex: when we play mp3 or any other
> compress format, we need audio pd support.

This needs to be in the commit message.


--
With best wishes
Dmitry