the following patch was just integrated into master:
commit a3119e5835e4b8fd510d046c56a3bf2bf43a5c0d
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Jun 18 14:28:03 2014 +1000
drivers/elog: Unmangle header include out of pre-proc cond
Change-Id: Ic4905d8a6908a30602382f5846f1dc2c0dbe2431
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6068
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/6068 for details.
-gerrit
the following patch was just integrated into master:
commit b57fef9f3e2b3cb8d8c8c5a3552c94a28ff81d15
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Tue Jun 17 20:13:08 2014 +1000
src/ec: Sanitize headers and comment #endif pairings
Comment #endif /* FOO */ pairings.
Alphabetise headers and remove any #if CONFIG_ guards around them.
Background rational:
Remove guarding the inclusion of headers based on CONFIG_ options. This
*potentially* could hide issues such as functions being swapped from
under our feet, since different runtime behaviour could be declared with
the same function same name and type-signature. Hence, depending on the
header we happen to get may change runtime behaviour.
Change-Id: Ic61bdfb64d99f0e2998c6451ae6686915b7bb3d4
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6059
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/6059 for details.
-gerrit
the following patch was just integrated into master:
commit 0ddb82671cae52571e92b7b22cf088939d887d50
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Tue Jun 17 18:37:08 2014 +1000
src/console: Sanitize headers and IS_ENABLED usage
Alphabetise headers and remove any #if CONFIG_ guards around them.
Use #if IS_ENABLED(CONFIG_FOO) over #if CONFIG_FOO where applicable.
Change-Id: I2a616bcfb8470a1fa21c9e26271e81cca835272a
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6057
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/6057 for details.
-gerrit
the following patch was just integrated into master:
commit 6c99250c3fb97ccaa03d4f0c158caffdc53ff995
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Fri Jun 20 21:19:06 2014 +1000
device/pci_device.c: Sanitize headers
Change-Id: I6254f4ab767952cc8ff31bb462c7037b027442ba
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6079
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/6079 for details.
-gerrit
the following patch was just integrated into master:
commit 618ddfeea557f076fb38f21671dcabe34411f583
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Fri Jun 20 21:49:39 2014 +1000
device/{cardbus,agp}.h: Missing header for device_t type
Missing header for the ramstage version of device_t which is a struct
ptr.
Change-Id: Ie2a30b75ee1d0513397276b81e8df1d995707f6f
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6080
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/6080 for details.
-gerrit
the following patch was just integrated into master:
commit b393fa09e565af929474c442a62589ad1d73acbf
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Sat Jun 21 23:50:53 2014 +0300
AGESA S3: Fix ACPISCRATCH in CBMEM
After commit
2ca2afe ACPI S3 support: Add acpi_s3_resume_allowed()
ACPISCRATCH region in CBMEM was no longer allocated, causing
AGESA platforms to fail S3 resume.
IS_ENABLED() did not evaluate true here with non-zero parameter.
Also avoid multiple defined defaults for HIGH_SCRATCH_MEMORY_SIZE.
Change-Id: Id99e4bee91581b8ac3d1ec44763b2d792b721832
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-on: http://review.coreboot.org/6093
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Reviewed-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Reviewed-by: Martin Roth <gaumless(a)gmail.com>
See http://review.coreboot.org/6093 for details.
-gerrit
the following patch was just integrated into master:
commit 4f7cb87df2a4b2d1da6c1e759a3a69bc36626850
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Thu Jun 19 03:48:42 2014 +0300
AGESA: Move config parameters for non-volatile S3 data
These parameters are not specific to the southbridge device, but
the implementation of S3 storage defined by CPU code.
Change-Id: Ic341cc2b7669cf8e3e920c48473826ec03fc7d8d
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-on: http://review.coreboot.org/6081
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/6081 for details.
-gerrit
the following patch was just integrated into master:
commit 207880cd1127acd6f5f0f2241d753aa6b1c39da0
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Tue Dec 10 09:03:17 2013 +0200
Declare acpi_is_wakeup_early() only once
Change-Id: I5314d76168c40a6327d4a9ac3b4f4fb05497d6fc
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-on: http://review.coreboot.org/4525
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/4525 for details.
-gerrit
the following patch was just integrated into master:
commit 206f37043ed4c8581c7351399c267434653ec13b
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Sat Jun 14 21:01:22 2014 +0300
i945 boards: Drop disabled ram_check() calls
This code would not get enabled just by flipping the options in menuconfig,
also ramcheck() no longer test the range like the parameters would imply.
We should add non-destructive ram_check() on S3 resume path to verify
memory controller configuration has been properly recovered.
Change-Id: Ie4675c4770146c4312cdfbc81afa19f243f90ee4
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-on: http://review.coreboot.org/6027
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net>
Reviewed-by: Idwer Vollering <vidwer(a)gmail.com>
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/6027 for details.
-gerrit