On Mon, Jul 06, 2020 at 03:03:41PM +0300, Gal Pressman wrote: > The common kernel pattern is to check for error, not success. > Flip the if statement accordingly and keep the main flow unindented. > > Signed-off-by: Gal Pressman <galpress@xxxxxxxxxx> > --- > drivers/infiniband/core/verbs.c | 25 +++++++++++++------------ > 1 file changed, 13 insertions(+), 12 deletions(-) > Thanks, Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>