[PATCHv6 00/13] MSI support for Marvell EBU PCIe driver
Jason Cooper
jason at lakedaemon.net
Wed Aug 7 11:55:18 EDT 2013
On Wed, Aug 07, 2013 at 08:59:56AM +0200, Thierry Reding wrote:
> On Tue, Aug 06, 2013 at 04:02:56PM -0400, Jason Cooper wrote:
> > Thomas,
> >
> > In order to get some testing in -next, I've *tentatively* applied this
> > series in the following way:
> >
> > +---+ mvebu/msi_common (1,2,3,10)
> > |
> > +---+---+ mvebu/msi_irq (4-9)
> > |
> > +---+---+ mvebu/msi_mvebu (11-13)
> >
> > I had to make one change to patch #9, I protected the last two blocks
> > with #ifdef CONFIG_PCI_MSI to prevent a build failure at that point.
> >
> > mvebu/msi_common should be appropriate for Thierry to base off, however,
> > I make no guarantees about the stability of the branch until I send a
> > PR for it. Code-wise, it should be fine, but the commit-ids may change
> > due to adding Tested-by's, etc.
>
> I noticed that patch 10 has my Reviewed-by, even though my comments
> weren't addressed. They were about stylistic issues but I'd still like
> to see them fixed now rather than having to fix them up later. Also I
> think I gave my Tested-by on those patches somewhere already and I think
> it's unlikely that anybody else will test them, so with the consistency
> style fixes applied I don't see any reason why that branch can't be
> stabilized right away.
Ahh, thanks. I did miss your tested-by. We did get one other from
Daniel Price (which sounded like a pretty cool setup, btw), so I'll add
yours before I tag it.
wrt your Reviewed-by, that came with the patch and may be from a
previous revision. At any rate, I saw your comments about the
whitespace/alignment, and I'll fixup v7 as I pull it in.
thx,
Jason.
More information about the linux-arm-kernel
mailing list