On Sun, Jan 10, 2021 at 11:19:02AM +0000, Max Gurtovoy wrote: > No need to add semicolon after closing bracket. > > Reviewed-by: Israel Rukshin <israelr@xxxxxxxxxx> > Signed-off-by: Max Gurtovoy <mgurtovoy@xxxxxxxxxx> > --- > drivers/infiniband/ulp/isert/ib_isert.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/infiniband/ulp/isert/ib_isert.c b/drivers/infiniband/ulp/isert/ib_isert.c > index 5958929b7dec..96514f675427 100644 > --- a/drivers/infiniband/ulp/isert/ib_isert.c > +++ b/drivers/infiniband/ulp/isert/ib_isert.c > @@ -1991,7 +1991,7 @@ isert_set_dif_domain(struct se_cmd *se_cmd, struct ib_sig_domain *domain) > if (se_cmd->prot_type == TARGET_DIF_TYPE1_PROT || > se_cmd->prot_type == TARGET_DIF_TYPE2_PROT) > domain->sig.dif.ref_remap = true; > -}; > +} The same goes for iser_set_dif_domain() and iser_cleanup_handler(). Thanks > > static int > isert_set_sig_attrs(struct se_cmd *se_cmd, struct ib_sig_attrs *sig_attrs) > -- > 2.25.4 >
