From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f43.google.com (mail-wr1-f43.google.com [209.85.221.43]) (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 8983626B2C8 for ; Mon, 25 Aug 2025 12:02:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756123366; cv=none; b=hGwcYag7OK/Rno92+EnaFMm1WnJVLGdsi2BJyrzL0iP1BLNtT8kaWaaZ8sEIg+8T8VbZHGpLW1EfOPSr6TMw4gXSZqaYklyU2zWDQqbbdAYz8XzOpSpgEEEvX8WPL1kXSemt4OpWQSXelUtL7o6AG6SH3dNPUMRudTGDfT5i5MQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756123366; c=relaxed/simple; bh=kN3O4oAZ9kJwN6mPygYUfzC342hF1oirE9ZK//UXyWA=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=k2XfThzFsTmIXyzmFWJS9aC94oeibavUUoBx8DhL7gCHEk6CBrTHdrHCUATjDTJUJivQTrspIfvH7s+SYF8kvjZ8wkiBStaXglSYIX3QMJGW524WmnhUW5df/XOF5Z7kvpRt0nCEYZUyoKvJQv5zi7m9HZim8BvbHhER9f92LkY= 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=SE2BL/wZ; arc=none smtp.client-ip=209.85.221.43 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="SE2BL/wZ" Received: by mail-wr1-f43.google.com with SMTP id ffacd0b85a97d-3c68ac7e238so1353392f8f.1 for ; Mon, 25 Aug 2025 05:02:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1756123360; x=1756728160; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=IJvbqf/yt1tsn43VAwo8VG/RGkLizif52M9SXD1q9EE=; b=SE2BL/wZ8ZQHFH9NPi6CMvQrmp5ooM5cHc9ESIWihbo1tnXjTbzDJa0ULyO9JYzPyT sG5e2BD+gRPA5rtyK/sm8X1IASJyvm512i1KW5V68GbA8GfxQht+riDTFvsRY76/Cakq AB7bQyO7G+nF0h9Jrql/38zZHVtFPZqQrv06trPa/GV2MH0dUATtslHglUtLNXJQwccU 8RAB/cY4URQouu+0VVoXy8YThpifeIznF9q/u1hVux3LqCgNi8l4ebKR0nszxWzH9ulJ MNXSNe2/mtrWZfAOauHqXFK6bTua0F+HWRl4/iF2bS7jjoxIRKAdmcMXOvhSgDGnQAC+ 7VRw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756123360; x=1756728160; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=IJvbqf/yt1tsn43VAwo8VG/RGkLizif52M9SXD1q9EE=; b=CrY3cHq71ebCNNvitv6Gsm4yPp5wLM9jpa/ddkRMZpYZVwdLR9OFn5GEJvkAkAkK70 qTcI3fBFhxz/+ZoukA2XKyFXts7oTihGryzqxuRvF20Prx3npOEwaWLkwNFLvxu328YD BqsOSKejRMqrrl6RsL+pQf8wHaBC5JypDx3y+TRG1nzl1lz1stlPMeSXrqkOuh+ZSvBo qSHy46C+cInEcHPG4xa9Jpg94w0Sx38bX2g2BnlnDwmeU3AV6eS6i6nl0Zu9G8URr+Cs Pn1eCblrNniEMOlhSwCL8m/vUHoLzgfzvbQfCWQKlN/PoPyoxNNgE8uCFSSUAehCA4du Xxqw== X-Forwarded-Encrypted: i=1; AJvYcCUI9jz6whL3tw/sQrEJJScmKnzPc1u+AG7COYoiIrVUm8kpugkfvaedgmyEjiNmjdUAa/wMA7UUOVOx@vger.kernel.org X-Gm-Message-State: AOJu0YxQ0Bl1weSo/amXnArhtx1OvVhfYR0zIdXayGqGzZFdu5WWyL+a 5rbfP/SqmY9coOSnK2xrkSVBjkLwTjTMJsJbStG3fA2lUZTF24YzaHFYjqaAZwmZR6A= X-Gm-Gg: ASbGncvh4SQazETf6NvcNG9zZHR/TAa3rN5fSFN6PUrd6RoHLTxyV6qKBKelDHSd2Eh q1su+Oz+QOnPPJMN1L+YxKEn6iuLpX/Z3V0TcvJDb4nmdHF783aQ/brncBM5cIP/xfYM/DTnsLr fHQHEw0XsR8sG8ecyJ8k4rMvBDbRQE1SuXULHS2vDfl1E0/5AkRQN4nDWWn4LTiFsWrmqh9wNvO 2ZJaMPM9a1m6mO+Qu8mK6YDJrmDumkv2x1K3wBspVhKub3E+AUtv1+t2p2An2xgzD/0mlf05N1u fJCzNt91lZycFSw3qte7u5E+SndfucHWJBrpyTjbbAde64ZcE5xXUWlPzqKKI6qilK+bHxmKv5x VuGs4cmjWmM8JyN1i3NQySscfii5klk4= X-Google-Smtp-Source: AGHT+IHNjJgvFd7vseEGAj3DNMBQJ1GiUVz6n+LYqvwtjb1i5BLxin1zHA6RIo01dWv3lsPQdAUdIQ== X-Received: by 2002:a05:6000:310b:b0:3a4:f70e:abda with SMTP id ffacd0b85a97d-3c5db8aae86mr8234480f8f.10.1756123360040; Mon, 25 Aug 2025 05:02:40 -0700 (PDT) Received: from [192.168.0.251] ([82.76.204.34]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3c711211bd7sm11550954f8f.38.2025.08.25.05.02.37 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 25 Aug 2025 05:02:39 -0700 (PDT) Message-ID: Date: Mon, 25 Aug 2025 13:02:37 +0100 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/3] firmware: exynos-acpm: add DVFS protocol To: Krzysztof Kozlowski , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Peter Griffin , =?UTF-8?Q?Andr=C3=A9_Draszik?= , Michael Turquette , Stephen Boyd , Alim Akhtar , Sylwester Nawrocki , Chanwoo Choi Cc: linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org, willmcvicker@google.com, kernel-team@android.com References: <20250819-acpm-clk-v1-0-6bbd97474671@linaro.org> <20250819-acpm-clk-v1-2-6bbd97474671@linaro.org> Content-Language: en-US From: Tudor Ambarus In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 8/24/25 6:11 PM, Krzysztof Kozlowski wrote: > On 19/08/2025 13:45, Tudor Ambarus wrote: >> Add ACPM DVFS protocol handler. It constructs DVFS messages that >> the APM firmware can understand. >> >> Signed-off-by: Tudor Ambarus >> --- >> drivers/firmware/samsung/Makefile | 4 +- >> drivers/firmware/samsung/exynos-acpm-dvfs.c | 85 ++++++++++++++++++++++ >> drivers/firmware/samsung/exynos-acpm-dvfs.h | 21 ++++++ >> drivers/firmware/samsung/exynos-acpm.c | 5 ++ >> .../linux/firmware/samsung/exynos-acpm-protocol.h | 10 +++ >> 5 files changed, 124 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/firmware/samsung/Makefile b/drivers/firmware/samsung/Makefile >> index 7b4c9f6f34f54fd731886d97a615fe1aa97ba9a0..80d4f89b33a9558b68c9083da675c70ec3d05f19 100644 >> --- a/drivers/firmware/samsung/Makefile >> +++ b/drivers/firmware/samsung/Makefile >> @@ -1,4 +1,6 @@ >> # SPDX-License-Identifier: GPL-2.0-only >> >> -acpm-protocol-objs := exynos-acpm.o exynos-acpm-pmic.o >> +acpm-protocol-objs := exynos-acpm.o >> +acpm-protocol-objs += exynos-acpm-pmic.o >> +acpm-protocol-objs += exynos-acpm-dvfs.o >> obj-$(CONFIG_EXYNOS_ACPM_PROTOCOL) += acpm-protocol.o >> diff --git a/drivers/firmware/samsung/exynos-acpm-dvfs.c b/drivers/firmware/samsung/exynos-acpm-dvfs.c >> new file mode 100644 >> index 0000000000000000000000000000000000000000..ee457c1a3de2ff2e4395d9fc3ff4c13294473b2d >> --- /dev/null >> +++ b/drivers/firmware/samsung/exynos-acpm-dvfs.c >> @@ -0,0 +1,85 @@ >> +// SPDX-License-Identifier: GPL-2.0-only >> +/* >> + * Copyright 2020 Samsung Electronics Co., Ltd. >> + * Copyright 2020 Google LLC. >> + * Copyright 2025 Linaro Ltd. >> + */ >> + >> +#include >> +#include >> +#include >> +#include >> +#include >> + >> +#include "exynos-acpm.h" >> +#include "exynos-acpm-dvfs.h" >> + >> +#define ACPM_DVFS_ID GENMASK(11, 0) >> +#define ACPM_DVFS_REQ_TYPE GENMASK(15, 0) >> + >> +enum exynos_acpm_dvfs_func { >> + ACPM_DVFS_FREQ_REQ, >> + ACPM_DVFS_FREQ_GET, >> +}; > > These are actual values for hardware/firmware? If so, please use rather > defines. yes, they are. Will do, thanks!