Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19690 )
Change subject: src/southbridge/i82801gx: Include a default SeaBIOS config file
......................................................................
Patch Set 1:
I suppose the best thing to do now is to abandon this and document on the wiki/documentation how users can get a speedup if only using sata?
--
To view, visit https://review.coreboot.org/19690
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6a6e79ea6cd0a14070b8092aefa477b9e3a0c4b1
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kevin O'Connor <kevin(a)koconnor.net>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
Kevin O'Connor has posted comments on this change. ( https://review.coreboot.org/19690 )
Change subject: src/southbridge/i82801gx: Include a default SeaBIOS config file
......................................................................
Patch Set 1:
> > The SeaBIOS ATA DMA setting isn't safe on PATA, nor is it safe on
> > SATA to PATA adapters.
>
> Thanks for the clarification.
>
> How about a whitelist in SeaBIOS then? For the ICH7 that would be
> DID
> 0x27c0 and 0x27c4 if I read the spec update correctly. If we'd want
> to be extra sure, we could check that combined mode is disabled
> (PCI+0x90 bits 0..1 should be zero).
A whitelist would be fine, but what would we whitelist? How would we know if a port is PATA, SATA, or PATA over SATA bridge? It's been a while since I looked at the ATA specs, so it may be possible to do something sane. My recollection was that enabling DMA on ATA was controller specific and thus too much of a hassle to support in SeaBIOS.
--
To view, visit https://review.coreboot.org/19690
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6a6e79ea6cd0a14070b8092aefa477b9e3a0c4b1
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kevin O'Connor <kevin(a)koconnor.net>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
Kevin O'Connor has posted comments on this change. ( https://review.coreboot.org/19690 )
Change subject: src/southbridge/i82801gx: Include a default SeaBIOS config file
......................................................................
Patch Set 1:
> > > (1 comment)
> > >
> > > > Why can’t this be enabled by default for coreboot?
> > >
> > > Why isn't it the global default in SeaBIOS?
> >
> > The SeaBIOS ATA DMA setting isn't safe on PATA, nor is it safe on
> > SATA to PATA adapters. On those devices, a complicated
> controller
> > specific negotiation must be done to determine the DMA level.
> The
> > SeaBIOS ATA DMA support works fine on regular SATA devices, but
> > almost all new machines use an AHCI controller now.
>
> Many boards with this southbridge do feature PATA ports, but don't
> have ACHI for SATA...
If this southbridge has PATA ports, then I would not proceed with this patch. The SeaBIOS DMA support is not complete on real PATA drives.
--
To view, visit https://review.coreboot.org/19690
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6a6e79ea6cd0a14070b8092aefa477b9e3a0c4b1
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kevin O'Connor <kevin(a)koconnor.net>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/19810 )
Change subject: mb/lenovo/*/devicetree: Add BDC detection support
......................................................................
Patch Set 1:
Tested on T400/T500. Detected the installed BDC.
--
To view, visit https://review.coreboot.org/19810
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If33ef88fb808f36b050393fa83eb1b541ce936b9
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/19570 )
Change subject: mb/*/romstage: Don't lock ETR3 CF9GR in early romstage
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/19570/1/src/mainboard/gigabyte/ga-b75m-d3h/…
File src/mainboard/gigabyte/ga-b75m-d3h/romstage.c:
PS1, Line 156: 0x10000
> huh? bit16 should be set in ACPI when USB ports are routed to the
I could not find any documentation about this bit. Do you have some ?
--
To view, visit https://review.coreboot.org/19570
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iea091511f0d2a6128d3a19e9413090c85e4c2e57
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: Yes