* [PATCH] x86 pci acpi: fix type mismatch
@ 2011-05-14 17:27 Németh Márton
2011-06-01 18:51 ` Jesse Barnes
0 siblings, 1 reply; 2+ messages in thread
From: Németh Márton @ 2011-05-14 17:27 UTC (permalink / raw)
To: Thomas Gleixner, Ingo Molnar, H. Peter Anvin, x86, Jesse Barnes,
Bjorn Helgaas, Andrew Morton
Cc: Julia Lawall, LKML
From: Márton Németh <nm127@freemail.hu>
The flags field of struct resource from linux/ioport.h is "unsigned long". Change
the "type" parameter of coalesce_windows() function to match that field. This fixes
the following warning messages when compiling with "make C=1 W=1 bzImage modules":
arch/x86/pci/acpi.c: In function ‘coalesce_windows’:
arch/x86/pci/acpi.c:198: warning: conversion to ‘long unsigned int’ from ‘int’ may change the sign of the result
arch/x86/pci/acpi.c:203: warning: conversion to ‘long unsigned int’ from ‘int’ may change the sign of the result
Signed-off-by: Márton Németh <nm127@freemail.hu>
---
--- linux-2.6.39-rc6/arch/x86/pci/acpi.c.orig 2011-05-04 04:59:13.000000000 +0200
+++ linux-2.6.39-rc6/arch/x86/pci/acpi.c 2011-05-14 19:12:42.000000000 +0200
@@ -188,7 +188,7 @@ static bool resource_contains(struct res
return false;
}
-static void coalesce_windows(struct pci_root_info *info, int type)
+static void coalesce_windows(struct pci_root_info *info, unsigned long type)
{
int i, j;
struct resource *res1, *res2;
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] x86 pci acpi: fix type mismatch
2011-05-14 17:27 [PATCH] x86 pci acpi: fix type mismatch Németh Márton
@ 2011-06-01 18:51 ` Jesse Barnes
0 siblings, 0 replies; 2+ messages in thread
From: Jesse Barnes @ 2011-06-01 18:51 UTC (permalink / raw)
To: Németh Márton
Cc: Thomas Gleixner, Ingo Molnar, H. Peter Anvin, x86, Bjorn Helgaas,
Andrew Morton, Julia Lawall, LKML
On Sat, 14 May 2011 19:27:33 +0200
Németh Márton <nm127@freemail.hu> wrote:
> From: Márton Németh <nm127@freemail.hu>
>
> The flags field of struct resource from linux/ioport.h is "unsigned long". Change
> the "type" parameter of coalesce_windows() function to match that field. This fixes
> the following warning messages when compiling with "make C=1 W=1 bzImage modules":
>
> arch/x86/pci/acpi.c: In function ‘coalesce_windows’:
> arch/x86/pci/acpi.c:198: warning: conversion to ‘long unsigned int’ from ‘int’ may change the sign of the result
> arch/x86/pci/acpi.c:203: warning: conversion to ‘long unsigned int’ from ‘int’ may change the sign of the result
>
> Signed-off-by: Márton Németh <nm127@freemail.hu>
> ---
> --- linux-2.6.39-rc6/arch/x86/pci/acpi.c.orig 2011-05-04 04:59:13.000000000 +0200
> +++ linux-2.6.39-rc6/arch/x86/pci/acpi.c 2011-05-14 19:12:42.000000000 +0200
> @@ -188,7 +188,7 @@ static bool resource_contains(struct res
> return false;
> }
>
> -static void coalesce_windows(struct pci_root_info *info, int type)
> +static void coalesce_windows(struct pci_root_info *info, unsigned long type)
> {
> int i, j;
> struct resource *res1, *res2;
Applied to my for-linus tree, thanks.
--
Jesse Barnes, Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-06-01 18:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-14 17:27 [PATCH] x86 pci acpi: fix type mismatch Németh Márton
2011-06-01 18:51 ` Jesse Barnes
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox