[coreboot-gerrit] [S] Change in coreboot[master]: soc/amd/noncar/memlayout.ld: Warn about incorrect reset vector

[Arthur Heymans (Code Review)]

Attention is currently required from: Jason Glenesk, Marshall Dawson, Matt DeVillier, Fred Reitberger, Felix Held.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64951 )

Change subject: soc/amd/noncar/memlayout.ld: Warn about incorrect reset vector
......................................................................

Patch Set 13:

(2 comments)

File src/soc/amd/common/block/cpu/noncar/memlayout_x86.ld:

https://review.coreboot.org/c/coreboot/+/64951/comment/c2ae7c58_e7c57d7f 
PS11, Line 150: be
> be or end with?

Well in 16 bit real mode IP really is 0xfff0. A shadow CS deals with the other bits.

https://review.coreboot.org/c/coreboot/+/64951/comment/25dffdf6_f77f3ba6 
PS11, Line 150: 0xfff0
> > shouldn't this be 0xffff? […]

Done