On Tue, Feb 28, 2023 at 07:09:50PM +0000, Mark Brown wrote: > On Mon, Feb 27, 2023 at 08:53:39PM -0500, William Breathitt Gray wrote: > > > A patch to pass the device regmap and irq_drv_data as a parameters for > > the struct regmap_irq_chip set_type_config() is included. This is needed > > by idio_24_set_type_config() in order to update the type configuration > > on the device as well as irq_drv_data for idio_24_handle_mask_sync(). > > The values from the config buffer are supposed to be written out in > regmap_irq_sync_unlock() - why is something custom needed here? The PCIe-IDIO-24 "COS Enable" serves a dual purpose of interrupt enabling/disabling as well as configuring the interrupt types. Since this register is used for masking, config buffer would clobber the register if we use it in this particular case. Instead, we ignore the config buffer and configure the type directly for the device (handling the case where interrupts are masked and shouldn't be enabled). William Breathitt Gray
Attachment:
signature.asc
Description: PGP signature
