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 A4832EB64D8 for ; Tue, 13 Jun 2023 21:38:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238478AbjFMViq (ORCPT ); Tue, 13 Jun 2023 17:38:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42708 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229559AbjFMVip (ORCPT ); Tue, 13 Jun 2023 17:38:45 -0400 Received: from mail-ej1-x62e.google.com (mail-ej1-x62e.google.com [IPv6:2a00:1450:4864:20::62e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CA1A198 for ; Tue, 13 Jun 2023 14:38:43 -0700 (PDT) Received: by mail-ej1-x62e.google.com with SMTP id a640c23a62f3a-977ed383b8aso2212866b.3 for ; Tue, 13 Jun 2023 14:38:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1686692322; x=1689284322; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=IjC8sLTBfnrVHtu3dchJNU/FgXzyF+J0/8Jze1O1iI8=; b=AzPT/wZgr7P0DAlj9wX7buFMwigcsG5VkN5YC46JszhbWZpQOMBnCVL1CGFrzmZeOT Yz1QkEOXoaZQOMUJeGAAkC0LGXGETn0ux03RePC5Qq6T9BPXxd+Me4IWCkg5CSYYQcqH t/a1dpR4nmcP9MzGMtoqtVbQ3MVNa3tgU47r/gOXlvB97wW0TRSee4EnK6stQyxk7Ulm O+QubKI24lPmUcxuds3zWVjhhI28F3r2KGcuBMzkLhaV6goEd2zFsN9LadnDdl/ZPtZl bzyF3QsqXk1IOEzTY+E+MD2zpzq5B4h40GwOAAXqT2qsimPfVXek/cFCRCKca4fCKOqI JeJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686692322; x=1689284322; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=IjC8sLTBfnrVHtu3dchJNU/FgXzyF+J0/8Jze1O1iI8=; b=WEcYvomoLPqls1cpKXWYFt5MEMwpxqfMnuv+5n7z98UHKIELtN5F+XCJn5ZahVWoPC 1nJn3L5PPTmoA48OwUIArI0B54ke7JJkzb4hQO+lxPde7pvyIs3GlXyvV2P/692pXAit un63EIud0nCpvv2Fu6+DsByeyt2yBOF+EFwgod6rMORW6nQ9+9/qclQXxJPhYZnGVRMQ Xh83eqQIJPMAAf6VCOdPRH37pIeLsslrbqwdx/puo8Mu9yRjXe0AsYKO3tM8AKWMN3hH 9aTsUGH5cN1c20YYwCCjZj7NEQZqcqUQbv+5NIBiHPIBVcGwBJ4ZLeMUWyuNqhmB9cSU Ulxg== X-Gm-Message-State: AC+VfDzSXRV4NlwNLP9pQCDsfaSSUZEbdxZiMR62aC5laxUHd6+XVtau E21u2jD0pKIhM1hT5QuQ7zaH/A== X-Google-Smtp-Source: ACHHUZ65Rx8zx8S8O4fyQjWDJKbkfBSxlojJnhgAncSog4C1GQJK1oHej6a0lu74m26bO+xbiEsNPQ== X-Received: by 2002:a17:906:eece:b0:959:6fb2:1c3b with SMTP id wu14-20020a170906eece00b009596fb21c3bmr13125627ejb.39.1686692322273; Tue, 13 Jun 2023 14:38:42 -0700 (PDT) Received: from [192.168.1.20] ([178.197.219.26]) by smtp.gmail.com with ESMTPSA id v6-20020a17090606c600b009745d484519sm7135381ejb.70.2023.06.13.14.38.40 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 13 Jun 2023 14:38:41 -0700 (PDT) Message-ID: <57082b2e-7f89-4e90-492e-54d7bebb427b@linaro.org> Date: Tue, 13 Jun 2023 23:38:40 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [PATCH v2 7/8] dt-bindings: rtc: isl12022: add #clock-cells property Content-Language: en-US To: Rasmus Villemoes , Alessandro Zummo , Alexandre Belloni Cc: Andy Shevchenko , devicetree@vger.kernel.org, Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-rtc@vger.kernel.org, linux-kernel@vger.kernel.org References: <20230612113059.247275-1-linux@rasmusvillemoes.dk> <20230613130011.305589-1-linux@rasmusvillemoes.dk> <20230613130011.305589-8-linux@rasmusvillemoes.dk> <208d06d1-58e3-748e-de28-075dc0cd28de@linaro.org> <13f61130-ca44-cacb-15f7-835a74dda26f@rasmusvillemoes.dk> From: Krzysztof Kozlowski In-Reply-To: <13f61130-ca44-cacb-15f7-835a74dda26f@rasmusvillemoes.dk> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-rtc@vger.kernel.org On 13/06/2023 22:25, Rasmus Villemoes wrote: > On 13/06/2023 21.10, Krzysztof Kozlowski wrote: >> On 13/06/2023 15:00, Rasmus Villemoes wrote: > >>> diff --git a/Documentation/devicetree/bindings/rtc/intersil,isl12022.yaml b/Documentation/devicetree/bindings/rtc/intersil,isl12022.yaml >>> index d5d3a687a34d..a9ef68b5fdcd 100644 >>> --- a/Documentation/devicetree/bindings/rtc/intersil,isl12022.yaml >>> +++ b/Documentation/devicetree/bindings/rtc/intersil,isl12022.yaml >>> @@ -11,6 +11,13 @@ maintainers: >>> >>> allOf: >>> - $ref: rtc.yaml# >>> + # If #clock-cells is present, interrupts must not be present >>> + - if: >>> + required: >>> + - '#clock-cells' >>> + then: >>> + properties: >>> + interrupts: false >> >> Entire allOf block should be like in example-schema, so before >> unevaluatedProperties. Please put it in correct place in your first >> patch so here it does not have to be moved. >> > > OK. That first patch was basically a copy-paste of c690048ed59b, and > e.g. ingenic,rtc.yaml has a similar non-trivial allOf block between > maintainers and properties. Is there somehow I could have known it > should be right before unevaluatedProperties? The trivial - with a $ref - we keep often at the top. But once it starts growing, should be at the bottom. Since you know it will grow, just put it at the bottom (not total bottom, but like in example-schema, so after required:). Best regards, Krzysztof