Re: [igt-dev] [PATCH i-g-t 8/8] README.md: Accept the i915_drm.h standalone update.

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Thu, Oct 07, 2021 at 11:27:22AM +0300, Petri Latvala wrote:
> On Wed, Oct 06, 2021 at 12:14:44PM -0400, Rodrigo Vivi wrote:
> > Looking to our git log we can see that the most common case
> > is the update of the single i915_drm.h file.
> > 
> > There are some advantages of doing this standalone update:
> > 
> > 1. Make reviews easier without the noise of other headers.
> > 2. Avoid the developer to have to touch other areas that are
> >    not immediately related to targeted work.
> > 
> > Cc: Petri Latvala <petri.latvala@intel.com>
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> 
> Reviewed-by: Petri Latvala <petri.latvala@intel.com>

Thank you. Pushed the remaining patches just now.

> 
> 
> > ---
> >  README.md | 14 ++++++++++++++
> >  1 file changed, 14 insertions(+)
> > 
> > diff --git a/README.md b/README.md
> > index 62bef46d..66f2bc0f 100644
> > --- a/README.md
> > +++ b/README.md
> > @@ -159,6 +159,20 @@ These should be updated all together by:
> >  Then, commit with a note of which exact commit from airlied's branch
> >  was used to generate them.
> >  
> > +### `include/drm-uapi/i915_drm.h`
> > +
> > +Imported i915_drm.h uapi headers from airlied's drm-next branch.
> > +
> > +In some cases updating a single uapi file is needed as our history
> > +shows. So in this case, it should be done by:
> > +
> > +    # From the kernel dir with a drm/drm-next commit checked out:
> > +    $ make INSTALL_HDR_PATH=<dest-dir> headers_install
> > +    $ cp <dest-dir>/include/drm/i915_drm.h <igt-dir>/include/drm-uapi/
> > +
> > +Then, commit with a note of which exact commit from airlied's branch
> > +was used to generate it.
> > +
> >  ### `include/linux-uapi/sync_file.h`
> >  
> >  Imported non-DRM uapi headers from airlied's drm-next branch.
> > -- 
> > 2.31.1
> >