From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oo1-f67.google.com (mail-oo1-f67.google.com [209.85.161.67]) (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 6E60B35CBB2 for ; Sat, 31 Jan 2026 19:46:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.161.67 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769888793; cv=none; b=aVUE1CoYBuam1cdUVYwJyE2EfRhQQLgvvsCZcE3oqAuYWHiz3endtdyYKFRaXhiuFyGU/bZaxBWZOCkOagBbsHIfQVYzfLLxFNQaT1YQKbXNC13Ag5BTylqKC3ujC70z+jQvW8PG1rhkCbo7Vj0rNyNWneMpLXJ6D/JLDnYkVhc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769888793; c=relaxed/simple; bh=K8YQX8LaOueyeKPe9xY7pRAyIGpZfWO0T3uKpztWwW0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=aJ5wZa4IQCMBMvuaprqRqQ//ubJJeyA8z3nzraS1XdhAC/A2Hj6yqQJi2iLPgtq0BdGrzEmguBS/JLqsE73iq9oSu38I8ZZ1UBWtZkGrzfzfDcRuPBxQu4cK6RDqom5zVdW5M1hQsK3IBQLU5hHA03pN3cCIu2Hzk14lcH/HG9s= 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=sAcfGwyH; arc=none smtp.client-ip=209.85.161.67 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="sAcfGwyH" Received: by mail-oo1-f67.google.com with SMTP id 006d021491bc7-662efd1bdd4so1861051eaf.0 for ; Sat, 31 Jan 2026 11:46:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1769888790; x=1770493590; 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=USE7814nD7qzzRAMgxZuZkB6FtznoRYP0SohwFcy9+I=; b=sAcfGwyH1OaXM2tfj6vThUweIkIeXGNDFBb1OUzUrMtHzF7y9szC/6DLBEcu+wG9a7 llDsdNxciCcUv54eHZWzQOWwjuD1aHzfJuGrNqU6TzlkY1QuPKAW7B03VCSGKVGaZ4WP QCw2fHOM0e2eFx4mmVmS4T84h7NImteSmA+teYFRg/DaCrQspUDe4iL8B0MT7ES4gxB3 XsVhyKe8syqNUxQ8LSyBL9JV8ZDoGPIyJhsfBpIcLelMOoEUAoeHbi2XjMgE+7X74tyy SOxgQ/HxpsXgeyM+w2vHYizIttm3bq3dAkfsO5DHimJDdlYCrXp1M/Sv4wgos6OqEMn/ WEDw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769888790; x=1770493590; h=content-transfer-encoding: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; bh=USE7814nD7qzzRAMgxZuZkB6FtznoRYP0SohwFcy9+I=; b=F/6JK4vHx+KTBKZLTI60OpEomLKLd7DgQjnSVaWRtwhHQDkbvSAAfTinIZPoQDqfFo /ES1+8JqN+qvO6RsOvZNIneYSKxS/Bw6iotsBMXPtfzfc6BacRGvL/y5wIwMeZoifmBw 9wxjbGomwDyX9H1MsNoXhxbdncNlSsclJtyd+Q/0GvlTuicwCTQx0dWQUNesWXtZRy7E i3Ef1JsUZ0/B4FcFOFXEHH90uTEdBS6QX4tOMCz/1uIXzgRgfIoivsER0TPG49mzwegB vM0X+WWOz1xV+rB20RLYf6Jiwmo3zFCbz65s6ps/QXxZjLY1SFZ3wysKkarP5KWNADd2 PcfA== X-Forwarded-Encrypted: i=1; AJvYcCUyje76t56GPnGHG3N4vn8ePBXcsizkwPQscUGdk3OnBGE1DDNXW1isvqNTMCl/AmENCqvLtjoCz8jsL9w=@vger.kernel.org X-Gm-Message-State: AOJu0YwxYNvUjDsGrAj5JJVE95LtzGeQUI9d9br8OduDs0tEBJCKRlZ7 ZCSEgp7qOlfyqbqPcsvywiBMzr0T/VfOy71HT0WS13KUi1jgLWgposj+HAmaVXXGCAU= X-Gm-Gg: AZuq6aITb7tLPmVFJkYd7A7cSLggjYFghr9bR3+9ZW6X8RXjdj1DTfno6aH2RYmWptC hk/e3PvzvOTc47ZXKmVhwwg++TjDLzMfR8eO6kUdjH9wMydiwS+PUKkL8s3iL9Tl1Ur3AzMeHVo j8BqPyCHV3zQAX26TcYWCyPh3cqgB0Gou+QdLI5y7vtFpcFuLTJZw5O7XsGlZFTbiid45qEpNnu tfPKRvLWruHj0t765Gq4Bz4+6j6vkte/ybWtvgRitUvvWMnOrQUId4zjqFxkrmq/T/rktPsBxOs uOu3IbL5gPWebI40q212YmRDLRM90AJfDJP9a3uYiiQYmAQK/jVbA7U3SbGbtBKG3gto+6Ma5ve rE92KeZoWc0qy5JBIh5X5EcCZmmCgU6kx6eYqiyWcCdRcXe6fl4Se0r70ekD1xRQwztQAEjmVSX SI48GzqJUP9loW+j18/OaIdEgPW9UmidNjIDd/l5evfj4z3J8mgN/6rGBaL3HL X-Received: by 2002:a05:6820:4810:b0:661:1bee:284a with SMTP id 006d021491bc7-663103a4b9amr2983263eaf.28.1769888790423; Sat, 31 Jan 2026 11:46:30 -0800 (PST) Received: from ?IPV6:2600:8803:e7e4:500:c7ef:51f2:4f7a:6cd7? ([2600:8803:e7e4:500:c7ef:51f2:4f7a:6cd7]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-662f9a18776sm6975873eaf.12.2026.01.31.11.46.29 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 31 Jan 2026 11:46:29 -0800 (PST) Message-ID: Date: Sat, 31 Jan 2026 13:46:28 -0600 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 v1 01/10] dt-bindings: regulator: cpcap-regulator: convert to schema To: Svyatoslav Ryhel , Jonathan Cameron , =?UTF-8?Q?Nuno_S=C3=A1?= , Andy Shevchenko , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Dmitry Torokhov , Lee Jones , Pavel Machek , Liam Girdwood , Mark Brown , Alexandre Belloni , Dixit Parmar , Tony Lindgren Cc: linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-input@vger.kernel.org, linux-leds@vger.kernel.org, linux-rtc@vger.kernel.org References: <20260125134302.45958-1-clamor95@gmail.com> <20260125134302.45958-2-clamor95@gmail.com> Content-Language: en-US From: David Lechner In-Reply-To: <20260125134302.45958-2-clamor95@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 1/25/26 7:42 AM, Svyatoslav Ryhel wrote: > Convert devicetree bindings for the Motorola CPCAP MFD regulator subnode > from TXT to YAML format. Main functionality preserved and added compatible > for CPCAP regulator set found in the Mot board. > > Signed-off-by: Svyatoslav Ryhel > --- > .../bindings/regulator/cpcap-regulator.txt | 35 ------------- > .../regulator/motorola,cpcap-regulator.yaml | 51 +++++++++++++++++++ > 2 files changed, 51 insertions(+), 35 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/regulator/cpcap-regulator.txt > create mode 100644 Documentation/devicetree/bindings/regulator/motorola,cpcap-regulator.yaml > > diff --git a/Documentation/devicetree/bindings/regulator/cpcap-regulator.txt b/Documentation/devicetree/bindings/regulator/cpcap-regulator.txt > deleted file mode 100644 > index 36f5e2f5cc0f..000000000000 > --- a/Documentation/devicetree/bindings/regulator/cpcap-regulator.txt > +++ /dev/null > @@ -1,35 +0,0 @@ > -Motorola CPCAP PMIC voltage regulators > ------------------------------------- > - > -Requires node properties: > -- "compatible" value one of: > - "motorola,cpcap-regulator" > - "motorola,mapphone-cpcap-regulator" > - "motorola,xoom-cpcap-regulator" > - > -Required regulator properties: > -- "regulator-name" > -- "regulator-enable-ramp-delay" > -- "regulator-min-microvolt" > -- "regulator-max-microvolt" > - > -Optional regulator properties: > -- "regulator-boot-on" > - > -See Documentation/devicetree/bindings/regulator/regulator.txt > -for more details about the regulator properties. > - > -Example: > - > -cpcap_regulator: regulator { > - compatible = "motorola,cpcap-regulator"; > - > - cpcap_regulators: regulators { > - sw5: SW5 { Old example is missing the required regulator-names property. > - regulator-min-microvolt = <5050000>; > - regulator-max-microvolt = <5050000>; > - regulator-enable-ramp-delay = <50000>; > - regulator-boot-on; > - }; > - }; > -}; > diff --git a/Documentation/devicetree/bindings/regulator/motorola,cpcap-regulator.yaml b/Documentation/devicetree/bindings/regulator/motorola,cpcap-regulator.yaml > new file mode 100644 > index 000000000000..b73d32a86904 > --- /dev/null > +++ b/Documentation/devicetree/bindings/regulator/motorola,cpcap-regulator.yaml > @@ -0,0 +1,51 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/regulator/motorola,cpcap-regulator.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Motorola CPCAP PMIC regulators > + > +maintainers: > + - Svyatoslav Ryhel > + > +description: > + This module is part of the Motorola CPCAP MFD device. For more details > + see Documentation/devicetree/bindings/mfd/motorola,cpcap.yaml. The > + regulator controller is represented as a sub-node of the PMIC node > + on the device tree. > + > +properties: > + compatible: > + enum: > + - motorola,cpcap-regulator > + - motorola,mapphone-cpcap-regulator > + - motorola,mot-cpcap-regulator > + - motorola,xoom-cpcap-regulator > + > + regulators: > + type: object > + > + patternProperties: > + "$[A-Z0-9]+^": Why not put the valid names here? Or does the node name not actually matter? (in which case lower case could be allowed too.) "^(SW1|SW2|...)$": And $,^ are swapped. > + $ref: /schemas/regulator/regulator.yaml# > + type: object > + description: > + Valid regulator names are SW1, SW2, SW3, SW4, SW5, VCAM, VCSI, > + VDAC, VDIG, VFUSE, VHVIO, VSDIO, VPLL, VRF1, VRF2, VRFREF, VWLAN1, > + VWLAN2, VSIM, VSIMCARD, VVIB, VUSB, VAUDIO > + If these apply to the regulator-name property, it can be written instead as: properties: regulator-name: enum: - SW1 - SW2 ... Not sure if it is strictly needed, but this would document the optional property: regulator-boot-on: true > + required: > + - regulator-name > + - regulator-enable-ramp-delay > + - regulator-min-microvolt > + - regulator-max-microvolt > + > + unevaluatedProperties: false > + > +required: > + - compatible > + > +additionalProperties: false > + Example should go here. > +...