From nobody Tue Dec 16 07:11:28 2025 Received: from mail-wr1-f41.google.com (mail-wr1-f41.google.com [209.85.221.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 361B81E495 for ; Tue, 14 Jan 2025 19:17:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.41 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736882251; cv=none; b=HcuzTW+j/9XqFnfbviMi8mta9GB4xA7awPRLsX0uNm8RAXBXbSWYrf1+EOrtvwn9EQS1d2hWRU3gm+0Yy8cgfUo9ovA9vHt7ArNn/51X71DJymKYX5uf3up/rpgnJfe8YnTQ0Mo6QEb+Z2OmbTYP1UwKDdbvssBW/Q+dGs8g/xY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736882251; c=relaxed/simple; bh=WXcKX3hyiCbFpF9dstj8z/jaicZ9JRFr7RTEYEj2UDk=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=j/E8Hvuv1ZDLdmhcnaYqZcQwsWfWlCJ2cN/ScsZEerrF4sB2C6B3etJJBNlmc4nGNsemO29n0YtUkYvx8Lj8pjMLqVvwxRekJx3oFkWG/n8plMqRX+HgTjUnVyBCkVVSpIT+NoRQmC/6cOywf4PMrQExzs8ChxnUF9tyq0i2O6k= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=eY8tG1ML; arc=none smtp.client-ip=209.85.221.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="eY8tG1ML" Received: by mail-wr1-f41.google.com with SMTP id ffacd0b85a97d-3862c78536bso333622f8f.2 for ; Tue, 14 Jan 2025 11:17:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736882247; x=1737487047; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=V06lBlmBxfsjJKJMHVQIInwZH2ivoya6DqMfzagHCMc=; b=eY8tG1MLbXAjH/laJe8bMgOfvGQfM9KI6kr3auIACLO2evX3Kx4Ac8JehtSSscwpLp wOwJMTinzUykeYs/h4RfRfXjnI8mvpoD3HVWJJ37Y0b5I+uOeEU3J+NrT/ovI3nAPBQH +zhcBeB9GSUJqlv5HXKWUWEzqN4XS7tPT4uo1RbX/vo+8jDjUBqZRj3W36n0hamgrf/K p3FnR5wGqlHzI6b+JJ7tZ0wQWzGLwmxqobSNZQo1TubTOqLn8jVtNXheDL/4fRGB+KQu mS01L9fEfIv1TdgqPwoMjm58uLIs6/GGH0JZ76eOBHrBsKrtsVTydzdbX0eef/9Gkiuy Dndw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736882247; x=1737487047; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=V06lBlmBxfsjJKJMHVQIInwZH2ivoya6DqMfzagHCMc=; b=ual0U4jO8cgz8s6+3iKrmdNCNQ2K05ELpoTbQpuQ395c2CyHn0NJ6LHhxwrO2FDkix NAnjQvhBCSR1Pw/Ya7QmECfX5Qz8ppQpt+o8Xqspb7x5EiKGgXutaL4bNI86e8MG4sLX ANjizogDxSL/ozQGBu7P+KXWdr+vB/tuX8pBLVXQSahRjscNxrznEQxVBZjsqDYb2I73 mKlfZ08yDlSbl+/7BYF4jJI1Qf6bGjPxnRlN/FFnwpeNSslKneTLAmOlY4Jm6Q/wiA/+ gmQYs3hK98djcyz+5rLIrdnnhPp/A1ul0DqrQuKmRUGZKbubHPt/VRT+BMWfcegT8C53 D3zQ== X-Forwarded-Encrypted: i=1; AJvYcCX7/KmACkuDxYVmQl6+FiSQjQnS1kC/XuNO+0g1cS3DwkDbvWTumCSMGJasdeABKSxdL8SV1VRWu5bBq18=@vger.kernel.org X-Gm-Message-State: AOJu0YwzejV/SXL50qL5f9VqGShCf80zNnHztjX2pl2huKVsr8QN9J/u TekFMzSY4xps8on1nWX6fQ3rHzvR16635kRCnI5UzNmuwqMKOQAP/IwstkCamP8= X-Gm-Gg: ASbGnculWEkwzp1bKEHK8qdnB+u+cbMR4iUO3y2beUzbDxCt5gg9+XGdOkXFA3UflrU Phrl1tDTTeUlc/CcTupdE4P/ph4nzNu0ZG89GPpcBlFegVg7ajPwwaXfbBK0nVRg5VG/PR9NhEu alSv0VyonTalv+0T0khvleBVUUzUHkSCy43TFxI92TZfhf7nYIjOV18ra/TsqkKJWxiDTBVC/t/ 4iOvEM8oaD6/vfwJW8vZ8OpiOTcOfcK9aPEJffF8XOS0TbBmuBqWo8Vep/OGGL9yb3JAxw= X-Google-Smtp-Source: AGHT+IGUuvJd37PBNvk/Af4zVCTFBkmKr2VZr39NQdU1icUhyecsGR7h9tETuRGt6OlfcQkAdHOQXg== X-Received: by 2002:a5d:6c69:0:b0:38a:9ed4:9ffc with SMTP id ffacd0b85a97d-38a9ed4b06fmr5620244f8f.10.1736882247587; Tue, 14 Jan 2025 11:17:27 -0800 (PST) Received: from krzk-bin.. ([178.197.223.165]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-38a8e4b82a4sm15814423f8f.72.2025.01.14.11.17.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Jan 2025 11:17:26 -0800 (PST) From: Krzysztof Kozlowski To: Rob Clark , Abhinav Kumar , Dmitry Baryshkov , Sean Paul , Marijn Suijten , David Airlie , Simona Vetter , linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, linux-kernel@vger.kernel.org Cc: Krzysztof Kozlowski Subject: [PATCH] drm/msm: Use str_enable_disable-like helpers Date: Tue, 14 Jan 2025 20:17:24 +0100 Message-ID: <20250114191724.861601-1-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Replace ternary (condition ? "enable" : "disable") syntax with helpers from string_choices.h because: 1. Simple function call with one argument is easier to read. Ternary operator has three arguments and with wrapping might lead to quite long code. 2. Is slightly shorter thus also easier to read. 3. It brings uniformity in the text - same string. 4. Allows deduping by the linker, which results in a smaller binary file. Signed-off-by: Krzysztof Kozlowski Reviewed-by: Abhinav Kumar --- .../drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 5 ++-- drivers/gpu/drm/msm/disp/mdp5/mdp5_ctl.c | 3 ++- drivers/gpu/drm/msm/dp/dp_ctrl.c | 25 ++++++++++--------- drivers/gpu/drm/msm/dp/dp_display.c | 4 +-- drivers/gpu/drm/msm/dp/dp_drm.c | 5 ++-- 5 files changed, 23 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c b/drivers= /gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c index e9bbccc44dad..da9994a79ca2 100644 --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c @@ -5,6 +5,7 @@ =20 #define pr_fmt(fmt) "[drm:%s:%d] " fmt, __func__, __LINE__ #include +#include #include "dpu_encoder_phys.h" #include "dpu_hw_interrupts.h" #include "dpu_hw_pingpong.h" @@ -261,7 +262,7 @@ static int dpu_encoder_phys_cmd_control_vblank_irq( =20 DRM_DEBUG_KMS("id:%u pp:%d enable=3D%s/%d\n", DRMID(phys_enc->parent), phys_enc->hw_pp->idx - PINGPONG_0, - enable ? "true" : "false", refcount); + str_true_false(enable), refcount); =20 if (enable) { if (phys_enc->vblank_refcount =3D=3D 0) @@ -285,7 +286,7 @@ static int dpu_encoder_phys_cmd_control_vblank_irq( DRM_ERROR("vblank irq err id:%u pp:%d ret:%d, enable %s/%d\n", DRMID(phys_enc->parent), phys_enc->hw_pp->idx - PINGPONG_0, ret, - enable ? "true" : "false", refcount); + str_true_false(enable), refcount); } =20 return ret; diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_ctl.c b/drivers/gpu/drm/msm= /disp/mdp5/mdp5_ctl.c index 666de99a46a5..fc183fe37f56 100644 --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_ctl.c +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_ctl.c @@ -3,6 +3,7 @@ * Copyright (c) 2014-2015 The Linux Foundation. All rights reserved. */ =20 +#include #include "mdp5_kms.h" #include "mdp5_ctl.h" =20 @@ -233,7 +234,7 @@ int mdp5_ctl_set_encoder_state(struct mdp5_ctl *ctl, return -EINVAL; =20 ctl->encoder_enabled =3D enabled; - DBG("intf_%d: %s", intf->num, enabled ? "on" : "off"); + DBG("intf_%d: %s", intf->num, str_on_off(enabled)); =20 if (start_signal_needed(ctl, pipeline)) { send_start_signal(ctl); diff --git a/drivers/gpu/drm/msm/dp/dp_ctrl.c b/drivers/gpu/drm/msm/dp/dp_c= trl.c index 9c463ae2f8fa..d8633a596f8d 100644 --- a/drivers/gpu/drm/msm/dp/dp_ctrl.c +++ b/drivers/gpu/drm/msm/dp/dp_ctrl.c @@ -11,6 +11,7 @@ #include #include #include +#include =20 #include #include @@ -1366,9 +1367,9 @@ int msm_dp_ctrl_core_clk_enable(struct msm_dp_ctrl *m= sm_dp_ctrl) =20 drm_dbg_dp(ctrl->drm_dev, "enable core clocks \n"); drm_dbg_dp(ctrl->drm_dev, "stream_clks:%s link_clks:%s core_clks:%s\n", - ctrl->stream_clks_on ? "on" : "off", - ctrl->link_clks_on ? "on" : "off", - ctrl->core_clks_on ? "on" : "off"); + str_on_off(ctrl->stream_clks_on), + str_on_off(ctrl->link_clks_on), + str_on_off(ctrl->core_clks_on)); =20 return 0; } @@ -1385,9 +1386,9 @@ void msm_dp_ctrl_core_clk_disable(struct msm_dp_ctrl = *msm_dp_ctrl) =20 drm_dbg_dp(ctrl->drm_dev, "disable core clocks \n"); drm_dbg_dp(ctrl->drm_dev, "stream_clks:%s link_clks:%s core_clks:%s\n", - ctrl->stream_clks_on ? "on" : "off", - ctrl->link_clks_on ? "on" : "off", - ctrl->core_clks_on ? "on" : "off"); + str_on_off(ctrl->stream_clks_on), + str_on_off(ctrl->link_clks_on), + str_on_off(ctrl->core_clks_on)); } =20 static int msm_dp_ctrl_link_clk_enable(struct msm_dp_ctrl *msm_dp_ctrl) @@ -1416,9 +1417,9 @@ static int msm_dp_ctrl_link_clk_enable(struct msm_dp_= ctrl *msm_dp_ctrl) =20 drm_dbg_dp(ctrl->drm_dev, "enable link clocks\n"); drm_dbg_dp(ctrl->drm_dev, "stream_clks:%s link_clks:%s core_clks:%s\n", - ctrl->stream_clks_on ? "on" : "off", - ctrl->link_clks_on ? "on" : "off", - ctrl->core_clks_on ? "on" : "off"); + str_on_off(ctrl->stream_clks_on), + str_on_off(ctrl->link_clks_on), + str_on_off(ctrl->core_clks_on)); =20 return 0; } @@ -1435,9 +1436,9 @@ static void msm_dp_ctrl_link_clk_disable(struct msm_d= p_ctrl *msm_dp_ctrl) =20 drm_dbg_dp(ctrl->drm_dev, "disabled link clocks\n"); drm_dbg_dp(ctrl->drm_dev, "stream_clks:%s link_clks:%s core_clks:%s\n", - ctrl->stream_clks_on ? "on" : "off", - ctrl->link_clks_on ? "on" : "off", - ctrl->core_clks_on ? "on" : "off"); + str_on_off(ctrl->stream_clks_on), + str_on_off(ctrl->link_clks_on), + str_on_off(ctrl->core_clks_on)); } =20 static int msm_dp_ctrl_enable_mainlink_clocks(struct msm_dp_ctrl_private *= ctrl) diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/d= p_display.c index 24dd37f1682b..b200efd97947 100644 --- a/drivers/gpu/drm/msm/dp/dp_display.c +++ b/drivers/gpu/drm/msm/dp/dp_display.c @@ -11,6 +11,7 @@ #include #include #include +#include #include #include =20 @@ -343,8 +344,7 @@ static int msm_dp_display_send_hpd_notification(struct = msm_dp_display_private *d { if ((hpd && dp->msm_dp_display.link_ready) || (!hpd && !dp->msm_dp_display.link_ready)) { - drm_dbg_dp(dp->drm_dev, "HPD already %s\n", - (hpd ? "on" : "off")); + drm_dbg_dp(dp->drm_dev, "HPD already %s\n", str_on_off(hpd)); return 0; } =20 diff --git a/drivers/gpu/drm/msm/dp/dp_drm.c b/drivers/gpu/drm/msm/dp/dp_dr= m.c index d3e241ea6941..8364f3834531 100644 --- a/drivers/gpu/drm/msm/dp/dp_drm.c +++ b/drivers/gpu/drm/msm/dp/dp_drm.c @@ -3,6 +3,7 @@ * Copyright (c) 2017-2020, The Linux Foundation. All rights reserved. */ =20 +#include #include #include #include @@ -25,7 +26,7 @@ static enum drm_connector_status msm_dp_bridge_detect(str= uct drm_bridge *bridge) dp =3D to_dp_bridge(bridge)->msm_dp_display; =20 drm_dbg_dp(dp->drm_dev, "link_ready =3D %s\n", - (dp->link_ready) ? "true" : "false"); + str_true_false(dp->link_ready)); =20 return (dp->link_ready) ? connector_status_connected : connector_status_disconnected; @@ -41,7 +42,7 @@ static int msm_dp_bridge_atomic_check(struct drm_bridge *= bridge, dp =3D to_dp_bridge(bridge)->msm_dp_display; =20 drm_dbg_dp(dp->drm_dev, "link_ready =3D %s\n", - (dp->link_ready) ? "true" : "false"); + str_true_false(dp->link_ready)); =20 /* * There is no protection in the DRM framework to check if the display --=20 2.43.0