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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C3DD4C433EF for ; Sun, 27 Feb 2022 12:42:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230239AbiB0Mnb (ORCPT ); Sun, 27 Feb 2022 07:43:31 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52898 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230186AbiB0Mnb (ORCPT ); Sun, 27 Feb 2022 07:43:31 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A3CF713DDE; Sun, 27 Feb 2022 04:42:53 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 441DAB80C9B; Sun, 27 Feb 2022 12:42:52 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8C5DDC340E9; Sun, 27 Feb 2022 12:42:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1645965770; bh=lgoi3zu7RH4Rt2lZePFxBiGxyY3ADqDf6BS0ZPsdrMA=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=BayYw+tkIVJ+zcOKfmBAvjzOfVlPm4fjRg7M8vRebocmvHKa6h2KMCwlxKx36IY3J bhEAQnfLJuG2j9S0H3UbdX5qdelAZUKXVNfk0lQCvlS4JKDerqkkxYwQS14n8fMyS4 dlOox4s5x+XUbGZ//ITSoH+GgkKH7su+XIGRvbphrlaYD/R7VLIv4hEJMGKKjYMDH9 MHktsu4dMq9PMb3wZ3tt90lOuQTapXNJePq5yAp/gFmcPACSDpbYEa7wLLLd5Bk0zH xO0hh9MGOdXzKTeGFMAKP2wG43KXPLn7t4q7ilE0Nrn/bRVccM9sPepk6Rv16KMqzb PsWq5gEgFLraA== Date: Sun, 27 Feb 2022 12:49:53 +0000 From: Jonathan Cameron To: Nuno =?UTF-8?B?U8Oh?= Cc: , , Rob Herring , Lars-Peter Clausen , Michael Hennerich Subject: Re: [PATCH v4 0/3] Add support for LTC2688 Message-ID: <20220227124953.02ab01fc@jic23-huawei> In-Reply-To: <20220225130129.69-1-nuno.sa@analog.com> References: <20220225130129.69-1-nuno.sa@analog.com> X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Fri, 25 Feb 2022 14:01:26 +0100 Nuno S=C3=A1 wrote: Hi Nuno, Given we are close to the end of this cycle and Andy has been heavily invol= ved in review of this one so I want to give more time for Andy to potentially t= ake another look.. Hence, I'm going to do something unusual and push out an extra-testing bran= ch with this on so we can get through the autobuilder tests in parallel with that extra = time. So, applied to the new extra-testing branch of iio.git with the intent to a= pply it to togreg later in a day or two subject to any last minute feedback. Thanks, Jonathan > The ABI defined for this driver has some subtleties that were previously > discussed in this RFC [1]. This might not be the final state but, > hopefully, we are close to it: >=20 > toggle mode channels: >=20 > * out_voltageY_toggle_en > * out_voltageY_raw0 > * out_voltageY_raw1 > * out_voltageY_symbol >=20 > dither mode channels: >=20 > * out_voltageY_dither_en > * out_voltageY_dither_raw > * out_voltageY_dither_raw_available > * out_voltageY_dither_offset > * out_voltageY_dither_frequency > * out_voltageY_dither_frequency_available > * out_voltageY_dither_phase > * out_voltageY_dither_phase_available >=20 > Default channels won't have any of the above ABIs. A channel is toggle > capable if the devicetree 'adi,toggle-mode' flag is set. For dither, the > assumption is more silent. If 'adi,toggle-mode' is not given and a > channel is associated with a TGPx pin through 'adi,toggle-dither-input', > then the channel is assumed to be dither capable (there's no point in > having a dither capable channel without an input clock). >=20 > changes in v2: >=20 > ltc2688: > * Use local buffer for regmap read. Do not assume that reg is part of > larger buffer; > * Renamed GPIO to "clr" so that is consistent with the datasheet; > * Renamed 'mask' and 'm' to info. 'mask' is a thing from the past; > * Removed 'LTC2688_CHAN_TOGGLE()' and defined to static ext_info arrays; > * Use 'regmap_set_bits' to set external ref; > * Use FIELD_{PREP|GET} for dither amplitude and channel calibbias where > only 13bits are used; > * Use 'regmap_write()' instead of update_bits for channels settings; > * Init 'val' at the beginning of the channel configuration loop > (and drop mask); > * Comment 'ltc2688_reg_writable()' to account for the special condition; > * Kmemdup default channels so that it can be safely changed per probed > device; > * Replace extended info multiplexer functions by individual functions; > * Use raw0 ABI for toggle channels; > * Use dedicated offset ABI for dither channels; > * Misc changes (spell fixes, blank lines...); > * Have a clock property per channel. Note that we this I moved to OF > since we now have to use 'devm_get_clk_from_child()' which is using > device_node. Note that I could use 'to_of_node()' but mixing of.h and > property.h does not feel like a good idea. >=20 > ABI: > * Added out_voltageY_raw0 ABI for toggle mode; > * Added out_voltageY_dither_offset. >=20 > Bindings: > * Use standard microvolt unit; > * Added constrains for adi,output-range-microvolt and removed negative > values from the dts example; > * Moved clocks to the channel object; > * Dropped clock-names; > * Add a dependency between 'adi,toggle-dither-input' and 'clocks'. >=20 > Changes in v3: >=20 > ltc2688: > * Fix mismatch between functions and function pointers detected by kern= el > test bot;=20 > * Always use if (ret) when ret > 0 has no meaning; > * Rename ltc2688_bulk_disable -> ltc2688_disable_regulators; > * Report dither phase in radians rather than degrees. >=20 > ABI: > * Specify units for dither_phase and dither_freqency;=20 > * Say why its useful to have dither_en and toggle_en; > * Combine out_voltageY_raw0 and out_voltageY_raw1; > * Fix some description issues in out_voltageY_raw{0|1} and > out_voltageY_symbol. >=20 > Bindings: > * Remove mentions to ABI (linux specifix); > * Slightly rephrased VREF and adi,toggle-dither-input properties and > suggested. > =20 > changes in v4: >=20 > ltc2688: > * Use reg_size + val_size instead of plain 3 in regmap; > * Use out_unlock instead of unlock in goto labels; > * Add comma to LTC2688_CHANNEL(), ltc2688_regmap_bus and > ltc2688_regmap_bus; > * Use __clear_bit() instead of clear_bit(); > * Flip the logic in vref regulator so that error condition is handled > first; > * Change to device API. With this, we need to_of_node() > for devm_get_clk_from_child(). >=20 > ABI: > * Update kernel version. >=20 > Bindings: > * Add Rob's Rb tag. >=20 > [1]: https://marc.info/?l=3Dlinux-iio&m=3D163662843603265&w=3D2 >=20 > Nuno S=C3=A1 (3): > iio: dac: add support for ltc2688 > iio: ABI: add ABI file for the LTC2688 DAC > dt-bindings: iio: Add ltc2688 documentation >=20 > .../ABI/testing/sysfs-bus-iio-dac-ltc2688 | 86 ++ > .../bindings/iio/dac/adi,ltc2688.yaml | 146 +++ > MAINTAINERS | 9 + > drivers/iio/dac/Kconfig | 11 + > drivers/iio/dac/Makefile | 1 + > drivers/iio/dac/ltc2688.c | 1071 +++++++++++++++++ > 6 files changed, 1324 insertions(+) > create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-dac-ltc2688 > create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ltc2688= .yaml > create mode 100644 drivers/iio/dac/ltc2688.c >=20