On 02.08.2008 16:50, Stefan Reinauer wrote:
> Index: src/arch/i386/lib/console_print.c
> ===================================================================
> --- src/arch/i386/lib/console_print.c (revision 3461)
> +++ src/arch/i386/lib/console_print.c (working copy)
> @@ -58,58 +58,59 @@
> }
> }
>
> -#define NOINLINE __attribute__((noinline))
> -static void print_emerg_char(unsigned char byte) {
> __console_tx_char(BIOS_EMERG, byte); }
> -static void print_emerg_hex8(unsigned char value){
> __console_tx_hex8(BIOS_EMERG, value); }
> -static void print_emerg_hex16(unsigned short value){
> __console_tx_hex16(BIOS_EMERG, value); }
> -static void print_emerg_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_EMERG, value); }
> -static void print_emerg(const char *str) { __console_tx_string(BIOS_EMERG,
> str); }
> +#define STATIC
I don't understand that change. You unconditionally define STATIC
as empty token instead of simply removing it.
>
> -static void print_alert_char(unsigned char byte) {
> __console_tx_char(BIOS_ALERT, byte); }
> -static void print_alert_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_ALERT, value); }
> -static void print_alert_hex16(unsigned short value){
> __console_tx_hex16(BIOS_ALERT, value); }
> -static void print_alert_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_ALERT, value); }
> -static void print_alert(const char *str) { __console_tx_string(BIOS_ALERT,
> str); }
> +STATIC void print_emerg_char(unsigned char byte) {
> __console_tx_char(BIOS_EMERG, byte); }
> +STATIC void print_emerg_hex8(unsigned char value){
> __console_tx_hex8(BIOS_EMERG, value); }
> +STATIC void print_emerg_hex16(unsigned short value){
> __console_tx_hex16(BIOS_EMERG, value); }
> +STATIC void print_emerg_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_EMERG, value); }
> +STATIC void print_emerg(const char *str) { __console_tx_string(BIOS_EMERG,
> str); }
>
> -static void print_crit_char(unsigned char byte) {
> __console_tx_char(BIOS_CRIT, byte); }
> -static void print_crit_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_CRIT, value); }
> -static void print_crit_hex16(unsigned short value){
> __console_tx_hex16(BIOS_CRIT, value); }
> -static void print_crit_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_CRIT, value); }
> -static void print_crit(const char *str) { __console_tx_string(BIOS_CRIT,
> str); }
> +STATIC void print_alert_char(unsigned char byte) {
> __console_tx_char(BIOS_ALERT, byte); }
> +STATIC void print_alert_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_ALERT, value); }
> +STATIC void print_alert_hex16(unsigned short value){
> __console_tx_hex16(BIOS_ALERT, value); }
> +STATIC void print_alert_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_ALERT, value); }
> +STATIC void print_alert(const char *str) { __console_tx_string(BIOS_ALERT,
> str); }
>
> -static void print_err_char(unsigned char byte) { __console_tx_char(BIOS_ERR,
> byte); }
> -static void print_err_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_ERR, value); }
> -static void print_err_hex16(unsigned short value){
> __console_tx_hex16(BIOS_ERR, value); }
> -static void print_err_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_ERR, value); }
> -static void print_err(const char *str) { __console_tx_string(BIOS_ERR, str);
> }
> +STATIC void print_crit_char(unsigned char byte) {
> __console_tx_char(BIOS_CRIT, byte); }
> +STATIC void print_crit_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_CRIT, value); }
> +STATIC void print_crit_hex16(unsigned short value){
> __console_tx_hex16(BIOS_CRIT, value); }
> +STATIC void print_crit_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_CRIT, value); }
> +STATIC void print_crit(const char *str) { __console_tx_string(BIOS_CRIT,
> str); }
>
> -static void print_warning_char(unsigned char byte) {
> __console_tx_char(BIOS_WARNING, byte); }
> -static void print_warning_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_WARNING, value); }
> -static void print_warning_hex16(unsigned short value){
> __console_tx_hex16(BIOS_WARNING, value); }
> -static void print_warning_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_WARNING, value); }
> -static void print_warning(const char *str) {
> __console_tx_string(BIOS_WARNING, str); }
> +STATIC void print_err_char(unsigned char byte) { __console_tx_char(BIOS_ERR,
> byte); }
> +STATIC void print_err_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_ERR, value); }
> +STATIC void print_err_hex16(unsigned short value){
> __console_tx_hex16(BIOS_ERR, value); }
> +STATIC void print_err_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_ERR, value); }
> +STATIC void print_err(const char *str) { __console_tx_string(BIOS_ERR, str);
> }
>
> -static void print_notice_char(unsigned char byte) {
> __console_tx_char(BIOS_NOTICE, byte); }
> -static void print_notice_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_NOTICE, value); }
> -static void print_notice_hex16(unsigned short value){
> __console_tx_hex16(BIOS_NOTICE, value); }
> -static void print_notice_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_NOTICE, value); }
> -static void print_notice(const char *str) { __console_tx_string(BIOS_NOTICE,
> str); }
> +STATIC void print_warning_char(unsigned char byte) {
> __console_tx_char(BIOS_WARNING, byte); }
> +STATIC void print_warning_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_WARNING, value); }
> +STATIC void print_warning_hex16(unsigned short value){
> __console_tx_hex16(BIOS_WARNING, value); }
> +STATIC void print_warning_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_WARNING, value); }
> +STATIC void print_warning(const char *str) {
> __console_tx_string(BIOS_WARNING, str); }
>
> -static void print_info_char(unsigned char byte) {
> __console_tx_char(BIOS_INFO, byte); }
> -static void print_info_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_INFO, value); }
> -static void print_info_hex16(unsigned short value){
> __console_tx_hex16(BIOS_INFO, value); }
> -static void print_info_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_INFO, value); }
> -static void print_info(const char *str) { __console_tx_string(BIOS_INFO,
> str); }
> +STATIC void print_notice_char(unsigned char byte) {
> __console_tx_char(BIOS_NOTICE, byte); }
> +STATIC void print_notice_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_NOTICE, value); }
> +STATIC void print_notice_hex16(unsigned short value){
> __console_tx_hex16(BIOS_NOTICE, value); }
> +STATIC void print_notice_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_NOTICE, value); }
> +STATIC void print_notice(const char *str) { __console_tx_string(BIOS_NOTICE,
> str); }
>
> -static void print_debug_char(unsigned char byte) {
> __console_tx_char(BIOS_DEBUG, byte); }
> -static void print_debug_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_DEBUG, value); }
> -static void print_debug_hex16(unsigned short value){
> __console_tx_hex16(BIOS_DEBUG, value); }
> -static void print_debug_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_DEBUG, value); }
> -static void print_debug(const char *str) { __console_tx_string(BIOS_DEBUG,
> str); }
> +STATIC void print_info_char(unsigned char byte) {
> __console_tx_char(BIOS_INFO, byte); }
> +STATIC void print_info_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_INFO, value); }
> +STATIC void print_info_hex16(unsigned short value){
> __console_tx_hex16(BIOS_INFO, value); }
> +STATIC void print_info_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_INFO, value); }
> +STATIC void print_info(const char *str) { __console_tx_string(BIOS_INFO,
> str); }
>
> -static void print_spew_char(unsigned char byte) {
> __console_tx_char(BIOS_SPEW, byte); }
> -static void print_spew_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_SPEW, value); }
> -static void print_spew_hex16(unsigned short value){
> __console_tx_hex16(BIOS_SPEW, value); }
> -static void print_spew_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_SPEW, value); }
> -static void print_spew(const char *str) { __console_tx_string(BIOS_SPEW,
> str); }
> +STATIC void print_debug_char(unsigned char byte) {
> __console_tx_char(BIOS_DEBUG, byte); }
> +STATIC void print_debug_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_DEBUG, value); }
> +STATIC void print_debug_hex16(unsigned short value){
> __console_tx_hex16(BIOS_DEBUG, value); }
> +STATIC void print_debug_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_DEBUG, value); }
> +STATIC void print_debug(const char *str) { __console_tx_string(BIOS_DEBUG,
> str); }
>
> +STATIC void print_spew_char(unsigned char byte) {
> __console_tx_char(BIOS_SPEW, byte); }
> +STATIC void print_spew_hex8(unsigned char value) {
> __console_tx_hex8(BIOS_SPEW, value); }
> +STATIC void print_spew_hex16(unsigned short value){
> __console_tx_hex16(BIOS_SPEW, value); }
> +STATIC void print_spew_hex32(unsigned int value) {
> __console_tx_hex32(BIOS_SPEW, value); }
> +STATIC void print_spew(const char *str) { __console_tx_string(BIOS_SPEW,
> str); }
> +
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
--
coreboot mailing list
[email protected]
http://www.coreboot.org/mailman/listinfo/coreboot
