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

[Keith Hui]

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.

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