Re: [PATCH 5/5] spi: ti-qspi: cleanup wait_for_completion return handling

2015-02-02 Thread Mark Brown
On Mon, Feb 02, 2015 at 03:30:36AM -0500, Nicholas Mc Guire wrote: > - ret = wait_for_completion_timeout(>transfer_complete, > - QSPI_COMPLETION_TIMEOUT); > - if (ret == 0) { > + if

[PATCH 5/5] spi: ti-qspi: cleanup wait_for_completion return handling

2015-02-02 Thread Nicholas Mc Guire
return type of wait_for_completion_timeout is unsigned long not int, this patch uses the return value of wait_for_completion_timeout in the condition directly rather than assigning it to an incorrect type variable. Signed-off-by: Nicholas Mc Guire --- The return type of

Re: [PATCH 5/5] spi: ti-qspi: cleanup wait_for_completion return handling

2015-02-02 Thread Mark Brown
On Mon, Feb 02, 2015 at 03:30:36AM -0500, Nicholas Mc Guire wrote: - ret = wait_for_completion_timeout(qspi-transfer_complete, - QSPI_COMPLETION_TIMEOUT); - if (ret == 0) { + if

[PATCH 5/5] spi: ti-qspi: cleanup wait_for_completion return handling

2015-02-02 Thread Nicholas Mc Guire
return type of wait_for_completion_timeout is unsigned long not int, this patch uses the return value of wait_for_completion_timeout in the condition directly rather than assigning it to an incorrect type variable. Signed-off-by: Nicholas Mc Guire hof...@osadl.org --- The return type of