From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oa1-f41.google.com (mail-oa1-f41.google.com [209.85.160.41]) (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 470F64DC52B for ; Thu, 3 Sep 2026 15:30:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788449405; cv=none; b=oquSdcYnLCk4SOBU2YvQgl4MdHhlzlOaRGFTiPWakHscbcnD/RUN25EBosC3lYT+oKkqaH0ileUTUz4/ZLkVWQJwqdx30zX0Sl4GD6zr631s7qGxqDSYEKKaiDUXXILQ8inTkjMbKvARW25uGY4MxVLWuuFzfJUvexO6FkjKEz0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788449405; c=relaxed/simple; bh=V65lDMJi4pB0ovS/PwcrWiReO5oAmLVmduccTtJtLR8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=nGimlIGSr3ERiXnawGXLV7KsiLgDs17oEkLUSRbDHQd6W52kuU/3sfDTchYtfvYBExyhUk6I+bmh4WRbq5uTBrwiCdhEXZ43kX/zcssMtkjbqK/tEHmscI/QTGtWNE1c/FXBs6o/5hvLLUIV6DEGGiBrcJbTO7jp0H3dxeDm3+I= 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 header.i=@baylibre.com header.b=KouNON8K; arc=none smtp.client-ip=209.85.160.41 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 header.i=@baylibre.com header.b="KouNON8K" Received: by mail-oa1-f41.google.com with SMTP id 586e51a60fabf-46ad7304688so1158420fac.0 for ; Thu, 03 Sep 2026 08:30:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1788449403; x=1789054203; darn=vger.kernel.org; h=content-transfer-encoding:content-type: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 :content-type; bh=xLiFaZ2E/NhzIjkyqhk0xpKIELhAH9ZpSaji+KDMIeg=; b=KouNON8KR+MtWPdB1YNPqf/YDOk/xWk+QSjARjyFno/oURjCiLi9Jr+T+UqIiTiALH /INrXb1m69DDbaMSGlAPeIkjb9CwT02KPbegfs4H9FLlcMRUPfjF2/8ophoSTHjJ8EWl oUQALiIWfJymm7y8i/xlFLuiTJRkaX8J5n+UDiDJ4eLhT79FX1I1dbpAtDHPyo5kn7At Y1mq1AsgcANuIwuZ19OnF1Haobq9tyQDfvbTesq9OreDKQTjAbPltoAjW0jLi58q9BMy +vsaOinTYjfPyf0xmLehzRWOviWp363hhjxZZ+WqbRqQkhuH1AHscy50F12unu9GBJRe xq0g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788449403; x=1789054203; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=xLiFaZ2E/NhzIjkyqhk0xpKIELhAH9ZpSaji+KDMIeg=; b=qawCrolrG7QBT0agYU6QD09Lppi3SRM4o6g48EQVup7U3Ca24uF8NArrTllnST5cQT BAwDdsQN4ftb9HOWJTYwVRt02wI00lyJSgnmCQGM8YrYL/WXp1LXOlDuVfjByqW4M1YZ sKwPsZsEZ8LXKCB88Fo2gpESD1+gMoMycbtcawoKCAusbO+lzoyHtPaOu9RggXflw9UT 2xfNFAvks/ZK1BtJRkdF9JrqAB8y4ybMG4+j6yMy6pDi1+eLHjpMZae0x9akLkDse9Q5 2qbVwU6NM9cKps3OFov8FPMp1jxIDdYNe4FOM8KwGnFgCdkfgeao3uaD3U4lQb0ZaF93 1h5A== X-Forwarded-Encrypted: i=1; AKwUvBxKolAJge6PCJEwvF2s2AgOdMzPmeJshqKA+eIvhC2KnqXnCBaPG1j2X5OoY7NtyQVlqIkoXhvXzVLxD0w=@vger.kernel.org X-Gm-Message-State: AFuF++kOLDOKY74A06p+wP/2DtB86LzjVLA/KUpABSCEb4EMKXX2GmWS fT2jQ9uMdufywNb6OxyKWjCg8/w3RVW58g797+ckid9oZMCDeJqjavIujknrAYWfwKo= X-Gm-Gg: AYBFou3QPLUAaWNGHVTNl94QldV63PpgHOAFyY2HSG2lSLZeFL7nxVlBiCXHxYo8eR0 8d6rhaMogy1xZbLiknnzJ8YMws2wAgdJP4OnpucxTZpTYrPvLXF+Bv26OmfhoKRgycGk/e+vXTk FHxVr6avpVVLjYfCtLzU6MbrAFp4CaHC2mX8caYsNE6fmQsIz+g0QORU4kWgvWQV6HmMJFPFdcE MK+6Zc9K8TQEwlcear3c0uAY/IKabFDXbSo/o3sKg7G/QC8t0U5xYz1nwIoWM8TNsdvhXeJKJ0l jOZuVjPkfsmsB9VZIyNe0aFDofRAvsW5doHlG44gL1PGwmgguxF3d2Je/YssRv9D+wIuu8auWwr /t70Moj2wtR1uhJ+mK85gln6PCA48hjnj8/hCZ2COfTfbi/lHTR/18dDg6i6cNmStDohM/trRR1 z4rTC9mbQxOkmRI0A6qdZztZGxVYGcrVXscuUyjFdGaYJAexglnG6ENCl8INDPcXCV26VYOb1nF ssXO9HHDpaPTBGcRQW6mMJUibW1uyRe84aeTH9O+5C1VPA= X-Received: by 2002:a05:6820:810:b0:6b1:aa44:fc27 with SMTP id 006d021491bc7-6b5ef34b541mr1496418eaf.5.1788449402980; Thu, 03 Sep 2026 08:30:02 -0700 (PDT) Received: from ?IPV6:2600:8803:e7e4:500:e24:471f:c835:8147? ([2600:8803:e7e4:500:e24:471f:c835:8147]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-6b6dbedc38bsm137817eaf.4.2026.09.03.08.30.02 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 03 Sep 2026 08:30:02 -0700 (PDT) Message-ID: <61d64649-603d-4a55-a789-babe655a2de8@baylibre.com> Date: Thu, 3 Sep 2026 10:30:01 -0500 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 1/2] dt-bindings: iio: adc: ti,ads1015: Add label property To: Flaviu Nistor Cc: Andy Shevchenko , Jonathan Cameron , Nuno Sa , Krzysztof Kozlowski , Rob Herring , Conor Dooley , Daniel Baluta , Antoniu Miclaus , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org References: <60dc58c6-dfe2-4377-9a73-f65cda56179c@baylibre.com> <20260903151939.5117-1-flaviu.nistor@gmail.com> Content-Language: en-US From: David Lechner In-Reply-To: <20260903151939.5117-1-flaviu.nistor@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 9/3/26 10:19 AM, Flaviu Nistor wrote: > On Wed, Sep 2, 2026 at 9:31 PM, David Lechner wrote: >> On 9/2/26 12:22 PM, Flaviu Nistor wrote: >>> Add support for an optional label property per channel similar to other adc >>> chips. This allows assigning distinct names for each channel which can >>> match the schematic signal name. >>> >>> Signed-off-by: Flaviu Nistor >>> --- >>> Documentation/devicetree/bindings/iio/adc/ti,ads1015.yaml | 6 ++++++ >>> 1 file changed, 6 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads1015.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads1015.yaml >>> index 718f633c6e04..777a75c21d7f 100644 >>> --- a/Documentation/devicetree/bindings/iio/adc/ti,ads1015.yaml >>> +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads1015.yaml >>> @@ -66,6 +66,11 @@ patternProperties: >>> - minimum: 0 >>> maximum: 7 >>> >>> + label: >>> + $ref: /schemas/types.yaml#/definitions/string >>> + description: >>> + A descriptive name for this channel, like "vcc_ram" or "CH3". >>> + >> >> Apparently this chip has quite old dt-bindings. Normally, we would have >> included adc.yaml to get this and other channel properties. >> >> And we would have used diff-channels and single-channel from that to >> describe which pins are associated with which channel rather than having >> a magic number for reg. And the ti,datarate looks suspicious as sampling >> frequency is usually we control at runtime. >> >> Anyway, none of that is really relevant to this patch. We'll see what >> Jonathan says about using adc.yaml here vs. adding a label property. >> > > Thanks for your feedback. I can also use: > > $ref: adc.yaml > unevaluatedProperties: false > > and move additionalProperties: false at the end of patternProperties block. It can't have both unevaluatedProperties and additionalProperties (hopefully the dt_binding_check would catch that). > Any other idea? > > Best regards, > Flaviu Nistor > >>> ti,gain: >>> $ref: /schemas/types.yaml#/definitions/uint32 >>> minimum: 0 >>> @@ -125,6 +130,7 @@ examples: >>> reg = <4>; >>> ti,gain = <3>; >>> ti,datarate = <5>; >>> + label = "CH3"; >>> }; >>> }; >>> };