On Mon, Jan 20, 2025 at 11:48:00PM +0530, Mukunda,Vijendar wrote: > On 20/01/25 23:25, Mark Brown wrote: > >> +static irqreturn_t acp70_irq_handler(int irq, void *dev_id) > >> +{ > > This really does seem *very* similar to the acp63 code... > Compared to PS, in ACP70 wake interrupt logic added. > New register fields got introduced inACP70 compared to PS. > Please refer below patch. > https://lore.kernel.org/lkml/20250120100130.3710412-18-Vijendar.Mukunda@xxxxxxx/ > Let this patch series go as initial version for ACP70 platform. > We will revisit the code and implement common helper functions in the next cycle. That does feel like quirks and new features rather than a completely distinct IP.
Attachment:
signature.asc
Description: PGP signature
- Follow-Ups:
- Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- From: Mario Limonciello
- Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- References:
- [PATCH V2 00/23] ASoC: amd: acp70: add soundwire and acp pdm support
- From: Vijendar Mukunda
- [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- From: Vijendar Mukunda
- Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- From: Mark Brown
- Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- From: Mukunda,Vijendar
- [PATCH V2 00/23] ASoC: amd: acp70: add soundwire and acp pdm support
- Prev by Date: Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- Next by Date: Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- Previous by thread: Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- Next by thread: Re: [PATCH V2 12/23] ASoC: amd: acp70: add acp ip interrupt handler
- Index(es):