[S] Change in coreboot[master]: src/soc/intel/xeon_sp/ras/hest.c: fix HEST table length

[Arthur Heymans (Code Review)]

Attention is currently required from: Jonathan Zhang, Johnny Lin, Christian Walter, Li Jinfeng, Tim Chu.

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

Change subject: src/soc/intel/xeon_sp/ras/hest.c: fix HEST table length
......................................................................

Patch Set 3: Code-Review-1

(2 comments)

Patchset:

PS3: 
The whole table generation is just plain wrong. I don't think it makes sense to just fix the returned without fixing the actual table genereation.

File src/soc/intel/xeon_sp/ras/hest.c:

https://review.coreboot.org/c/coreboot/+/69500/comment/bbdb5420_91cae2c8 
PS3, Line 62: void *current;
            : 	current = (void *)(hest);
            : 	void *next = current;
            : 	next = hest + 1;
Arithmetics on pointers to void is not defined in C. Gcc does accept it and sets size to 1 but it's not portable. Also it looks plain wrong. next will be right in the middle of the header. Look at the stoneyridge code on how it might be done.