From nobody Tue May 13 01:09:32 2025
Received: from mail-ej1-f44.google.com (mail-ej1-f44.google.com
 [209.85.218.44])
	(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 C7F841DF254
	for <linux-kernel@vger.kernel.org>; Sat,  5 Apr 2025 09:27:44 +0000 (UTC)
Authentication-Results: smtp.subspace.kernel.org;
 arc=none smtp.client-ip=209.85.218.44
ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;
	t=1743845266; cv=none;
 b=WgMJ/apyFDesbha1bj9goUDaHPBNJgXl9y1CL/IXPRxIzZFAZ1x2qONAZTld3mayQeDiHWTjRZatXzx4Nuq41Fgjr54ypJtLEXwAyOThMw0Sc0BqFZh5MuBoOIhHsxNrBVmEd5Sgj9US9HgHS9OJ9pfO2IlkU9Qi8k3lWhgz76Q=
ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org;
	s=arc-20240116; t=1743845266; c=relaxed/simple;
	bh=PeMUN5MaPR/iihGuaAYk1+RqNZDBvhqS1HijQAO+fo4=;
	h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:
	 MIME-Version:Content-Type;
 b=fVLHnE/IAKBrfmKZ+lMpTEajeecs4OYSC+FTVf1sYFxQnuLULC6qQBmgOJTPb+ZjUXW/ulGWB24ae5g95QFplaWZHS+fTzjV5AqwpNknFvR8fxVaRgou+mC4QKsNMiQsODBlq3ydN6G5uwukHLfaCybeQyq9bifTj3Ensv32hUw=
ARC-Authentication-Results: i=1; smtp.subspace.kernel.org;
 dmarc=none (p=none dis=none) header.from=baylibre.com;
 spf=pass smtp.mailfrom=baylibre.com;
 dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com
 header.i=@baylibre-com.20230601.gappssmtp.com header.b=ycS583EC;
 arc=none smtp.client-ip=209.85.218.44
Authentication-Results: smtp.subspace.kernel.org;
 dmarc=none (p=none dis=none) header.from=baylibre.com
Authentication-Results: smtp.subspace.kernel.org;
 spf=pass smtp.mailfrom=baylibre.com
Authentication-Results: smtp.subspace.kernel.org;
	dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com
 header.i=@baylibre-com.20230601.gappssmtp.com header.b="ycS583EC"
Received: by mail-ej1-f44.google.com with SMTP id
 a640c23a62f3a-ac2bfcd2a70so369193766b.0
        for <linux-kernel@vger.kernel.org>;
 Sat, 05 Apr 2025 02:27:44 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1743845263;
 x=1744450063; darn=vger.kernel.org;
        h=content-transfer-encoding:mime-version:references:in-reply-to
         :message-id:date:subject:cc:to:from:from:to:cc:subject:date
         :message-id:reply-to;
        bh=zZnZ1QBH4c/cR2r7Z2I0IWQDQQpPtp/xF22TjNG8zFs=;
        b=ycS583ECOuUzKWKMfrY1W1xmumbs15pUWtPkZ76FQ86PZugNIy1oVVg3Md3h9QtYfM
         zkqwX0+aAg7WdNIvCyn+hAyXuMuSBDEjzl01FmeohcXh6F92BOpB3e5T71cByn6G/Cbi
         955P1+2ehCyiTS+xqBnpx34pMta46pJ0LPSCfTCt0xr2iamS/IiX2y0ziBljWyNbIP2q
         i5D9ON69ZgFJEWPZNTAUyS2wQYxRBDU5daUXmoflbeMGy+tHpa0CBPgz9kpZVGVnMOCn
         02kvx8G2QibNh52qs/Q92VPWyBe7YZiy1RE7SMuozkws2OQ5rtynAQnW2nx1UNGEj6Q/
         LcxQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20230601; t=1743845263; x=1744450063;
        h=content-transfer-encoding:mime-version:references:in-reply-to
         :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc
         :subject:date:message-id:reply-to;
        bh=zZnZ1QBH4c/cR2r7Z2I0IWQDQQpPtp/xF22TjNG8zFs=;
        b=VtwuZEZ2tAViWTdphhGfeRv972QQmOp/h59ClkX0UWialGU0ZwqmYvEVwnlJYsxr4H
         tT226lnxfgqOF8BBp4RHFNsKrLe29v1MgcOsuiIM9Bh3+LUBT1m0+AFumtbt5dT19WnX
         R4KwgMex0+ZZjNguBZmI6hK5iz/jAOMZ23YtNyImld4DSaJjxnpR6I77WO2vY7Zb6l3t
         FY0nn46+cwBIEuefhjRdTO0b84/Da6efP6sSv3JR07t0iiWO5Eike3oHkPSfiUfJ7OGK
         9dvXyAXBH6qRD/oZGzVypBh9UoJ1D+xfQR+zZiRZxZ+K6EjSFLk7USd+cQAh0BnCywBw
         xGZA==
X-Forwarded-Encrypted: i=1;
 AJvYcCVJLyGvDi1moKrXE1TWDrJ87MO3YST3B4FDKrXonJgzkzYjJ8GpMemHGlYWtORkm+2VDgSYXeebIiAfXsQ=@vger.kernel.org
X-Gm-Message-State: AOJu0Yy5JuH/5l31+8V3QkR/qKVEHdPH5BYF3bHZrQX2zTm4OJ7Zxf/m
	x19kJ1u8JBaIusmFfZdkKyezkSKepjU5blPny1INVGWBJOQo8zUWlLhHlMXIXFYXnL0Fgr+cWtm
	J
X-Gm-Gg: ASbGnct2dMwTxOAe1o/4HZuML/r3+w/267PuvI2zAO/C3huFlbe2IHVNJIFCbzD/nFD
	n5Zeujrzpz9TF/UaPvMXOE+1iNCOGp0eYoM7It4G0watMEZ/Tb5ZcEcKq6gsX6vuQr5C1riEs98
	/7qXsAHcUQBcgvUJ6MvmbhKGS01m32Ipd3ptcqhLICqDiUdS90qXoNx60Kkq4NpdkryEaGlAI0/
	PVlolWb7Z2Ud8rMxM4TeQz+xvCG2jSUPur9YF/b//RsJmzd8FnFZi8i3wWmIf2p0vbFg9jsDYJT
	FWVWbe/8OXaoRzpiDkpISB7yaIdlz2WFS4l4DT7t/lqmWjEUaA==
X-Google-Smtp-Source: 
 AGHT+IGWO7dBoEwfH+vhvetzVchvOebwHG0LSEkTGJ+mVXy0B0Ix45di+z7EJEIkw1x3jHja6bWBjg==
X-Received: by 2002:a17:907:7f07:b0:ac3:8a3a:e924 with SMTP id
 a640c23a62f3a-ac7d19903fdmr555132166b.39.1743845263101;
        Sat, 05 Apr 2025 02:27:43 -0700 (PDT)
Received: from localhost ([2a02:8071:b783:6940:36f3:9aff:fec2:7e46])
        by smtp.gmail.com with UTF8SMTPSA id
 a640c23a62f3a-ac7bfea0eadsm394579766b.67.2025.04.05.02.27.42
        (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
        Sat, 05 Apr 2025 02:27:42 -0700 (PDT)
From: =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= <u.kleine-koenig@baylibre.com>
To: linux-pwm@vger.kernel.org
Cc: Fabrice Gasnier <fabrice.gasnier@foss.st.com>,
	Maxime Coquelin <mcoquelin.stm32@gmail.com>,
	Alexandre Torgue <alexandre.torgue@foss.st.com>,
	linux-stm32@st-md-mailman.stormreply.com,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH 4/6] pwm: stm32: Emit debug output also for corner cases of
 the rounding callbacks
Date: Sat,  5 Apr 2025 11:27:15 +0200
Message-ID: 
 <fe154e79319da5ff4159cdc71201a9d3b395e491.1743844730.git.u.kleine-koenig@baylibre.com>
X-Mailer: git-send-email 2.47.2
In-Reply-To: <cover.1743844730.git.u.kleine-koenig@baylibre.com>
References: <cover.1743844730.git.u.kleine-koenig@baylibre.com>
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
X-Developer-Signature: v=1; a=openpgp-sha256; l=1852;
 i=u.kleine-koenig@baylibre.com; h=from:subject:message-id;
 bh=PeMUN5MaPR/iihGuaAYk1+RqNZDBvhqS1HijQAO+fo4=;
 b=owEBbQGS/pANAwAKAY+A+1h9Ev5OAcsmYgBn8Pd6gBS/U7sm8gCc++Qe+H9FPvkAFOyatKX8B
 qZYNTVb/8WJATMEAAEKAB0WIQQ/gaxpOnoeWYmt/tOPgPtYfRL+TgUCZ/D3egAKCRCPgPtYfRL+
 TiyKB/0dJZmWMtAaRs0WJ3PXT9d3VqFRF+ApoExKeBVC+2ES9YhZvF7McGhAzK48ik7Ly+LbIdd
 Il/cB2eBN8HrWNjZoHv/RVydoIwXGPyxm+7EVsFLjp6QVEbKfRjnonbncS3QTgmSKMIUL6e6JSC
 Exlph+FquJCDEjgnQZiyDNnKFMpId9mwZfG4STy0nFW4ohy/2CIVisdMr+VJ/Ku9COUq3a/FB+F
 QNpy10ntJAQfeJnxjycByIlnMBpPR+ouavHhm3lAae1baYZOSbounM633kPFBC43UuT9ehZ8ryi
 RmE7IGRwAQ/z49wXa8pqeOAbyygY5+bxUJ0PuQwUNrsaJoRK
X-Developer-Key: i=u.kleine-koenig@baylibre.com; a=openpgp;
 fpr=0D2511F322BFAB1C1580266BE2DCDD9132669BD6
Content-Transfer-Encoding: quoted-printable

When you're interested in the actual register settings the driver
chooses or interprets you want to see them also for calls that hit
corner cases.

Make sure that all calls to stm32_pwm_round_waveform_tohw() and
stm32_pwm_round_waveform_fromhw() emit the debug message about the
register settings.

Signed-off-by: Uwe Kleine-K=C3=B6nig <u.kleine-koenig@baylibre.com>
---
 drivers/pwm/pwm-stm32.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/drivers/pwm/pwm-stm32.c b/drivers/pwm/pwm-stm32.c
index c6625f51a199..dca5d09d80b9 100644
--- a/drivers/pwm/pwm-stm32.c
+++ b/drivers/pwm/pwm-stm32.c
@@ -180,11 +180,11 @@ static int stm32_pwm_round_waveform_tohw(struct pwm_c=
hip *chip,
=20
 	wfhw->ccr =3D min_t(u64, ccr, wfhw->arr + 1);
=20
+out:
 	dev_dbg(&chip->dev, "pwm#%u: %lld/%lld [+%lld] @%lu -> CCER: %08x, PSC: %=
08x, ARR: %08x, CCR: %08x\n",
 		pwm->hwpwm, wf->duty_length_ns, wf->period_length_ns, wf->duty_offset_ns,
 		rate, wfhw->ccer, wfhw->psc, wfhw->arr, wfhw->ccr);
=20
-out:
 	clk_disable(priv->clk);
=20
 	return ret;
@@ -236,17 +236,16 @@ static int stm32_pwm_round_waveform_fromhw(struct pwm=
_chip *chip,
 			wf->duty_length_ns =3D ccr_ns;
 			wf->duty_offset_ns =3D 0;
 		}
-
-		dev_dbg(&chip->dev, "pwm#%u: CCER: %08x, PSC: %08x, ARR: %08x, CCR: %08x=
 @%lu -> %lld/%lld [+%lld]\n",
-			pwm->hwpwm, wfhw->ccer, wfhw->psc, wfhw->arr, wfhw->ccr, rate,
-			wf->duty_length_ns, wf->period_length_ns, wf->duty_offset_ns);
-
 	} else {
 		*wf =3D (struct pwm_waveform){
 			.period_length_ns =3D 0,
 		};
 	}
=20
+	dev_dbg(&chip->dev, "pwm#%u: CCER: %08x, PSC: %08x, ARR: %08x, CCR: %08x =
@%lu -> %lld/%lld [+%lld]\n",
+		pwm->hwpwm, wfhw->ccer, wfhw->psc, wfhw->arr, wfhw->ccr, rate,
+		wf->duty_length_ns, wf->period_length_ns, wf->duty_offset_ns);
+
 	return 0;
 }
=20
--=20
2.47.2