From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f46.google.com (mail-lf1-f46.google.com [209.85.167.46]) (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 CF93F2FD696 for ; Mon, 9 Feb 2026 07:13:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770621220; cv=none; b=XgR21ldeJHXQ41SNQYujYlksSlh84vzohLhTbxWTr1/Lwvmm+ZtJs7wUTt8A7WHLN9widxS8XZVTZO/MBnVMmLxBqNuSx0QHlKJ/IfDIx7m3QD2leeFNXkdUlbQqs1hz2LRvwP1H8z6kVaIm05IdUzVwRs2T0cuy4dXdbholCpo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770621220; c=relaxed/simple; bh=wjhPJySClgeUMyCJuXTdPR22ZbR2363nYldOD0ul2/0=; h=Date:From:To:CC:Subject:In-Reply-To:References:Message-ID: MIME-Version:Content-Type; b=q2mxrtxuSoCfGPl8j8C9tLFbHgFVnRBr+aaMUTv4rRQh5z+RrLbWRA/JtRV/7KQRweXAzo+G6wujgo5N3wwZvSAHqx8qAJ2MBhqb/KioitAl6oJPEMxrPH6cfa4UrNjf2Lg6/cHK2EnQ9G+14Mh5AUqLCKDbXyb5cH2ynTVJBXM= 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=eRooKwF0; arc=none smtp.client-ip=209.85.167.46 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="eRooKwF0" Received: by mail-lf1-f46.google.com with SMTP id 2adb3069b0e04-59ddb31ddcaso4616336e87.2 for ; Sun, 08 Feb 2026 23:13:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1770621218; x=1771226018; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to; bh=gJTcZlZbFHgUhIybFH4cWwk7Nh3ta/uxgKGllk07sVY=; b=eRooKwF0vCfpmtbR5z7o7JrYBoQ3suRDgEGIIh3ia554zWhryLlSc5nwMgjkVIuivr l2kbaO6rE2HXqw2VoJwORSshlBHbFw2gmKRMFRBuMY4wvbwmlxgZBWhQecUZPnBQNquu w8B4o19nfDZbaIJ1j4+OgNLmk42FhSmxFKhEv7ZBhSvE8oD0T/QcEsM+lLWlqv15ZdN6 Et+FsptXbXtwLYo4t3Urk2UTpPZO+kdyU9z+0bAmonHSlHqwzI5FEIN3wiVeno2GfTrb /ZXOrk9I8WBhTEifYiyEuC6sP6+pOBhqS7Z/wK23E7u3bleduzQAm++OPs7xl8AvSLZ/ eSHQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1770621218; x=1771226018; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=gJTcZlZbFHgUhIybFH4cWwk7Nh3ta/uxgKGllk07sVY=; b=w0KnDyeb0QPJFjGUXmdw+v3cSVPSZaM62eTCmZFNMOli8lDHBPPXr1E62TcosCLwA7 4ias0JmLgTWEb64GH6zrS551WShiynE6Upuhfl5USxoVtkFOZXQE5eTINArGuktW+ncs g0q1DWRryNjWYw+4Avjiuq1YEjKpqy2FjHn0iSZq7OqCOqQW2wfRvklb+nDhjiEayh/X ahsBky8VwMs95ymDIjBNNjdQ0ulEYyM33Pvzd89qLr+TuMijAwT52LZSw6+DX9kr9Y/d BFnUFtYfTHhJYmeRD5kReK2JKX/KTFjrXoDMCl69j1JUlgoNRzoodU/8RNo1jfDY/32P khRA== X-Forwarded-Encrypted: i=1; AJvYcCUkGJNx/CpoghrsX19RBjO2h3xxRQKoebn8NgeIxLTrTqHm8l415VaVjUaRd64Dpvtl7xldv+DX08d9DuQ=@vger.kernel.org X-Gm-Message-State: AOJu0Yw4GdfFuJrHHB7BhB16qGiCuxJU2ozh0xDXecIYKhoIzvzwzWB8 vdvXA56FsS7tI7bdKC3Nq5kKyvxWQPY29U65GV0f1d6oNSh5OPlz7u5F X-Gm-Gg: AZuq6aKxYUj9r7GX9+saSpxO+ypqJauaY3NJQ3QGaqG+9idJKu1JS7tUcb9KFuE29nW nQjmNLGdvICmjKHpzgBfVAvaLOP++vRm6dx+YPnLNCyhzmukv2Zk5Mpz3LLbfYu1k/UJb1AOOLM B57O3TK1LNigVAZ/K3yHgd6l3UhmrOkU2nw4uvGjia6kood01E/5mSqNxODK0cxj04gVaCeEU9P tLnWHqX7hHIyjyQXkdhzmjDXWsncGlOqZoRW5sBMbMAnwHke2+K3HHDSAJeB0z/XLcSbOa3MI9S ZVvFL8a07vrPWdIXUfBxSkPadnlhYUfDrg9U2AM+FatvGjBkuFRRDCfkDqSvSE3Zf7WIXlZbBTR GkVD3vE+/To3cqRdPSjYXBRkMLwefnfW5r4ugWmy/yZr9az57k/Il90778puFajF7m8pxEWYHID wzOSSdCmvUqpxMq5tbh+sQ X-Received: by 2002:a05:6512:a90:b0:59e:49f3:658c with SMTP id 2adb3069b0e04-59e49f36647mr2144179e87.49.1770621217578; Sun, 08 Feb 2026 23:13:37 -0800 (PST) Received: from ehlo.thunderbird.net ([178.137.105.239]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-59e44d29975sm2395771e87.66.2026.02.08.23.13.36 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 08 Feb 2026 23:13:36 -0800 (PST) Date: Mon, 09 Feb 2026 09:13:35 +0200 From: Svyatoslav Ryhel To: Rob Herring , David Lechner CC: Jonathan Cameron , =?ISO-8859-1?Q?Nuno_S=E1?= , Andy Shevchenko , Krzysztof Kozlowski , Conor Dooley , Dmitry Torokhov , Lee Jones , Pavel Machek , Liam Girdwood , Mark Brown , Dixit Parmar , Tony Lindgren , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-input@vger.kernel.org, linux-leds@vger.kernel.org Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_v2_01/11=5D_dt-bindings=3A_regulat?= =?US-ASCII?Q?or=3A_cpcap-regulator=3A_convert_to_DT_schema?= User-Agent: K-9 Mail for Android In-Reply-To: References: <20260206172845.145407-1-clamor95@gmail.com> <20260206172845.145407-2-clamor95@gmail.com> <20260206230922.GA254792-robh@kernel.org> <51eddfc0-9f23-4a7f-8729-19775d834c0b@baylibre.com> Message-ID: <20E2821E-3A3B-425B-8E5A-AEC36A2AB9CE@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable 9 =D0=BB=D1=8E=D1=82=D0=BE=D0=B3=D0=BE 2026=E2=80=AF=D1=80=2E 00:37:41 GMT= +02:00, Rob Herring =D0=BF=D0=B8=D1=88=D0=B5: >On Sat, Feb 7, 2026 at 11:32=E2=80=AFAM David Lechner wrote: >> >> On 2/7/26 11:25 AM, Svyatoslav Ryhel wrote: >> > =D1=81=D0=B1, 7 =D0=BB=D1=8E=D1=82=2E 2026=E2=80=AF=D1=80=2E =D0=BE 1= 9:19 David Lechner =D0=BF=D0=B8=D1=88=D0=B5: >> >> >> >> On 2/7/26 3:20 AM, Svyatoslav Ryhel wrote: >> >>> =D1=81=D0=B1, 7 =D0=BB=D1=8E=D1=82=2E 2026=E2=80=AF=D1=80=2E =D0=BE= 01:09 Rob Herring =D0=BF=D0=B8=D1=88=D0=B5: >> >>>> >> >>>> On Fri, Feb 06, 2026 at 07:28:35PM +0200, Svyatoslav Ryhel wrote: >> >>>>> Convert devicetree bindings for the Motorola CPCAP MFD regulator = subnode >> >>>>> from TXT to YAML format=2E Main functionality preserved=2E >> >>>>> >> >> >> >> =2E=2E=2E >> >> >> >>>>> +properties: >> >>>>> + compatible: >> >>>>> + enum: >> >>>>> + - motorola,cpcap-regulator >> >>>>> + - motorola,mapphone-cpcap-regulator >> >>>>> + - motorola,xoom-cpcap-regulator >> >>>>> + >> >>>>> + regulators: >> >>>>> + type: object >> >>>>> + >> >>>>> + patternProperties: >> >>>>> + "$[A-Z0-9]+^": >> >>>> >> >>>> I thought it was said on the last version to list the names=2E Con= sidering >> >>>> you already have them below, better to put them in schema than pro= se=2E >> >>>> And pretty much all regulator bindings define the names=2E >> >>>> >> >>> >> >>> What exactly do you propose? All those names will not fit into a >> >>> single unbreakable line (more than 125 columns in length btw)=2E Pa= tch >> >>> checker will complain about this=2E Duplicate pattern properties fo= r >> >>> every few names? Please be a bit more specific=2E Thank you! >> >> >> >> Making checkpatch happy is not a hard requirement=2E If you have to >> >> go over 125 characters for technical reasons, no one is going to >> >> complain=2E >> >> >> >> I already gave an example on the last version=2E It is just a regex >> >> expression=2E (And note that you didn't fix the $ and ^ either=2E ^ >> >> means beginning of the line and $ means the end of the line=2E So >> >> the pattern currently won't match anything=2E) >> >> >> >> patternProperties: >> >> "^(SW1|SW2|SW3|SW4|SW5|=2E=2E=2E)$": >> >> >> >> >> >> And I hope it is obvious that you are meant to replace =2E=2E=2E wit= h the >> >> rest of the names separated by |=2E >> >> >> > >> > David, thank you for your suggestions and I did not want to offend yo= u >> > nor Rob, or ignore=2E Using this large string IMHO feels a bit wrong= =2E No >> > offence, I will use it if required=2E At the same time I was >> > experimenting with possible representation and come up with smth like >> > this: >> > >> > patternProperties: >> > "^SW[1-6]$": >> > type: object >> > $ref: /schemas/regulator/regulator=2Eyaml# >> > unevaluatedProperties: false >> > >> > required: >> > - regulator-name >> > - regulator-enable-ramp-delay >> > - regulator-min-microvolt >> > - regulator-max-microvolt >> > >> > "^V(CAM|CSI|DAC|DIG|FUSE|HVIO|SDIO|PLL|RF1|RF2|RFREF)$": >> > type: object >> > $ref: /schemas/regulator/regulator=2Eyaml# >> > unevaluatedProperties: false >> > >> > required: >> > - regulator-name >> > - regulator-enable-ramp-delay >> > - regulator-min-microvolt >> > - regulator-max-microvolt >> > >> > "^V(WLAN1|WLAN2|SIM|SIMCARD|VIB|USB|AUDIO)$": >> > type: object >> > $ref: /schemas/regulator/regulator=2Eyaml# >> > unevaluatedProperties: false >> > >> > required: >> > - regulator-name >> > - regulator-enable-ramp-delay >> > - regulator-min-microvolt >> > - regulator-max-microvolt >> > >> > Will this be acceptable? It passes checks >> >> I think we would rather have the long match string instead of >> repeating the rest of it multiple times=2E > >Yes=2E > >105 chars: > > "^(SW[1-6]|V(CAM|CSI|DAC|DIG|FUSE|HVIO|SDIO|PLL|RF[12]|RFREF|WLAN[12]= |SIM|SIMCARD|VIB|USB|AUDIO))$": > >There is the yamllint of 110 which does have to be followed=2E > Amazing! Thank you both for your suggestions=2E >Rob