From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ot1-f41.google.com (mail-ot1-f41.google.com [209.85.210.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 1A7F73AB48F for ; Tue, 12 May 2026 14:13:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778595199; cv=none; b=nxjHg2aZAM69ebP+PnBKW1G9PS6gPQCHOFSJTVxV7HFRKk+rXvv8lhXgBS+pUtrZlcRwLvWU9yi+KxtqRIskBBj5m+CwLJ4HXorblbVeUcd/356gkyqw6YW4SZdSnrQAjWbwnpJLPmztQHV/XPTak6s+KJBuZFuoR0qQpghjLEY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778595199; c=relaxed/simple; bh=Dq7Xn2iwHrokGpyReWmPWnvaDDe9J7rZKMtFuqJTkzs=; h=Message-ID:Date:MIME-Version:Subject:From:To:Cc:References: In-Reply-To:Content-Type; b=ZJewzHU3bCvRf6ckCeL3a02OCwYOWYGcs8NwGvntTQ/6dqBW03Q3K08m9pMpQC6lStGqIl2QT20pf0q8eijODT4Ydz8x4A68I5TB8xdSbqrnWjYfpl1YFa2zrbcr1aUt/m5+rvbibPfGUZx4xmB5ze6XFTifCdSZ9gIsLi/impM= 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.20251104.gappssmtp.com header.i=@baylibre-com.20251104.gappssmtp.com header.b=wbjqGHqV; arc=none smtp.client-ip=209.85.210.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.20251104.gappssmtp.com header.i=@baylibre-com.20251104.gappssmtp.com header.b="wbjqGHqV" Received: by mail-ot1-f41.google.com with SMTP id 46e09a7af769-7de4a9cb8eeso4748441a34.0 for ; Tue, 12 May 2026 07:13:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20251104.gappssmtp.com; s=20251104; t=1778595194; x=1779199994; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:content-language:references :cc:to:from:subject:user-agent:mime-version:date:message-id:from:to :cc:subject:date:message-id:reply-to; bh=kgNJhNIeorV7YFB4VYbzC5DWm0skgWvwJAbHsFTJdPA=; b=wbjqGHqV3gvG324B/2jF+Ak3z0wntnnoBWrQK8QCPEjhoaZa8Ivfz4uTCbvV833ary 0gx0t03uRwfnSEX7zuTOxKDkGBHMEencVgQX7WU5vMfWlh7Jnly3gutnbYMr/gjjgXKq z7MdUK2YdvHr01ttKPadyb1uTOn37Pxi+zdrPh+QEkKee0qe4U0pl3tB+65TJXGsqKSK zOfEX2xNXMs/wDoKy2xrXeDFYL9BAVRUibTBHMBWNajU4W9/Gr0Q6PtouDwxVG1Cyn/2 LVy5V+uMrO9+Gcrzqhb6oEaAR3yFVR0hr5KYsLdNXG2C4lTk+ViNswfNlyzr1TJg5LwN eYEw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778595194; x=1779199994; h=content-transfer-encoding:in-reply-to:content-language:references :cc:to:from:subject:user-agent:mime-version:date:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=kgNJhNIeorV7YFB4VYbzC5DWm0skgWvwJAbHsFTJdPA=; b=UWA24tzoapki+kkpDLgMqcPqdNRX9xPDEq63FNuh12cYZW4Cha55pE24OL4S6me0wJ f88S4tx/q419ZFZuXXI3GSiPm0G1dwIFDfp4M3bFl76eQXmcprNz/ldlM/my7dhndtdH pb0TwOXvoA6vRWHZDufqGXD70MYsakn9Y4hFJ4LsO+DlR2JTh4xY/twVdWXwFJk1gZ+y ZGH91QxbN6UkUHDsW6FvcmpSVydA+Mxspn9r3oRhQpRfNLRcjVLu3CCOdD3TCTkIiDOH GZs8PzrQ2+LzniOiUPAk8QOTH+i1aQKme7XHZd5XXS4DTh09AM3jzoyTbxfdx2vC0gr/ L20A== X-Forwarded-Encrypted: i=1; AFNElJ8Uv6AzkZ431SW8xD3MMKlcuHVdPmYD1EOL/9s3qxsa44kWjY6BJHJxZtvcwX3IiCFcEuCt6DCyV4VCwew=@vger.kernel.org X-Gm-Message-State: AOJu0YwA0CTJ0HoeNHvzvI/fuLrsXD57R6XmIgJMjFyPlxoP8cQwT72G ab7BxtfOnKb8CvOH0+kkhKuAsxwfKBzpClcoAsbqzU3ElKtKqt9l4IaAxSz5ANHdcnE= X-Gm-Gg: Acq92OGRTvYq0OCpYlvq122if+dyKEVzlVYsjQQTlxWIetv7JhV2x9LDs7hyh7CRVOH ziLseLbhBEj9LQi4U9moiUAzfh6hZR546h/3divuAUfJCEiGmohlWCtEd9TQdbwcW12o97xqpSd 1sxMVHqpdKkwwlMBSE5wMHtvgVJawy5DAAagizLQ7sAF0jrZH4Z+YDOYerg9g0Az70Qacaa2oV9 YWu2F61olIjbGKvAk8ZmMriwu8roWSZNKY2owbvYCrLjamEqxoM/etiJA0c94CWJypcUbmmHcUW BsKLzsESpgDXTDaofG/DiJ/VM/KV8L4WQEODuBYsiLrUChRkIVaRU7VYfxhaFGYHzKm3yO2dQbo 4S8xtJhuZh5CMg/ACxykt25gNYneFVNqEsdsTE3lSSBoTnJ9yQ+2OyDR75DxUFjF3PVVqkFrnTx AMh133H3ptfuDCS/EMAYBYuNqHoMIP//aq8rT4EuEuBI/hCYNhwMjcVJyHp9X1D5HQPbTJaAf5T g== X-Received: by 2002:a05:6830:dcc:b0:7db:a184:2b39 with SMTP id 46e09a7af769-7e3c3d98299mr1933172a34.11.1778595194041; Tue, 12 May 2026 07:13:14 -0700 (PDT) Received: from ?IPV6:2600:8803:e7e4:500:baa4:f7fb:528a:2457? ([2600:8803:e7e4:500:baa4:f7fb:528a:2457]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7e367be1168sm8998431a34.5.2026.05.12.07.13.12 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 12 May 2026 07:13:13 -0700 (PDT) Message-ID: Date: Tue, 12 May 2026 09:13:12 -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 v3] dt-bindings: iio: adc: Convert xilinx-xadc bindings to YAML schema From: David Lechner To: Rob Herring Cc: Jonathan Cameron , Pramod Maurya , =?UTF-8?Q?Nuno_S=C3=A1?= , Andy Shevchenko , Krzysztof Kozlowski , Conor Dooley , Michal Simek , Lars-Peter Clausen , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20260510083219.70224-1-pramod.nexgen@gmail.com> <20260510120141.118057-1-pramod.nexgen@gmail.com> <20260511171554.6541042b@jic23-huawei> <7baf9ca5-50ff-4131-995b-70ee094ed247@baylibre.com> Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 5/12/26 8:58 AM, David Lechner wrote: > On 5/12/26 7:14 AM, Rob Herring wrote: >> On Mon, May 11, 2026 at 11:24 AM David Lechner wrote: >>> >>> On 5/11/26 11:15 AM, Jonathan Cameron wrote: >>>> On Sun, 10 May 2026 08:01:36 -0400 >>>> Pramod Maurya wrote: >>>> >>>>> Convert the Xilinx XADC and UltraScale System Monitor device tree binding >>>>> from the legacy plain-text format to a YAML schema, enabling automated >>>>> validation with dt-schema. >>>>> >>>>> The new binding covers the same hardware and compatible strings: >>>>> - xlnx,zynq-xadc-1.00.a (ZYNQ hardmacro) >>>>> - xlnx,axi-xadc-1.00.a (AXI softmacro) >>>>> - xlnx,system-management-wiz-1.3 (UltraScale System Management Wizard) >>>>> >>>>> Signed-off-by: Pramod Maurya >>>> Hi Pramod, >>>> >>>> Something went wrong with your sending of v3. I have two versions sent >>>> half a day apart and no idea how they are related. >>>> >>>> Anyhow one of them got feedback from Rob's bot so I'll assume we are >>>> getting a v4 and wait for that. >>>> >>>> Jonathan >>> >>> I think Rob will have to fix the bot to make an exception for the >>> legacy bindings. This should have been called out in the commit message >>> as requested in a previous revision. >> >> The bot is not the problem. It just runs validation. The schemas will >> have to either drop this check (comma's in nodenames) or exclude just >> this property. >> >> >> Rob > > Even though this is an existing text-based schema that has been around > for 12 years with this name already? Changing it could be a breaking > change to existing users. Although there aren't any in any .dts in the > kernel source. Pramod, Unless Rob changes his mind, the thing to do would be to use the standard channels property from adc.yaml when converting the binding instead of xlnx,channels. We will also need to add handling in the driver for "channels" while preserving "xlnx,channels" handling for backwards compatibility. xlnx,channels will just be undocumented. And all of this reasoning must be explained in the commit messages.