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

From: Mark Brown
Date: Mon Feb 02 2015 - 07:16:11 EST


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 (!wait_for_completion_timeout(&qspi->transfer_complete,
> + QSPI_COMPLETION_TIMEOUT)) {

Same indentation issue as with the earlier patch here. I also just
noticed that you're not CCing driver maintainers, please try to do that
so they get a chance to review things.

Attachment: signature.asc
Description: Digital signature