From nobody Mon Jun 8 09:48:25 2026 Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) (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 30899239E7E for ; Sat, 30 May 2026 05:28:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.171 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780118904; cv=none; b=W4cbPdxOxqjLmIIOAy4mBf/K/7Y1L5sGkGNKpckcs26gm6SLmCF+KagZGFI7Xxv/YwB6avsumZHC8aQaKFByI/SdZSOEu95ICX8mO1uhiln0D/XMNJpDFDe0uVe4cnCYD5RMMNmUJ9UdydR4RRi9ciYc3+fwfWxu7nBvrzafzGo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780118904; c=relaxed/simple; bh=MktIe/kyYqBWrJCdG9ppZaqxYKeQyaEQWEmvLUbvBxg=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=iMHaUKjJrKj0lsIu6A+GkmdtsyrtsWLwm2tCbYLaXjGyDFUf1zJFgf+pvS8YHMSZdEyfdPFnpcIxA/WSyWnQQGhfuSyjJvn6UPk0Wv8KbEXWrO/f2l1UvBOnQ3Cs2cIIWTwHFZSIBj6Yi/5h+/89GRlT6HNUEH1h2/LW5KZy8h4= 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=O2CZvf+Y; arc=none smtp.client-ip=209.85.214.171 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="O2CZvf+Y" Received: by mail-pl1-f171.google.com with SMTP id d9443c01a7336-2bf20f6be6bso7958325ad.3 for ; Fri, 29 May 2026 22:28:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1780118902; x=1780723702; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=SmY3471HweDt7C0JbHesY0+I32AoN6OAjy1ZuD0tCcY=; b=O2CZvf+YuIWncvTHFQB1uUXaBHOQ7YWgI0qkqd8FIXtken7nfNhlL5eQsAvpg9+pee nb9eWQOISoyVeCn/YfmnCd962fT5Zj04jPoOXDzZ/PCL3MMmzgR2SUWHp1aI+13/s6le BRQk9+CjfJUHpwC6drl/zmJiLcDHTxQo4oawD5Vlg1LqR6olIFdpLF4RwP7rsTe1acTU +F8qp6lFcAhkSJs3TFjoeEDfltIqYyk7HGR2kAMT1w2O4PIOfJ6o1baXHVcrSozDfMhN pJoMYtCR3q+hSQVvu3FlDLFHRU+3OfOas104ncRWM6rirMSVJktSGssDlG8OIl6Hcxfv P+fg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780118902; x=1780723702; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=SmY3471HweDt7C0JbHesY0+I32AoN6OAjy1ZuD0tCcY=; b=UYgM81JzKgp2MAj5TSyvf2mGzLSXjdyWRQ+HHw3HfdGGaNAkgFeGDevc+N39Cn1ZlR BuDaU94HroUcFK2V/eYKX19PCkyfFei8fp/LG30uNlDMc3vp9upEO4lWxSE6RtuMFxbm g3me3KWxj6i53allVGjIfei8At/5dRyml7dw3y8+QBwas5IgEyVPRs9q2uyb542xHCXk +Ed5pSewNY3932uhdcGUpI/V+oJ7Zo+pzzwPXh0kRxUmGCzq2RwXlkalS3JT//vEmbgF lBMfDFxwBzQtVyr2OJExRObNH46aUhIhYi3pfRaPADf5SjGvTD7JYM3gfNPGjabI6tp4 M3kQ== X-Forwarded-Encrypted: i=1; AFNElJ/oa28ORM1fW9mblxSk/F32O5eXXO++yuuyBx17M14Yr/2/FDv/u9Jx1d4xXymhbb++gDQ9yoru23gHXMU=@vger.kernel.org X-Gm-Message-State: AOJu0Yzy7xaZXZ40MF2V6c4htsLdKnV9D0nGWfWUPAqL6GjMye7/7cWF cXNuSgNHERryNChpMieaX5L/pSz0QPzgxWre3+M2Zpn22I1GsRUcXWn3 X-Gm-Gg: Acq92OEWao9FT3cyl74ogJELDMx1+lJ8KxTAVlICcnO2DZu8lqlQf6OeSv0UNlCs6Zl RjJ27dLezBYusCOW0cTyHg3n5cg4oovQ7LLLSWwACDhy00fI41coGp25FvUvThupKHJ9S9DB4jq jzIyBmhsKkAeoQKozq1kLbQaEk6tblmnByyb7zdUQPkWSZi2Ks9hzMHEaNbOgBh4sw7t0tZvVpm nUmPpiFsV/TyilIPIZWMc867+ZZ60Zs9YO//3TxdEV9l7SifMdf3/3N+FS8+YtY+9fzGAvD6sTb Kf3QZFY1N/isZ/x7xRukkkdzZHDycINHtsJpWABWXBQlEd9UsJ6J0/8QvdKm8JX8x3C/G75UlWB p4rbQRIcM7CfjVjTI6NGrA+hKU/y3MdZeKOBaOv3m7rFZJ718rL7p4NUldptq5Cj+iyqp1lTRMp bcvS2yMDqEk9c6hUx+TcmeNlN+28imvWrAcaHIsINRKqO/g2lLpcohXQyvtB2BuK8wTU9YsnLR4 wu26XCM0IiXkxI= X-Received: by 2002:a17:903:390d:b0:2bc:f202:54d7 with SMTP id d9443c01a7336-2bf3689d80emr35457005ad.38.1780118902331; Fri, 29 May 2026 22:28:22 -0700 (PDT) Received: from Ubuntu.. ([49.37.171.82]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2bf239fd6edsm47386275ad.18.2026.05.29.22.28.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 29 May 2026 22:28:21 -0700 (PDT) From: Manish Baing To: claudiu.beznea@tuxon.dev, lgirdwood@gmail.com, broonie@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, nicolas.ferre@microchip.com, alexandre.belloni@bootlin.com Cc: linux-sound@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, manishbaing2789@gmail.com Subject: [PATCH v3] ASoC: dt-bindings: sound: atmel_ac97c: Convert to DT schema Date: Sat, 30 May 2026 05:28:12 +0000 Message-ID: <20260530052812.115994-1-manishbaing2789@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Convert the Atmel AC97 controller binding from text format to YAML schema. The binding constraints were updated during conversion to align with actual driver behavior: - Added previously undocumented 'clocks' and 'clock-names' properties. These are made required, as the driver unconditionally requests them. - Removed 'ac97-gpios' from the required list, as the driver treats the reset pin as optional, and requiring it breaks existing .dtsi validation. - Fixed the ac97-gpios array in the example to use standard comma-separated formatting. Signed-off-by: Manish Baing --- Changes in v3: - Updated the commit message to include the reasoning behind the property c= hanges,=20 addressing Sashiko bot feedback. Changes in v2: - Added undocumented 'clocks' and 'clock-names' properties which are unconditionally required by the driver. - Removed 'ac97-gpios' from the required list, as the driver treats it as optional, fixing validation for existing dtsi files. .../sound/atmel,at91sam9263-ac97c.yaml | 70 +++++++++++++++++++ .../devicetree/bindings/sound/atmel_ac97c.txt | 20 ------ 2 files changed, 70 insertions(+), 20 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/atmel,at91sam92= 63-ac97c.yaml delete mode 100644 Documentation/devicetree/bindings/sound/atmel_ac97c.txt diff --git a/Documentation/devicetree/bindings/sound/atmel,at91sam9263-ac97= c.yaml b/Documentation/devicetree/bindings/sound/atmel,at91sam9263-ac97c.ya= ml new file mode 100644 index 000000000000..5f4ebe81bf90 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/atmel,at91sam9263-ac97c.yaml @@ -0,0 +1,70 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/atmel,at91sam9263-ac97c.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Atmel AC97 Controller + +maintainers: + - Nicolas Ferre + - Alexandre Belloni + +allOf: + - $ref: dai-common.yaml# + +properties: + compatible: + const: atmel,at91sam9263-ac97c + + reg: + maxItems: 1 + description: Address and length of the register set for the device. + + interrupts: + maxItems: 1 + description: Should contain the AC97 interrupt. + + clocks: + maxItems: 1 + + clock-names: + const: ac97_clk + + ac97-gpios: + minItems: 3 + maxItems: 4 + description: | + AC97 link GPIOs- sync, sdata_out, reset, and optional sdata_in. + The driver primarily uses the reset pin. + + pinctrl-0: true + + pinctrl-names: + const: default + +required: + - compatible + - reg + - interrupts + - clocks + - clock-names + +unevaluatedProperties: false + +examples: + - | + #include + #include + #include + + sound@fffa0000 { + compatible =3D "atmel,at91sam9263-ac97c"; + reg =3D <0xfffa0000 0x4000>; + interrupts =3D <18 IRQ_TYPE_LEVEL_HIGH 5>; + clocks =3D <&pmc PMC_TYPE_PERIPHERAL 2>; + clock-names =3D "ac97_clk"; + pinctrl-names =3D "default"; + pinctrl-0 =3D <&pinctrl_ac97>; + ac97-gpios =3D <&pioB 0 0>, <&pioB 2 0>, <&pioC 29 GPIO_ACTIVE_LOW= >; + }; diff --git a/Documentation/devicetree/bindings/sound/atmel_ac97c.txt b/Docu= mentation/devicetree/bindings/sound/atmel_ac97c.txt deleted file mode 100644 index b151bd902ce3..000000000000 --- a/Documentation/devicetree/bindings/sound/atmel_ac97c.txt +++ /dev/null @@ -1,20 +0,0 @@ -* Atmel AC97 controller - -Required properties: - - compatible: "atmel,at91sam9263-ac97c" - - reg: Address and length of the register set for the device - - interrupts: Should contain AC97 interrupt - - ac97-gpios: Please refer to soc-ac97link.txt, only ac97-reset is used -Optional properties: - - pinctrl-names, pinctrl-0: Please refer to pinctrl-bindings.txt - -Example: -sound@fffa0000 { - compatible =3D "atmel,at91sam9263-ac97c"; - pinctrl-names =3D "default"; - pinctrl-0 =3D <&pinctrl_ac97>; - reg =3D <0xfffa0000 0x4000>; - interrupts =3D <18 IRQ_TYPE_LEVEL_HIGH 5>; - - ac97-gpios =3D <&pioB 0 0 &pioB 2 0 &pioC 29 GPIO_ACTIVE_LOW>; -}; --=20 2.43.0