Hi, On Fri, Jun 2, 2023 at 11:13 AM Vijaya Krishna Nivarthi <quic_vnivarth@xxxxxxxxxxx> wrote: > > The select_fifo/dma_mode() functions in geni driver enable/disable > interrupts (secondary included) conditionally for non-uart modes, while > uart is supposed to manage this internally. > However, only uart uses secondary IRQs while spi, i2c do not care about > these at all making their enablement (or disablement) totally unnecessary > for these protos. > > Drop enabling/disabling secondary IRQs for non-uart modes. > This doesn't solve any observed problem but only gets rid of code pieces > that are not required. > > Signed-off-by: Vijaya Krishna Nivarthi <quic_vnivarth@xxxxxxxxxxx> > --- > drivers/soc/qcom/qcom-geni-se.c | 24 ++++-------------------- > 1 file changed, 4 insertions(+), 20 deletions(-) This seems like a nice cleanup to me. It's still odd that the general code has a special case for UART, but I guess the alternative is to duplicate the exact same logic for both the i2c and SPI drivers. I won't insist on that, though I wouldn't be opposed to it either. I guess one comment, though: should we also remove the code that messes with "SE_GENI_S_IRQ_EN" in geni_se_select_gpi_mode()? -Doug