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]>
