From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ot1-f48.google.com (mail-ot1-f48.google.com [209.85.210.48]) (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 E070713D607 for ; Thu, 13 Jun 2024 14:39:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718289564; cv=none; b=FDXllYtemHXDU0IIqa8mSh6vbDLSFf5Y3yzuUH2sfRkDtTiWx/4+eLdrx22FG9ap8KtSCohlpPsV+ImWA57VuJxZYkk/kjS+0TDVunj9lDvRDZVR/q/ZTG9Xozum0a0qE5CV/Ml2H2tEVRUU/DhxM6V/WU9I2RuOXbMe03AcMMs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718289564; c=relaxed/simple; bh=TGYwQdD0hydwVFCQEXTmPl0v48jG53So9o6AmWcYSLY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=kzVdZdwK+9GM4qe2HK2IxzTc6HL+ULKUP5YMRGztGnqINhYZH/wnckLYB1hXkw7Ba1VlOq9zZD/ePegu5W6Sg9A0Qs9uGxL8Lf9jFUVcvFqqCqMTwTe7158+cQEfaV6v9KD2H/Y40oaFjKs2BhbeQZhqIIrIOIIxDoJ7Gm8/b3o= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=RQESr830; arc=none smtp.client-ip=209.85.210.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="RQESr830" Received: by mail-ot1-f48.google.com with SMTP id 46e09a7af769-6f97a4c4588so602420a34.2 for ; Thu, 13 Jun 2024 07:39:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1718289562; x=1718894362; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=0N4XXQrV3Vj1p76CTTLMvxUKN2Z68IrEsu0illIw4/U=; b=RQESr830Y121jVe87be5ALDhgW/cxS+UIt8XUXNxzL7HJaprA4+itpz5lDxjZ7uUYu KSfdHKxslikVYRaMqEApdvn3AnEqww8FJIH88pHxS/hp2T/6Ra5o0/3jdIuELwi4VpOK TN/vJweO7iMO0nEegMFKVvfKaa+XYz3cHEkGgvVOf8Pf1E4/oi2Kgn8qwMRagbxWEHnr AlNfKlS6D/tCSUzMmQ8kfD4wLQsZIO0tt0hbberkbt0rV2wdLcfhCsMqdD2y9x0KE2vd FjJkrzzrEkgYQwD17YqnahGAc5010Ibz+jTJu+4fMCe01c1ZPWDyka1wiEvKEPagxzS3 AhKw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1718289562; x=1718894362; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=0N4XXQrV3Vj1p76CTTLMvxUKN2Z68IrEsu0illIw4/U=; b=E7h1Ld5F5nUpG2EPMRYe30RovO+zRcZVdIc1lBY/ECOH0Rm+2+rt5pTHdIoBAKFTtO WnMn44W//HfXLn8mD1GxNKIckjwm2ADSPHOaL+OWcTfj7hMo+YGVUxy2bItX9QF+80uF 83RPoGrMtEGgWdjBahmHFXeiGpCvfsB5Njn7TT7EV25/IVu8cXb6mT7HEFY7D12kAE4Z oQ1kTRHap/tH+fO4z5vkQYq/jIJvGtXjd/Ja8+ZLj/irJEWuReHkNV9euMX5Ofx1jjzD hBJvTuhWB3oK6hHJJ3wfAn/r9qjpQthXqJMmKbECRaOhftHu5OIOW9dif/NqmDhEg/+6 aExA== X-Forwarded-Encrypted: i=1; AJvYcCUekeu7vg6TAfBvbEX8tpgGyuzvuCzi6xPyXpyRHAR5QXtw8sq9hM7dKRp/NsHV1ydq/chzkdDJoCsGkHRLQGS8ASAG/UvvhKcQ X-Gm-Message-State: AOJu0Yz908dPR13uYMe7gXK0K2KzBpe6B1ZKYyldD7/llBjPrFWJEQqj le876f8mBQITP0JCSww8nnJx/UNu7CireOra+xQz1a3RRm4jL55TAoxsB73BrFs= X-Google-Smtp-Source: AGHT+IG95TqU5M84bOISdVcjzil531JdHikT5kRxpc3XBVOTexFZ4lLkN7JG2P7LUMplLqCWy/iU6w== X-Received: by 2002:a05:6830:603:b0:6f9:c46e:af75 with SMTP id 46e09a7af769-6fa1bf726a8mr4806080a34.11.1718289562014; Thu, 13 Jun 2024 07:39:22 -0700 (PDT) Received: from [192.168.0.142] (ip98-183-112-25.ok.ok.cox.net. [98.183.112.25]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-6fb5b8651d7sm236097a34.81.2024.06.13.07.39.21 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 13 Jun 2024 07:39:21 -0700 (PDT) Message-ID: Date: Thu, 13 Jun 2024 09:39:20 -0500 Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/3] dt-bindings: iio: adc: add AD4695 and similar ADCs To: Krzysztof Kozlowski , Jonathan Cameron , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Michael Hennerich , =?UTF-8?Q?Nuno_S=C3=A1?= , Jonathan Corbet , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org References: <20240612-iio-adc-ad4695-v1-0-6a4ed251fc86@baylibre.com> <20240612-iio-adc-ad4695-v1-1-6a4ed251fc86@baylibre.com> <94448c2c-e7b2-4191-858c-529b254994f1@kernel.org> Content-Language: en-US From: David Lechner In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 6/13/24 9:18 AM, Krzysztof Kozlowski wrote: > On 13/06/2024 15:57, David Lechner wrote: >> >>> >>>> + - const: adi,ad4695 >>>> + - items: >>>> + - const: adi,ad4697-wlcsp >>>> + - const: adi,ad4697 >>>> + # same chips with higher max sample rate >> >> I suppose one could make the argument that the programming model is >> the same on these too, but the maximum sampling frequency does seem >> like an important bit of information so that you don't try to set >> the conversion trigger rate too high. >> > > which property is that? I don't see differences in the driver, so I > don't get how these wlcsp compatibles allow you to control value of > conversion trigger. This comment is unrelated to the package type (WLCSP or LFCSP). What I mean is that e.g. AD4695 and AD4696 are virtually identical other than the maximum allowable sample rate (500 kSPS or 1 MSPS). So my thinking was that it would make sense to have: compatible = "ad4695"; for the lower sample rate chip and compatible = "ad4696", "ad4695"; for the higher sample rate chip since ad4696 can do everything that ad4695 does plus a bit more. We haven't implemented buffered reads in the driver yet, so there isn't anything currently to be seen there. But when we do, we probably want to limit the allowable value for the sampling_frequency attribute based on which version of the chip is present. (I would like to get Jonathan's opinion of if this is something we actually need to do or not.)