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

Rodrigo Vivi rodrigo.vivi at intel.com
Wed Oct 6 16:14:44 UTC 2021


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 at intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
---
 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



More information about the igt-dev mailing list