Hi, On Mon, Sep 24, 2018 at 4:52 PM Stephen Boyd <swb...@chromium.org> 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 <krama...@codeaurora.org> > Cc: Sagar Dharia <sdha...@codeaurora.org> > Cc: Girish Mahadevan <giri...@codeaurora.org> > Cc: Doug Anderson <diand...@chromium.org> > Signed-off-by: Stephen Boyd <swb...@chromium.org> > --- > 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 <diand...@chromium.org>
