[{"id":3673446,"web_url":"http://patchwork.ozlabs.org/comment/3673446/","msgid":"","list_archive_url":null,"date":"2026-04-04T14:25:41","subject":"Re: [PATCH v6 2/4] iio: adc: ad4691: add initial driver for AD4691\n family","submitter":{"id":87228,"url":"http://patchwork.ozlabs.org/api/people/87228/","name":"David Lechner","email":"dlechner@baylibre.com"},"content":"On 4/3/26 6:03 AM, Radu Sabau via B4 Relay wrote:\n> From: Radu Sabau \n> \n> Add support for the Analog Devices AD4691 family of high-speed,\n> low-power multichannel SAR ADCs: AD4691 (16-ch, 500 kSPS),\n> AD4692 (16-ch, 1 MSPS), AD4693 (8-ch, 500 kSPS) and\n> AD4694 (8-ch, 1 MSPS).\n> \n> The driver implements a custom regmap layer over raw SPI to handle the\n> device's mixed 1/2/3/4-byte register widths and uses the standard IIO\n> read_raw/write_raw interface for single-channel reads.\n> \n> The chip idles in Autonomous Mode so that single-shot read_raw can use\n> the internal oscillator without disturbing the hardware configuration.\n> \n> Three voltage supply domains are managed: avdd (required), vio, and a\n> reference supply on either the REF pin (ref-supply, external buffer)\n> or the REFIN pin (refin-supply, uses the on-chip reference buffer;\n> REFBUF_EN is set accordingly). Hardware reset is performed via\n> the reset controller framework; a software reset through SPI_CONFIG_A\n> is used as fallback when no hardware reset is available.\n> \n> Accumulator channel masking for single-shot reads uses ACC_MASK_REG via\n> an ADDR_DESCENDING SPI write, which covers both mask bytes in a single\n> 16-bit transfer.\n> \n> Signed-off-by: Radu Sabau \n> ---\nThis patch looks in good shape. Although I wouldn't mind using\nMEGA/MICRO, etc. more in numbers with more than 3 or 4 zeros.\n\nReviewed-by: David Lechner ","headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-pwm@vger.kernel.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=baylibre-com.20251104.gappssmtp.com\n header.i=@baylibre-com.20251104.gappssmtp.com header.a=rsa-sha256\n header.s=20251104 header.b=T+tHPtq/;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=172.105.105.114; helo=tor.lore.kernel.org;\n envelope-from=linux-pwm+bounces-8473-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=baylibre-com.20251104.gappssmtp.com\n header.i=@baylibre-com.20251104.gappssmtp.com header.b=\"T+tHPtq/\"","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=209.85.167.178","smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=baylibre.com","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=baylibre.com"],"Received":["from tor.lore.kernel.org (tor.lore.kernel.org [172.105.105.114])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fnyYr6plcz1yGr\n\tfor ; Sun, 05 Apr 2026 01:25:52 +1100 (AEDT)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby tor.lore.kernel.org (Postfix) with ESMTP id CDC88300E152\n\tfor ; Sat, 4 Apr 2026 14:25:49 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id D2135296BDC;\n\tSat, 4 Apr 2026 14:25:48 +0000 (UTC)","from mail-oi1-f178.google.com (mail-oi1-f178.google.com\n [209.85.167.178])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id EFBB6291864\n\tfor ; Sat, 4 Apr 2026 14:25:45 +0000 (UTC)","by mail-oi1-f178.google.com with SMTP id\n 5614622812f47-46fb6d65c28so338366b6e.0\n for ; Sat, 04 Apr 2026 07:25:45 -0700 (PDT)","from ?IPV6:2600:8803:e7e4:500:e14e:bcc6:3f95:26eb?\n ([2600:8803:e7e4:500:e14e:bcc6:3f95:26eb])\n by smtp.gmail.com with ESMTPSA id\n 5614622812f47-46f0f4e16a9sm2936944b6e.4.2026.04.04.07.25.41\n (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n Sat, 04 Apr 2026 07:25:43 -0700 (PDT)"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1775312748; cv=none;\n b=j/sXFk4bcXdtbKfsn+Dpv8TWe8JMIDB6okXyAV15U8W2W0hDslB6t2Q7ShR6goYAVciUdLnmfJYWoI0f4YAyDpq4+zDw8h/Ah6yXqVrdGk335HzedJ8rwIB4PY4WpFZUMuz1XPpXsTJmDYUyx0mh5G1+iNkQjSIk5DCHy5qcX/k=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1775312748; c=relaxed/simple;\n\tbh=YDHQQIkUtIMA5RuVXX+SK+ZopnAIz0IH3aZAqaHI9pk=;\n\th=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From:\n\t In-Reply-To:Content-Type;\n b=awrcqZbuehkfW16guhD5w5k9MlRie7ysVimswv2mpZTLXk5YrmPzz2y4Vai+FTJ6Fsu2rtsMCvs/q2IUjAitoIMWGREe+honJQuPXdoHIdk3ZOXjhTIV4kiBXc4bRv/osVaZJ17emp0RWWN0KeaZ9x25xLDLGx4ikwyfg95gMrQ=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=baylibre.com;\n spf=pass smtp.mailfrom=baylibre.com;\n dkim=pass (2048-bit key) header.d=baylibre-com.20251104.gappssmtp.com\n header.i=@baylibre-com.20251104.gappssmtp.com header.b=T+tHPtq/;\n arc=none smtp.client-ip=209.85.167.178","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=baylibre-com.20251104.gappssmtp.com; s=20251104; t=1775312745;\n x=1775917545; darn=vger.kernel.org;\n h=content-transfer-encoding:in-reply-to:from:content-language\n :references:cc:to:subject:user-agent:mime-version:date:message-id\n :from:to:cc:subject:date:message-id:reply-to;\n bh=DG8ZLzSkLL5KxGbpBt+cYKmm52XpYZesZWRmcIu235A=;\n b=T+tHPtq/ZJmrz/x0hon6ubvef3dyaaHhk1GRRXgW8y8HepFtZGYjfJcUQczd3MEkSt\n 2Agviun9Cf5EYKtk9j0IqUraHya5+cWg7jpTqs6odctBlL2Zz+DNQ/uNqQMe9wgH193o\n z3zeYYVklZaMQki3yiMddob47zunpO6AWKU8n5PWDQ6LxULv6X3f1wcyghb+It/+xjWX\n z9PNVkRx7eWxmjJNLwRnRuJlliHjZbQNuYTor2Ij+hbvfTGn47XIa2O2GDia9HEi0TFp\n QBOE28LuVqSi18lbs2VxGAN1lYamyOifHSPJ/JYonO7K4FYFJ/G3E8jlIr/y0xjtfZ+C\n UNXw==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1775312745; x=1775917545;\n h=content-transfer-encoding:in-reply-to:from:content-language\n :references:cc:to:subject:user-agent:mime-version:date:message-id\n :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id\n :reply-to;\n bh=DG8ZLzSkLL5KxGbpBt+cYKmm52XpYZesZWRmcIu235A=;\n b=nDoiG8fdhzP9n88Xa0W09e6N6ZgnkoUfcnR+uG4WHa8g8cnHeBYru5ytg50v14lx6U\n GAZKMJW5+h6gQbrviwdqORIuQZy1YiO9tiHScNarj5J1RiNTdeEYHw2T7fcIgTITaAr8\n qB9GyPt8fVARQteyIuav0Sl1HguawhgAcR8G+aDkDl79O5VUB81AclIFLbQkz92Lc/oJ\n CQI9mmtofE+xeHuHPQcQuPwp1FGHhWTfjxJI2ljsYz9ms3/cfZP40itX+tBXAEEVeXFo\n fPYC66eFwXwxI5nJiON+Vy8zBWeDLdKVVGOwSp8Puq+wb4I8hkWfMYwYW0aPqzJEj01N\n Gnug==","X-Forwarded-Encrypted":"i=1;\n AJvYcCVrShlEBSwKrHRZ7TnoXQBJ/VJWGXwZGkfAf9D+WnSGCLUEhqGCMQmztTahxX/a3iJufFXCJPjcEI0=@vger.kernel.org","X-Gm-Message-State":"AOJu0Yxtm7wSK+yBjwz1B0Xk5Dzy390v5hqUseZRmJJ9MtGmhVePcOkN\n\tmvqiSnmnieUdoaFuqVPOaaQoZ0T3Z34q3y+xbMoNbcWOR9HiKFhXRsxOhQmCMahHSyY=","X-Gm-Gg":"AeBDietbTPWspmvBLUDr4z6s7EskzK5h3ODNzD9O+MaJGt53DHla5mG3WCvFnmC49lo\n\tmg/wWYDWziz+sc6r+BzRkJS7J3XU+c/V18tP08UuhuOBwlbjmUZulxLVEwYLyqQDEpJrYpIBXJU\n\tIjj5GSRlVfIX2zStEGr6/55GvixygEXquNnXmvV3jrvLFH+ED23WOf8IudRMxhSgt9pER75cXR0\n\ttkLVoX0EbesGIa0RHTqh1Ay8NmsD6Blxdz32pv4KNPyx5mWXdRzTs80YfdksOUsdbX4TEKa7jwG\n\tn4rr3om+HGD17qMhFpGDTno3gcu7gR21Sue0n9swvIr72sLp97LTBe7u6jqyQXS8u+Ypr3dbnob\n\tSHTeaS5RuBWgUqZNZWLJf+I20KEWwceARgm6YbsYJUiG1bSGV+sfFvqiFaAJZLV7n/Ma5AiWtO4\n\tOobEWWnLYhY3MpYreU2I1Zdfp3/IgdmFo34E5zIGL5hWqRgOe13Rt3W1jg5ggMIqOQCb9YXdhk4\n\tg==","X-Received":"by 2002:a05:6808:1206:b0:46a:8dd5:3a4f with SMTP id\n 5614622812f47-46ef5002b8dmr3010289b6e.5.1775312744846;\n Sat, 04 Apr 2026 07:25:44 -0700 (PDT)","Message-ID":"","Date":"Sat, 4 Apr 2026 09:25:41 -0500","Precedence":"bulk","X-Mailing-List":"linux-pwm@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","Subject":"Re: [PATCH v6 2/4] iio: adc: ad4691: add initial driver for AD4691\n family","To":"radu.sabau@analog.com, Lars-Peter Clausen ,\n Michael Hennerich ,\n Jonathan Cameron ,\n =?utf-8?q?Nuno_S=C3=A1?= ,\n Andy Shevchenko , Rob Herring ,\n Krzysztof Kozlowski , Conor Dooley ,\n =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= ,\n Liam Girdwood , Mark Brown ,\n Linus Walleij , Bartosz Golaszewski ,\n Philipp Zabel , Jonathan Corbet ,\n Shuah Khan ","Cc":"linux-iio@vger.kernel.org, devicetree@vger.kernel.org,\n linux-kernel@vger.kernel.org, linux-pwm@vger.kernel.org,\n linux-gpio@vger.kernel.org, linux-doc@vger.kernel.org","References":"\n <20260403-ad4692-multichannel-sar-adc-driver-v6-0-fa2a01a57c4e@analog.com>\n <20260403-ad4692-multichannel-sar-adc-driver-v6-2-fa2a01a57c4e@analog.com>","Content-Language":"en-US","From":"David Lechner ","In-Reply-To":"\n <20260403-ad4692-multichannel-sar-adc-driver-v6-2-fa2a01a57c4e@analog.com>","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"7bit"}}]