Change in coreboot[master]: added a new fmap for the qemu-q35 emulation that contains only a read...

[Name of user not set (Code Review)]

Name of user not set #1002959 has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41689 )

Change subject: added a new fmap for the qemu-q35 emulation that contains only a read-only section
......................................................................

added a new fmap for the qemu-q35 emulation
that contains only a read-only section

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: A. Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/1

diff --git a/src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd b/src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
new file mode 100644
index 0000000..4d4fcd6
--- /dev/null
+++ b/src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
@@ -0,0 +1,14 @@
+FLASH@0xff000000 0x1000000 {
+	SI_BIOS 0x1000000 {
+		RW_VPD(PRESERVE) 0x1000
+
+		WP_RO {
+			FMAP 0x800
+			RO_FRID 0x40
+			RO_PADDING 0x7c0
+			RO_VPD(PRESERVE) 0x1000
+			GBB 0x1e000
+			COREBOOT(CBFS)
+		}
+	}
+}

[Name of user not set (Code Review)]

Hello build bot (Jenkins),

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

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

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

Change subject: added a new fmap for the qemu-q35 emulation that contains only a read-only section
......................................................................

added a new fmap for the qemu-q35 emulation
that contains only a read-only section

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: Andre Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/2

Change in coreboot[master]: added a new fmap for the qemu-q35 emulation

[Name of user not set (Code Review)]

Hello build bot (Jenkins),

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

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

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

Change subject: added a new fmap for the qemu-q35 emulation
......................................................................

added a new fmap for the qemu-q35 emulation

that contains only a read-only section

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: Andre Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/3

Change in coreboot[master]: Add new fmap

[Name of user not set (Code Review)]

Hello build bot (Jenkins),

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

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

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

Change subject: Add new fmap
......................................................................

Add new fmap

Add new fmap for the qemu-q35 emulation with vboot
that contains only a read-only section

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: Andre Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/4

Change in coreboot[master]: mb/emul/q35: Add new fmap

[Name of user not set (Code Review)]

Hello build bot (Jenkins),

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

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

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

Change subject: mb/emul/q35: Add new fmap
......................................................................

mb/emul/q35: Add new fmap

Add new fmap for the qemu-q35 emulation with vboot
that contains a RO partition only.

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: Andre Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/5

Change in coreboot[master]: mb/emul/q35: Add new fmap

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41689 )

Change subject: mb/emul/q35: Add new fmap
......................................................................

Patch Set 5: Code-Review+1

(2 comments)

https://review.coreboot.org/c/coreboot/+/41689/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/41689/5//COMMIT_MSG@7 
PS5, Line 7: emul
`emulation` or just `mb/qemu-q35`.

https://review.coreboot.org/c/coreboot/+/41689/5//COMMIT_MSG@10 
PS5, Line 10: that contains a RO partition only.
Please add your usecase.

Change in coreboot[master]: mb/qemu-q35: Add new fmap

[Name of user not set (Code Review)]

Hello build bot (Jenkins), Paul Menzel,

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

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

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

Change subject: mb/qemu-q35: Add new fmap
......................................................................

mb/qemu-q35: Add new fmap

Add new fmap for the qemu-q35 emulation with vboot
that contains a RO partition only.

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: Andre Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/6

Change in coreboot[master]: mb/qemu-q35: Add new fmap

[Name of user not set (Code Review)]

Hello build bot (Jenkins), Paul Menzel,

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

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

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

Change subject: mb/qemu-q35: Add new fmap
......................................................................

mb/qemu-q35: Add new fmap

Add new fmap for the qemu-q35 emulation with vboot
that contains a RO partition only.
Usecase: Boot with a RO-partition only.

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: Andre Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/7

Change in coreboot[master]: mb/qemu-q35: Add new fmap

[Name of user not set (Code Review)]

Hello build bot (Jenkins), Paul Menzel,

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

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

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

Change subject: mb/qemu-q35: Add new fmap
......................................................................

mb/qemu-q35: Add new fmap

Add new fmap for the qemu-q35 emulation with vboot
that contains a RO partition only.
Usecase: Boot with a RO-partition only.

Change-Id: Ie7c977ab3513be974e1e2ab39f6e05f068121df8
Signed-off-by: Andre Bach 
---
A src/mainboard/emulation/qemu-q35/vboot-ro-16M.fmd
1 file changed, 14 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/41689/8

[S] Change in coreboot[master]: mb/qemu-q35: Add new fmap

[Martin L Roth (Code Review)]

Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/41689?usp=email )

Change subject: mb/qemu-q35: Add new fmap
......................................................................

Abandoned

This patch has not been touched in over 12 months. Anyone who wants to take over work on this patch, please feel free to restore it and do any work needed to get it merged. If you create a new patch based on this work, please credit the original author.