[coreboot-gerrit] Change in coreboot[master]: soc/intel/broadwell: Mask lower 20 bits of TOLUD and TOLM in systemag...

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41471 )

Change subject: soc/intel/broadwell: Mask lower 20 bits of TOLUD and TOLM in systemagent.asl
......................................................................

Patch Set 6:
> Patch Set 6:
> > Patch Set 6:
> >
> > Why was this not fixed in the whole tree?
>
> Was any platform missed? I had done a grep for TLUD in *.asl files:

Um, yes, `git grep TLUD reveals them. :)
> Were there more platforms that required changes?

Haswell (which is most likely exactly the same as Broadwell
in separate dirs) and Sandy Bridge at least.

I would like to have some conclusion about the ME range,
though, before we continue. Should we drop that? It seems
safe enough, to me...