Change in coreboot[master]: AGESA: Refactor S3 support functions

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/19037 )

Change subject: AGESA: Refactor S3 support functions
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/19037/1/src/northbridge/amd/agesa/oem_s3.c
File src/northbridge/amd/agesa/oem_s3.c:

PS1, Line 67: if (len == 0 || len == (u32)-1ULL)
            : 		return AGESA_FATAL;
I don't understand this check.  This value is determined from a Kconfig option added to a constant, so I don't see how it's likely to be 0 or -1.

For Agesa, this is:
0xFFFF0000 + 0x1000
For the PI family this will also be:
0xFFFF0000 + 0x1000
For Family 10h, it's:
0x0 + 0x1000

There's no Kconfig prompt, so those values will always be fixed.