[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0/hand_off_block: Remove dead decrement

[Nico Huber (Code Review)]

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

Change subject: drivers/intel/fsp2_0/hand_off_block: Remove dead decrement
......................................................................

Patch Set 2:
> > I wonder if this is a bug or something that was left out in refactoring
>
> how to check?

Now the `size` parameter is not used at all. If this is a bug depends on your
quality standards. One could use the `size` parameter to check that the given
structure is sane, i.e. really fits into a block of the given size.

So can we trust FSP to write proper HOBs? No => bug, yes => no bug.

I would prefer to remove the size parameter entirely. After all, the code is
guarded by a debug option.