I've discovered that this is a far bigger issue than one change in one file. Most all of the invocations of ACPI_ERROR use %X to format an integer value. Most of these invocations were originally debug statements where all output was assumed to be in hex. In today's environment, we should follow standard conventions for such output -- meaning prefix all of the %X formats with 0x, or convert all %X to %u. So, I'm going to open a bugzilla on this issue so we can address all invocations of ACPI_ERROR (and ACPI_WARNING) at the same time. There are over 300 invocations of these macros across ACPICA. Bob >-----Original Message----- >From: devel-bounces@xxxxxxxxxx [mailto:devel-bounces@xxxxxxxxxx] On Behalf >Of Moore, Robert >Sent: Friday, February 19, 2010 9:09 AM >To: Chumbalkar, Nagananda; Len Brown; devel@xxxxxxxxxx >Cc: linux-acpi@xxxxxxxxxxxxxxx >Subject: Re: [Devel] [PATCH] ACPICA: exfield: minor format change in error >message (fwd) > >Thanks for your help, I'll roll this change into ACPICA today. > > >>-----Original Message----- >>From: Chumbalkar, Nagananda [mailto:Nagananda.Chumbalkar@xxxxxx] >>Sent: Friday, February 19, 2010 8:38 AM >>To: Moore, Robert; Len Brown; devel@xxxxxxxxxx >>Cc: linux-acpi@xxxxxxxxxxxxxxx >>Subject: RE: [Devel] [PATCH] ACPICA: exfield: minor format change in error >>message (fwd) >> >>[PATCH v2] ACPICA: exfield: minor format change in error message >> >>Change %X to %u. >> >>V1->V2: >>%d to %u >> >>Signed-off-by: Naga Chumbalkar <nagananda.chumbalkar@xxxxxx> >> >>diff --git a/drivers/acpi/acpica/exfield.c b/drivers/acpi/acpica/exfield.c >>index 1588a2d..b9400e7 100644 >>--- a/drivers/acpi/acpica/exfield.c >>+++ b/drivers/acpi/acpica/exfield.c >>@@ -281,7 +281,8 @@ acpi_ex_write_data_to_field(union acpi_operand_object >>*source_desc, >> >> if (source_desc->buffer.length < length) { >> ACPI_ERROR((AE_INFO, >>- "SMBus or IPMI write requires Buffer of length >>%X, found length %X", >>+ "SMBus or IPMI write requires Buffer of " >>+ "length %u, found length %u", >> length, source_desc->buffer.length)); >> >> return_ACPI_STATUS(AE_AML_BUFFER_LIMIT); >> >> >>-----Original Message----- >>From: Moore, Robert [mailto:robert.moore@xxxxxxxxx] >>Sent: Friday, February 19, 2010 1:49 AM >>To: Len Brown; devel@xxxxxxxxxx >>Cc: linux-acpi@xxxxxxxxxxxxxxx; Chumbalkar, Nagananda >>Subject: RE: [Devel] [PATCH] ACPICA: exfield: minor format >>change in error message (fwd) >> >>Yes. >> >>One nit: >>+ "length %d, found length %d", >> >>We usually use %u on these: >> >>+ "length %u, found length %u", >> >>>-----Original Message----- >>>From: devel-bounces@xxxxxxxxxx >>[mailto:devel-bounces@xxxxxxxxxx] On Behalf >>>Of Len Brown >>>Sent: Thursday, February 18, 2010 10:43 PM >>>To: devel@xxxxxxxxxx >>>Cc: linux-acpi@xxxxxxxxxxxxxxx; Naga Chumbalkar >>>Subject: [Devel] [PATCH] ACPICA: exfield: minor format change in error >>>message (fwd) >>> >>>Bob, >>>one for you. >>> >>>thanks, >>>Len Brown, Intel Open Source Technology Center >>> > >_______________________________________________ >Devel mailing list >Devel@xxxxxxxxxx >http://lists.acpica.org/listinfo/devel -- 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
