Messages in this thread | | | From | Doug Anderson <> | Date | Tue, 25 Sep 2018 14:49:45 -0700 | Subject | Re: [PATCH v3 3/3] i2c: i2c-qcom-geni: Simplify irq handler |
| |
Hi,
On Mon, Sep 24, 2018 at 4:52 PM Stephen Boyd <swboyd@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 <kramasub@codeaurora.org> > Cc: Sagar Dharia <sdharia@codeaurora.org> > Cc: Girish Mahadevan <girishm@codeaurora.org> > Cc: Doug Anderson <dianders@chromium.org> > Signed-off-by: Stephen Boyd <swboyd@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 <dianders@chromium.org>
| |