On 2018-09-26 03:19, Doug Anderson wrote:
Hi,
On Mon, Sep 24, 2018 at 4:52 PM Stephen Boyd <[email protected]>
wrote:
We don't need to use goto here, we can just collapse the if statement
and goto chain into multiple branches and then combine some duplicate
completion calls into one big if statement. Let's do it to clean up
code
some more.
Cc: Karthikeyan Ramasubramanian <[email protected]>
Cc: Sagar Dharia <[email protected]>
Cc: Girish Mahadevan <[email protected]>
Cc: Doug Anderson <[email protected]>
Signed-off-by: Stephen Boyd <[email protected]>
---
drivers/i2c/busses/i2c-qcom-geni.c | 70
+++++++++++++-----------------
1 file changed, 29 insertions(+), 41 deletions(-)
It doesn't gleam as powerfully the cleanups in patch 2/3 but this does
have a few nice readability improvements.
Reviewed-by: Douglas Anderson <[email protected]>
agree.
Reviewed-by: Alok Chauhan <[email protected]>
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora
Forum,a Linux Foundation Collaborative Project
