Module: Mesa
Branch: main
Commit: 0f974d1f907d1b3169fb1479ed81e2ed03789386
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=0f974d1f907d1b3169fb1479ed81e2ed03789386

Author: Alyssa Rosenzweig <[email protected]>
Date:   Wed Mar 15 17:36:17 2023 -0400

asahi: Convert to SPDX headers

Also drop my email address in the copyright lines and fix some "Copyright 208
Alyssa Rosenzweig" lines, I'm not *that* old. Together this drops a lot of
boilerplate without losing any meaningful licensing information. SPDX is already
in use for the MIT-licensed code in turnip, venus, and a few other scattered
parts of the tree, so this should be ok from a Mesa licensing standpoint.

This reduces friction to create new files, by parsing the copy/paste boilerplate
and being short enough you can easily type it out if you want.  It makes new
files seem less daunting: 20 lines of header for 30 lines of code is
discouraging, but 2 lines of header for 30 lines of code is reasonable for a
simple compiler pass. This has technical effects, as lowering the barrier to
making new files should encourage people to split code into more modular files
with (hopefully positive) effects on project compile time.

This helps with consistency between files. Across the tree we have at least a
half dozen variants of the MIT license text (probably more), plus code that uses
SPDX headers instead. I've already been using SPDX headers in Asahi manually, so
you can tell old vs new code based on the headers.

Finally, it means less for reviewers to scroll through adding files. Minimal
actual cognitive burden for reviewers thanks to banner blindness, but the big
headers still bloat diffs that add/delete files.

I originally proposed this in December (for much more of the tree) but someone
requested I wait until January to discuss. I've been trying to get in touch with
them since then. It is now almost April and, with still no response, I'd like to
press forward with this. So with a joint sign-off from the major authors of the
code in question, let's do this.

Signed-off-by: Asahi Lina <[email protected]>
Signed-off-by: Alyssa Rosenzweig <[email protected]>
Acked-by: Emma Anholt <[email protected]>
Acked-by: Daniel Stone <[email protected]>
Reviewed-by: Eric Engestrom <[email protected]>
Acked-by: Kenneth Graunke <[email protected]>
Acked-by: Rose Hudson <[email protected]>
Acked-by: Lyude Paul [over IRC: "yes I'm fine with that"]
Meh'd-by: Rob Clark <[email protected]>

Signed-off-by: Alyssa Rosenzweig <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22062>

---

 src/asahi/compiler/agx_builder.h.py                | 22 ++-------------
 src/asahi/compiler/agx_compile.c                   | 26 +++---------------
 src/asahi/compiler/agx_compile.h                   | 22 ++-------------
 src/asahi/compiler/agx_compiler.h                  | 24 ++--------------
 src/asahi/compiler/agx_dce.c                       | 22 ++-------------
 src/asahi/compiler/agx_debug.h                     | 24 ++--------------
 src/asahi/compiler/agx_liveness.c                  | 24 ++--------------
 src/asahi/compiler/agx_lower_64bit.c               | 22 ++-------------
 src/asahi/compiler/agx_lower_parallel_copy.c       | 24 ++--------------
 src/asahi/compiler/agx_lower_pseudo.c              | 22 ++-------------
 src/asahi/compiler/agx_minifloat.h                 | 22 ++-------------
 src/asahi/compiler/agx_nir_lower_texture.c         | 26 +++---------------
 src/asahi/compiler/agx_nir_opt_preamble.c          | 24 ++--------------
 src/asahi/compiler/agx_opcodes.c.py                | 22 ++-------------
 src/asahi/compiler/agx_opcodes.h.py                | 22 ++-------------
 src/asahi/compiler/agx_opcodes.py                  | 22 ++-------------
 src/asahi/compiler/agx_optimizer.c                 | 22 ++-------------
 src/asahi/compiler/agx_pack.c                      | 22 ++-------------
 src/asahi/compiler/agx_print.c                     | 24 ++--------------
 src/asahi/compiler/agx_register_allocate.c         | 22 ++-------------
 src/asahi/compiler/agx_validate.c                  | 24 ++--------------
 src/asahi/compiler/meson.build                     | 23 ++--------------
 src/asahi/compiler/test/agx_test.h                 | 25 ++---------------
 .../compiler/test/test-lower-parallel-copy.cpp     | 22 ++-------------
 src/asahi/compiler/test/test-lower-pseudo.cpp      | 22 ++-------------
 src/asahi/compiler/test/test-minifloat.cpp         | 22 ++-------------
 src/asahi/compiler/test/test-optimizer.cpp         | 22 ++-------------
 src/asahi/drm-shim/asahi_noop.c                    | 24 ++--------------
 src/asahi/drm-shim/meson.build                     | 21 ++------------
 src/asahi/layout/layout.c                          | 22 ++-------------
 src/asahi/layout/layout.h                          | 22 ++-------------
 src/asahi/layout/meson.build                       | 21 ++------------
 src/asahi/layout/tests/test-layout.cpp             | 22 ++-------------
 src/asahi/layout/tiling.c                          | 24 ++--------------
 src/asahi/lib/agx_bo.c                             | 22 ++-------------
 src/asahi/lib/agx_bo.h                             | 24 ++--------------
 src/asahi/lib/agx_device.c                         | 22 ++-------------
 src/asahi/lib/agx_device.h                         | 22 ++-------------
 src/asahi/lib/agx_formats.c                        | 22 ++-------------
 src/asahi/lib/agx_formats.h                        | 22 ++-------------
 src/asahi/lib/agx_iokit.h                          | 22 ++-------------
 src/asahi/lib/agx_ppp.h                            | 20 +-------------
 src/asahi/lib/decode.c                             | 26 +++---------------
 src/asahi/lib/decode.h                             | 24 ++--------------
 src/asahi/lib/gen_pack.py                          | 26 +++---------------
 src/asahi/lib/hexdump.h                            | 22 ++-------------
 src/asahi/lib/meson.build                          | 23 ++--------------
 src/asahi/lib/pool.c                               | 24 ++--------------
 src/asahi/lib/pool.h                               | 22 ++-------------
 src/asahi/lib/tests/test-packing.cpp               | 22 ++-------------
 src/asahi/lib/wrap.c                               | 22 ++-------------
 src/asahi/meson.build                              | 23 ++--------------
 src/gallium/drivers/asahi/agx_blit.c               | 26 +++---------------
 src/gallium/drivers/asahi/agx_disk_cache.c         | 25 +++--------------
 src/gallium/drivers/asahi/agx_disk_cache.h         | 24 ++--------------
 src/gallium/drivers/asahi/agx_fence.c              | 32 +++++-----------------
 src/gallium/drivers/asahi/agx_fence.h              | 24 ++--------------
 src/gallium/drivers/asahi/agx_pipe.c               | 24 ++--------------
 src/gallium/drivers/asahi/agx_public.h             | 20 +-------------
 src/gallium/drivers/asahi/agx_state.c              | 24 ++--------------
 src/gallium/drivers/asahi/agx_state.h              | 22 ++-------------
 src/gallium/drivers/asahi/agx_uniforms.c           | 20 +-------------
 src/gallium/drivers/asahi/meson.build              | 21 ++------------
 src/gallium/winsys/asahi/drm/asahi_drm_public.h    | 24 ++--------------
 src/gallium/winsys/asahi/drm/asahi_drm_winsys.c    | 24 ++--------------
 src/gallium/winsys/asahi/drm/meson.build           | 21 ++------------
 66 files changed, 166 insertions(+), 1351 deletions(-)

Diff:   
http://cgit.freedesktop.org/mesa/mesa/diff/?id=0f974d1f907d1b3169fb1479ed81e2ed03789386

Reply via email to