From 2188994c81d687b6692516126b311a8e4195cb70 Mon Sep 17 00:00:00 2001 From: "Juan A. Suarez Romero" Date: Tue, 8 Jul 2025 12:40:23 +0200 Subject: [PATCH] broadcom: remove obvious comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It is obvious that we check the return value to see if kernel supports the feature, so no need to add a comment. Signed-off-by: Juan A. Suarez Romero Reviewed-by: Iago Toral Quiroga Reviewed-by: Maíra Canal Part-of: --- src/broadcom/common/v3d_device_info.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/broadcom/common/v3d_device_info.c b/src/broadcom/common/v3d_device_info.c index 867f6bae0f0..9d781f0c0d3 100644 --- a/src/broadcom/common/v3d_device_info.c +++ b/src/broadcom/common/v3d_device_info.c @@ -100,14 +100,10 @@ v3d_get_device_info(int fd, struct v3d_device_info* devinfo, v3d_ioctl_fun drm_i devinfo->compat_rev = (hub_ident3.value >> 16) & 0xff; ret = drm_ioctl(fd, DRM_IOCTL_V3D_GET_PARAM, &max_perfcnt); - if (ret != 0) { - /* Kernel doesn't have support to return the maximum number of - * performance counters. - */ + if (ret != 0) devinfo->max_perfcnt = 0; - } else { + else devinfo->max_perfcnt = max_perfcnt.value; - } return true; }