[coreboot-gerrit] Change in coreboot[master]: mb/ti/beaglebone: Initialize DDR3

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44387 )

Change subject: mb/ti/beaglebone: Initialize DDR3
......................................................................

Patch Set 12:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44387/12/src/mainboard/ti/beaglebon... 
File src/mainboard/ti/beaglebone/ddr3.h:

https://review.coreboot.org/c/coreboot/+/44387/12/src/mainboard/ti/beaglebon... 
PS12, Line 29: const struct ctrl_ioregs ioregs_bonelt = {
I'd avoid defining structs in headers (this can cause problems in the long run). Just put them in romstage.c or make a new raminit.c file?