[coreboot-gerrit] Change in coreboot[master]: vc/cavium/bdk/libbdk-hal: Fix eye data memory leak

[Jacob Garber (Code Review)]

Jacob Garber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34591 )

Change subject: vc/cavium/bdk/libbdk-hal: Fix eye data memory leak
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34591/1/src/vendorcode/cavium/bdk/l... 
File src/vendorcode/cavium/bdk/libbdk-hal/bdk-qlm.c:

https://review.coreboot.org/c/coreboot/+/34591/1/src/vendorcode/cavium/bdk/l... 
PS1, Line 365:  {
> Looks like this vendorcode uses a different coding style... I'd be consistent with it.

Done

https://review.coreboot.org/c/coreboot/+/34591/1/src/vendorcode/cavium/bdk/l... 
PS1, Line 366: eye_data
> Very minor: cast this as "(void*)eye_data" to be consistent with the free() call at the end of the f […]

The (void *) cast at the end is needed to get rid of the const qualifier, which isn't needed here.