From patchwork Fri Jan 21 14:24:58 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Nuno Sa X-Patchwork-Id: 1582599 Return-Path: X-Original-To: incoming-dt@patchwork.ozlabs.org Delivered-To: patchwork-incoming-dt@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org (client-ip=23.128.96.18; helo=vger.kernel.org; envelope-from=devicetree-owner@vger.kernel.org; receiver=) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by bilbo.ozlabs.org (Postfix) with ESMTP id 4JgM9b47vvz9t2p for ; Sat, 22 Jan 2022 01:25:39 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1350934AbiAUOZh (ORCPT ); Fri, 21 Jan 2022 09:25:37 -0500 Received: from mx0a-00128a01.pphosted.com ([148.163.135.77]:5830 "EHLO mx0a-00128a01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1350885AbiAUOZg (ORCPT ); Fri, 21 Jan 2022 09:25:36 -0500 Received: from pps.filterd (m0167088.ppops.net [127.0.0.1]) by mx0a-00128a01.pphosted.com (8.16.1.2/8.16.1.2) with ESMTP id 20LC6V1b015511; Fri, 21 Jan 2022 09:25:18 -0500 Received: from nwd2mta4.analog.com ([137.71.173.58]) by mx0a-00128a01.pphosted.com (PPS) with ESMTPS id 3dqj1bayqu-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 21 Jan 2022 09:25:17 -0500 Received: from ASHBMBX8.ad.analog.com (ASHBMBX8.ad.analog.com [10.64.17.5]) by nwd2mta4.analog.com (8.14.7/8.14.7) with ESMTP id 20LEPGep048199 (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 21 Jan 2022 09:25:16 -0500 Received: from ASHBMBX9.ad.analog.com (10.64.17.10) by ASHBMBX8.ad.analog.com (10.64.17.5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.14; Fri, 21 Jan 2022 09:25:15 -0500 Received: from zeus.spd.analog.com (10.66.68.11) by ashbmbx9.ad.analog.com (10.64.17.10) with Microsoft SMTP Server id 15.2.986.14 via Frontend Transport; Fri, 21 Jan 2022 09:25:15 -0500 Received: from NSA-L01.ad.analog.com ([10.32.224.71]) by zeus.spd.analog.com (8.15.1/8.15.1) with ESMTP id 20LEP5pj017001; Fri, 21 Jan 2022 09:25:07 -0500 From: =?utf-8?q?Nuno_S=C3=A1?= To: , CC: Rob Herring , Jonathan Cameron , Lars-Peter Clausen , Michael Hennerich Subject: [PATCH v3 0/3] Add support for LTC2688 Date: Fri, 21 Jan 2022 15:24:58 +0100 Message-ID: <20220121142501.151-1-nuno.sa@analog.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 X-ADIRuleOP-NewSCL: Rule Triggered X-Proofpoint-GUID: pUxgP4iy1SMVZRDkMl-Lc1gWtFDS01EF X-Proofpoint-ORIG-GUID: pUxgP4iy1SMVZRDkMl-Lc1gWtFDS01EF X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.816,Hydra:6.0.425,FMLib:17.11.62.513 definitions=2022-01-21_06,2022-01-21_01,2021-12-02_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 lowpriorityscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 malwarescore=0 mlxscore=0 spamscore=0 bulkscore=0 priorityscore=1501 clxscore=1015 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000 definitions=main-2201210097 Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org 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: toggle mode channels: * out_voltageY_toggle_en * out_voltageY_raw0 * out_voltageY_raw1 * out_voltageY_symbol dither mode channels: * 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 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). changes in v2: 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. ABI: * Added out_voltageY_raw0 ABI for toggle mode; * Added out_voltageY_dither_offset. 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'. Changes in v3: ltc2688: * Fix mismatch between functions and function pointers detected by kernel test bot; * 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. ABI: * Specify units for dither_phase and dither_freqency; * 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. Bindings: * Remove mentions to ABI (linux specifix); * Slightly rephrased VREF and adi,toggle-dither-input properties and suggested. [1]: https://marc.info/?l=linux-iio&m=163662843603265&w=2 Nuno Sá (3): iio: dac: add support for ltc2688 iio: ABI: add ABI file for the LTC2688 DAC dt-bindings: iio: Add ltc2688 documentation .../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 | 1070 +++++++++++++++++ 6 files changed, 1323 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