From nobody Tue Dec 16 05:59:22 2025 Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) (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 57B7D1FAC5C for ; Tue, 14 Jan 2025 19:06:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736881568; cv=none; b=cRaT/Q+ljx6Y/CuXXrsKTB1Qk0uouwIZMvKR1+odooxjbigFBwSzBNBpxH4XETNYAuI1d1mb3pwqtkmTJB7r1RaCuFv5XXg59rlu/4QJ7k42hWcsmkfzD3q2jcvAsBKjHZe7Qy7UuXieXB+hUzlvtOU2TmzMMZw2V0JDbt+JKFA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736881568; c=relaxed/simple; bh=LXLQiJqWt+6VSxSgWHu0HnY8Id/SQVdnYvhKrOX0X7o=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=LsKvMuUCO2i+FN0T7qcHCH/myC7diJCgpGFNMVAICMADNfR0GzOs8RAuVQ0AH/cORgFdViMBm29QxTh9b63aZe2+qQ2vMKBAt3XOZdbpmORUHUtyTd7pR0ociQD5YYWWlevezgNFBEbpRAWJk0B7uztQSaIZLUg7Nw0oTVSNQzY= 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=Bxq3MzXx; arc=none smtp.client-ip=209.85.221.46 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="Bxq3MzXx" Received: by mail-wr1-f46.google.com with SMTP id ffacd0b85a97d-385d851e7c3so272879f8f.3 for ; Tue, 14 Jan 2025 11:06:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736881565; x=1737486365; 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=lN5rUEMysp3+AroTy4SlvvD7Pduqgen38rd5O/+MvPY=; b=Bxq3MzXxtNeyVBTpsf3utpz6NkwjrQz6mV1qh68FgOIgezYUFS8qIAF+XT5jsAvT7c h66i7mApvhJtNIHdJHBrRIlRu3Pyge9jgIl6xACMxTmN3+plyc39SkY91AuChSZ+n8Mc quGEVWWJ1cmJ0HbgYMgmWFusfVCVK7IeM3/fMX6IXGIzGSRuyG1fYFqpd1WBKDwbZQUy 4zBsGgvVJPi5+5IcuzYgTajTm/Ipm1Vnh2PKvd2rotEHPMdyCRsLO+THADdQ7g1URFQi FKpGT/+m3YfLDN8Gb+SnuvH0n5dEOnfNmyHDyAeQazPnTf/Rk5RPhaTnWYbO4rv47Swe TgWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736881565; x=1737486365; 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=lN5rUEMysp3+AroTy4SlvvD7Pduqgen38rd5O/+MvPY=; b=rtDhPZzJPy64Y1F1YwGjnCdZ2ljvluHGL4LkrgLGqfaH8/pUdOtwvR+dDqx8A3ud0p S2moo1928g2xHy3m/XSfcuQdMvnPn6CJTKnmSZP8J6IWS9+kUYd2J1bIMc/LkEYSCsA9 w+w05RKkV5LjWl9nJN0YRwcA2Ztxpjx8qzQLWFlEJUnBcTTWWMSbWCvoa8hHLIEqrZlZ ulk4RbecMYRJrYNTSNdeScAjezZI3gg8JhkFFjOKboQbZ4SvbJlbxG371yUqdVXjJVv8 tufQb8OviguTatWSYXP2xrg8/Krp3LKvX87nSmk+g2fXjHETS9JcrZ24SRzQkUHU9LTI fWGQ== X-Forwarded-Encrypted: i=1; AJvYcCV3JHEDqf5oTTiPqWqrEUrs9ecH2x2YFulWPGKybpcySA4Wu+ofn4CTocXwYm8ng3S0EhCfgXszi4CBXFY=@vger.kernel.org X-Gm-Message-State: AOJu0YxVyjuTsYX4bmfO8bXXaJ0M7Y10s/oC1Ac6Nx9PX+ccAZFTRiSu 87fr/gY2JLR5Wuk+KUsgO4caym3uoVQGXumDydwen+OKfWMs8EH/mIK3gA8O8C0= X-Gm-Gg: ASbGncu/4E+6ygUcg6Bi/9s3LZVjpjjpr8GuLNr4VLnAaU/9ttdc4baocWW6t5BCmxN yzLlSTqd57IpVd5gxkxO22NlQ6aqMjPQAAUIRGhXIwHmeF5ijS6z4Zw2I/arkF3lzhf1rU0wRbP PWTk7rzd17I8gDdQtETAe0yYe3wWn5wMGRltsQGAUkj+hrJrtDg18DTa5bdlGWNnE4dekD1X6Tz UcC/UEL9xh5vRhIdzHxLov68qZTnS4vah4NwkIPqFAl+adis6wYZSh45yvl7DusxoFTIRU= X-Google-Smtp-Source: AGHT+IE6pa0lthPJtWzhB15VzlOKuOcoh6DUo2csVHRrLlrQDBEdBv9yOOVd7c4Cp/ivr1V81kDl7A== X-Received: by 2002:a05:6000:705:b0:37d:4aa2:5cfe with SMTP id ffacd0b85a97d-38a872d9f96mr8638256f8f.6.1736881564689; Tue, 14 Jan 2025 11:06:04 -0800 (PST) Received: from krzk-bin.. ([178.197.223.165]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-38a8e4b81a4sm15958883f8f.68.2025.01.14.11.06.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Jan 2025 11:06:04 -0800 (PST) From: Krzysztof Kozlowski To: "Rafael J. Wysocki" , Viresh Kumar , Michael Ellerman , Nicholas Piggin , Christophe Leroy , Naveen N Rao , Madhavan Srinivasan , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Cc: Krzysztof Kozlowski Subject: [PATCH v2] cpufreq: Use str_enable_disable-like helpers Date: Tue, 14 Jan 2025 20:06:00 +0100 Message-ID: <20250114190600.846651-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 Acked-by: Viresh Kumar --- Changes in v2: 1. Fix enable->enabled --- drivers/cpufreq/cpufreq.c | 7 ++++--- drivers/cpufreq/powernv-cpufreq.c | 3 ++- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 418236fef172..1076e37a18ad 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -25,6 +25,7 @@ #include #include #include +#include #include #include #include @@ -602,12 +603,12 @@ static ssize_t store_boost(struct kobject *kobj, stru= ct kobj_attribute *attr, =20 if (cpufreq_boost_trigger_state(enable)) { pr_err("%s: Cannot %s BOOST!\n", - __func__, enable ? "enable" : "disable"); + __func__, str_enable_disable(enable)); return -EINVAL; } =20 pr_debug("%s: cpufreq BOOST %s\n", - __func__, enable ? "enabled" : "disabled"); + __func__, str_enabled_disabled(enable)); =20 return count; } @@ -2812,7 +2813,7 @@ int cpufreq_boost_trigger_state(int state) write_unlock_irqrestore(&cpufreq_driver_lock, flags); =20 pr_err("%s: Cannot %s BOOST\n", - __func__, state ? "enable" : "disable"); + __func__, str_enable_disable(state)); =20 return ret; } diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cp= ufreq.c index 8de759247771..ae79d909943b 100644 --- a/drivers/cpufreq/powernv-cpufreq.c +++ b/drivers/cpufreq/powernv-cpufreq.c @@ -18,6 +18,7 @@ #include #include #include +#include #include #include #include @@ -281,7 +282,7 @@ static int init_powernv_pstates(void) pr_info("cpufreq pstate min 0x%x nominal 0x%x max 0x%x\n", pstate_min, pstate_nominal, pstate_max); pr_info("Workload Optimized Frequency is %s in the platform\n", - (powernv_pstate_info.wof_enabled) ? "enabled" : "disabled"); + str_enabled_disabled(powernv_pstate_info.wof_enabled)); =20 pstate_ids =3D of_get_property(power_mgt, "ibm,pstate-ids", &len_ids); if (!pstate_ids) { --=20 2.43.0