Hi Ashutosh, On 2023-04-26 at 13:40:28 -0700, Dixit, Ashutosh wrote: > On Tue, 25 Apr 2023 09:24:04 -0700, Vinay Belgaumkar wrote: > > > > diff --git a/tests/i915/i915_pm_freq_api.c b/tests/i915/i915_pm_freq_api.c > > new file mode 100644 > > index 00000000..17adacbc > > --- /dev/null > > +++ b/tests/i915/i915_pm_freq_api.c > > @@ -0,0 +1,153 @@ > > +// SPDX-License-Identifier: MIT > > +/* > > + * Copyright © 2023 Intel Corporation > > + */ > > + > > +#include <dirent.h> > > +#include <errno.h> > > +#include <fcntl.h> > > +#include <inttypes.h> > > +#include <stdlib.h> > > +#include <sys/stat.h> > > +#include <sys/syscall.h> > > +#include <sys/types.h> > > +#include <unistd.h> > > + > > +#include "drmtest.h" > > The series is merged but before merging I removed all the #include's above, > they are not needed. Please do not do this, at least send it to trybot and look at GitLab.Pipeline status. There are platforms where compilation failed due to missing fcntl.h, see https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/issues/137 It is fixed now with https://patchwork.freedesktop.org/series/117047/ Regards, Kamil > > > +#include "i915/gem.h" > > +#include "igt_sysfs.h" > > +#include "igt.h" > > Thanks. > -- > Ashutosh
