Re: [PATCH v2 4/4] arm64: dts: qcom: shikra: Add support for AudioCoreCC and AudioCoreCSR nodes
Dmitry Baryshkov <[email protected]>
| Newsgroups | org.kernel.vger.linux-clk,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <gotzbjedro6lkzpk7xfszzymg4rd36jb3qojwlm5yelvi355za@jufl7tenh26b> |
On Mon, Jul 13, 2026 at 06:01:34PM +0530, Imran Shaik wrote: > > > On 13-07-2026 05:44 pm, Konrad Dybcio wrote: > > On 7/13/26 2:07 PM, Imran Shaik wrote: > > > > > > > > > On 09-07-2026 05:30 pm, Konrad Dybcio wrote: > > > > On 7/8/26 8:25 PM, Imran Shaik wrote: > > > > > Add support for Audio Core Clock Controller (AudioCoreCC) and Audio Core > > > > > CSR nodes on Qualcomm Shikra SoC. The Audio Core Clocks and Resets support > > > > > differs across Shikra variants based on Audio subsystem enablement as > > > > > follows: > > > > > - CQM variant: The QAIF driver runs on HLOS, hence both clocks and resets > > > > > are required to be supported on HLOS. > > > > > - CQS variant: The QAIF driver runs on the Modem, and required clocks are > > > > > handled on Modem, so from HLOS only resets are needed. > > > > > - IQS variant: no soundwire codes, hence no clocks/resets are needed. > > > > > > > > > > Signed-off-by: Imran Shaik <[email protected]> > > > > > --- > > > > > > > > For the sake of simplification, can the resets be enabled > > > > globally by default? > > > > > > > > > > By default (PoR), those resets are under HW control. For audio use cases, software desserts/asserts for switching to SW control. Hence, these cannot be enabled globally by default. > > > > What does enabling the provider node have to do with changing the > > state of the resources? > > > > My bad, I interpreted it in wrong way. > > We have IQS variant, where we don't have the soundwire codes, and hence the > respective clocks/resets are not required to be supported. Due to this, I > have kept the status disabled, and enabling only where it is required. Would it make sense to do it differently - disable them for the IQS? Is SoundWire not supported on the IQS at all or is it not a part of the EVK? > > > FWIW there's nothing like reset_sync_state > > > > Sure, I can keep the audiocore_csr node enabled by default. > > > Konrad > -- With best wishes Dmitry