Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1690879 > unrolled thread
| Started by | Jakub Kicinski <jakub.kicinski@netronome.com> |
|---|---|
| First post | 2017-07-19 03:40 +0200 |
| Last post | 2017-07-19 04:00 +0200 |
| Articles | 3 — 2 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: [PATCH] aic7xxx: fix firmware build with O=path Jakub Kicinski <jakub.kicinski@netronome.com> - 2017-07-19 03:40 +0200
Re: [PATCH] aic7xxx: fix firmware build with O=path "Martin K. Petersen" <martin.petersen@oracle.com> - 2017-07-19 03:50 +0200
Re: [PATCH] aic7xxx: fix firmware build with O=path Jakub Kicinski <jakub.kicinski@netronome.com> - 2017-07-19 04:00 +0200
| From | Jakub Kicinski <jakub.kicinski@netronome.com> |
|---|---|
| Date | 2017-07-19 03:40 +0200 |
| Subject | Re: [PATCH] aic7xxx: fix firmware build with O=path |
| Message-ID | <u4M81-1zI-11@gated-at.bofh.it> |
On Wed, 28 Jun 2017 21:17:16 -0700, Jakub Kicinski wrote: > Building firmware with O=path was apparently broken in aic7 for ever. > Message of the previous commit to the Makefile (from 2008) mentions > this unfortunate state of affairs already. Fix this, mostly to make > randconfig builds more reliable. > > Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com> I just noticed this patch got marked as "Deferred, archived" in patchwork. I'm not entirely sure by whom and why? Could someone explain what is the issue? I would really appreciate if we could merge this patch and have this driver not break builds randomly...
[toc] | [next] | [standalone]
| From | "Martin K. Petersen" <martin.petersen@oracle.com> |
|---|---|
| Date | 2017-07-19 03:50 +0200 |
| Message-ID | <u4MhI-1Dj-17@gated-at.bofh.it> |
| In reply to | #1690879 |
Jakub, > I just noticed this patch got marked as "Deferred, archived" in > patchwork. I'm not entirely sure by whom and why? Could someone > explain what is the issue? I would really appreciate if we could > merge this patch and have this driver not break builds randomly... It's an indication that the patch sat around without any reviews for way too long. People generally don't go chasing around in their mail archives for several week old patches to review. The best thing to do for patches I mark as deferred is to resubmit them so they get back on people's radar. -- Martin K. Petersen Oracle Linux Engineering
[toc] | [prev] | [next] | [standalone]
| From | Jakub Kicinski <jakub.kicinski@netronome.com> |
|---|---|
| Date | 2017-07-19 04:00 +0200 |
| Message-ID | <u4Mrn-1Gv-1@gated-at.bofh.it> |
| In reply to | #1690888 |
On Tue, 18 Jul 2017 21:45:49 -0400, Martin K. Petersen wrote: > Jakub, > > > I just noticed this patch got marked as "Deferred, archived" in > > patchwork. I'm not entirely sure by whom and why? Could someone > > explain what is the issue? I would really appreciate if we could > > merge this patch and have this driver not break builds randomly... > > It's an indication that the patch sat around without any reviews for way > too long. > > People generally don't go chasing around in their mail archives for > several week old patches to review. The best thing to do for patches I > mark as deferred is to resubmit them so they get back on people's radar. OK, I will repost then. Thanks for the explanation! I hope someone will find time to review, the driver is a bit old :)
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web