Change in coreboot[master]: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using g...

List overview All Threads
Download

newer

older

Change in coreboot[master]:...

Furquan Shaikh (Code Review)

29 May 2020 29 May '20

10:53 p.m.

Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41879 )

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

Differences in auto-generated SPD from current SPD are as follows: Part: MT53E512M32D2NP-046 WT:F Byte# Current New Explanation 9 0x40 0x00 Unused by MRC. 19 0x0F 0xFF As per JEDEC spec, tckMax should be 100ns. So, value should be 0xFF as per datasheet. 123 0x00 0xE5 Fine offset for taa. Expected value is 0xE5 as per datasheet. 125 0xE1 0xE0 Fine offset for tckMin. As per datasheet tckMin is 0.468ns. So, this comes out to be 0xE0. Additionally, manufacturer name bytes are set to 0.

Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Signed-off-by: Furquan Shaikh furquan@google.com --- A src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc A src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt A src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt 3 files changed, 10 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/41879/1

diff --git a/src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc b/src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc new file mode 100644 index 0000000..add0919 --- /dev/null +++ b/src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc @@ -0,0 +1,6 @@ +## SPDX-License-Identifier: GPL-2.0-or-later +## This is an auto-generated file. Do not edit!! + +MEM_TYPE = lp4x +SPD_SOURCES = +SPD_SOURCES += spd-8.hex # ID = 0(0b0000) Parts = MT53E512M32D2NP-046 WT:F diff --git a/src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt b/src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt new file mode 100644 index 0000000..93934ae --- /dev/null +++ b/src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt @@ -0,0 +1,2 @@ +DRAM Part Name ID to assign +MT53E512M32D2NP-046 WT:F 0 (0000) diff --git a/src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt b/src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt new file mode 100644 index 0000000..08daad4 --- /dev/null +++ b/src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt @@ -0,0 +1,2 @@ +MT53E512M32D2NP-046 WT:F +NT6AP256T32AV-J2

Attachments:

attachment.htm (text/html — 5.2 KB)

Show replies by date

Furquan Shaikh (Code Review)

30 May 30 May

1:06 a.m.

Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Marco Chen, Aamir Bohra, Aaron Durbin, Karthik Ramasubramanian,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41879

to look at the new patch set (#2).

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/41879/2

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 2 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: newpatchset

Furquan Shaikh (Code Review)

2 Jun 2 Jun

9:41 a.m.

Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Marco Chen, Aamir Bohra, Aaron Durbin, Karthik Ramasubramanian,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41879

to look at the new patch set (#3).

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/41879/3

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 3 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: newpatchset

Karthik Ramasubramanian (Code Review)

3 Jun 3 Jun

1:33 a.m.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41879 )

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

Patch Set 3: Code-Review+2

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 3 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Tue, 02 Jun 2020 23:33:49 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

Furquan Shaikh (Code Review)

2:08 a.m.

Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Marco Chen, Aamir Bohra, Aaron Durbin, Karthik Ramasubramanian,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41879

to look at the new patch set (#4).

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/41879/4

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 4 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: newpatchset

Furquan Shaikh (Code Review)

7:46 a.m.

Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Marco Chen, Aamir Bohra, Aaron Durbin, Karthik Ramasubramanian,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41879

to look at the new patch set (#5).

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

Differences in auto-generated SPD from current SPD are as follows: Part: MT53E512M32D2NP-046 WT:F Byte# Current New Explanation 4 0x15 0x16 This part has only 1 die. Hence, density per die is 16Gb. 6 0x90 0x04 1 die in package and 2 channels per die. 9 0x40 0x00 Unused by MRC. 19 0x0F 0xFF As per JEDEC spec, tckMax should be 100ns. So, value should be 0xFF as per datasheet. 123 0x00 0xE5 Fine offset for taa. Expected value is 0xE5 as per datasheet. 125 0xE1 0xE0 Fine offset for tckMin. As per datasheet tckMin is 0.468ns. So, this comes out to be 0xE0. Additionally, manufacturer name bytes are set to 0.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/41879/5

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 5 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: newpatchset

Furquan Shaikh (Code Review)

6:47 p.m.

Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Marco Chen, Aamir Bohra, Aaron Durbin, Karthik Ramasubramanian,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41879

to look at the new patch set (#8).

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/41879/8

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 8 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: newpatchset

Nick Vaccaro (Code Review)

7:30 p.m.

Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41879 )

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

Patch Set 9: Code-Review+1

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 9 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Nick Vaccaro nvaccaro@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Wed, 03 Jun 2020 17:30:46 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

Furquan Shaikh (Code Review)

10:28 p.m.

Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Nick Vaccaro, Marco Chen, Aamir Bohra, Aaron Durbin, Karthik Ramasubramanian,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41879

to look at the new patch set (#10).

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

Differences in auto-generated SPD from current SPD are as follows: Part: MT53E512M32D2NP-046 WT:E Byte# Current New Explanation 4 0x15 0x16 This part has only 1 die. Hence, density per die is 16Gb. 6 0x90 0x04 1 die in package and 2 channels per die. 9 0x40 0x00 Unused by MRC. 19 0x0F 0xFF As per JEDEC spec, tckMax should be 100ns. So, value should be 0xFF as per datasheet. 123 0x00 0xE5 Fine offset for taa. Expected value is 0xE5 as per datasheet. 125 0xE1 0xE0 Fine offset for tckMin. As per datasheet tckMin is 0.468ns. So, this comes out to be 0xE0. Additionally, manufacturer name bytes are set to 0.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/41879/10

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 10 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Nick Vaccaro nvaccaro@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: newpatchset

Karthik Ramasubramanian (Code Review)

4 Jun 4 Jun

12:25 a.m.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41879 )

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

Patch Set 10: Code-Review+2

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 10 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Nick Vaccaro nvaccaro@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Wed, 03 Jun 2020 22:25:06 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

Patrick Georgi (Code Review)

6 Jun 6 Jun

11:30 a.m.

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41879 )

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go

This change adds mem_list_variant.txt that contains the list of memory parts used by wheelie and Makefile.inc generated by gen_part_id.go using mem_list_variant.txt.

In the final change of the series, all dedede variants will be switched from using the current SPDs to new auto-generated SPDs.

Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Signed-off-by: Furquan Shaikh furquan@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/41879 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Karthik Ramasubramanian kramasub@google.com --- A src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc A src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt A src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt 3 files changed, 8 insertions(+), 0 deletions(-)

Approvals: build bot (Jenkins): Verified Karthik Ramasubramanian: Looks good to me, approved

diff --git a/src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc b/src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc new file mode 100644 index 0000000..573213b --- /dev/null +++ b/src/mainboard/google/dedede/variants/wheelie/memory/Makefile.inc @@ -0,0 +1,5 @@ +## SPDX-License-Identifier: GPL-2.0-or-later +## This is an auto-generated file. Do not edit!! + +SPD_SOURCES = +SPD_SOURCES += spd-1.hex # ID = 0(0b0000) Parts = MT53E512M32D2NP-046 WT:E diff --git a/src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt b/src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt new file mode 100644 index 0000000..a825c3a --- /dev/null +++ b/src/mainboard/google/dedede/variants/wheelie/memory/dram_id.generated.txt @@ -0,0 +1,2 @@ +DRAM Part Name ID to assign +MT53E512M32D2NP-046 WT:E 0 (0000) diff --git a/src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt b/src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt new file mode 100644 index 0000000..20887c1 --- /dev/null +++ b/src/mainboard/google/dedede/variants/wheelie/memory/mem_list_variant.txt @@ -0,0 +1 @@ +MT53E512M32D2NP-046 WT:E

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 11 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Nick Vaccaro nvaccaro@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: merged

9elements QA (Code Review)

7 Jun 7 Jun

7:39 p.m.

9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41879 )

Change subject: mb/google/dedede/var/wheelie: Use auto-generated Makefile.inc using gen_part_id.go ......................................................................

Patch Set 11:

Automatic boot test returned (PASS/FAIL/TOTAL): 4/0/4 Emulation targets: "QEMU x86 q35/ich9" using payload TianoCore : SUCCESS : https://lava.9esec.io/r/5094 "QEMU x86 q35/ich9" using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/5093 "QEMU x86 i440fx/piix4" using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/5092 "QEMU AArch64" using payload LinuxBoot_u-root_kexec : SUCCESS : https://lava.9esec.io/r/5091

Please note: This test is under development and might not be accurate at all!

-- To view, visit https://review.coreboot.org/c/coreboot/+/41879 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: If307bfb1d376e32af08af4f020f9e125f6a415dd Gerrit-Change-Number: 41879 Gerrit-PatchSet: 11 Gerrit-Owner: Furquan Shaikh furquan@google.com Gerrit-Reviewer: Aamir Bohra aamir.bohra@intel.com Gerrit-Reviewer: Aaron Durbin adurbin@chromium.org Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com Gerrit-Reviewer: Marco Chen marcochen@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Nick Vaccaro nvaccaro@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: 9elements QA hardwaretestrobot@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Sun, 07 Jun 2020 17:39:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

1430

days inactive

1439

days old

coreboot-gerrit@coreboot.org

11 comments

5 participants

tags (0)

participants (5)

9elements QA (Code Review)
Furquan Shaikh (Code Review)
Karthik Ramasubramanian (Code Review)
Nick Vaccaro (Code Review)
Patrick Georgi (Code Review)