summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2018-10-31lib: Make libunwind optional againDaniel Vetter
Apparently it's really not great to require it for a bunch of platforms. Requested by Sean and Eric. v2: Use combo option (Petri). v3: Fix the right option (Petri) v4: try a bit harder ... v5: Even more simplification (Dylan) Cc: Dylan Baker <dylan@pnwbakers.com> Cc: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Eric Anholt <eric@anholt.net> (v1) Reviewed-by: Petri Latvala <petri.latvala@intel.com> Cc: Sean Paul <sean@poorly.run> Cc: Eric Anholt <eric@anholt.net> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-31igt/gem_ppgtt: Unroll 32b memsetChris Wilson
Manually unroll the 32b memset in create_bo() to set a cacheline at a time, for a 2x speed improve of the whole test. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Michał Winiarski <michal.winiarski@intel.com>
2018-10-30igt/gem_tiled_fence_blits: Remember to mark up fence blitsChris Wilson
Older platforms require fence registers to perform blits, and so userspace is expected to mark up the objects to request fences be assigned. Fixes: ff2db94acb53 ("igt/gem_tiled_fence_blits: Remove libdrm_intel dependence") Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108591 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2018-10-30gitlab-ci: Build Docker images if Dockerfiles have changedArkadiusz Hiler
The jobs that build container images and upload them to the registry do not have to be manual any longer. By leveraging the new only-changes feature (GitLab 11.4), we can trigger those jobs conditionally on any alterations to the Dockerfiles. Cc: Daniel Vetter <daniel@ffwll.ch> Cc: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-29tests/kms_content_protection: restore for real before failingDaniel Vetter
Nothing happens if you don't commit ... v2: rebased!? Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108550 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108549 Cc: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Ramalingam C <ramalingam.c@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-26lib/kmod: Unload snd_hdmi_lpe_audioVille Syrjälä
Unload the snd_hdmi_lpe_audio module. Otherwise i915 can't be unloaded. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-26kms_plane_alpha_blend: Start CRC after enabling display.Maarten Lankhorst
This will fix the -EIO from trying to start CRC on a disabled pipe. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108146 Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed-by: Imre Deak <imre.deak@intel.com>
2018-10-26igt/gem_tiled_fence_blits: Remove libdrm_intel dependenceChris Wilson
Modernise the test to use igt's ioctl library as opposed to the antiquated libdrm_intel. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2018-10-25tests/kms_content_protection: Avoid dmesg falloutDaniel Vetter
Subsequent tests have become really unhappy. Paper over this for now. If this doesn't work, then I think we need to revert. v2: Use Ram's suggestion for comment. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108550 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108549 Cc: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Ramalingam C <ramalingam.c@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-25tools/intel_watermark: Recognize cnl+Ville Syrjälä
Pick the skl+ code path for cnl+. And since this tool has nothing to do with pch let's also replace the has_pch_split check with gen>=5 check. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-10-25README|CONTRIBUTING: Move to .mdDaniel Vetter
Looks so much better in the gitlab UI. Maybe we want to split out some of these README into subdirectories ... v2: Polish layout a bit. Changes are only whitespace, but README is so complicated reworked that git's rename detection doesn't spot the similarities anymore. Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-25CONTRIBUTING: Point towards gitlabPetri Latvala
Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-25kms_content_protection: Add Content Protection testRamalingam C
Pretty simple test: - picks the hdcp capable output with suitable pipe and apply modeset. - checks the connected sink's hdcp capability through debugfs - apply a FB and wait for the flip completion. - clears the content protection property - verifies that it clears - sets the content protection property to desired - verifies that it transitions to enabled - incase of timeout three reattempts are implemented - clear the content protection property and modeset on the crtc Above steps are repeated on all HDCP capable connectors for both legacy and atomic subtests. v2: dynamic subtests are dropped [Daniel] v3: debugfs is used to detect the sink's hdcp capability [Daniel] data structure is made as global variable. v4: debugfs file from connector's debugfs dir is used [Daniel] v5: i915_debugfs_connector_dir() usage is modified [Chris] Signed-off-by: Sean Paul <seanpaul@chromium.org> Signed-off-by: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-25lib/debugfs: function to open connector debugfs dirRamalingam C
Function to open a debugfs directory of a connector associated to a device. v2: instead of string manipulation openat used [Chris] Signed-off-by: Ramalingam C <ramalingam.c@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Do meson build with oldest supported mesonArkadiusz Hiler
Documentation building differs for meson < 0.47, so let's make sure that it works. Also remove a comment that has slipped through. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Compare test-list.txt(s) generated by meson and autotoolsArkadiusz Hiler
Let's make sure that we haven't missed any tests with any of the supported build systems. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Compile using autotoolsArkadiusz Hiler
Let's make sure that IGT compiles with autotools. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24gitlab-ci: Parametrize images used with CI_ variablesArkadiusz Hiler
The container stage was already using CI_ variables, pushing to the right registry. Let's do that for the pulling containers too. Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-24tools/intel_vbt_decode: Dump DSI panel rotationVille Syrjälä
Dump out the rotation field from the MIPI config block. Cc: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com>
2018-10-24tests/kms_rotation_crc: Don't leave crc runningJuha-Pekka Heikkila
Leaving crc running nobody reading it may fail tests randomly with dmesg comment "*ERROR* Overflow of CRC buffer, userspace reads too slow." v2 (Maarten Lankhorst): Don't change clean up code but just add flag for starting crc and stop it at end of test. bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105748 Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2018-10-23Add GitLab's CI/CD configurationArkadiusz Hiler
The CI/CD pipeline is configured the following way: 1. Build docker images and add them to repo's registry (manual step for now, automation needs GitLab to be updated). 2. Build igt with meson on Debian and Fedora. 3. Run `ninja tests` on Fedora. 4. Build and publish docs as an artifact, for GitLab Pages. v2: Typos and fully-featured meson build. v3: Use variables instead of hard-coding registry and project. Cc: Petri Latvala <petri.latvala@intel.com> Cc: Daniel Vetter <daniel@ffwll.ch> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-10-23runner: Add "root" also to the "totals" objectPetri Latvala
Turns out the same information is looked up in different places in different code paths. Piglit's summary module looks up total counts in ['totals']['root'], CI looks up ['totals']['']. The latter is inherited from piglit, so this has probably changed at some point. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108486 Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Andi Shyti <andi.shyti@intel.com> Tested-by: Andi Shyti <andi.shyti@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-10-23tests: Introduce i915 directoryArkadiusz Hiler
We can already move all the tests with distinct prefixes: gem_, gen3_ and i915_. pm_ and drv_ tests will follow in batches, so we can do the adjustments in the reporting/filtering layer of the CI system. v2: Fix test-list.txt generation with meson v3: Fix docs build (Petri) Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Tested-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
2018-10-19runner: Add --overall-timeoutPetri Latvala
With --overall-timeout $foo, the runner will stop executing new tests when $foo seconds have already been used. A resumed run will start over with no time used, using the same timeout. This allows for executing a long list of tests piecemeal, in about $foo length executions. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106127 Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-10-18tools/dpcd_reg: Introduce dump as the default operationRodrigo Vivi
For now this only imports the registers that were used on i915's debugfs dpcd dump. Later this could be extended. With this, we should be able to kill i915_dpcd from the kernel debugfs and rely solely on dpcd_reg for dpcd dumps. Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Tarun Vyas <tarun.vyas@intel.com>
2018-10-17tests/kms_selftest: Integrate kernel selftest test-drm_modesetDeepak Rawat
Call kernel selftest module test-drm_modeset for testing KMS. v2: - Add test alphabetically. - Add test to meson build. v3: Rename to kms_selftest. Signed-off-by: Deepak Rawat <drawat@vmware.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-17lib: Don't call igt_require_fb_modifiers() when no modifierDeepak Rawat
vmwgfx and amdgpu doesn't support fb modifiers, yet kms_addfb() requires modifier support. Since many tests don't need this to run, the requirement can be made less broad. Therefore, tighten the requirement to cases where modifiers are actually needed. v2: * In create_fb() calls to kms_addfb(), remove the modifier flag iff the driver doesn't support modifiers and the modifer is 0 * Don't modify the flag in kms_addfb(). Signed-off-by: Deepak Rawat <drawat@vmware.com> Signed-off-by: Leo Li <sunpeng.li@amd.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-17lib/igt_fb: Check for cairo surface successDeepak Rawat
For vmwgfx cairo surface creation fails due to stride mismatch, add a igt_require_f() for surface. v2: Check for surface creation failure. Signed-off-by: Deepak Rawat <drawat@vmware.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-17lib/igt_fb: Call dumb_destroy ioctl in case of dumb buffersDeepak Rawat
vmwgfx does not support GEM interface so calling gem_close on vmwgfx results in error. v2: Use drmIoctl with error when ioctl() failed. v3: Seperate ioctl wrapper. Signed-off-by: Deepak Rawat <drawat@vmware.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-16lib/chamelium: Check drmModeGetResources()Chris Wilson
If KMS is not available, drmModeGetResoures() reports NULL and so we cannot use this device to find the chamelium connector. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
2018-10-16tests: Skip testdisplay when KMS is disabledJosé Roberto de Souza
This test should be skiped when KMS is disabled. Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: José Roberto de Souza <jose.souza@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-10-16lib: Skip tests that require fb modifiers when KMS is disabledJosé Roberto de Souza
When KMS is disabled, drmGetCap() for DRM_CAP_ADDFB2_MODIFIERS() will return -1 and set errno as EOPNOTSUPP, the current assert was only checking for a sucess result or invalid argument causing prime_vgem@basic-fence-flip test to fail when KMS is disabled. Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: José Roberto de Souza <jose.souza@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-10-16tests/kms_ccs: Avoid using plane sizes which exceed hw capabilityJuha-Pekka Heikkila
Max sprite plane width at 2000 to avoid going over hw watermark limits. v2: (Ville Syrjälä) comment on source added plane width limitation. bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105458 Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2018-10-16tests/kms_frontbuffer_tracking: Do not skip on 4k displays.Maarten Lankhorst
On 4k displays the horizontal resolution is 3840, which when you add 500 gives a stride of 17408 (3840 + 500 + 12 (tile alignment)). Decrease the extra offset to 248, so we end up at exactly 16384, which is the maximum supported stride for FBC. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
2018-10-15lib: sync i915_pciids.h with kernelRodrigo Vivi
One more AML ID added and WHL IDs reorganized. Align with commit c0c46ca461f1 ("drm/i915/aml: Add new Amber Lake PCI ID"), including commit c1c8f6fa731b ("drm/i915: Redefine some Whiskey Lake SKUs") v2: Also sync intel_device_info.c (CI and Jose) Cc: José Roberto de Souza <jose.souza@intel.com> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
2018-10-15libs: Add rendercopy support for GEN11Kalamarz, Lukasz
This patch introduces a render copy shader for GEN11. The plumbing is same as with GEN9, so we can reuse it, extracting the common parts, and wrapping it in GEN-specific helpers. v2: Added gen11 shader source path next to its binary form Signed-off-by: Lukasz Kalamarz <lukasz.kalamarz@intel.com> Cc: Michał Winiarski <michal.winiarski@intel.com> Cc: Antonio Argenziano <antonio.argenziano@intel.com> Cc: Lucas De Marchi <lucas.demarchi@intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: Katarzyna Dec <katarzyna.dec@intel.com> Reviewed-by: Katarzyna Dec <katarzyna.dec@intel.com>
2018-10-15runner: Fix results when there's warns and dmesg-warnsPetri Latvala
After setting the result object text, the string retrieved from the old object is invalid. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-10-15tests/perf: adapt report timing delta acceptance for ICLLionel Landwerlin
The behavior of the OA unit is a tiny bit different on ICL. It appears to be a bit sloppier on the timings of its OA reports (missing the deadline by one period quite often). Let's add an acceptance delta. v2: Use larger acceptance delta only on ICL (José) Tweak indentation (José) Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
2018-10-11Add TODO.rstDaniel Vetter
First attempt at collecting a bunch of cleanup tasks. I think using the kernel as an example, it's much better if we keep these here than in our own internal $bug_db instance. v2: Fix typo (Rodrigo). Cc: Petri Latvala <petri.latvala@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2018-10-10tests: Add chamelium formats subtests to vc4 test listsMaxime Ripard
Now that we have support for the format sub-tests, enable them in the vc4 chamelium test lists. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Acked-by: Eric Anholt <eric@anholt.net>
2018-10-08chamelium: Add format subtestsMaxime Ripard
Now that we have everything in place, we can add the support for the subtests testing the output of planes setup with formats other than XR24. Since YUV will be a bit trickier to handle, start with various common RGB formats. Reviewed-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-10-08chamelium: Add format supportMaxime Ripard
In order to introduce CRC subtests for DRM formats, we need to take an intermediate step. The current code will generate a pattern in XR24, and will try to compare the CRC returned by the Chamelium for each frames. This CRC is computed on an XR24 format as well, so it works. However, as soon as we will start implementing other formats, if we just change the format of the pattern, the raw content of the buffer, and therefore the CRC's won't match anymore. In order to address that, we will need an intermediate step, and we will now still create the XR24 pattern, and compute its CRC, then convert it to the format we want to test, and finally retrieve the CRC from the Chamelium to compare it with the one from the XR24 pattern. The current code is converted to the new prototype that will take the fourcc of the format to test, even though we're still using XR24 everywhere for now. Reviewed-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-10-08chamelium: Change our pattern for a custom one if neededMaxime Ripard
The current pattern being used is the one generated through the igt_create_color_pattern_fb. However, in order to deal with multiple formats and the upsampling / downsampling issues that might arise from converting back and forth between formats, we will need to have a pattern with quite precise color values, and without any shades or gradient of colors. Let's create a function that will generate that pattern in the chamelium code if we need to convert the framebuffer to a smaller depth, and use the current pattern otherwise. The easiest way to do that will be to only use values that would have the same part on the common most significant bits (5, to deal with most formats) and have the same bit repeated on the least significant bits that are going to be dropped and / or padded when converting between formats. Pixman will fill the lowest bits with 1, and our hardware (this has been tested on a Raspberry Pi's VC4) is able to support that, so the easiest is to just use all 1's for our components in order to still be able to compute the CRCs. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-10-08chamelium: Split CRC test function in twoMaxime Ripard
We have two use cases in our current sub-test suites: the tests that test all the modes exposed by the driver, and the ones just picking up one. Instead of having to deal with this two cases in the same function as it is currently done, move the part that test a single mode into a separate function, and just call it for every mode that we want to test if needs be. This will result in a simpler function that will be easier to extend to support formats. Reviewed-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-10-08fb: Add more formatsMaxime Ripard
We're going to need some DRM formats, and we're going to need the igt_fb code to handle them. Since it relies on the format_desc structure to map the DRM fourcc to the pixman and cairo formats, we need to add these new formats to that structure. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-10-08fb: Add support for conversions through pixmanMaxime Ripard
Pixman allows for much more conversions than cairo, and we don't want to open code conversions routine for the common formats. Let's plug pixman in our conversion function so that we can benefit from it when possible. Reviewed-by: Eric Anholt <eric@anholt.net> Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-10-08tests: kms_plane: Disable XBGR8888Maxime Ripard
For some reason, the XBGR8888 CRC check will fail in the kms_plane tests. Since that format will be enabled and checked by that test in the next commit, make sure we don't introduce a regression for no particular reason. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Martin Peres <martin.peres@linux.intel.com>
2018-10-08igt: Make pixman mandatoryMaxime Ripard
So far, pixman was used exclusively when the Chamelium support was enabled. However, since we're going to use it as one of the backend to do the igt_fb conversions between formats, we'll need it all the time. Make that explicit. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-10-08fb: Add format conversion routineMaxime Ripard
The chamelium format subtests will need to convert the reference pattern to the format to be tested on the DRM device. However, Cairo is very limited when it comes to format, and while pixman has much more support for formats, it's still falling short compared to what DRM exposes, especially on the YUV side. Plus, since we want to run CRC checks on the frame, we can't afford having conversions back and forth between RGB24, as the current API is doing. In order to abstract this away, let's create a function that will convert a igt_fb structure to another DRM format and return the converted igt_fb. For now, we will use only cairo to do the conversion, but we will use other libraries or roll our own routines to convert to more exotic formats and abstract it away from the users. Reviewed-by: Eric Anholt <eric@anholt.net> Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
2018-10-08fb: Create common function to convert frame formatsMaxime Ripard
The current code to convert between two buffer formats is quite tied to the cairo surface infrastructure. Since we'll want to reuse it, make that function more generic by introducing a common structure that passes all the arguments and a common function that will call the right functions we needed. Reviewed-by: Eric Anholt <eric@anholt.net> Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-11-01 04:24:06 +0000