From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 4C73D150984; Sun, 8 Sep 2024 10:41:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725792115; cv=none; b=SRSGM6j1JNOeeTEeRShRG/IKJoeA8PlvH8GXABsW+t6VZXa1zvHdTQVOGyWhKzIFEsUAFkFAXelrHrviN0dvKzXenJ7MoRGOSGMszX5gPiodU0vpO2W3HnNVvCQyaSK56GDBzpTPUhJzwMJ1AB6BmFAthao+LXco3rBgbhxN4Lo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725792115; c=relaxed/simple; bh=+M6JdLABBdfXrmO8zUKmmKac7j/TQa7IzYDjOIGqSeE=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Qb0yklmSSoq/LH6+iT/h/HsMJft4Di5H8xtp2uPrLDXOrwDaANfbXTGfZhGhuXr9sg/1XNr/BTVEpyfq4ddbuOl9AQb1CmZOJXvaR8EeRjdQji/7FYGHXatTWsxpuFbwYa5u7NBCd155yYjyZKSBxXo4VLl3ZSuTbc+Nq7v6HY4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=SGkuBqin; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="SGkuBqin" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E8B8DC4CEC3; Sun, 8 Sep 2024 10:41:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1725792114; bh=+M6JdLABBdfXrmO8zUKmmKac7j/TQa7IzYDjOIGqSeE=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=SGkuBqin0MlEDug6vExUXXwXRFVFCtcoZOgP1Cl1YTAmSKXOvjqseTQqEIhahXH0b DJ37PMjMU6hEEEuvk686jHdG/MqRCBUoIkw3v8zHnqVuWQt1WOh29u1esh64z5WDxY DihdzjiljsXcVqgz3KKvITmD1CGNxhkLxEQ4swpYw2GwA3F8+dWBVU8JQoI/wPxb+H Rkj/WF58VxLmhT86PAA83hUaJ4lL55PCJ3ov0OWDQVcAkhQ3Fa4kg+kRjreZkJs3ub hDQq7mdyWXmSRwih/T8uNEGl9YNdC0K4pwB1vzcQml5P5JkTeDZVqfwhCq1FqqfAHF x6enYTyWsd2QA== Date: Sun, 8 Sep 2024 11:41:43 +0100 From: Jonathan Cameron To: Dumitru Ceclan via B4 Relay Cc: dumitru.ceclan@analog.com, Lars-Peter Clausen , Michael Hennerich , Rob Herring , Krzysztof Kozlowski , Conor Dooley , mitrutzceclan@gmail.com, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Conor Dooley , Nuno Sa Subject: Re: [PATCH v3 0/3] Add support for AD4113 Message-ID: <20240908114143.414a9367@jic23-huawei> In-Reply-To: <20240817111902.2ed6b98a@jic23-huawei> References: <20240812-ad4113-v3-0-046e785dd253@analog.com> <20240817111902.2ed6b98a@jic23-huawei> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.43; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-iio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Sat, 17 Aug 2024 11:19:02 +0100 Jonathan Cameron wrote: > On Mon, 12 Aug 2024 11:13:13 +0300 > Dumitru Ceclan via B4 Relay wrote: >=20 > > This patch series adds support for the AD4113 ADC within the existing > > AD7173 driver. > >=20 > > The AD4113 is a low power, low noise, 16-bit, =CE=A3-=CE=94 analog-to-d= igital > > converter (ADC) that integrates an analog front end (AFE) for four > > fully differential or eight single-ended inputs. > >=20 > > The part is not released yet and the documentation is not public. > > Register map is identical to AD4114 besides the lower width data > > register and the GPIO register. > >=20 > > Particularities of this model: > > - 16 bit data register > > - no temperature sensor > > - no current inputs > > - input buffers > > - internal reference > > - external reference REF-/REF+ > > - no second external reference REF2-/REF2+ > > - no AVDD2 supply > > - 2 GPIO pins with config bits starting at a higher position in register > > - 8 VINx inputs with voltage divider > > - 16 channel registers and 8 setup registers > >=20 > > Signed-off-by: Dumitru Ceclan =20 > Hi. >=20 > Series is fine, but I don't yet have the fix=20 > [PATCH] iio: adc: ad7173: Fix incorrect compatible string > in the upstream of my togreg branch. >=20 > Hence this will have to wait a little while for that to be present. > Otherwise this will create a fiddly merge for linux-next etc. Tree juggling didn't go entirely according to plan so unfortunately this has slipped back to next cycle unless there is a delay in the 6.12 merge window opening. Sorry about that, and I'll make sure to queued it up nice and early after rc1. >=20 > Jonathan >=20 > > --- > > Changes in v3: > > - lowercase chip ID > > - add patch to correctly order chip IDs defines > > - picked up RB and ACK tags > > - Link to v2: https://lore.kernel.org/r/20240809-ad4113-v2-0-2a70c101a1= f4@analog.com > >=20 > > Changes in v2: > > - correctly set realbits and storagebits to 16 in iio_chan_spec > > - describe bindings restrictions in commit message due to lack of > > sufficient diff context > > - describe model differences better in cover letter > > - Link to v1: https://lore.kernel.org/r/20240807-ad4113-v1-0-2d338f702c= 7b@analog.com > >=20 > > --- > > Dumitru Ceclan (3): > > dt-bindings: adc: ad7173: add support for ad4113 > > iio: adc: ad7173: order chipID by value > > iio: adc: ad7173: add support for ad4113 > >=20 > > .../devicetree/bindings/iio/adc/adi,ad7173.yaml | 3 ++ > > drivers/iio/adc/ad7173.c | 38 ++++++++++++++= ++++++-- > > 2 files changed, 39 insertions(+), 2 deletions(-) > > --- > > base-commit: 1c61e13d7dc9003662bd7fd6064dfea67e64b014 > > change-id: 20240725-ad4113-baa63ff99245 > >=20 > > Best regards, =20 >=20 >=20