On Thu, 16 Oct 2008, David Miller wrote: > From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > Date: Thu, 16 Oct 2008 09:31:29 +0200 (CEST) > > > On Wed, 15 Oct 2008, David Miller wrote: > > > > kernel/resource.c: In function '__reserve_region_with_split': > > > > kernel/resource.c:554: warning: format '%llx' expects type 'long long unsigned int', but argument 3 has type 'resource_size_t' > > > > kernel/resource.c:554: warning: format '%llx' expects type 'long long unsigned int', but argument 4 has type 'resource_size_t' > > > > kernel/resource.c:554: warning: format '%llx' expects type 'long long unsigned int', but argument 6 has type 'resource_size_t' > > > > kernel/resource.c:554: warning: format '%llx' expects type 'long long unsigned int', but argument 7 has type 'resource_size_t' > > > > > > Known issue, Ben wants to add a new variant of %pX in order to print resources so that > > > resource_size_t vs. unsigned long stuff doesn't matter like this any more. > > > > Will still give a warning, as resource_size_t is not a pointer. > > The idea is to pass in a pointer to the resource struct, > and the %pX variant specified says what part to print. Neat! So we can also have a separate variant to print the resource range. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html