[coreboot] Re: Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Aaron Durbin]

On Thu, May 14, 2020 at 2:25 PM Keith Hui  wrote:
> Hi Aaron,
>
> I think I have success after applying 41363 as well. Please see
> attached and tell us if this is what you set out to do.

That looks correct, and what I was expecting. Thanks for the help gathering
logs.
> Thanks
> Keith
>
> On Thu, May 14, 2020 at 4:01 PM Aaron Durbin  wrote:
> > Hi Keith,
> >
> > Did the newresalloc.log file contain this patch?
>
> https://review.coreboot.org/c/coreboot/+/41363
> > I ask because I see the fixed resources on the domain that are dram:
> >   DOMAIN: 0000 resource base 0 size a0000 align 0 gran 0 limit 0 flags
>
> e0004200 index a
> > DOMAIN: 0000 resource base c0000 size 2ff40000 align 0 gran 0 limit 0
>
> flags e0004200 index b
> > However, I don't see them being removed the address space selection:
> > DOMAIN: 0000 mem: base: 0 size: 0 align: 0 gran: 0 limit: fffffffff
> > update_constraints: PCI: 00:04.0 02 base ff800000 limit ffffffff mem
>
> (fixed)
> > Resource ranges:
> > Base: 0, Size: ff800000, Tag: 200
> > Base: 100000000, Size: f00000000, Tag: 100200
> >
> > update_constraints isn't called for the resources above.
> >
> > On Thu, May 14, 2020 at 1:32 PM Keith Hui  wrote:
> > > Hi Aaron,
> > >
> > > Here are two more logs at SPEW. Old log is a separate branch where I
> > > reverted all the problem commits; new log has everything.
> > >
> > > Thanks
> > > Keith