On Wed, Feb 11, 2015 at 11:20:33PM -0800, Patrick Lai wrote: > On 2/11/2015 6:53 PM, Mark Brown wrote: > >On Wed, Feb 11, 2015 at 05:05:52PM -0800, Kenneth Westfield wrote: > >>Replacing DSP-based drivers with LPASS-based drivers would be something that > >>should be handled by Kconfig selections. For the DT, the DSP-related > >No, it shouldn't be. We should have the ability to build a single > >kernel image which will run on many systems, including both your > >system with a DSP and other systems without. > Is there expectation that DTB flashed onto the system would define nodes to > bind with both LPASS-based driver and DSP-based driver? I hope not as we want > to keep LPASS-based driver & DSP-based driver mutually exclusive. DTB time selections are a separate thing to Kconfig changes like Kenneth was proposing. They're more viable though it'd be a lot better to avoid needing them, designing out the possibility of doing something is often a sure fire way of finding a user. > >The selection of DSP use sounds like something which isn't part of the > >description of the hardware but rather a runtime policy decision (at > >least in so far as non-DSP is ever an option). > Put aside IPQ8064, I would say it is actually more of build time policy > decision for QC SoCs with DSP. XPU is programmed by trust zone to allow certain > LPASS registers to be accessed by the chosen processor. If ADSP and app > processor would have to have access to audio interfaces and DMA, resources > partition(i.e # of DMAs go to ADSP while rest of DMA go to app processor is > decided after analyzing expected concurrency use case. For case of 8016, MDSP > would simply expect it has access to all audio subsystem except digital core of > CODEC. But is it possible to configure the TrustZone firmware to leave things open? That's the tricky case. Actually, can we read the configuration TrustZone did? That might be the best answer here, the DT can describe the silicon and then we can check at runtime which bits of it we're actually allowed to talk to.
Attachment:
signature.asc
Description: Digital signature
