Hello Patrick Rudolph, Aaron Durbin, Julius Werner, Paul Menzel, build bot (Jenkins), Philipp Hug, Patrick Georgi, Furquan Shaikh, ron minnich, David Guckian, Vanny E, Huang Jin, York Yang, Lee Leahy, Jonathan Neuschäfer, Nico Huber, David Guckian, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/17656
to look at the new patch set (#9).
Change subject: buildsystem: Promote rules.h to default include
......................................................................
buildsystem: Promote rules.h to default include
Does not fix 3rdparty/, *.S or *.ld or yet.
Change-Id: I66b48013dd89540b35ab219d2b64bc13f5f19cda
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
---
M Makefile.inc
M src/arch/arm64/boot.c
M src/arch/riscv/boot.c
M src/arch/riscv/stages.c
M src/arch/x86/acpi_s3.c
M src/arch/x86/exception.c
M src/arch/x86/include/arch/acpi.h
M src/arch/x86/include/arch/cpu.h
M src/arch/x86/include/arch/early_variables.h
M src/arch/x86/include/arch/exception.h
M src/arch/x86/include/arch/io.h
M src/arch/x86/include/arch/memlayout.h
M src/arch/x86/rdrand.c
M src/commonlib/storage/pci_sdhci.c
M src/console/console.c
M src/console/init.c
M src/console/post.c
M src/cpu/intel/microcode/microcode.c
M src/cpu/x86/32bit/entry32.inc
M src/cpu/x86/pae/pgtbl.c
M src/drivers/intel/fsp1_1/cache_as_ram.inc
M src/drivers/intel/fsp1_1/include/fsp/util.h
M src/drivers/intel/fsp2_0/include/fsp/info_header.h
M src/drivers/net/ne2k.c
M src/drivers/spi/spi_flash.c
M src/drivers/uart/uart8250io.c
M src/drivers/uart/uart8250mem.c
M src/include/bootstate.h
M src/include/cbmem.h
M src/include/console/cbmem_console.h
M src/include/console/console.h
M src/include/console/flash.h
M src/include/console/ne2k.h
M src/include/console/qemu_debugcon.h
M src/include/console/spi.h
M src/include/console/spkmodem.h
M src/include/console/uart.h
M src/include/console/usb.h
M src/include/device/device.h
M src/include/device/pci.h
M src/include/device/pnp.h
M src/include/memlayout.h
M src/include/stddef.h
M src/lib/bootmode.c
M src/lib/cbmem_common.c
M src/lib/ext_stage_cache.c
M src/lib/imd_cbmem.c
M src/lib/prog_loaders.c
M src/lib/romstage_handoff.c
M src/lib/timestamp.c
M src/mainboard/google/cyan/chromeos.c
M src/mainboard/google/dragonegg/chromeos.c
M src/mainboard/google/eve/chromeos.c
M src/mainboard/google/fizz/chromeos.c
M src/mainboard/google/glados/chromeos.c
M src/mainboard/google/kahlee/ec.c
M src/mainboard/google/octopus/ec.c
M src/mainboard/google/poppy/chromeos.c
M src/mainboard/google/reef/ec.c
M src/mainboard/google/sarien/chromeos.c
M src/mainboard/google/storm/mmu.c
M src/mainboard/intel/cannonlake_rvp/chromeos.c
M src/mainboard/intel/coffeelake_rvp/chromeos.c
M src/mainboard/intel/glkrvp/ec.c
M src/mainboard/intel/icelake_rvp/chromeos.c
M src/mainboard/intel/kblrvp/chromeos.c
M src/mainboard/intel/kunimitsu/chromeos.c
M src/mainboard/intel/strago/chromeos.c
M src/northbridge/intel/fsp_rangeley/northbridge.h
M src/northbridge/intel/gm45/gm45.h
M src/northbridge/intel/sandybridge/sandybridge.h
M src/security/vboot/bootmode.c
M src/security/vboot/vboot_common.c
M src/security/vboot/vboot_loader.c
M src/soc/amd/common/block/pi/agesawrapper.c
M src/soc/amd/stoneyridge/include/soc/pci_devs.h
M src/soc/amd/stoneyridge/include/soc/southbridge.h
M src/soc/intel/apollolake/include/soc/pci_devs.h
M src/soc/intel/apollolake/pmutil.c
M src/soc/intel/braswell/acpi.c
M src/soc/intel/braswell/include/soc/iosf.h
M src/soc/intel/braswell/include/soc/nvs.h
M src/soc/intel/braswell/include/soc/smm.h
M src/soc/intel/braswell/pmutil.c
M src/soc/intel/braswell/spi.c
M src/soc/intel/braswell/tsc_freq.c
M src/soc/intel/cannonlake/include/soc/pci_devs.h
M src/soc/intel/cannonlake/pmutil.c
M src/soc/intel/common/block/p2sb/p2sb.c
M src/soc/intel/denverton_ns/include/soc/pci_devs.h
M src/soc/intel/icelake/include/soc/pci_devs.h
M src/soc/intel/icelake/pmutil.c
M src/soc/intel/skylake/include/soc/nvs.h
M src/soc/intel/skylake/include/soc/pch.h
M src/soc/intel/skylake/include/soc/pci_devs.h
M src/soc/intel/skylake/pmutil.c
M src/southbridge/amd/rs780/rs780.h
M src/southbridge/intel/bd82x6x/early_pch_common.c
98 files changed, 1 insertion(+), 98 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/17656/9
--
To view, visit https://review.coreboot.org/c/coreboot/+/17656
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I66b48013dd89540b35ab219d2b64bc13f5f19cda
Gerrit-Change-Number: 17656
Gerrit-PatchSet: 9
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: David Guckian
Gerrit-Reviewer: David Guckian <david.guckian(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Huang Jin <huang.jin(a)intel.com>
Gerrit-Reviewer: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Lee Leahy <leroy.p.leahy(a)intel.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Philipp Hug <philipp(a)hug.cx>
Gerrit-Reviewer: Vanny E <vanessa.f.eusebio(a)intel.com>
Gerrit-Reviewer: York Yang <york.yang(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: ron minnich <rminnich(a)gmail.com>
Gerrit-MessageType: newpatchset
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30320 )
Change subject: sb/intel/i82801gx: Implement PCIe coalescing
......................................................................
Patch Set 12: Code-Review+2
(2 comments)
https://review.coreboot.org/#/c/30320/12/src/southbridge/intel/i82801gx/pci…
File src/southbridge/intel/i82801gx/pcie.c:
https://review.coreboot.org/#/c/30320/12/src/southbridge/intel/i82801gx/pci…
PS12, Line 30: int coalesce;
Caching `coalesce` doesn't seem necessary as you can also read it
from `config` when checking `rpc.ports[0].enable`.
https://review.coreboot.org/#/c/30320/12/src/southbridge/intel/i82801gx/pci…
PS12, Line 125: struct device *dev = pcidev_path_on_root(PCI_DEVFN(31, 0));
same as `pcidev_on_root(31, 0)`
--
To view, visit https://review.coreboot.org/c/coreboot/+/30320
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6d7ddef96e4f45e163f7017175398a0938a18273
Gerrit-Change-Number: 30320
Gerrit-PatchSet: 12
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Tristan Corrick <tristan(a)corrick.kiwi>
Gerrit-Comment-Date: Sun, 13 Jan 2019 22:41:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28640 )
Change subject: mb/clevo/n130wu: Add mainboard
......................................................................
Patch Set 65:
> Patch Set 63:
>
> Please add the variant in a separate commit on top.
Done.
--
To view, visit https://review.coreboot.org/c/coreboot/+/28640
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I364f5849ef88f43b85efbd7a635a27e54d08c513
Gerrit-Change-Number: 28640
Gerrit-PatchSet: 65
Gerrit-Owner: Felix Singer <migy(a)darmstadt.ccc.de>
Gerrit-Reviewer: Felix Singer <migy(a)darmstadt.ccc.de>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-CC: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-CC: Shaleen jain <shaleen(a)jain.sh>
Gerrit-Comment-Date: Sun, 13 Jan 2019 20:57:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28640 )
Change subject: mb/clevo/n130wu: Add mainboard
......................................................................
Patch Set 65:
(4 comments)
https://review.coreboot.org/#/c/28640/65/src/mainboard/clevo/n130wu/gpio.h
File src/mainboard/clevo/n130wu/gpio.h:
https://review.coreboot.org/#/c/28640/65/src/mainboard/clevo/n130wu/gpio.h@…
PS65, Line 168: /* I2C5_SDA/ISH_I2C2_SDA */ _PAD_CFG_STRUCT(GPP_F10, 0x44000b02, 0x2000000),
line over 80 characters
https://review.coreboot.org/#/c/28640/65/src/mainboard/clevo/n130wu/gpio.h@…
PS65, Line 169: /* I2C5_SCL/ISH_I2C2_SCL */ _PAD_CFG_STRUCT(GPP_F11, 0x44000b02, 0x2000000),
line over 80 characters
https://review.coreboot.org/#/c/28640/65/src/mainboard/clevo/n130wu/pei_dat…
File src/mainboard/clevo/n130wu/pei_data.c:
https://review.coreboot.org/#/c/28640/65/src/mainboard/clevo/n130wu/pei_dat…
PS65, Line 31: 0x33, 0x00, 0xFF, 0x00, 0xFF, 0x00}};
space required after that close brace '}'
https://review.coreboot.org/#/c/28640/65/src/mainboard/clevo/n130wu/pei_dat…
PS65, Line 40: {1, 0, 4, 5, 2, 3, 6, 7}};
space required after that close brace '}'
--
To view, visit https://review.coreboot.org/c/coreboot/+/28640
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I364f5849ef88f43b85efbd7a635a27e54d08c513
Gerrit-Change-Number: 28640
Gerrit-PatchSet: 65
Gerrit-Owner: Felix Singer <migy(a)darmstadt.ccc.de>
Gerrit-Reviewer: Felix Singer <migy(a)darmstadt.ccc.de>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-CC: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-CC: Shaleen jain <shaleen(a)jain.sh>
Gerrit-Comment-Date: Sun, 13 Jan 2019 20:35:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment