[coreboot-gerrit] [L] Change in coreboot[main]: mb/hp: Add HP Compaq 8300 Elite SFF

[Paul Menzel (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Felix Held, Michał Zieliński, Patrick Rudolph.

Paul Menzel has posted comments on this change by Michał Zieliński. ( https://review.coreboot.org/c/coreboot/+/67666?usp=email )

Change subject: mb/hp: Add HP Compaq 8300 Elite SFF
......................................................................

Patch Set 8:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/67666/comment/80dc5307_2067f67d?usp... :
PS8, Line 2: Author:     Michał Zieliński
> I don't have one.

Oh, sorry. I just looked at your first name and drew the wrong conclusions. :/

Are you reachable via the GitHub address? It contains *noreply*? Do you have an email address for development?

File Documentation/mainboard/hp/compaq_8300_sff.md:

https://review.coreboot.org/c/coreboot/+/67666/comment/4f2a0434_9e97e36e?usp... :
PS8, Line 43: Once the coreboot rom is flashed, one should remove the jumper.
> Removing a jumper enables write protection back on. […]

Acknowledged