fix(PeriphDrivers): Resolve incorrect DMA request in DMA-based SPI transactions for all parts

[JeonChangmin]

Description

This pull request addresses an issue in the DMA-based SPI transaction implementation where the DMA request would not be executed under specific conditions.

Problem Details

• Issue: When MXC_SPI_MasterTransactionDMA is called with txData == NULL and rxData != NULL in SPI_WIDTH_STANDARD (full duplex) mode, the function sets reqselTx to -1. However, in MXC_SPI_RevA1_MasterTransactionDMA, MXC_SPI_RevA1_TransSetup may additionally set txData based on the request. This leads to the use of an invalid request selector, causing the DMA to not be executed and resulting in an infinite wait state.

• Call Stack:

  • MXC_SPI_MasterTransactionDMA
  • Determines reqselTx and reqselRx based on the device and data presence.
  • MXC_SPI_RevA1_MasterTransactionDMA
  • Uses MXC_SPI_RevA1_TransSetup to configure the request, potentially modifying req to include both Tx and Rx data.
  • MXC_SPI_RevA1_TransSetup
  • Adjusts the request to comply with SPI protocol, potentially setting both Tx and Rx data.

Solution

• Always pass valid reqselRx and reqselTx values from MXC_SPI_MasterTransactionDMA.
• Configure DMA in MXC_SPI_RevA1_MasterTransactionDMA after executing MXC_SPI_RevA1_TransSetup.

  Solution Rationale: By ensuring valid reqselRx and reqselTx values are always passed from MXC_SPI_MasterTransactionDMA, the MXC_SPI_RevA1_MasterTransactionDMA function can correctly configure the DMA settings based on the request. This prevents the DMA from not being executed due to invalid request selectors. Additionally, the MXC_SPI_RevA1_MasterTransactionDMA function always checks the null status of rxData and txData before using the request selectors, ensuring that no other issues arise from this change.

Additional Fixes

• Applied the same fix to MXC_SPI_SlaveTransactionDMA.
• Corrected potentially swapped reqselRx and reqselTx values in spi_me11.c and spi_me18.c files. Please confirm if this change was intended. (fix in this PR)

Tests

• Tested the changes on a MAX78000 device using spi_me17.c as a basis.

Checklist Before Requesting Review

• [x] PR Title follows correct guidelines.
• [x] Description of changes and all other relevant information.
• [x] Provide info on any relevant functional testing/validation.

[JeonChangmin]

Thanks for the review! I fixed the cases where spi_num and reqselTx/Rx didn't match as you suggested. There are more than AI87, so I fixed all of them, but please let me know if there is something else you intended.