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

Stephen Warren swarren at wwwdotorg.org
Mon May 11 19:45:36 PDT 2015


On 05/11/2015 01:38 PM, Martin Sperl wrote:
> 
>> 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?

Since the patch has been applied, you'll need to send an incremental patch.




More information about the linux-rpi-kernel mailing list