From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8A58A31354C for ; Mon, 10 Nov 2025 14:48:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762786095; cv=none; b=CNjmf20RQDbc2Lwf4OsHcbQT521EwXBLYMKOSQHPnjmEJ/WKsh/XzaAKLLTdTieUMll7j4bpkYhz997IP8uROW81E4cde2Ps2PfB2ydCCQwa9lMudVlIX8l749GSvgI2i/BJlOc/ueFxgwDtO+/q0OVS1OoFbphanKzjATkdtbc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762786095; c=relaxed/simple; bh=LPHuH34GPzz+FS33GWGiucjUVrgKTliQiQDa8mDrPto=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=ivl7eDXXik1h1tiCQBKMoC3I4ipAs390xvggZULMKZf1V038wM3hPWQiY8fvBsApsVZq8xqp3jDpfRbMnN8qgSi0ZvG9G/RPI+I+5XF43rH/eTSx54wIdKYNHFffmnEnY49KRR77bd4UNuEjFdVaZKD7fv4W18FbCx+GD/zaAY4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=mrKtTsa/; arc=none smtp.client-ip=209.85.128.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="mrKtTsa/" Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-47774d3536dso14246195e9.0 for ; Mon, 10 Nov 2025 06:48:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1762786091; x=1763390891; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=LPHuH34GPzz+FS33GWGiucjUVrgKTliQiQDa8mDrPto=; b=mrKtTsa/Waj1IsC00cI2/hZIW3lo0cdVW7+7mKXNZtIH+0sINwOiud66lJSMlFl8Lu NSegLjc9zyP/bcP4NLVFR+4+B28aocJbymeEVpNYIbBcsutZQCOQepec8dkpVYrvGMMg vP+gWyegzSO6Zm46s12EwF/AS1aPcASDEhAbjNhtXu+VnvZIkkwucFIKSr53XwvFrGgv NB4YEX7XTSmGkCo646UzXMp1x9zuSfLEZxGD6WKGX9nyKeuh5fRJodobCLquBl+JTgPp wwdufxIdXyE5kBIOol8fdoGRNht5nrMPZLv/YUhJ44ekIqC0gx5/7SsFRRcZmrROyzsE 0Hiw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762786091; x=1763390891; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=LPHuH34GPzz+FS33GWGiucjUVrgKTliQiQDa8mDrPto=; b=ogrblSpkZCy8GNyBWJx+Xt+9e9l7r7qRVIe/cD10LLOgEMGquUVsK4uw5iJzma1EER EO8vvbgd583q7PKaAwPgSr2mDBKOb4L2s5z+VqCtUd4oT8F+154sxIxXxl2+PPig6m5o AJkjnYmpW1x6P2ZBSKMzyaQkVMbqNDt+5+50hnAOSiuTCVzwzj8dZxap5VXdtVbhopua aHdyaXDV3jIW+7gD7Ac2sJGjVzcEJBw9c4/tu8VBuuRaRjLIJ2Omtt6snHx+Y4L5S0s6 Sf1fN+lGgitAa7WOq+JyZt75I1s9CxAXDptSQhfAmXr4r9+wz/Ny27ot7z38R8Vgxvju kYyw== X-Forwarded-Encrypted: i=1; AJvYcCUelDv2N93zm+OZeMMJ0ZpJ5OY0OxDVQvJu5efvRoRDLTb7mJ3v8IhhAaqW6Up2BvJVZhkgJOj5YmI=@vger.kernel.org X-Gm-Message-State: AOJu0Yy0lGNj00NTbmsgSRiqG9gVmFywc0u2qHWcDzhw9rwL2nbXGREF CD8VyedlSEK7mcoRDJNdxWFmed6WFMzbZv6WpUy81gWDB26nZ3GWKjgy X-Gm-Gg: ASbGncvjYKRTgpHJmD0luC32kh+43NlRwLbPt7MCM9z5UbPO0WM3PXgm0xSYs3C4Ltn UYzWXhwK8I90Lte+7x7dt9IJT9QkXKvtjw93wGmr6HOJHyOop0/lnQD0D86tzPrH2NVrzfYTsVT k4SwJ4LUUSuktlAspXZLCDcXR4P0nwfyOAhCXNTNyZ8JjSBROr/gSaq6kK3W10ATP+zsmk2FmWy EKNn1nDr4UubUOB4tYGXLI0YUERc/ExxQMVjwN7ayhgHmM3wRVsHoA/FqIPO12MP/LcW758IKQk qABxHYMoH0Uw5HAphP5IeqU+kEvjAIHK+vP/S9MavfaZuECifC6nsHb+g8Q28iqNEeWSk48FJ/R h7lHH7gXO0PDO9rCoqe15hglbiH6KOuVEafLH6jcON3ZPa4XffVwS/nkN4zekKc8YR1K24+19g2 L03TE3d233AJfqURBU/zA= X-Google-Smtp-Source: AGHT+IFyyPGi2h2HdORU5Y7qZAm0mzvGdm657Ujj5pRLIdfkz1ikPx4AlWnK/vjsN3l+QNuYFG0dhA== X-Received: by 2002:a05:600c:a30d:b0:471:ea1:a460 with SMTP id 5b1f17b1804b1-4776dcbe68cmr86535255e9.11.1762786090588; Mon, 10 Nov 2025 06:48:10 -0800 (PST) Received: from [192.168.1.187] ([161.230.67.253]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42b2e052f32sm15817388f8f.17.2025.11.10.06.48.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 10 Nov 2025 06:48:10 -0800 (PST) Message-ID: <5e0ea52e6a77a1d6af861ba5aaeeea5c3d514705.camel@gmail.com> Subject: Re: [PATCH v1 0/3] iio: adc: Add AD4134 minimum I/O support From: Nuno =?ISO-8859-1?Q?S=E1?= To: Marcelo Schmitt , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Cc: jic23@kernel.org, nuno.sa@analog.com, dlechner@baylibre.com, andy@kernel.org, Michael.Hennerich@analog.com, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, corbet@lwn.net, cosmin.tanislav@analog.com, marcelo.schmitt1@gmail.com Date: Mon, 10 Nov 2025 14:48:46 +0000 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.1 Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Mon, 2025-11-10 at 09:44 -0300, Marcelo Schmitt wrote: > This patch series adds basic support for ad4134. AD4134 is a very flexibl= e > device that can be configured in many different ways. This series aims to > support the simplest way of interfacing with AD4134 which is called minim= um I/O > mode in data sheet. This is essentially usual SPI with the addition of an= ODR > (Output Data Rate) GPIO which functions as conversion start signal in min= imum > I/O mode. The CS pin may be connected to a host controller CS pin or grou= nded. >=20 > This set provides just one feature: > - Single-shot ADC sample read. >=20 > [PATCH 1] Device tree documentation for AD4134. > [PATCH 2] IIO Linux driver for AD4134. > [PATCH 3] Initial IIO documentation. >=20 > There is a driver by Cosmin on ADI Linux tree that supports AD4134 in wir= ing > configurations suited for high speed data transfers. Even though the mini= mum I/O > support was initialy based on that high speed transfer driver, the result= ended > up becoming entirely different. Also, because the different wiring > configurations are likely going to use different resources and software > interfaces, the code for AD4134 support was split into ad4134-spi.c, > ad4134-common.h, and ad4134-common.c. I'm familiar with the odd way this part is implemented in ADI tree :). Ques= tion is, are you intending to support the high speed bits? I guess so, otherwise having = the above split would not make much sense. - Nuno S=C3=A1 >=20 > With best regards, > Marcelo >=20 > Marcelo Schmitt (3): > =C2=A0 dt-bindings: iio: adc: Add AD4134 > =C2=A0 iio: adc: Initial support for AD4134 > =C2=A0 Docs: iio: Add AD4134 >=20 > =C2=A0.../bindings/iio/adc/adi,ad4134.yaml=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 | 209 +++++++++++++ > =C2=A0Documentation/iio/ad4134.rst=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 |=C2=A0 58 = ++++ > =C2=A0Documentation/iio/index.rst=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 |=C2= =A0=C2=A0 1 + > =C2=A0MAINTAINERS=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= |=C2=A0=C2=A0 9 + > =C2=A0drivers/iio/adc/Kconfig=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0 |=C2=A0 15 + > =C2=A0drivers/iio/adc/Makefile=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0 |=C2=A0=C2=A0 2 + > =C2=A0drivers/iio/adc/ad4134-common.c=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | 200 ++++++++++++ > =C2=A0drivers/iio/adc/ad4134-common.h=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | 132 ++++++++ > =C2=A0drivers/iio/adc/ad4134-spi.c=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | 287 +++++= +++++++++++++ > =C2=A09 files changed, 913 insertions(+) > =C2=A0create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad= 4134.yaml > =C2=A0create mode 100644 Documentation/iio/ad4134.rst > =C2=A0create mode 100644 drivers/iio/adc/ad4134-common.c > =C2=A0create mode 100644 drivers/iio/adc/ad4134-common.h > =C2=A0create mode 100644 drivers/iio/adc/ad4134-spi.c >=20 >=20 > base-commit: c5411c8b9ed1caf53604bb1a5be3f487988efc98