Re: [PATCH] Change pci_raw_ops to pci_raw_read/write

From: Matthew Wilcox
Date: Sun Feb 10 2008 - 15:46:20 EST


On Sun, Feb 10, 2008 at 12:24:18PM -0800, Linus Torvalds wrote:
> On Sun, 10 Feb 2008, Yinghai Lu wrote:
> > >
> > > I suggest Ivan's patch be merged ASAP as it actually fixes bugs.
> > > This patch is just cleanup (and takes care of some future concerns).
> >
> > your patch and Ivan's patch should be merged in one...
>
> I really don't care whether they get merges as one or separately, but I
> think it should be merged _now_ (-rc1 is already delayed), and I'd like to
> see the final versions of both. Does anybody have them in a final
> agreed-upon format (preferably with that oddness in quirk_intel_irqbalance
> also fixed?)

I just looked at fixing that -- the reason seems to be that we don't
actually have the struct pci_dev at that point. I can fix it, but I
think it's actually buggy. I want to look at some chipset docs to
confirm that though.

I've attached the two patches that I believe are the ones we want. We
can (and should) fix quirk_intel_irqbalance separately.

--
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."