Turn global functions that are only used locally into static ones. This
reduces the number of -Wmissing-prototypes warnings.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/display/dc/clk_mgr/dcn315/dcn315_clk_mgr.c | 2 +-
drivers/gpu/drm/amd/display/dc/irq/dcn201/irq_service_dcn201.c | 2
Hi,
This series resolve some of the warnings that appear when compiling AMDGPU
with W=1.
Each patch is focused in a specific warning.
This is my First Patch for the GSoC Project Idea about increasing code
coverage of the DRM code[1].
Thanks for reviewing!
Best regards,
Arthur Grillo
[1
Add function prototypes to headers to reduce the number of
-Wmissing-prototypes warnings.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/display/dc/dcn31/dcn31_hubbub.h | 2 ++
drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hubbub.h | 2 ++
drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hubp.h
Add function prototypes to headers to reduce the number of
-Wmissing-prototypes warnings.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h
b/drivers/gpu/drm/amd/amdgpu
Add includes that were previously missing to reduce the number of
-Wmissing-prototypes warnings.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/display/dc/dcn32/dcn32_init.c | 1 +
drivers/gpu/drm/amd/display/dc/link/accessories/link_dp_trace.c | 1 +
2 files changed, 2
Remove arguments present on kernel-doc that are not present on the
function declaration and add the new ones if present.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c| 15 +++
drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c | 2 +-
.../gpu
|
locals->ODMCombineEnablePerState[i][k] = true;
|
^
Signed-off-by: Arthur Grillo
---
.../amd/display/dc/dml/dcn20/display_mode_vba_20.c | 9 +
.../amd/display/dc/dml/dc
Deal with return value of an allocation. This reduces the number of
-Wunused-but-set-variable warnings.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/amdgpu/amdgpu_mes.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mes.c
b/drivers/gpu/drm/amd
Remove local variables that were just set but were never used. This
decrease the number of -Wunused-but-set-variable warnings.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_link_encoder.c | 3 ---
drivers/gpu/drm/amd/display/dc/dcn201/dcn201_dpp.c | 7
|
locals->ODMCombineEnablePerState[i][k] = true;
|
^
Signed-off-by: Arthur Grillo
---
.../amd/display/dc/dml/dcn20/display_mode_vba_20.c | 9 +
.../amd/display/dc/dml/dc
Remove local variables that were just set but were never used. This
decrease the number of -Wunused-but-set-variable warnings.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_link_encoder.c | 3 +--
drivers/gpu/drm/amd/display/dc/dcn201/dcn201_dpp.c | 7
Remove a couple of local variables that are only set but never used,
also remove an static utility function that is never used in consequence
of the variable removal.
This decrease the number of -Wunused-but-set-variable warnings.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/amd/display/dc
Hi,
This series resolve some of the warnings that appear when compiling AMDGPU
with W=1.
Each patch is focused in a specific warning.
This is my First Patch for the GSoC Project Idea about increasing code
coverage of the DRM code[1].
Thanks for reviewing!
Best regards,
Arthur Grillo
[1
Extend the existing test cases to test the conversion from XRGB to
monochromatic.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 73 +++
1 file changed, 73 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c
b/drivers
As reported by Michał the drm_mm and drm_buddy unit tests lost the
printk with seed value after they being refactored into kunit. This
patch adds back this important information to assure reproducibility
converting them to use the kunit api.
Signed-off-by: Arthur Grillo
Reported-by: Michał
As reported by Michał the drm_mm and drm_buddy unit tests lost the
printk with seed value after they where refactored into KUnit. This
patch add back this important information to assure reproducibility and
convert them to the KUnit API.
Reported-by: Michał Winiarski
Signed-off-by: Arthur Grillo
As reported by Michał the drm_mm and drm_buddy unit tests lost the
printk with seed value after they were refactored into KUnit.
Add kunit_info with seed value information to assure reproducibility.
Reported-by: Michał Winiarski
Signed-off-by: Arthur Grillo
---
v1->v2:
https://lore.kernel.
As reported by Michał the drm_mm and drm_buddy unit tests lost the
printk with seed value after they were refactored into KUnit.
Add kunit_info with seed value information to assure reproducibility.
Reported-by: Michał Winiarski
Signed-off-by: Arthur Grillo
---
v1->v2:
https://lore.kernel.
As reported by Michał the drm_mm and drm_buddy unit tests lost the
printk with seed value after they were refactored into KUnit.
Add kunit_info with seed value information to assure reproducibility.
Reported-by: Michał Winiarski
Signed-off-by: Arthur Grillo
---
v1->v2:
https://lore.kernel.
coverage report, I uploaded on the link below:
https://grillo-0.github.io/coverage-reports/more-rect-tests/drivers/gpu/drm/drm_rect.c.gcov.html
Thanks for reviewing!
Best regards,
~Arthur Grillo
---
v1->v2:
https://lore.kernel.org/all/20230322140701.69852-1-arthurgri...@riseup.net/
- Cre
Insert parameterized tests for the drm_rect_intersect() to ensure
correctness and prevent future regressions.
Also, create a helper for testing if two drm_rects are equal.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_rect_test.c | 148
Insert parameterized test for the drm_rect_calc_hscale() to ensure
correctness and prevent future regressions. Besides the test for the
usual case, tests the exceptions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 72 +++
1 file changed, 72
Insert parameterized test for the drm_rect_calc_vscale() to ensure
correctness and prevent future regressions. Besides the test for the
usual case, tests the exceptions.
It uses the same struct from drm_rect_calc_hscale().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c
Insert a parameterized test for the drm_rect_rotate() to ensure
correctness and prevent future regressions.
All possible rotation modes are covered by the test.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_rect_test.c | 72 +++
1
Insert a parameterized test for the drm_rect_rotate_inv() to ensure its
correctness and prevent future regressions. The test covers all rotation
modes.
It uses the same test cases from drm_test_rect_rotate().
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests
!
Best regards,
~Arthur Grillo
---
v1->v2:
https://lore.kernel.org/all/20230322140701.69852-1-arthurgri...@riseup.net/
- Create parameterized tests for drm_rect_intersect().
- Use .16 binary fixed point values on the drm_rect_calc_hscale() and
drm_rect_calc_vscale() tests.
- Replace INT_MIN fo
Insert parameterized tests for the drm_rect_intersect() to ensure
correctness and prevent future regressions.
Also, create a helper for testing if two drm_rects are equal.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 148 ++
1 file changed
Insert parameterized test for the drm_rect_calc_hscale() to ensure
correctness and prevent future regressions. Besides the test for the
usual case, tests the exceptions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 72 +++
1 file changed, 72
Insert parameterized test for the drm_rect_calc_vscale() to ensure
correctness and prevent future regressions. Besides the test for the
usual case, tests the exceptions.
It uses the same struct from drm_rect_calc_hscale().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c
Insert a parameterized test for the drm_rect_rotate() to ensure
correctness and prevent future regressions.
All possible rotation modes are covered by the test.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 72 +++
1 file changed, 72
Insert a parameterized test for the drm_rect_rotate_inv() to ensure its
correctness and prevent future regressions. The test covers all rotation
modes.
It uses the same test cases from drm_test_rect_rotate().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 11
coverage report, I uploaded on the link below:
https://grillo-0.github.io/coverage-reports/more-rect-tests/drivers/gpu/drm/drm_rect.c.gcov.html
Thanks for reviewing!
Best regards,
~Arthur Grillo
---
v1->v2:
https://lore.kernel.org/all/20230322140701.69852-1-arthurgri...@riseup.net/
- Cre
Insert parameterized tests for the drm_rect_intersect() to ensure
correctness and prevent future regressions.
Also, create a helper for testing if two drm_rects are equal.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 148 ++
1 file changed
Insert parameterized test for the drm_rect_calc_hscale() to ensure
correctness and prevent future regressions. Besides the test for the
usual case, tests the exceptions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 72 +++
1 file changed, 72
Insert parameterized test for the drm_rect_calc_vscale() to ensure
correctness and prevent future regressions. Besides the test for the
usual case, tests the exceptions.
It uses the same struct from drm_rect_calc_hscale().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c
Insert a parameterized test for the drm_rect_rotate() to ensure
correctness and prevent future regressions.
All possible rotation modes are covered by the test.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 72 +++
1 file changed, 72
Insert a parameterized test for the drm_rect_rotate_inv() to ensure its
correctness and prevent future regressions. The test covers all rotation
modes.
It uses the same test cases from drm_test_rect_rotate().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 11
This patchset aims to insert a unit test for the conversion from
XRGB to monochromatic.
Previously this was a single patch, but as the changes grew from v1 to
v2, I thought it would be better to split it in two.
Thanks for reviewing!
Best regards,
~Arthur Grillo
---
v1->v2:
ht
Extend the existing test cases to test the conversion from XRGB to
monochromatic.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 62 +++
1 file changed, 62 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c
b/drivers
This test case uses an arbitrary pitch size, different of the default
one, to test if the conversions methods obey.
Change the "destination_pitch" colors to change the monochrome expected
result from being just zeros, as this makes the arbitrary pitch use
unusable.
Signed-off-by: Art
This patchset aims to insert a unit test for the conversion from
XRGB to monochromatic.
Previously this was a single patch, but as the changes grew from v1 to
v2, I thought it would be better to split it in two.
Thanks for reviewing!
Best regards,
~Arthur Grillo
---
v1->v2:
ht
Extend the existing test cases to test the conversion from XRGB to
monochromatic.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 62 +++
1 file changed, 62 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c
b/drivers
This test case uses an arbitrary pitch size, different of the default
one, to test if the conversions methods obey.
Change the "destination_pitch" colors to change the monochrome expected
result from being just zeros, as this makes the arbitrary pitch use
unusable.
Signed-off-by: Art
!
Best regards,
~Arthur Grillo
---
[1]: https://gcc.gnu.org/onlinedocs/gcc/Gcov.html#Gcov
---
Arthur Grillo (5):
drm/tests: Test drm_rect_intersect()
drm/tests: Test drm_rect_calc_hscale()
drm/tests: Test drm_rect_calc_vscale()
drm/tests: Test drm_rect_rotate()
drm/test: Test
Insert test for the drm_rect_intersect() function, it also create a
helper for comparing drm_rects more easily.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 30 +++
1 file changed, 30 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert test for the drm_rect_hscale() function, besides the test
for the usual case it also test for the exceptions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 58 +++
1 file changed, 58 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert test for the drm_rect_vscale() function, besides the test
for the usual case it also test for the exceptions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 56 +++
1 file changed, 56 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert test for the drm_rect_rotate() function, create test cases
for all the rotation modes.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 74 +++
1 file changed, 74 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_rect_test.c
b/drivers
Insert test for the drm_rect_rotate_inv() function, by expanding the
existing test framework for the drm_rect_rotate().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_rect_test.c
The drm_rect_intersect() already returns if the intersection is visible
or not, so the use of drm_rect_visible() is duplicate.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/i915/display/intel_overlay.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915
The drm_rect_intersect() already returns if the intersection is visible
or not, so the use of drm_rect_visible() is duplicate.
Signed-off-by: Arthur Grillo
---
v1->v2:
https://lore.kernel.org/all/20230324142533.6357-1-arthurgri...@riseup.net/
- Split the if condition.
---
drivers/gpu/drm/i
!
Best regards,
~Arthur Grillo
---
v1->v2:
https://lore.kernel.org/all/20230322140701.69852-1-arthurgri...@riseup.net/
- Create parameterized tests for drm_rect_intersect().
- Use .16 binary fixed point values on the drm_rect_calc_hscale() and
drm_rect_calc_vscale() tests.
- Replace INT_MIN fo
Insert test for the drm_rect_intersect() function, it also create a
helper for comparing drm_rects more easily.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 139 ++
1 file changed, 139 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert test for the drm_rect_vscale() function, besides the test
for the usual case it also test for the exceptions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 56 +++
1 file changed, 56 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert test for the drm_rect_rotate() function, create test cases
for all the rotation modes.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 72 +++
1 file changed, 72 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_rect_test.c
b/drivers
Insert test for the drm_rect_hscale() function, besides the test
for the usual case it also test for the exceptions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 57 +++
1 file changed, 57 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert test for the drm_rect_rotate_inv() function, by expanding the
existing test framework for the drm_rect_rotate().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_rect_test.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_rect_test.c
used through the LUT procedure
(Pekka)
- Change the switch statement to a cast to __u16* (Pekka)
- Make the apply_lut_to_channel_value return the computation result
(Pekka)
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/vkms/vkms_composer.c | 82
drm_color_lut` (Pekka)
- Remove unused variable (kernel test robot)
Signed-off-by: Arthur Grillo
Acked-by: Pekka Paalanen
---
drivers/gpu/drm/vkms/vkms_composer.c | 86
drivers/gpu/drm/vkms/vkms_crtc.c | 3 +
drivers/gpu/drm/vkms/vkms_drv.c | 20 ++-
drivers
://grillo-0.github.io/coverage-reports/gsoc-drm-format-test/drivers/gpu/drm/drm_format_helper.c.gcov.html
Arthur Grillo (6):
drm/format-helper: Test default pitch fallback
drm/format-helper: Add KUnit tests for drm_fb_swab()
drm/format-helper: Add KUnit tests for drm_fb_clip_offset()
drm
Test the default pitch fallback when NULL is passed as the dst_pitch on
the conversion procedures.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 132 --
1 file changed, 87 insertions(+), 45 deletions(-)
diff --git a/drivers/gpu/drm/tests
Insert parameterized test for the drm_fb_swab() to ensure correctness
and prevent future regressions.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 66 +++
1 file changed, 66 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert parameterized test for the drm_fb_clip_offset() to ensure
correctness and prevent future regressions.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 91 +++
1 file changed, 91 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert parameterized test for the drm_fb_build_fourcc_list() to ensure
correctness and prevent future regressions.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 143 ++
1 file changed, 143 insertions(+)
diff --git a/drivers/gpu/drm/tests
Insert parameterized test for the drm_fb_memcpy() to ensure correctness
and prevent future regressions. The test case can accept different
formats.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 391 ++
1 file changed, 391 insertions(+)
diff
The drm_fb_memcpy() supports multi-plane formats. To fully test it in
the future, add multi-plane support to the conversion_buf_size() helper.
Signed-off-by: Arthur Grillo
---
.../gpu/drm/tests/drm_format_helper_test.c| 28 +--
1 file changed, 14 insertions(+), 14 deletions
unction `fbcon_update_vcs':
.
.
.
make[3]: *** [../scripts/Makefile.vmlinux:36: vmlinux] Error 1
make[2]: *** [/home/grillo/projects/linux/Makefile:1238: vmlinux] Error 2
make[1]: *** [/home/grillo/projects/linux/Makefile:234: __sub-make] Error 2
make: *** [Makefile:234: __sub-make] Error 2
Fixes: c242
/2014
[15:05:50] RIP: 0010:drm_gem_private_object_init+0x60/0xc0
Fixes: e6303f323b1a ("drm: manager to keep track of GPUs VA mappings")
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_exec_test.c | 36 +--
1 file changed, 29 insertions(+), 7 deletion
/2014
[15:05:50] RIP: 0010:drm_gem_private_object_init+0x60/0xc0
Fixes: e6303f323b1a ("drm: manager to keep track of GPUs VA mappings")
Signed-off-by: Arthur Grillo
Tested-by: Danilo Krummrich
Acked-by: Danilo Krummrich
Reviewed-by: Maíra Canal
---
v1->v2:
https://lore.k
/2014
[15:05:50] RIP: 0010:drm_gem_private_object_init+0x60/0xc0
Fixes: e6303f323b1a ("drm: manager to keep track of GPUs VA mappings")
Signed-off-by: Arthur Grillo
Tested-by: Danilo Krummrich
Acked-by: Danilo Krummrich
Reviewed-by: Maíra Canal
---
v1->v2:
https://lore.k
previously had DRM=y
> and FB=n now has FB_CORE=y and FRAMEBUFFER_CONSOLE=y.
>
> This leads to unmet direct dependencies detected for FRAMEBUFFER_CONSOLE
> as reported by Arthur Grillo, e.g:
>
> WARNING: unmet direct dependencies detected for FRAMEBUFFER_CONSOLE
> Depends on
Support a 1D gamma LUT for each color channel on the VKMS driver. Add a
check for the LUT length by creating vkms_atomic_check().
Tested with:
igt@kms_color@gamma
igt@kms_color@legacy-gamma
igt@kms_color@invalid-gamma-lut-sizes
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/vkms
ff-by: Ruan Jinjie
> ---
> .../gpu/drm/tests/drm_format_helper_test.c| 28 +--
> 1 file changed, 14 insertions(+), 14 deletions(-)
Reviewed-by: Arthur Grillo
Best Regards,
~Arthur Grillo
>
> diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c
> b/drivers/gp
On 05/08/23 10:26, Maira Canal wrote:
> On 7/21/23 15:23, Arthur Grillo wrote:
>> Insert parameterized test for the drm_fb_memcpy() to ensure correctness
>> and prevent future regressions. The test case can accept different
>> formats.
>>
>> Signed-off-by: Arthu
://grillo-0.github.io/coverage-reports/gsoc-drm-format-test/drivers/gpu/drm/drm_format_helper.c.gcov.html
Signed-off-by: Arthur Grillo
---
v1->v2:
https://lore.kernel.org/r/20230721182316.560649-1-arthurgri...@riseup.net
- Change patch prefix to "drm/tests" (Maíra Canal)
- Simplif
Test the default pitch fallback when NULL is passed as the dst_pitch on
the conversion procedures.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 126 -
1 file changed, 81 insertions(+), 45 deletions(-)
diff --git a/drivers/gpu/drm
Insert parameterized test for the drm_fb_build_fourcc_list() to ensure
correctness and prevent future regressions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 148 +
1 file changed, 148 insertions(+)
diff --git a/drivers/gpu/drm
Insert parameterized test for the drm_fb_clip_offset() to ensure
correctness and prevent future regressions.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 91 ++
1 file changed, 91 insertions(+)
diff --git a
Insert parameterized test for the drm_fb_swab() to ensure correctness
and prevent future regressions.
Each expected color has it bytes reversed in order, so xrgb would be
bgrx.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 66
The drm_fb_memcpy() supports multi-plane formats. To fully test it in
the future, add multi-plane support to the conversion_buf_size() helper.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 28 +-
1 file
Insert parameterized test for the drm_fb_memcpy() to ensure correctness
and prevent future regressions. The test case can accept different
formats.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 391 +
1 file changed, 391 insertions
On 13/08/23 10:30, Maira Canal wrote:
> Hi Arthur,
>
> On 8/11/23 15:17, Arthur Grillo wrote:
>> Insert parameterized test for the drm_fb_memcpy() to ensure correctness
>> and prevent future regressions. The test case can accept different
>> formats.
>>
://grillo-0.github.io/coverage-reports/gsoc-drm-format-test/drivers/gpu/drm/drm_format_helper.c.gcov.html
Signed-off-by: Arthur Grillo
---
v1->v2:
https://lore.kernel.org/r/20230721182316.560649-1-arthurgri...@riseup.net
- Change patch prefix to "drm/tests" (Maíra Canal)
- Simplif
Test the default pitch fallback when NULL is passed as the dst_pitch on
the conversion procedures.
Reviewed-by: Maíra Canal
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 126 -
1 file changed, 81 insertions(+), 45 deletions(-)
diff
Insert parameterized test for the drm_fb_swab() to ensure correctness
and prevent future regressions.
Each expected color has it bytes reversed in order, so xrgb would be
bgrx.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 66
Insert parameterized test for the drm_fb_clip_offset() to ensure
correctness and prevent future regressions.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 91 ++
1 file changed, 91 insertions(+)
diff --git a
Insert parameterized test for the drm_fb_build_fourcc_list() to ensure
correctness and prevent future regressions.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 144 +
1 file changed, 144 insertions(+)
diff --git a/drivers/gpu/drm
The drm_fb_memcpy() supports multi-plane formats. To fully test it in
the future, add multi-plane support to the conversion_buf_size() helper.
Signed-off-by: Arthur Grillo
Reviewed-by: Maíra Canal
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 28 +-
1 file
Insert parameterized test for the drm_fb_memcpy() to ensure correctness
and prevent future regressions. The test case can accept different
formats.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 362 +
1 file changed, 362 insertions
lut_area` depends on the
layout of `struct drm_color_lut` (Pekka)
- Remove unused variable (kernel test robot)
v5:
- Mention that this will make it possible to run the test
igt@kms_plane@pixel-format (Maíra)
- s/had/has (Maíra)
Signed-off-by: Arthur Grillo
Acked-by: Pekka
Insert parameterized test for the drm_fb_blit() to ensure correctness
and prevent future regressions.
The test is done by adding a call to drm_fb_blit() on every format
that has support. Also, to fully test the function, add new format
conversion tests.
Signed-off-by: Arthur Grillo
---
drivers
.feakbzej-...@intel.com/
Signed-off-by: Arthur Grillo
---
include/drm/drm_drv.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/drm_drv.h b/include/drm/drm_drv.h
index 9850fe73b739..e2640dc64e08 100644
--- a/include/drm/drm_drv.h
+++ b/include/drm/drm_drv.h
@@ -584,7
tests for
drm_fb_build_fourcc_list()")
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c
b/drivers/gpu/drm/tests/drm_format_helper_test.c
ind
drm_fb_blit(). The second patch
adds those format conversion tests.
Signed-off-by: Arthur Grillo
---
Changes in v2:
- Split the patch into two (Maíra Canal)
- Link to v1:
https://lore.kernel.org/r/20230901-final-gsoc-v1-1-e310c7685...@riseup.net
---
Arthur Grillo (2):
drm/tests: Add calls to
Add a call to drm_fb_blit() on existing format conversion tests that
has support.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 142 +
1 file changed, 142 insertions(+)
diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c
b
To fully cover drm_fb_blit(), add format conversion tests that are only
supported through drm_fb_blit().
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/tests/drm_format_helper_test.c | 142 +
1 file changed, 142 insertions(+)
diff --git a/drivers/gpu/drm/tests
.petazz...@bootlin.com
Cc: seanp...@google.com
Cc: marc...@google.com
Cc: nicolejade...@google.com
Signed-off-by: Arthur Grillo
---
Arthur Grillo (7):
drm: Fix drm_fixp2int_round() making it add 0.5
drm/vkms: Add comments
drm/vkmm: Use drm_fixed api
drm/vkms: Fix compilat
/
Suggested-by: Pekka Paalanen
Signed-off-by: Arthur Grillo
---
include/drm/drm_fixed.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/drm_fixed.h b/include/drm/drm_fixed.h
index 0c9f917a4d4b..de3a79909ac9 100644
--- a/include/drm/drm_fixed.h
+++ b/include/drm/drm_fixed.h
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/vkms/vkms_formats.c | 47 +
1 file changed, 47 insertions(+)
diff --git a/drivers/gpu/drm/vkms/vkms_formats.c
b/drivers/gpu/drm/vkms/vkms_formats.c
index 44d9b9b3bdc3..55ed3f598bd7 100644
--- a/drivers/gpu/drm
With the new 32.32 values it makes more sense to use drm_fixed functions
than trying to recreate the wheel.
Signed-off-by: Arthur Grillo
---
drivers/gpu/drm/vkms/vkms_formats.c | 54 +++--
1 file changed, 28 insertions(+), 26 deletions(-)
diff --git a/drivers
1 - 100 of 192 matches
Mail list logo
