On Thu, 27 Apr 2023 09:24:58 -0700, Kamil Konieczny wrote: > > 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/ Hi Kamil, agreed, sorry about that. Lesson learnt. Thanks for fixing it quickly. Ashutosh
