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

[Jeff Li (Code Review)]

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

Jeff Li 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 6:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/69500/comment/3142aa44_a3996b25 
PS3, Line 62: void *current;
            : 	current = (void *)(hest);
            : 	void *next = current;
            : 	next = hest + 1;
> Arithmetics on pointers to void is not defined in C. […]

Thank you for pointing out this problem of arithmetics on pointers to void. I agree with you to avoid this.

By the way, the next won't be in the middle of the header, because `hest + 1' does skip the acpi_hest_t struct. And stoneyridge code also use arithmetics on pointers to void in function acpi_fill_hest.

To avoid arithmetics on pointers to void completely, is it possible to add CFLAGS -Werror=pointer-arith?