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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6A993C433F5 for ; Wed, 9 Mar 2022 13:59:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=BQUR/ptZbeHc5ALu+AEaRpcB9tf3joGGDVHnqKZjyjs=; b=OGWJfEcT24u2nL OkJtXM/94Qe7ljH4OdzFt1m2x283WV2BwREuVdbibzZBkh7vhP+9rqSDztMZKTVcdbTvkrKBpVz4p nALdKuBR3V3+P2PLxYkYrbfy5U+F95d+YH1wovFQze2yNi30rTsYChricb3llTg1Ed0DhqZvjJP7X ExwBNM6C4LAUcWK0S9UwvNltn1w6X46Auc7IYh4MgvvogSAmzy6dg4AjtAfsDcGpEaTLA1aotu14R uHFPUksbIP9MSiRipFgTcuJjuJ4TU3UYkUlb0JHy6szgnx30pAd9NAzwpVOlGnC8zJfLW52b/lJWj +KCnD6Izs+bvQBsC5Bgw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nRwpa-008qVt-S0; Wed, 09 Mar 2022 13:58:23 +0000 Received: from mail-pj1-x1036.google.com ([2607:f8b0:4864:20::1036]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nRwow-008q8g-FB for linux-arm-kernel@lists.infradead.org; Wed, 09 Mar 2022 13:57:44 +0000 Received: by mail-pj1-x1036.google.com with SMTP id mv5-20020a17090b198500b001bf2a039831so5310374pjb.5 for ; Wed, 09 Mar 2022 05:57:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=faIIfTxQAdrbYwzah/aXL8R3vzKjdbBY0/6Hv4kojO4=; b=ba7FoR/sP7LQDzjdOdh904xFCN5OCpME0XYdqUUPWFsQqPIyZdWJ9D2U7967Qd2wPM shujyXE3lX67ic5tFWgsfPcoB5Iy3CU/KsE83wxIDppLDo7ifwYHPWiKvcCDC8jkDVxn cMKBNHxGu2va0/R9F6EdN05AUbkNxcnjEsLXLlYP1y56Do/hjlEHb7gbQT8zRCJ7zl13 IGCZl7WSP1Qmx4SMAQzhO/IIh82NIXq/oMmTNiSpFEGUGXFSYm5iMCcwIfK5ciUPjf0n nXrO25hM6xwS64LhBlNI3OdaKh+6JjBE9DV1SjUozCu5ylJhEREeGXVbVERd8pFzLD2L Xg4w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=faIIfTxQAdrbYwzah/aXL8R3vzKjdbBY0/6Hv4kojO4=; b=lhNx85xDBBseKmBOyGjgAg3fvR7EqtwXn4fJFJ4SJ4CugAeFMbjYLrE5m+ftD1t+5Y Sk2R3/OkO3nRtjCFIGpDI7dzuNm7BR02Dm9ST/usUOci7wwvwGUyzO1R9GwDizSVssS4 xJnW/imDa0g0ng4Et0zpuagT7JytZjjrjiQn/MKf8x/TlKd8ZY4hBnZXx/W7TKEs8TnK spW/bQHYh6USACEfk+r2vo8SeWUT0y/KMayRIXiEnXoA/21DDB7OO0NlrFloP5tmQo0t 8FJrR/Ak3vIXpbduJnCrS2xqhTRpcztzBXKnqSvwcpvgvpTkdtTffneAwdJfx9vapF7j 4PbA== X-Gm-Message-State: AOAM532I2655+GXvxoucWp2CPupQxZQ8ujGuufisFTPt1GD/CGu7tsF3 7M1ySbH5rX7r0ppGEFsLEuY= X-Google-Smtp-Source: ABdhPJxNl9iNPU4HIEro6FlW6Jac4RZUF7NPRzFtZKB+VP7mfRKYdbEoz+zBYnqrki4rdFud5EnLEw== X-Received: by 2002:a17:90a:a591:b0:1bc:8015:4c9e with SMTP id b17-20020a17090aa59100b001bc80154c9emr10303253pjq.154.1646834260513; Wed, 09 Mar 2022 05:57:40 -0800 (PST) Received: from 9a2d8922b8f1 ([122.161.53.68]) by smtp.gmail.com with ESMTPSA id c18-20020a056a000ad200b004f0f9696578sm3571032pfl.141.2022.03.09.05.57.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Mar 2022 05:57:40 -0800 (PST) Date: Wed, 9 Mar 2022 19:27:34 +0530 From: Kuldeep Singh To: Krzysztof Kozlowski Cc: Mark Brown , linux-arm-kernel@lists.infradead.org, linux-spi@vger.kernel.org, devicetree@vger.kernel.org, Rob Herring , Robin Murphy , Linus Walleij Subject: Re: [PATCH v2 1/2] dt-bindings: spi: Update clocks property for ARM pl022 Message-ID: <20220309135734.GA87543@9a2d8922b8f1> References: <20220308072125.38381-1-singh.kuldeep87k@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220309_055742_540488_646A1309 X-CRM114-Status: GOOD ( 23.48 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Mar 08, 2022 at 09:27:48AM +0100, Krzysztof Kozlowski wrote: > On 08/03/2022 08:21, Kuldeep Singh wrote: > > Add missing minItems property to clocks in ARM pl022 bindings. > > > > This helps in resolving below warnings: > > clocks: [[4]] is too short > > clock-names: ['apb_pclk'] is too short > > > > Signed-off-by: Kuldeep Singh > > --- > > v2: > > - Keep actual warning and remove path to file > > - Reword commit message a bit > > It still misses information whether it is actually correct from PL022 > point of view to have just one clock. > > If the DTS are wrong, do not change the bindings to match such wrong > DTS. If the DTS is correct, please explain why bindings are wrong. Thanks Krzysztof for pointing it out. So far I was wondering spiclk clock-name was the only culprit there and had different perception w.r.t number of clocks. Anyway, below reference is from pl022 point of view: https://documentation-service.arm.com/static/5e8e3bc7fd977155116a936d?token= https://developer.arm.com/documentation/ddi0194/h/functional-overview/primecell-ssp-operation/clock-ratios The reference no where mention that single clock can exist. So far Amd seattle and LG platforms don't comply with binding and define single clock which require updations. So now from bindings perspective, I am looking forward to keep only one notation of sspclk and will respin a patch. Kindly provide comments on the same. Regards Kuldeep > > > > Documentation/devicetree/bindings/spi/spi-pl022.yaml | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Documentation/devicetree/bindings/spi/spi-pl022.yaml b/Documentation/devicetree/bindings/spi/spi-pl022.yaml > > index 6d633728fc2b..7d36e15db5b3 100644 > > --- a/Documentation/devicetree/bindings/spi/spi-pl022.yaml > > +++ b/Documentation/devicetree/bindings/spi/spi-pl022.yaml > > @@ -34,6 +34,7 @@ properties: > > maxItems: 1 > > > > clocks: > > + minItems: 1 > > maxItems: 2 > > This does not match clock-names which requires two clocks. It's not > correct now. > > > > > clock-names: > > > Best regards, > Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel