This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git
commit 1b5aeb1a0844b319542e5607d1115f284df338cf Author: David Sidrane <[email protected]> AuthorDate: Wed Oct 25 03:23:49 2023 -0700 imxrt:lpi2c DMA transaction only need the status conditioned With DMA enabled on some I2C channels but not all the Non DMA channels were failing. The cause was condition the status with only the enabled interrupts on non DMA chennels. This conditioning needs to only happen in DMA enabled channels --- arch/arm/src/imxrt/imxrt_lpi2c.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/src/imxrt/imxrt_lpi2c.c b/arch/arm/src/imxrt/imxrt_lpi2c.c index f3414c316d..e46bf1eda9 100644 --- a/arch/arm/src/imxrt/imxrt_lpi2c.c +++ b/arch/arm/src/imxrt/imxrt_lpi2c.c @@ -1357,12 +1357,12 @@ static int imxrt_lpi2c_isr_process(struct imxrt_lpi2c_priv_s *priv) #ifdef CONFIG_IMXRT_LPI2C_DMA uint32_t current_status = status; - /* Condition the status with only the enabled interrupts */ - - status &= imxrt_lpi2c_getenabledints(priv); - if (priv->dma != NULL) { + /* Condition the status with only the enabled interrupts */ + + status &= imxrt_lpi2c_getenabledints(priv); + /* Is there an Error condition */ if (current_status & LPI2C_MSR_LIMITED_ERROR_MASK)
