Re: [PATCH i-g-t] include inttypes.h for PRI defines

From: Mike Frysinger <vapier@gentoo.org>
To: Petri Latvala <petri.latvala@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH i-g-t] include inttypes.h for PRI defines
Date: Wed, 10 Jan 2018 17:12:33 -0500	[thread overview]
Message-ID: <20180110221233.GC7217@vapier> (raw)
In-Reply-To: <20180110104224.rf2s4c2y26drvwf6@platvala-desk.ger.corp.intel.com>


[-- Attachment #1.1: Type: text/plain, Size: 979 bytes --]

On 10 Jan 2018 12:42, Petri Latvala wrote:
> On Tue, Jan 09, 2018 at 10:50:53PM -0500, Mike Frysinger wrote:
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96620
> > Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> > ---
> >  lib/igt_fb.c | 1 +
> >  1 file changed, 1 insertion(+)
> 
> 
> Files matching "PRI[diuoxX]" but not "inttypes.h":
> 
> lib/igt_fb.c
> lib/igt_kms.c
> lib/intel_os.c
> tests/prime_nv_pcopy.c
> tools/hsw_compute_wrpll.c
> tools/intel_gpu_top.c
> tools/intel_opregion_decode.c
> tools/intel_residency.c
> tools/intel_watermark.c
> 
> Looks like they get inttypes.h here via
> 
>  igt.h and pals -> intel_chipset.h -> pciaccess.h -> inttypes.h
> 
> The mystery is why the other files haven't slapped you with the same
> issue. Regardless, same change should be done in all of those. Can you
> expand this patch?

yeah i patched the one that was failing rather than looking deeper.
i can update them all.
-mike

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx