Re: [PATCH 2/2] staging: greybus: loopback: convert loopback to use generic async operations

2017-11-05 Thread Bryan O'Donoghue
On 05/11/17 12:24, Johan Hovold wrote: + if (error) { gb->error++; + gb->iteration_count++; + } And this looks like an unrelated bug fix that should go in it's own patch, right? I pinged

Re: [PATCH 2/2] staging: greybus: loopback: convert loopback to use generic async operations

2017-11-05 Thread Johan Hovold
On Sun, Nov 05, 2017 at 03:27:39AM +, Bryan O'Donoghue wrote: > Loopback has its own internal method for tracking and timing out > asynchronous operations however previous patches make it possible to use > functionality provided by operation.c to do this instead. Using the code in >

[PATCH 2/2] staging: greybus: loopback: convert loopback to use generic async operations

2017-11-04 Thread Bryan O'Donoghue
Loopback has its own internal method for tracking and timing out asynchronous operations however previous patches make it possible to use functionality provided by operation.c to do this instead. Using the code in operation.c means we can completely subtract the timer, the work-queue, the kref and