From patchwork Tue Jul 2 03:00:19 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kim Seer Paller X-Patchwork-Id: 13718907 Received: from mx0b-00128a01.pphosted.com (mx0a-00128a01.pphosted.com [148.163.135.77]) (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 69E4110A0C; Tue, 2 Jul 2024 03:01:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=148.163.135.77 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719889273; cv=none; b=nFT+piA57x5KrAP2+OOu6zID77RX3WhuyEUvi5nB/uaQYwwMRrXqPtuaEzKf96zhInuNfUh+U0pLRugj8HVNHCmirTNx5texjNuuxm0S2E7adK+XqjBiYfk0usJbn2Z2Is/IWYMiEdF0KnCBBYlIu0QKeG3JQCkK2FKxJAwV/mM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719889273; c=relaxed/simple; bh=ovL+Ia/Bm/fPmOhqWgFuW1Wh/9Q/auFV3N/BCa98UfY=; h=From:To:CC:Subject:Date:Message-ID:MIME-Version:Content-Type; b=iougUESHbge3ba/rpTkCzfQycQQIX5pxEW2raR77GcmYTgs9xq8C3sr3xjhxetMiVVm5KXiewU4WNniyqqpIH811rNWBF0CYMdFv0boxHJKdJStfGtY34Xf/bbJePmPLIOmklOI+lknI6ZzJu75nhho4fEWiSVW5lre86qbmy58= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=analog.com; spf=pass smtp.mailfrom=analog.com; dkim=pass (2048-bit key) header.d=analog.com header.i=@analog.com header.b=MkwGWEdv; arc=none smtp.client-ip=148.163.135.77 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=analog.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=analog.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=analog.com header.i=@analog.com header.b="MkwGWEdv" Received: from pps.filterd (m0375855.ppops.net [127.0.0.1]) by mx0b-00128a01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 461NoXJ9002311; Mon, 1 Jul 2024 23:00:54 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=analog.com; h=cc :content-transfer-encoding:content-type:date:from:message-id :mime-version:subject:to; s=DKIM; bh=2v12Qee/feYPEHcKk9+LHcTEkTj f3647wxZW/EkxQh4=; b=MkwGWEdv5lwyi7fjUK/wam/pOjutsIw0lUh5nq8Ibh3 shGRFtfk+70uI8jluYTaNMDkaqfoTdgRhpKIwfYB9Km/9146EQVl4ukclTOOyPVd 9dl4UYNPXyEOat4fvG/b97epxYGXIceIk1N3A3w1V2wtz8NEIm6ZTmopdASHzrRx oSgtH/TeL7FM+Z0ulSM0JlKD+5nExBkRAqjZulFmritcOjxZTtGxJFs9aK10TKmU Tw4yr0xq1eyajmVM/huCUcR4vHnQFIi5RJ6HTsNzYhS2RhrYYYcigV3xi0HHae24 Nog9BWhWTBh0iEBEVPnsawh5FN5LzL81KCJ7qe+/8/Q== Received: from nwd2mta3.analog.com ([137.71.173.56]) by mx0b-00128a01.pphosted.com (PPS) with ESMTPS id 4046g1gkfr-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 01 Jul 2024 23:00:54 -0400 (EDT) Received: from ASHBMBX9.ad.analog.com (ASHBMBX9.ad.analog.com [10.64.17.10]) by nwd2mta3.analog.com (8.14.7/8.14.7) with ESMTP id 46230qSM041137 (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Mon, 1 Jul 2024 23:00:52 -0400 Received: from ASHBCASHYB5.ad.analog.com (10.64.17.133) by ASHBMBX9.ad.analog.com (10.64.17.10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.14; Mon, 1 Jul 2024 23:00:51 -0400 Received: from ASHBMBX9.ad.analog.com (10.64.17.10) by ASHBCASHYB5.ad.analog.com (10.64.17.133) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.14; Mon, 1 Jul 2024 23:00:51 -0400 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; Mon, 1 Jul 2024 23:00:51 -0400 Received: from kim-VirtualBox.ad.analog.com (KPALLER2-L03.ad.analog.com [10.117.220.28]) by zeus.spd.analog.com (8.15.1/8.15.1) with ESMTP id 46230WbL032046; Mon, 1 Jul 2024 23:00:35 -0400 From: Kim Seer Paller To: , , CC: Jonathan Cameron , David Lechner , Lars-Peter Clausen , Liam Girdwood , Mark Brown , Dimitri Fedrau , Krzysztof Kozlowski , "Rob Herring" , Conor Dooley , "Michael Hennerich" , =?utf-8?q?Nuno_S=C3=A1?= , Kim Seer Paller Subject: [PATCH v5 0/6] Add driver for LTC2664 and LTC2672 Date: Tue, 2 Jul 2024 11:00:19 +0800 Message-ID: <20240702030025.57078-1-kimseer.paller@analog.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-iio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ADIRuleOP-NewSCL: Rule Triggered X-Proofpoint-ORIG-GUID: fSYSYUapmuLhq6ZQWR32VnlpZAsaYAHu X-Proofpoint-GUID: fSYSYUapmuLhq6ZQWR32VnlpZAsaYAHu X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.680,FMLib:17.12.28.16 definitions=2024-07-01_23,2024-07-01_01,2024-05-17_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 adultscore=0 priorityscore=1501 bulkscore=0 mlxscore=0 mlxlogscore=999 lowpriorityscore=0 impostorscore=0 phishscore=0 suspectscore=0 clxscore=1015 spamscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2406140001 definitions=main-2407020020 Generalize the ABI documentation for DAC. The ABI defined for toggle mode channels: LTC2664: * out_voltageY_toggle_en * out_voltageY_raw0 * out_voltageY_raw1 * out_voltageY_symbol LTC2672: * out_currentY_toggle_en * out_currentY_raw0 * out_currentY_raw1 * out_currentY_symbol Default channels won't have any of the above ABIs. A channel is toggle capable if the devicetree 'adi,toggle-mode' flag is set. changes in v5: ltc2664: * Removed return statement for error code in ltc2664_dac_code_read making it a void function. * Refactored voltage regulator error handling and logic setting for vref_mv. * Added Reviewed-by tag. Bindings: * Added dac.yaml and generalized DAC common properties. * Modified ltc2664 binding adding constraints for the adi,manual-span-operation-config and output-range-microvolt properties. changes in v4: ltc2664: * Added comments for each field in the ltc2664_chan struct. * Changed global_toggle data type to bool and updated vref and rfsadj variables to include units. * Added 0,0 entry in ltc2672_span_helper to removed the id field from the ltc2664_chip_info struct. * Used mul_u64_u32_div helper function from linux/math64.h to avoid integer overflow during scale calculation. * Refactored code to use a single template for channel instead of separate channel arrays. * Used the devm_regulator_get_enable_read_voltage API for simplifying voltage retrieval. ABI: sysfs-bus-iio: * Added commit message for the ABI changes. sysfs-bus-iio-dac: * Updated the description of toggle_en to clarify autonomous toggling. * Fixed inconsistent use of spacing and tabs. Bindings: * Dropped Reviewed-by tag. * Updated the description for both bindings to include both 12-bit and 16-bit versions. changes in v3: ltc2664: * Added span sanity check for no match. * Initialized the variable 'span' to fix build warning. * Added Reported-by and Closes by tag. ABI: * Modified descriptions to make it more generalize. * Removed MAINTAINERS file entry. Bindings: * Changed clr-gpios to reset-gpios. * Added output range and reset code description for 'adi,manual-span-operation-config' property in ltc2664 binding. * Removed the $ref for 'adi,output-range-microamp' due to dt-schema warning in ltc2672 binding. Added Reported-by and Closes by tag. * Modified io-channels description and added maxItems constraint. changes in v2: ltc2664: * Updated struct ltc2664_chip_info to include device-specific data for scale, offset, measurement type, internal vref, manual span support, and rfsadj support. * Added a read-only extended info attribute powerdown_mode to indicate the state that the DAC output enters when the device is powered down. * Refactored code for setting the span into separate function and directly returning the span. * Adjusted memory allocation for st->iio_channels to include null terminator. * Spaces have been added after { and before }. Each pair of values is now placed on a separate line. ABI: * Generalized the ABI documentation for DAC. * Added DAC 42kohm_to_gnd powerdown mode. Bindings: * Created separate bindings for ltc2664 and ltc2672. * Added v-pos-supply and v-neg-supply regulator properties. * Renamed vref-supply to ref-supply based on the datasheet. * Added io-channels property and specifying the pin for multiplexer output. * Added vdd0-vdd4 supply properties for ltc2672, although they are not currently supported in the driver. * Changed clr-gpios description based on the datasheet. * Used 4 spaces for example indentation. Kim Seer Paller (6): iio: ABI: Generalize ABI documentation for DAC iio: ABI: add DAC 42kohm_to_gnd powerdown mode dt-bindings: iio: dac: Generalize DAC common properties dt-bindings: iio: dac: Add adi,ltc2664.yaml dt-bindings: iio: dac: Add adi,ltc2672.yaml iio: dac: ltc2664: Add driver for LTC2664 and LTC2672 Documentation/ABI/testing/sysfs-bus-iio | 1 + Documentation/ABI/testing/sysfs-bus-iio-dac | 61 ++ .../ABI/testing/sysfs-bus-iio-dac-ltc2688 | 31 - .../bindings/iio/dac/adi,ltc2664.yaml | 176 +++++ .../bindings/iio/dac/adi,ltc2672.yaml | 160 ++++ .../devicetree/bindings/iio/dac/dac.yaml | 50 ++ MAINTAINERS | 10 + drivers/iio/dac/Kconfig | 11 + drivers/iio/dac/Makefile | 1 + drivers/iio/dac/ltc2664.c | 736 ++++++++++++++++++ 10 files changed, 1206 insertions(+), 31 deletions(-) create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-dac create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ltc2664.yaml create mode 100644 Documentation/devicetree/bindings/iio/dac/adi,ltc2672.yaml create mode 100644 Documentation/devicetree/bindings/iio/dac/dac.yaml create mode 100644 drivers/iio/dac/ltc2664.c base-commit: 3587914bf61df7924933530353d840378cdc4973