print expects C types, so conversion to them helps (use (unsigned) for %X or (unsigned long) for %lX). Regards, Alex. Moore, Robert wrote: > Problem is to make one version of the code compile everywhere with no warnings. > >> -----Original Message----- >> From: linux-acpi-owner@xxxxxxxxxxxxxxx [mailto:linux-acpi- >> owner@xxxxxxxxxxxxxxx] On Behalf Of Sergio Monteiro Basto >> Sent: Tuesday, September 19, 2006 6:30 PM >> To: acpi devel >> Cc: David Brownell >> Subject: [Fwd: [patch 2.6.18-rc7] ACPI: build warnings begone (x86_64)] >> >> Hi, I think this should be reported on >> ACPI mailing list , so I am Forwarding to this list >> >> -------- Forwarded Message -------- >>> From: David Brownell <david-b@xxxxxxxxxxx> >>> To: Brown, Len <len.brown@xxxxxxxxx> >>> Cc: linux-kernel@xxxxxxxxxxxxxxx >>> Subject: [patch 2.6.18-rc7] ACPI: build warnings begone (x86_64) >>> Date: Wed, 13 Sep 2006 21:00:33 -0700 >>> >>> This makes various ACPI compiler warnings go away (x86_64), >>> making builds be warning-free again. >>> >>> Signed-off-by: David Brownell <dbrownell@xxxxxxxxxxxxxxxxxxxxx> >>> >>> Index: g26/drivers/acpi/executer/exmutex.c >>> =================================================================== >>> --- g26.orig/drivers/acpi/executer/exmutex.c 2006-07-14 >> 10:03:04.000000000 -0700 >>> +++ g26/drivers/acpi/executer/exmutex.c 2006-09-13 13:31:10.000000000 - >> 0700 >>> @@ -266,10 +266,10 @@ acpi_ex_release_mutex(union acpi_operand >>> walk_state->thread->thread_id) >>> && (obj_desc->mutex.os_mutex != ACPI_GLOBAL_LOCK)) { >>> ACPI_ERROR((AE_INFO, >>> - "Thread %X cannot release Mutex [%4.4s] acquired by >> thread %X", >>> - (u32) walk_state->thread->thread_id, >>> + "Thread %p cannot release Mutex [%4.4s] acquired by >> thread %p", >>> + walk_state->thread->thread_id, >>> acpi_ut_get_node_name(obj_desc->mutex.node), >>> - (u32) obj_desc->mutex.owner_thread->thread_id)); >>> + obj_desc->mutex.owner_thread->thread_id)); >>> return_ACPI_STATUS(AE_AML_NOT_OWNER); >>> } >>> >>> Index: g26/drivers/acpi/tables/tbget.c >>> =================================================================== >>> --- g26.orig/drivers/acpi/tables/tbget.c 2006-07-14 >> 10:03:04.000000000 -0700 >>> +++ g26/drivers/acpi/tables/tbget.c 2006-09-13 13:37:27.000000000 - >> 0700 >>> @@ -324,7 +324,7 @@ acpi_tb_get_this_table(struct acpi_point >>> >>> if (header->length < sizeof(struct acpi_table_header)) { >>> ACPI_ERROR((AE_INFO, >>> - "Table length (%X) is smaller than minimum (%X)", >>> + "Table length (%X) is smaller than minimum (%zX)", >>> header->length, sizeof(struct acpi_table_header))); >>> >>> return_ACPI_STATUS(AE_INVALID_TABLE_LENGTH); >>> Index: g26/drivers/acpi/utilities/utmutex.c >>> =================================================================== >>> --- g26.orig/drivers/acpi/utilities/utmutex.c 2006-07-14 >> 10:03:04.000000000 -0700 >>> +++ g26/drivers/acpi/utilities/utmutex.c 2006-09-13 >> 13:39:24.000000000 -0700 >>> @@ -258,8 +258,8 @@ acpi_status acpi_ut_acquire_mutex(acpi_m >>> acpi_gbl_mutex_info[mutex_id].thread_id = this_thread_id; >>> } else { >>> ACPI_EXCEPTION((AE_INFO, status, >>> - "Thread %X could not acquire Mutex [%X]", >>> - (u32) this_thread_id, mutex_id)); >>> + "Thread %p could not acquire Mutex [%X]", >>> + this_thread_id, mutex_id)); >>> } >>> >>> return (status); >>> Index: g26/drivers/acpi/tables/tbrsdt.c >>> =================================================================== >>> --- g26.orig/drivers/acpi/tables/tbrsdt.c 2006-07-14 >> 10:03:04.000000000 -0700 >>> +++ g26/drivers/acpi/tables/tbrsdt.c 2006-09-13 13:38:39.000000000 - >> 0700 >>> @@ -187,7 +187,7 @@ acpi_status acpi_tb_validate_rsdt(struct >>> >>> if (table_ptr->length < sizeof(struct acpi_table_header)) { >>> ACPI_ERROR((AE_INFO, >>> - "RSDT/XSDT length (%X) is smaller than minimum >> (%X)", >>> + "RSDT/XSDT length (%X) is smaller than minimum >> (%zX)", >>> table_ptr->length, >>> sizeof(struct acpi_table_header))); >>> >>> - >>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" >> in >>> the body of a message to majordomo@xxxxxxxxxxxxxxx >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> Please read the FAQ at http://www.tux.org/lkml/ >> -- >> Sérgio M.B. > - > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
