From nobody Thu Oct 2 02:05:48 2025 Received: from smtpout-02.galae.net (smtpout-02.galae.net [185.246.84.56]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 54E0615ECCC for ; Wed, 24 Sep 2025 09:06:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.246.84.56 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758704816; cv=none; b=JOhrIcXxCFCXb7jjdzO7q4cfWSSvf3UDBK1mrYDRRfbVpOqxLXdgEVhZVkKMkxJr6FP3Dl5OJ+W70/VOT79pKQaP5DS9WmZFJTjwjTELWlo+7SrF0A3w6A+h+xecWqJXE/lrfO6OURCtw1Nes+QEVBxDXF3wHxdZgpVpTYHfgNI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758704816; c=relaxed/simple; bh=a547ooMEsxEG/7bHmCeVbD/kqXHtT87FdYIiTphxlwg=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=Sxw4p5r9FnJDJniEVd+n/DtUZdf7ZL9SKmTY9P/7hNfCM7TbyOtaNj9HFiLb82O/fOPbV6YV1Q7QirXh9FxawEG+kh/TCtxfopQaLTYFDMEqAP+bcJT0Oy7mgKnB8QH57+XuzXnHxqTagDkxojarqBrSHM8CU7QxuPndXsGMug0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=rDDeVtSV; arc=none smtp.client-ip=185.246.84.56 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="rDDeVtSV" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-02.galae.net (Postfix) with ESMTPS id 9B13D1A0EED for ; Wed, 24 Sep 2025 09:06:50 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 6BE2D60634; Wed, 24 Sep 2025 09:06:50 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 5EF14102F18C5; Wed, 24 Sep 2025 11:06:48 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1758704809; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding; bh=V2DD00oh1THLAH4vzd8aYJmPGiuNTiVRW2t0eRhScD4=; b=rDDeVtSVEhxdQ31Kq66eCQMvoBSze17xUsmSfRJb1aAD4WDKKegY2Iu6YtIa7Q2tpJsxqU jKBY0+GC3QZT0id0e092IdL5Qm12b0kk/iC8wFagO2cjPSqha2kapU7Izyw6HwsNSq2rhN Ti9ryf0l+XdcvUNFnfUa3xLGQo+v9lXqQwYo02I//k+eIgu1Q1NvfjB9U71kLKq5s6CA0N XvwE4VZms79ulZnt/yZ1OlMqtERyA731JGUHOaTozW0VWPSi1Ux0invdqL4VrSYSR+GhSv kGpEA8ozw0Zlvu1taOLa9djwtVOLWDwLOHdy305k5pSythyWnXV7hN8Encw/pA== From: Mathieu Dubois-Briand Date: Wed, 24 Sep 2025 11:06:40 +0200 Subject: [PATCH] pwm: max7360: Clean MAX7360 code Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250924-mdb-max7360-pwm-optimize-v1-1-5959eeed20d8@bootlin.com> X-B4-Tracking: v=1; b=H4sIAJ+002gC/x3MTQ5AMBBA4avIrE1Spf6uIhalg1mUphWEuLvG8 lu890AgzxSgTR7wdHDgbY3I0gTGRa8zIZtokEIq0cgCrRnQ6qvKS4HutLi5nS3fhMNoNClZT6p qIObO08TXv+769/0A+AyjEWoAAAA= X-Change-ID: 20250924-mdb-max7360-pwm-optimize-bcdae528f579 To: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= Cc: Thomas Petazzoni , linux-pwm@vger.kernel.org, linux-kernel@vger.kernel.org, Mathieu Dubois-Briand X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1758704808; l=1246; i=mathieu.dubois-briand@bootlin.com; s=20241219; h=from:subject:message-id; bh=a547ooMEsxEG/7bHmCeVbD/kqXHtT87FdYIiTphxlwg=; b=vWP85clr7MmOw5Vbuo3FLJYcnrREbMxrFB/6fwf2puzvpNW3haywUQB1M5wYVl3nCBQXHBOa3 2Nob72i8/G4AQXIIExLqTzFXPJXNiPcjbOJNSkvqyjlqNosmYGgodoJ X-Developer-Key: i=mathieu.dubois-briand@bootlin.com; a=ed25519; pk=1PVTmzPXfKvDwcPUzG0aqdGoKZJA3b9s+3DqRlm0Lww= X-Last-TLS-Session-Version: TLSv1.3 Duty steps computation can never end in values higher than MAX7360_PWM_MAX: remove useless use of min(). Signed-off-by: Mathieu Dubois-Briand Suggested-by: Uwe Kleine-K=C3=B6nig --- This was discussed in the MAX7360 series[1], and I believe we can indeed remove this check. [1] https://lore.kernel.org/all/20250824-mdb-max7360-support-v14-0-435cfda2= b1ea@bootlin.com/ --- drivers/pwm/pwm-max7360.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pwm/pwm-max7360.c b/drivers/pwm/pwm-max7360.c index ebf93a7aee5be4cfdac89601b78e6d20955f4877..16261958ce7ffb261b5a7cc7b1d= 47f63d6d398c2 100644 --- a/drivers/pwm/pwm-max7360.c +++ b/drivers/pwm/pwm-max7360.c @@ -75,7 +75,7 @@ static int max7360_pwm_round_waveform_tohw(struct pwm_chi= p *chip, duty_steps =3D MAX7360_PWM_MAX - 1; } =20 - wfhw->duty_steps =3D min(MAX7360_PWM_MAX, duty_steps); + wfhw->duty_steps =3D duty_steps; wfhw->enabled =3D !!wf->period_length_ns; =20 if (wf->period_length_ns && wf->period_length_ns < MAX7360_PWM_PERIOD_NS) --- base-commit: ce7f1a983b074f6cf8609068088ca3182c569ee4 change-id: 20250924-mdb-max7360-pwm-optimize-bcdae528f579 Best regards, --=20 Mathieu Dubois-Briand