Hi Eduard,

Unfortunately the mailing list is refusing patches with extension
.patch, we need to rename it to .txt to get it here.

BTW, you can submit a Pull Request directly to
https://github.com/apache/incubator-nuttx and we could review it.

BR,

Alan

On 10/16/20, Eduard Niesner <niesneredu...@gmail.com> wrote:
> Hi all,
>
> I am not familiar with the code from nuttx\arch\arm\src\stm32h7\stm32_spi.c
> but I believe that there is an issue.
> "txresult" is defined and used only if the CONFIG_STM32H7_SPI_DMA is
> defined.
> But in the spi_interrupt function, the txresult is used regardless of
> whether CONFIG_STM32H7_SPI_DMA is defined or not.
>
> This generates a build issue when you configure SPI without
> CONFIG_STM32H7_SPI_DMA.
>
> Is there anyone that knows more about this?
> I implemented a fix and it seems to be working - but since I am not
> familiar with the code I am not sure if this is the right thing to do.
> I attached a patch.
>
> Thanks,
> Edi
>

Reply via email to