[PATCH] spi: bcm2835: enable dma modes for transfers meeting certain conditions

Martin Sperl kernel at martin.sperl.org
Mon May 11 12:38:56 PDT 2015


> On 11.05.2015, at 20:24, Mark Brown <broonie at kernel.org> wrote:
> 
> Applied, thanks.

Found one typo with the logic - it only does one specific
check for TX twice instead of for rx_buf and tx_buf:
	if (((u32)tfr->tx_buf % 4 == 0) && ((u32)tfr->tx_buf % 4 == 0))

On top there have been some build-reports about SZ_4K not being defined
so I guess that there is an include <linux/sizes.h> missing.

as well as a warning about a pointer cast to u32 on 64 bit systems.
	if (((u32)tfr->tx_buf % 4 == 0) && ((u32)tfr->tx_buf % 4 == 0))
	if (((u32)tfr->rx_buf % SZ_4K) + tfr->len > SZ_4K) {
I assume fixable by using size_t instead of u32.

Do you want me to send a patch to fix those, or shall I resend the full
patch with those issues fixed?

Sorry for the inconvenience

Martin



More information about the linux-rpi-kernel mailing list