drm/i915/perf: improve invalid OA format debug message
A minor improvement to debugging output Signed-off-by: Robert Bragg <robert@sixbynine.org> Reviewed-by: Matthew Auld <matthew.auld@intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170511154345.962-7-lionel.g.landwerlin@intel.com Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
This commit is contained in:

committed by
Chris Wilson

parent
0dd860cf73
commit
52c57c263f
@@ -1902,11 +1902,13 @@ static int read_properties_unlocked(struct drm_i915_private *dev_priv,
|
|||||||
break;
|
break;
|
||||||
case DRM_I915_PERF_PROP_OA_FORMAT:
|
case DRM_I915_PERF_PROP_OA_FORMAT:
|
||||||
if (value == 0 || value >= I915_OA_FORMAT_MAX) {
|
if (value == 0 || value >= I915_OA_FORMAT_MAX) {
|
||||||
DRM_DEBUG("Invalid OA report format\n");
|
DRM_DEBUG("Out-of-range OA report format %llu\n",
|
||||||
|
value);
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
if (!dev_priv->perf.oa.oa_formats[value].size) {
|
if (!dev_priv->perf.oa.oa_formats[value].size) {
|
||||||
DRM_DEBUG("Invalid OA report format\n");
|
DRM_DEBUG("Unsupported OA report format %llu\n",
|
||||||
|
value);
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
props->oa_format = value;
|
props->oa_format = value;
|
||||||
|
Reference in New Issue
Block a user