Since I was CC-ed, I'll add in my opinion:

While Geert already pointed out the spelling mistake (_in_or_our >> 
_in_or_out), since that function is only just for qspi versions, a better 
function name should have been "qspi_pio_transfer_in_or_out"



However....

On 11/8/2016, Arnd Bergmann wrote:
> This simplifies it again by keeping the two separate, which then ends up
> avoiding that warning.

I agree with Arnd's method of NOT adding a new "rspi_pio_transfer_in_or_our" 
function and instead just doing it in the existing qspi_transfer_ functions.



Side note: The RSPI in the RZ/A1 devices also have FIFOs which can be used to 
reduce the number of interrupts in pio transfers, so maybe someday I'll make a 
similar change for non-qspi devices as well.


Chris

Reply via email to