Re: [PATCH] NTB: test: remove a duplicate check

2019-06-18 Thread Dan Carpenter
It's not a huge deal obviously but your commit was a6bed7a54165 ("NTB: Introduce NTB MSI Test Client") but you know that if I had sent a patch called ("NTB: remove a duplicate check") people would have correctly complained because the patch prefix is too vague. What I'm saying is we do this all th

Re: [PATCH] NTB: test: remove a duplicate check

2019-06-18 Thread Logan Gunthorpe
,, On 2019-06-18 11:32 p.m., Dan Carpenter wrote: > We already verified that the "nm->isr_ctx" allocation succeeded so there > is no need to check again here. > > Fixes: a6bed7a54165 ("NTB: Introduce NTB MSI Test Client") > Signed-off-by: Dan Carpenter Hmm, yup, not sure how that slipped throug

[PATCH] NTB: test: remove a duplicate check

2019-06-18 Thread Dan Carpenter
We already verified that the "nm->isr_ctx" allocation succeeded so there is no need to check again here. Fixes: a6bed7a54165 ("NTB: Introduce NTB MSI Test Client") Signed-off-by: Dan Carpenter --- Hey Logan, can pick a patch prefix when you're introducing a new module? "[PATCH] NTB/test: Introduc