Re: [PATCH v2 0/2] Fix minor issue in SoundWire slave IRQ mapping

[Pierre-Louis Bossart]

On 4/29/25 05:18, Charles Keepax wrote:
> Currently there would be problems if multiple devices on the same bus
> attempted to use SoundWire IRQ handling rather than the IRQ callback
> mechanism. So far only cs42l43 uses this system so this hasn't caused
> any problems.
>
> Thanks,
> Charles
>
> Changes since v1:
>   - Don't reuse the new IDA for the dev_num
>   - Expand the number of devices allowed on a bus to 16
>
> Charles Keepax (2):
>    soundwire: bus: Simplify sdw_assign_device_num()
>    soundwire: bus: Add internal slave ID and use for IRQs

LGTM,

Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxx>

>   drivers/soundwire/bus.c       | 31 ++++++++++++++-----------------
>   drivers/soundwire/bus_type.c  | 10 ++++++++++
>   drivers/soundwire/irq.c       |  6 +++---
>   include/linux/soundwire/sdw.h |  6 ++++++
>   4 files changed, 33 insertions(+), 20 deletions(-)