RE: [PATCH] drivers/firmware/edd: Treat "XPRS" host bus type thesame as "PCI"

From: Zou, Yi
Date: Wed Sep 28 2011 - 15:08:01 EST


>
> On Wed, Sep 28, 2011 at 01:56:42PM -0500, Zou, Yi wrote:
> > > On Wed, 2011-09-28 at 09:28 -0700, Matt Domsch wrote:
> > > > On Tue, Sep 27, 2011 at 05:32:08PM -0500, Michael Chan wrote:
> > > > > PCI Express devices will return "XPRS" host bus type during BIOS
> EDD
> > > > > call. "XPRS" should be treated just like "PCI" so that the
> proper
> > > > > pci_dev symlink will be created. Scripts such as fcoe-edd.sh
> will
> > > > > then work correctly.
> > > > >
> > > > > Cc: Matt Domsch <Matt_Domsch@xxxxxxxx>
> > > > > Signed-off-by: Michael Chan <mchan@xxxxxxxxxxxx>
> > > >
> > > > Agreed, this looks correct. Thanks for the patch. Feel free to
> > > > submit it through whichever maintainer tree is convenient for you.
> > > >
> > > > Reviewed-by: Matt Domsch <Matt_Domsch@xxxxxxxx>
> > > >
> > >
> > > Yi, could you submit this patch through the open-fcoe tree? Thanks.
> > >
> > Sure, I will pull this into open-fcoe.org's tree and queue it up for
> next
> > push to scsi-misc as long as edd folks are ok with it. Just curious,
> this
> > is from edd4.0, so might be helpful to point that out in description...
> > I can fix that when I apply the patch to fcoe-next tree.
>
> It's actually a bug from when I first wrote the code. EDD 3.0 has the
> "XPRS" bus type defined.
>
> --
> Matt Domsch
> Technology Strategist
> Dell | Office of the CTO

Really, hmm...the one v3.0rev0.8 I have only has PCI/ISA, it's really old as
1998. I only found XPRS in the 4.0 draft. Anyway, I will apply as is then.

yi


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/