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

[Angel Pons (Code Review)]

Angel Pons 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 1:

(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.

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 function. Not sure if this makes any meaningful difference.