Hi Kuninori, On Mon, Sep 09, 2024 at 04:42:45AM GMT, Kuninori Morimoto wrote: > rcar_i2c_probe() has priv->devtype operation, but handling (A) and (C) > in same place is more understandable ( (A) and (B) are independent). > > (A) if (priv->devtype < I2C_RCAR_GEN3) { > ... > } > > (B) ... > > (C) if (priv->devtype >= I2C_RCAR_GEN3) { > ... > } > > Let's merge it with if-else > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > Reviewed-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Merged to i2c/i2c-host. Thanks, Andi