topic DMA GetRemainingBytes bug report in MCUXpresso SDK https://community.nxp.com/t5/MCUXpresso-SDK/DMA-GetRemainingBytes-bug-report/m-p/703794#M290 <HTML><HEAD></HEAD><BODY><P>Hello</P><P>Bug in <STRONG>fsl_dma.c</STRONG> of LPC546xx SDK v2.2.0;</P><P>function <SPAN style="color: #800080;">uint32_t</SPAN><SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="font-weight: 600; color: #00677c;">DMA_GetRemainingBytes</SPAN>(<SPAN style="color: #800080;">DMA_Type</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>*<SPAN style="color: #092e64;">base</SPAN>,<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #800080;">uint32_t</SPAN><SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #092e64;">channel</SPAN>)</P><P></P><P>return statement doesn't perform masking and bit shifting as in other functions of same source file. Also "+1" performed for register value, instead of actual XFERCOUNT value.</P><P></P><P>fix:</P><PRE style="-qt-block-indent: 0; text-indent: 0px; margin: 0px;"><SPAN style="color: #808000;">return</SPAN><SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #092e64;">base</SPAN>-&gt;<SPAN style="color: #800000;">CHANNEL</SPAN>[<SPAN style="color: #092e64;">channel</SPAN>].<SPAN style="color: #800000;">XFERCFG</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>+<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">1</SPAN>;</PRE><P>change to</P><PRE style="-qt-block-indent: 0; text-indent: 0px; margin: 0px;"><SPAN style="color: #808000;">return</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>((<SPAN style="color: #092e64;">base</SPAN>-&gt;<SPAN style="color: #800000;">CHANNEL</SPAN>[<SPAN style="color: #092e64;">channel</SPAN>].<SPAN style="color: #800000;">XFERCFG</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>&amp;<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">DMA_CHANNEL_XFERCFG_XFERCOUNT_MASK</SPAN>)<SPAN style="color: #c0c0c0;"> </SPAN>&gt;&gt;<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">DMA_CHANNEL_XFERCFG_XFERCOUNT_SHIFT</SPAN>)<SPAN style="color: #c0c0c0;"> </SPAN>+<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">1</SPAN>;</PRE></BODY></HTML> Wed, 11 Oct 2017 15:47:22 GMT mstumbra 2017-10-11T15:47:22Z DMA GetRemainingBytes bug report https://community.nxp.com/t5/MCUXpresso-SDK/DMA-GetRemainingBytes-bug-report/m-p/703794#M290 <HTML><HEAD></HEAD><BODY><P>Hello</P><P>Bug in <STRONG>fsl_dma.c</STRONG> of LPC546xx SDK v2.2.0;</P><P>function <SPAN style="color: #800080;">uint32_t</SPAN><SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="font-weight: 600; color: #00677c;">DMA_GetRemainingBytes</SPAN>(<SPAN style="color: #800080;">DMA_Type</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>*<SPAN style="color: #092e64;">base</SPAN>,<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #800080;">uint32_t</SPAN><SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #092e64;">channel</SPAN>)</P><P></P><P>return statement doesn't perform masking and bit shifting as in other functions of same source file. Also "+1" performed for register value, instead of actual XFERCOUNT value.</P><P></P><P>fix:</P><PRE style="-qt-block-indent: 0; text-indent: 0px; margin: 0px;"><SPAN style="color: #808000;">return</SPAN><SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #092e64;">base</SPAN>-&gt;<SPAN style="color: #800000;">CHANNEL</SPAN>[<SPAN style="color: #092e64;">channel</SPAN>].<SPAN style="color: #800000;">XFERCFG</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>+<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">1</SPAN>;</PRE><P>change to</P><PRE style="-qt-block-indent: 0; text-indent: 0px; margin: 0px;"><SPAN style="color: #808000;">return</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>((<SPAN style="color: #092e64;">base</SPAN>-&gt;<SPAN style="color: #800000;">CHANNEL</SPAN>[<SPAN style="color: #092e64;">channel</SPAN>].<SPAN style="color: #800000;">XFERCFG</SPAN><SPAN style="color: #c0c0c0;"> </SPAN>&amp;<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">DMA_CHANNEL_XFERCFG_XFERCOUNT_MASK</SPAN>)<SPAN style="color: #c0c0c0;"> </SPAN>&gt;&gt;<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">DMA_CHANNEL_XFERCFG_XFERCOUNT_SHIFT</SPAN>)<SPAN style="color: #c0c0c0;"> </SPAN>+<SPAN style="color: #c0c0c0;"> </SPAN><SPAN style="color: #000080;">1</SPAN>;</PRE></BODY></HTML> Wed, 11 Oct 2017 15:47:22 GMT https://community.nxp.com/t5/MCUXpresso-SDK/DMA-GetRemainingBytes-bug-report/m-p/703794#M290 mstumbra 2017-10-11T15:47:22Z Re: DMA GetRemainingBytes bug report https://community.nxp.com/t5/MCUXpresso-SDK/DMA-GetRemainingBytes-bug-report/m-p/703795#M291 <HTML><HEAD></HEAD><BODY><P>Hi Michael,</P><P></P><P>Thank you for reporting this, I have sent this report to the SDK development team.</P><P></P><P>Best Regards!<BR />Carlos Mendoza<BR />Technical Support Engineer</P></BODY></HTML> Thu, 19 Oct 2017 16:34:53 GMT https://community.nxp.com/t5/MCUXpresso-SDK/DMA-GetRemainingBytes-bug-report/m-p/703795#M291 Carlos_Mendoza 2017-10-19T16:34:53Z