Philipp Ammann has posted comments on this change. ( https://review.coreboot.org/21421 )
Change subject: mainboard/intel/d525mw: Initial commit
......................................................................
Patch Set 1:
> Please do something like https://review.coreboot.org/#/c/21292/
> to support similar but slightly different devices.
I'll have a look at that, thanks.
> This port is worthless without a ddr3 raminit. Are you going to
> implement that?
I didn't notice that the D510MO uses DDR2. Whoops...my apologies. Can you folks help me with that?
--
To view, visit https://review.coreboot.org/21421
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3510879edc4542589d1e6754bc2e11ae2723e2ac
Gerrit-Change-Number: 21421
Gerrit-PatchSet: 1
Gerrit-Owner: Philipp Ammann
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Philipp Ammann
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 06 Sep 2017 15:09:19 +0000
Gerrit-HasComments: No
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/21421 )
Change subject: mainboard/intel/d525mw: Initial commit
......................................................................
Patch Set 1: Code-Review-1
Please do something like https://review.coreboot.org/#/c/21292/
to support similar but slightly different devices.
This port is worthless without a ddr3 raminit. Are you going to implement that?
--
To view, visit https://review.coreboot.org/21421
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3510879edc4542589d1e6754bc2e11ae2723e2ac
Gerrit-Change-Number: 21421
Gerrit-PatchSet: 1
Gerrit-Owner: Philipp Ammann
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 06 Sep 2017 14:57:00 +0000
Gerrit-HasComments: No