From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CF843CD98F0 for ; Sun, 21 Jun 2026 12:56:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=7BZ17cjFlry83TGansWhd9Db8FHG6/Qgrb4teQkt9TU=; b=oXtoPu0hrFfHjEcZyfT1QFdbvZ 8eT3/gHaCj9Yw8YevC/2yMn281ESSDMm6JDnP1oIaKGxw3nFgZmxIFoO498tL/CaAqwk9KqBsoV0e 4GvD0pbYRkcEMO53aFtvV+2dR9HMvkBIwufLN+Ym9UglQXgEVcrjC5RUQtkb8uFc1qh82fch+sbEe IWWjb745cGWiBPQrjigc2TbXHfuNJ8qN3Cq2rR0jEeVF1q0WWXqIXUoPvJ3aW/rYFFhmSEzKyDpMk OGl2F5SNNmI9HazGvJJDyg0ZWeOfS3FVcV6wx6YgfyHQF+3CiSgJwJzKsUBcnVK4EyHIYpS6HZ6QO EoEgoiLA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wbHiU-00000003wNp-44o7; Sun, 21 Jun 2026 12:56:02 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wbHiU-00000003wNf-1KZj for linux-arm-kernel@lists.infradead.org; Sun, 21 Jun 2026 12:56:02 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id AB8F44040D; Sun, 21 Jun 2026 12:56:01 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B7DAC1F000E9; Sun, 21 Jun 2026 12:55:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1782046561; bh=7BZ17cjFlry83TGansWhd9Db8FHG6/Qgrb4teQkt9TU=; h=Date:From:To:Cc:Subject:In-Reply-To:References; b=Z/o4lN77OqFzxFLKvGnHfUEt6l5Ua77jIJ7Dm0UNIKzV3uO3NLplC9wwxG4fQ9cJu MLfz+UT8a7Jp46YfwPvH6RzAOKGrHrS1Ru9GmfrjlajcExrbDfo2aL5HtiaU4NKp3J Le96uuPx5cvJiYrq51ws0Ps1qrAkJMGnx+jVhnHwRtbgjcljzXJvPBTXF/4316sTZn 0uxPdhSu7qDye3mjfydXHI8BOwhMUAgK+lQiNmcR6ZD/syFreqhke0B+WF0flbOwRY Tuna9/G0oJ0Dcm6VhcoBBJ5iZ8CBDIFf4ZCg7HmPW+dqJypsFG//xcDCcC0iB9MKAw QZTwrcbluXD0Q== Date: Sun, 21 Jun 2026 13:55:49 +0100 From: Jonathan Cameron To: Angelo Dureghello Cc: Greg Ungerer , Geert Uytterhoeven , Steven King , Arnd Bergmann , Maxime Coquelin , Alexandre Torgue , David Lechner , Nuno =?UTF-8?B?U8Oh?= , Andy Shevchenko , Greg Ungerer , linux-m68k@lists.linux-m68k.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-iio@vger.kernel.org, Michael Turquette , Stephen Boyd , Brian Masney Subject: Re: [PATCH v5 09/10] iio: dac: add mcf54415 DAC Message-ID: <20260621135549.75e152a7@jic23-huawei> In-Reply-To: References: <20260610-wip-stmark2-dac-v5-0-b76b83366d5c@baylibre.com> <20260610-wip-stmark2-dac-v5-9-b76b83366d5c@baylibre.com> <20260611114800.009d9797@jic23-huawei> X-Mailer: Claws Mail 4.4.0 (GTK 3.24.52; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 15 Jun 2026 20:20:21 +0900 Angelo Dureghello wrote: > Hi Jonathan, > > On Thu, Jun 11, 2026 at 11:48:00AM +0100, Jonathan Cameron wrote: > > On Wed, 10 Jun 2026 22:35:14 +0200 > > Angelo Dureghello wrote: > > > > > From: Angelo Dureghello > > > > > > Add basic version of mcf54415 DAC driver. DAC is embedded in the SoC and > > > DAC configuration registers are mapped in the internal IO address space. > > > > > > The DAC accepts a 12-bit digital signal and creates a monotonic 12-bit > > > analog output varying from DAC_VREFL to DAC_VREFH. The DAC module > > > consists of a conversion unit, an output amplifier, and the associated > > > digital control blocks. Default register values for DAC_VREFL and DAC_VREFH > > > are respectively 0 and 0xfff, left untouched in this initial version. > > > > > > This initial version of the driver is minimalistic, "output raw" only, to > > > be extended in the future. DMA and external sync are disabled, default mode > > > is high speed, default format is right-justified 12-bit on 16-bit word. > > > > > > Signed-off-by: Angelo Dureghello > > I'm lazy so didn't check earlier versions but assume the two bits > > of feedback from Sashiko are false positives: > > https://sashiko.dev/#/patchset/20260610-wip-stmark2-dac-v5-0-b76b83366d5c%40baylibre.com > > > > The one about clock underflow if resume fails, and then devm cleanup happens later > > is a bit nasty. > > > > I did a bit of digging and maybe it is better to just leave the clock on? > > The status dev.power.is_suspended is set to false whether or not resume succeeded > > and I believe a following suspend will not take into account that resume failed. > > > > I'm not set up to poke the combinations but it might be worth trying that. > > +CC common clk people who may immediately know what the right answer is. > > > > was about testing this, there are no bus faults on read/write of registers > with clock disabled, nor warnings in dmesg on double disable. Great. That makes life easier. I'm curious to why we don't get a double disable though... Guess I'm missing something in the flows. > > Anyway, i see now from arch Kconfig that is not possible to have CONFIG_PM > for this specifig CPU (with MMU, PM is force-disabled), so would remove pm_ops. > Ok ? I suspect that's something that might get 'fixed' at somepoint, so I'd keep the PM stuff in this driver as long as you can check it basically works. Jonathan > > Will fix all other things you pointyed out in v6. > > > Otherwise just a few minor style comments inline. > > > > Thanks, > > > > Jonathan > > > > thanks, > regards, > angelo >