From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0DA4EC433FE for ; Tue, 22 Nov 2022 11:11:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232140AbiKVLLs convert rfc822-to-8bit (ORCPT ); Tue, 22 Nov 2022 06:11:48 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58618 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232934AbiKVLLr (ORCPT ); Tue, 22 Nov 2022 06:11:47 -0500 Received: from muru.com (muru.com [72.249.23.125]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id DC6272FC2F; Tue, 22 Nov 2022 03:11:45 -0800 (PST) Received: from localhost (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 396CB8061; Tue, 22 Nov 2022 11:01:31 +0000 (UTC) Date: Tue, 22 Nov 2022 13:11:43 +0200 From: Tony Lindgren To: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= Cc: Krzysztof Kozlowski , Rob Herring , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org, Thierry Reding , Nishanth Menon , Vignesh Raghavendra Subject: Re: [PATCH v2 1/2] dt-bindings: pwm: Allow decimal format in addition to hex format Message-ID: References: <20221121115525.36362-1-tony@atomide.com> <20221121144000.7weeyfbbulltfxmd@pengutronix.de> <20221122093650.oouke2utlkvdd7r6@pengutronix.de> <20221122105710.5aow5qsri6pikgri@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8BIT In-Reply-To: <20221122105710.5aow5qsri6pikgri@pengutronix.de> Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org * Uwe Kleine-König [221122 10:47]: > On Tue, Nov 22, 2022 at 12:37:26PM +0200, Tony Lindgren wrote: > > * Uwe Kleine-König [221122 09:26]: > > > On Tue, Nov 22, 2022 at 06:43:55AM +0200, Tony Lindgren wrote: > > > > * Krzysztof Kozlowski [221121 16:05]: > > > > > On 21/11/2022 15:48, Tony Lindgren wrote: > > > > > > * Uwe Kleine-König [221121 14:30]: > > > > > >> On Mon, Nov 21, 2022 at 01:55:24PM +0200, Tony Lindgren wrote: > > > > > >>> Let's allow node numbering in decimal format too. > > > > > >>> > > > > > >>> Simple human-readable increments/IDs are usually decimal, hex is only for > > > > > >>> addresses as noted by Krzysztof Kozlowski . > > > > > >>> > > > > > >>> Cc: Thierry Reding > > > > > >>> Cc: Uwe Kleine-König > > > > > >>> Suggested-by: Krzysztof Kozlowski > > > > > >>> Signed-off-by: Tony Lindgren > > > > > >>> --- > > > > > >>> Documentation/devicetree/bindings/pwm/pwm.yaml | 2 +- > > > > > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > >>> > > > > > >>> diff --git a/Documentation/devicetree/bindings/pwm/pwm.yaml b/Documentation/devicetree/bindings/pwm/pwm.yaml > > > > > >>> --- a/Documentation/devicetree/bindings/pwm/pwm.yaml > > > > > >>> +++ b/Documentation/devicetree/bindings/pwm/pwm.yaml > > > > > >>> @@ -13,7 +13,7 @@ select: false > > > > > >>> > > > > > >>> properties: > > > > > >>> $nodename: > > > > > >>> - pattern: "^pwm(@.*|-[0-9a-f])*$" > > > > > >>> + pattern: "^pwm(@.*|-([0-9a-f]|1[0-5]))*$" > > > > > >> > > > > > >> I wonder why you don't make this: > > > > > >> > > > > > >> + pattern: "^pwm(@.*|-[0-9a-f]*)$" > > > > > > > > > > Yes, the '*' should be within (). > > > > > > > > Sorry I guess I don't follow. So for what type of naming is the second '*' > > > > actually needed here, or is it needed at all? > > > > > > > > We only want to match the following: > > > > > > > > pwm@1234 > > > > > > > > pwm-0 > > > > ... > > > > pwm-f > > > > > > > > And now also: > > > > > > > > pwm-0 > > > > ... > > > > pwm-15 > > > > > > > > Is there yet another format I'm missing? > > > > > > I think there is a misunderstanding about the type of pattern here. Do > > > we talk about a shell pattern where "[0-9a-f]*" matches a name starting > > > with a hex digit followed by any number of any caracter, or is it a > > > regexp where the * means "any number of the preceding pattern", so > > > effectively a word consisting of only hex digits. > > > > > > The .* part makes me think we're talking about a regexp, so the old > > > pattern ("^pwm(@.*|-[0-9a-f])*$") means that > > > > > > pwm-1-2-3-4 > > > > > > is allowed but > > > > > > pwm-10 > > > > > > isn't. And I think what we want is: > > > > > > "^pwm(@.*|-[0-9a-f]*)$" > > > > Heh so this fails for just node name pwm: > > > > pwm: $nodename:0: 'pwm' does not match '^pwm(@.*|-[0-9a-f]*)$' > > > > Maybe that's why the second '..)*$' was there originally for? > > And adding it back there fixes pwm but allows pwm-1-2-3.. > > > > > (or maybe better "^pwm(@.+|-[0-9a-f]+)$" ?) > > > > And this too fails for just pwm node name. > > Then let's pick: > > > "^pwm(@.+|-[0-9a-f]+)?$" That works for pwm and pwm-11, but now fails for pwm11 :) Tony