[coreboot-gerrit] Change in ...coreboot[master]: qcs405: Update memory layout

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32108 )

Change subject: qcs405: Update memory layout
......................................................................

Patch Set 1:

There's nothing wrong with this but it shouldn't be necessary. The kernel buffer in depthcharge only starts being used *after* depthcharge is entered, at which point both the ramstage and cbfs_cache from coreboot are no longer used. So it should not be a problem to overlap them. The only thing you need to watch out for is that the depthcharge program itself (i.e. what comes after CONFIG_BASE_ADDRESS) doesn't overlap with those regions.