Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35742 )
Change subject: mb/facebook/fbg1701: Remove ONBOARD_MICRON_MEM ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35742/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35742/3//COMMIT_MSG@9 PS3, Line 9: Config ONBOARD_MICRON_MEM and ONBOARD_SAMSUNG_MEM are available. : These configs are used to determine if Samsung or Micron onboard memory is : assembled. This can not detected run-time. : : One CONFIG value will be enough to determine if Samsung memory is : assembled. : Only oldest HW revision contains Samsung module, so set : CONFIG_ONBOARD_SAMSUNG memory to default No. If I run it through Vim, the text looks like below.
``` Config ONBOARD_MICRON_MEM and ONBOARD_SAMSUNG_MEM are available. These configs are used to determine if Samsung or Micron onboard memory is assembled. This can not detected run-time.
One CONFIG value will be enough to determine if Samsung memory is assembled. Only oldest HW revision contains Samsung module, so set CONFIG_ONBOARD_SAMSUNG memory to default No. ```