diff options
| author | Emma Anholt <emma@anholt.net> | 2022-12-05 11:18:55 -0800 |
|---|---|---|
| committer | Emma Anholt <emma@anholt.net> | 2022-12-05 13:03:12 -0800 |
| commit | f902203daa984c820f100847dd3dce161b13d4a7 (patch) | |
| tree | 9347ee8a352d2f1e98e616efa48b177a3af6a6bd /src/gallium/winsys/i915/drm | |
| parent | a42feab5ae8e16fdf0d9c1d33247c6c372e3646f (diff) | |
i915: Fix probing regression.
The device id was incorrectly getting passed as a pointer-to-pointer, and
also assert's expression doesn't get called on release builds.
Fixes: dfd20f002f38 ("intel: Add and use intel_gem_get_param()")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20166>
Diffstat (limited to 'src/gallium/winsys/i915/drm')
| -rw-r--r-- | src/gallium/winsys/i915/drm/i915_drm_winsys.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/gallium/winsys/i915/drm/i915_drm_winsys.c b/src/gallium/winsys/i915/drm/i915_drm_winsys.c index 74f1cfd0b33..38110d347b5 100644 --- a/src/gallium/winsys/i915/drm/i915_drm_winsys.c +++ b/src/gallium/winsys/i915/drm/i915_drm_winsys.c @@ -19,7 +19,8 @@ static void i915_drm_get_device_id(int fd, unsigned int *device_id) { - assert(intel_gem_get_param(fd, I915_PARAM_CHIPSET_ID, (int *)&device_id)); + ASSERTED bool ret = intel_gem_get_param(fd, I915_PARAM_CHIPSET_ID, (int *)device_id); + assert(ret); } static int |