Re: [PATCH 07/10] irqchip/cs42l43: Add support for the cs42l43 IRQs

From: Marc Zyngier
Date: Fri May 12 2023 - 11:10:22 EST


On Fri, 12 May 2023 13:28:35 +0100,
Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
> The CS42L43 is an audio CODEC with integrated MIPI SoundWire interface
> (Version 1.2.1 compliant), I2C, SPI, and I2S/TDM interfaces designed
> for portable applications. It provides a high dynamic range, stereo
> DAC for headphone output, two integrated Class D amplifiers for
> loudspeakers, and two ADCs for wired headset microphone input or
> stereo line input. PDM inputs are provided for digital microphones.
>
> The IRQ chip provides IRQ functionality both to other parts of the
> cs42l43 device and to external devices that wish to use its IRQs.

Sorry, but this isn't much of an interrupt controller driver. A modern
interrupt controller driver is firmware-driven (DT or ACPI, pick your
poison), uses irq domains, and uses the irqchip API.

This is just a another variant of the board-file theme, which has
nothing to do with the irqchip subsystem.

M.

--
Without deviation from the norm, progress is not possible.