Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5872
-gerrit
commit 06880f655609f2980450e6a8580e02c1971cee0c
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Mon May 26 17:38:23 2014 +1000
Intel: Add common header file for CAR setup
When passing '-ffreestanding' the 'main' romstage.c may no longer
necessarily be considered the entry point.
From the C specification in 5.1.2.1 Freestanding environment;
"In a freestanding environment (in which C program execution may take
place without any benefit of an operating system), the name and type of
the function called at program startup are implementation-defined."
Clang complains about these being missing as Clang is somewhat more
strict about the spec than GNU/GCC is. An advantage here is that a
different entry-point type-signature shall now be warned about at
compile time.
Change-Id: I467001adabd47958c30c9a15e3248e42ed1151f3
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/include/cpu/intel/car.h | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/src/include/cpu/intel/car.h b/src/include/cpu/intel/car.h
new file mode 100644
index 0000000..dc89ffc
--- /dev/null
+++ b/src/include/cpu/intel/car.h
@@ -0,0 +1,7 @@
+#ifndef _CPU_INTEL_CAR_H
+#define _CPU_INTEL_CAR_H
+
+/* std signature of entry-point to romstage.c */
+void main(unsigned long bist);
+
+#endif /* _CPU_INTEL_CAR_H */
the following patch was just integrated into master:
commit f2f7f03aff2df2471fce08f48bb0a6583263158e
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Fri Apr 4 15:05:28 2014 +0300
console: Add console for GDB
Connection of UARTs to GDB stub got lost in the console transition
process, bring it back. In theory, GDB stub should work also over
usbdebug, but that solution is not really tested at all yet.
Change-Id: I90e05e8132889e788b92e055ee191f35add43bbc
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-on: http://review.coreboot.org/5343
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/5343 for details.
-gerrit
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5872
-gerrit
commit 42ac2818689e0e2f1bdc3d5c6610f4f413229324
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Mon May 26 17:38:23 2014 +1000
mainboard: -ffreestanding so 'main' is not necessarily entry
From the C specification in 5.1.2.1 Freestanding environment;
"In a freestanding environment (in which C program execution may take
place without any benefit of an operating system), the name and type of
the function called at program startup are implementation-defined."
Clang complains about these being missing as Clang is somewhat more
strict about the spec than GNU/GCC is. An advantage here is that a
different entry-point type-signature shall now be warned about at
compile time.
Change-Id: I467001adabd47958c30c9a15e3248e42ed1151f3
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/include/stddef.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/include/stddef.h b/src/include/stddef.h
index d0dad62..eda2806 100644
--- a/src/include/stddef.h
+++ b/src/include/stddef.h
@@ -30,6 +30,7 @@ typedef unsigned int wint_t;
#ifdef __PRE_RAM__
#define ROMSTAGE_CONST const
+void main(unsigned long bist); /* std signature of entry-point to romstage.c */
#else
#define ROMSTAGE_CONST
#endif
the following patch was just integrated into master:
commit c009601f29847aa91ec2f5a89a02f46a2119c5a4
Author: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Date: Mon May 5 18:56:33 2014 +0300
AGESA fam12 fam14 fam15: Declare local callouts static
Change-Id: I2ff70cafdd808a235ed4f0663e182d306f493c7e
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Reviewed-on: http://review.coreboot.org/5685
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick(a)georgi-clan.de>
See http://review.coreboot.org/5685 for details.
-gerrit