* [PATCH] ntb_transport: Remove unnecessary call to ntb_peer_spad_read. [not found] <297aec6b47924ce9889abe4d6c51c173@mspexmb1.Beer.Town> @ 2016-12-08 17:03 ` Steven Wahl 2016-12-08 17:16 ` Allen Hubbe 0 siblings, 1 reply; 3+ messages in thread From: Steven Wahl @ 2016-12-08 17:03 UTC (permalink / raw) To: linux-ntb@googlegroups.com The results were previously ignored, anyway. Signed-off-by: Steve Wahl <Steve.Wahl@dell.com> --- drivers/ntb/ntb_transport.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/ntb/ntb_transport.c b/drivers/ntb/ntb_transport.c index 4eb8adb..cd9a392 100644 --- a/drivers/ntb/ntb_transport.c +++ b/drivers/ntb/ntb_transport.c @@ -960,7 +960,6 @@ static void ntb_qp_link_work(struct work_struct *work) ntb_peer_spad_write(nt->ndev, QP_LINKS, val | BIT(qp->qp_num)); /* query remote spad for qp ready bits */ - ntb_peer_spad_read(nt->ndev, QP_LINKS); dev_dbg_ratelimited(&pdev->dev, "Remote QP link status = %x\n", val); /* See if the remote side is up */ -- 2.7.4 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [PATCH] ntb_transport: Remove unnecessary call to ntb_peer_spad_read. 2016-12-08 17:03 ` [PATCH] ntb_transport: Remove unnecessary call to ntb_peer_spad_read Steven Wahl @ 2016-12-08 17:16 ` Allen Hubbe 2016-12-21 16:48 ` Jon Mason 0 siblings, 1 reply; 3+ messages in thread From: Allen Hubbe @ 2016-12-08 17:16 UTC (permalink / raw) To: Wahl, Steve, linux-ntb From: Steven Wahl > The results were previously ignored, anyway. > > Signed-off-by: Steve Wahl <Steve.Wahl@dell.com> Fixes: e26a5843f7f5014ae4460030ca4de029a3ac35d3 Acked-by: Allen Hubbe <Allen.Hubbe@dell.com> > --- > drivers/ntb/ntb_transport.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/ntb/ntb_transport.c b/drivers/ntb/ntb_transport.c > index 4eb8adb..cd9a392 100644 > --- a/drivers/ntb/ntb_transport.c > +++ b/drivers/ntb/ntb_transport.c > @@ -960,7 +960,6 @@ static void ntb_qp_link_work(struct work_struct *work) > ntb_peer_spad_write(nt->ndev, QP_LINKS, val | BIT(qp->qp_num)); > > /* query remote spad for qp ready bits */ > - ntb_peer_spad_read(nt->ndev, QP_LINKS); > dev_dbg_ratelimited(&pdev->dev, "Remote QP link status = %x\n", val); > > /* See if the remote side is up */ > -- > 2.7.4 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ntb_transport: Remove unnecessary call to ntb_peer_spad_read. 2016-12-08 17:16 ` Allen Hubbe @ 2016-12-21 16:48 ` Jon Mason 0 siblings, 0 replies; 3+ messages in thread From: Jon Mason @ 2016-12-21 16:48 UTC (permalink / raw) To: Allen Hubbe; +Cc: Wahl, Steve, linux-ntb On Thu, Dec 08, 2016 at 12:16:45PM -0500, Allen Hubbe wrote: > From: Steven Wahl > > The results were previously ignored, anyway. > > > > Signed-off-by: Steve Wahl <Steve.Wahl@dell.com> > > Fixes: e26a5843f7f5014ae4460030ca4de029a3ac35d3 > > Acked-by: Allen Hubbe <Allen.Hubbe@dell.com> This patch also didn't apply cleanly. I hand applied it, as it was a simple patch. Thanks, Jon > > > --- > > drivers/ntb/ntb_transport.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/ntb/ntb_transport.c b/drivers/ntb/ntb_transport.c > > index 4eb8adb..cd9a392 100644 > > --- a/drivers/ntb/ntb_transport.c > > +++ b/drivers/ntb/ntb_transport.c > > @@ -960,7 +960,6 @@ static void ntb_qp_link_work(struct work_struct *work) > > ntb_peer_spad_write(nt->ndev, QP_LINKS, val | BIT(qp->qp_num)); > > > > /* query remote spad for qp ready bits */ > > - ntb_peer_spad_read(nt->ndev, QP_LINKS); > > dev_dbg_ratelimited(&pdev->dev, "Remote QP link status = %x\n", val); > > > > /* See if the remote side is up */ > > -- > > 2.7.4 > > -- > You received this message because you are subscribed to the Google Groups "linux-ntb" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-ntb+unsubscribe@googlegroups.com. > To post to this group, send email to linux-ntb@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/linux-ntb/002001d25176%24da7cabc0%248f760340%24%40dell.com. > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-12-21 16:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <297aec6b47924ce9889abe4d6c51c173@mspexmb1.Beer.Town>
2016-12-08 17:03 ` [PATCH] ntb_transport: Remove unnecessary call to ntb_peer_spad_read Steven Wahl
2016-12-08 17:16 ` Allen Hubbe
2016-12-21 16:48 ` Jon Mason
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.