Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1313962 > unrolled thread
| Started by | "Hubbe, Allen" <Allen.Hubbe@emc.com> |
|---|---|
| First post | 2016-01-21 08:00 +0100 |
| Last post | 2016-01-21 17:40 +0100 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
RE: [Resend PATCH V5 1/1] NTB: Add support for AMD PCI-Express Non-Transparent Bridge "Hubbe, Allen" <Allen.Hubbe@emc.com> - 2016-01-21 08:00 +0100
RE: [Resend PATCH V5 1/1] NTB: Add support for AMD PCI-Express Non-Transparent Bridge "Yu, Xiangliang" <Xiangliang.Yu@amd.com> - 2016-01-21 08:30 +0100
Re: [Resend PATCH V5 1/1] NTB: Add support for AMD PCI-Express Non-Transparent Bridge Jon Mason <jdmason@kudzu.us> - 2016-01-21 17:40 +0100
| From | "Hubbe, Allen" <Allen.Hubbe@emc.com> |
|---|---|
| Date | 2016-01-21 08:00 +0100 |
| Subject | RE: [Resend PATCH V5 1/1] NTB: Add support for AMD PCI-Express Non-Transparent Bridge |
| Message-ID | <qThnQ-1tW-1@gated-at.bofh.it> |
From: Xiangliang Yu <Xiangliang.Yu@amd.com> > Signed-off-by: Xiangliang Yu <Xiangliang.Yu@amd.com> Yes. > Reviewed-by: Jon Mason <jdmason@kudzu.us> Maybe, but that's for Jon to decide. If he accepts it, he will add signed-off-by, but again, that's for Jon to decide. > Reviewed-by: Allen Hubbe <Allen.Hubbe@emc.com> Adding my reviewed-by is hardly a reason to resend the whole patch.
[toc] | [next] | [standalone]
| From | "Yu, Xiangliang" <Xiangliang.Yu@amd.com> |
|---|---|
| Date | 2016-01-21 08:30 +0100 |
| Message-ID | <qThQS-1VS-5@gated-at.bofh.it> |
| In reply to | #1313962 |
> From: Xiangliang Yu <Xiangliang.Yu@amd.com> > > > Signed-off-by: Xiangliang Yu <Xiangliang.Yu@amd.com> > > Yes. > > > Reviewed-by: Jon Mason <jdmason@kudzu.us> > > Maybe, but that's for Jon to decide. If he accepts it, he will add signed-off-by, > but again, that's for Jon to decide. Jon also spend a lot of time to review the code, I think should show his working On this patch. > > Reviewed-by: Allen Hubbe <Allen.Hubbe@emc.com> > > Adding my reviewed-by is hardly a reason to resend the whole patch. It will make work easier if merging.
[toc] | [prev] | [next] | [standalone]
| From | Jon Mason <jdmason@kudzu.us> |
|---|---|
| Date | 2016-01-21 17:40 +0100 |
| Message-ID | <qTqr8-7RN-11@gated-at.bofh.it> |
| In reply to | #1313971 |
On Thu, Jan 21, 2016 at 2:24 AM, Yu, Xiangliang <Xiangliang.Yu@amd.com> wrote: >> From: Xiangliang Yu <Xiangliang.Yu@amd.com> >> >> > Signed-off-by: Xiangliang Yu <Xiangliang.Yu@amd.com> >> >> Yes. >> >> > Reviewed-by: Jon Mason <jdmason@kudzu.us> >> >> Maybe, but that's for Jon to decide. If he accepts it, he will add signed-off-by, >> but again, that's for Jon to decide. > > Jon also spend a lot of time to review the code, I think should show his working > On this patch. I appreciate you trying to give me credit for the review of this code, but this is not how it works. Please read Documentation/SubmittingPatches "Reviewed-by:, instead, indicates that the patch has been reviewed and found acceptable according to the Reviewer's Statement:" While I am content with this patch, you cannot say that for me :) Thanks, Jon > >> > Reviewed-by: Allen Hubbe <Allen.Hubbe@emc.com> >> >> Adding my reviewed-by is hardly a reason to resend the whole patch. > > It will make work easier if merging. >
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web