[coreboot] [PATCH] Fix RS690 MMCONFIG access

Myles Watson mylesgw at gmail.com
Fri Mar 20 20:41:28 CET 2009



> -----Original Message-----
> From: Marc Jones [mailto:marcj303 at gmail.com]
> 
> On Thu, Mar 19, 2009 at 4:32 AM, Myles Watson <mylesgw at gmail.com> wrote:
> 
> > Here's my 2 cents:
> > 1. Overwrite read resources so it doesn't touch your MMConfig BAR
> > (sizing it would not be good)
> > 2. Overwrite set_resources so that it:
> >     a. disables MMConfig accesses
> >     b. writes the new value to the MMConfig BAR
> >     c. enables MMConfig accesses
> 
> I assume you mean to change which ever 690 device operations struct
> it is. Then check the address for the bar and skip it if mmconfig
> otherwise do a normal read/write. That sounds good to me.

Thanks for clarifying.  It was a little early :)

Skip it for read_resources.  Disable, write, then enable using conf1 pci
config accesses for set_resources.  Otherwise use the default.

I think that will allow you to use mmconfig accesses everywhere else.

Thanks,
Myles





More information about the coreboot mailing list