| Submitter | Felipe Contreras |
|---|---|
| Date | 2009-10-18 22:54:33 |
| Message ID | <1255906474-25091-7-git-send-email-felipe.contreras@gmail.com> |
| Download | mbox | patch |
| Permalink | /patch/54659/ |
| State | New |
| Headers | show |
Comments
On Mon, 19 Oct 2009, Felipe Contreras wrote: > drivers/acpi/acpica/tbfadt.c: In function ?acpi_tb_create_local_fadt?: > arch/x86/include/asm/string_32.h:74: warning: array subscript is above array bounds Same is in your previous patch -- what gcc is that? I don't see any warning with 4.3.1 $ make drivers/acpi/acpica/tbfadt.o CHK include/linux/version.h CHK include/linux/utsrelease.h SYMLINK include/asm -> include/asm-x86 CALL scripts/checksyscalls.sh CC drivers/acpi/acpica/tbfadt.o $
Patch
diff --git a/include/acpi/actypes.h b/include/acpi/actypes.h index c1a0eb4..7d989f1 100644 --- a/include/acpi/actypes.h +++ b/include/acpi/actypes.h @@ -438,8 +438,8 @@ typedef unsigned long long acpi_integer; #define ACPI_SET_BIT(target,bit) ((target) |= (bit)) #define ACPI_CLEAR_BIT(target,bit) ((target) &= ~(bit)) -#define ACPI_MIN(a, b) (((a)<(b))?(a):(b)) -#define ACPI_MAX(a, b) (((a)>(b))?(a):(b)) +#define ACPI_MIN(a, b) min(a, b) +#define ACPI_MAX(a, b) max(a, b) /* Size calculation */
drivers/acpi/acpica/tbfadt.c: In function ‘acpi_tb_create_local_fadt’: arch/x86/include/asm/string_32.h:74: warning: array subscript is above array bounds Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- include/acpi/actypes.h | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)