[<prev] [next>] [day] [month] [year] [list]
Message-ID: <5A9BD224CEA58D4CB62235967D650C160EBFC2C6@orsmsx509.amr.corp.intel.com>
Date: Wed, 28 Sep 2011 12:07:55 -0700
From: "Zou, Yi" <yi.zou@...el.com>
To: Matt Domsch <Matt_Domsch@...l.com>
CC: Michael Chan <mchan@...adcom.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"devel@...n-fcoe.org" <devel@...n-fcoe.org>
Subject: RE: [PATCH] drivers/firmware/edd: Treat "XPRS" host bus type the
same as "PCI"
>
> 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@...l.com>
> > > > > Signed-off-by: Michael Chan <mchan@...adcom.com>
> > > >
> > > > 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@...l.com>
> > > >
> > >
> > > 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists