Attention is currently required from: Jamie Chen, Stanley Wu, Marco Chen.
Derek Huang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69925 )
Change subject: mb/google/octopus/variants/phaser: Add memory ram id=3 to 8GB size ......................................................................
Patch Set 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/69925/comment/f0ab56c4_d0ac43ef PS2, Line 7: Add memory ram id=3 to 8GB size Implement variant_memory_sku()
https://review.coreboot.org/c/coreboot/+/69925/comment/5630116e_9ab9d8c7 PS2, Line 9: Add RAM ID=3 define to 8GB size same as RAM ID 1. This change override memory ID 3 to 1 to workaround the incorrect memory straps in hardware.
File src/mainboard/google/octopus/variants/phaser/memory.c:
https://review.coreboot.org/c/coreboot/+/69925/comment/0aa71f93_87010833 PS2, Line 1: /* : * This file is part of the coreboot project. : * : * Copyright 2017 Intel Corp. : * : * This program is free software; you can redistribute it and/or modify : * it under the terms of the GNU General Public License as published by : * the Free Software Foundation; version 2 of the License. : * : * This program is distributed in the hope that it will be useful, : * but WITHOUT ANY WARRANTY; without even the implied warranty of : * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the : * GNU General Public License for more details. : */ /* SPDX-License-Identifier: GPL-2.0-or-later */