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 B67E8C4321E for ; Fri, 2 Dec 2022 10:58:58 +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=3/uBa9YrLF8GmkT1K5fJ27hIauBik+R0mF1jU27wELs=; b=v+QGjrnPK58yrM kknvivzGSj1db/bena95uS0NqlMU9MSuUOsHmOCMBy1k2tfsIGJBuCha5uDx/gFdfd+qrTLuQthbX TPl2Ly/7p7U1O35c6haX5dAxRKaDZez52XwS3GWcVCLZP3yi0ONBwOc9OsXzdSzQe60CaaBSRQp3Y qOTquoibOndDeghSXkKYRsBv5elLSBxd42gLr9dbyPtk2ukfdkX46YD81fLMk4z8EHte0ZdAsT/GT fWkzdlJu0lpquZvQ+orlDrQm5aG+O1TRWCbgrBaztEP3KhwcKO2WfaLL85Mnn2Sc7wtwML/gRSo4g fXZePO9pLD+VdmBmD7Ew==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p13kj-00FndQ-Ky; Fri, 02 Dec 2022 10:58:45 +0000 Received: from mx.sberdevices.ru ([45.89.227.171]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p13ke-00FnZg-QZ; Fri, 02 Dec 2022 10:58:43 +0000 Received: from s-lin-edge02.sberdevices.ru (localhost [127.0.0.1]) by mx.sberdevices.ru (Postfix) with ESMTP id CB99C5FD0B; Fri, 2 Dec 2022 13:58:35 +0300 (MSK) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sberdevices.ru; s=mail; t=1669978715; bh=bmtQOp0MHFjIRwhVsuo8y6uKrcRollt9lDuoEi9K2Ns=; h=Date:From:To:Subject:Message-ID:MIME-Version:Content-Type; b=sDgpn/xX2RPFtFDVGuLMmeVg7uLVdiBcZaROU6x3DAGgZM0FchEFeCx3JRUpYdwY9 y0yN84dSkfMyxuZZx5BqNzh6pjlUzcngmgoy7o3gSei2fdiZaCs5SqM1MiSuh8ENyr 4Q9BGy/bvHKtkCPe7M+FE2HhijT3NzguM1r8bkpMBxuUn6oTb13fzxElpr6Z+ihNdV Si1fMikx9d8XNdgMs5pwxrKkmv/S6Y8nd2pIjbIHKLXOO4BCi17f5m7qlR9lQNN8gJ QU0PJlCJ7QtMAGiuJKKIh9fzjGqMACArv3WmsnL7QrIZdZBOGAbqvVkBYxf+aXuCrG nVZ6pfv+kampw== Received: from S-MS-EXCH01.sberdevices.ru (S-MS-EXCH01.sberdevices.ru [172.16.1.4]) by mx.sberdevices.ru (Postfix) with ESMTP; Fri, 2 Dec 2022 13:58:35 +0300 (MSK) Date: Fri, 2 Dec 2022 13:58:34 +0300 From: Dmitry Rokosov To: Krzysztof Kozlowski CC: Rob Herring , , , , , , , , , , , , , , , , Subject: Re: [PATCH v8 03/11] dt-bindings: clock: meson: add A1 peripheral clock controller bindings Message-ID: <20221202105834.4avip2trjbm3tpn7@CAB-WSD-L081021> References: <20221201225703.6507-1-ddrokosov@sberdevices.ru> <20221201225703.6507-4-ddrokosov@sberdevices.ru> <166995398331.2089708.18189499743713568568.robh@kernel.org> <20221202094940.elhjbz7tbgxesr52@CAB-WSD-L081021> <640ad325-91c4-109a-5ccd-05ac653a29bb@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <640ad325-91c4-109a-5ccd-05ac653a29bb@linaro.org> User-Agent: NeoMutt/20220415 X-Originating-IP: [172.16.1.6] X-ClientProxiedBy: S-MS-EXCH01.sberdevices.ru (172.16.1.4) To S-MS-EXCH01.sberdevices.ru (172.16.1.4) X-KSMG-Rule-ID: 4 X-KSMG-Message-Action: clean X-KSMG-AntiSpam-Status: not scanned, disabled by settings X-KSMG-AntiSpam-Interceptor-Info: not scanned X-KSMG-AntiPhishing: not scanned, disabled by settings X-KSMG-AntiVirus: Kaspersky Secure Mail Gateway, version 1.1.2.30, bases: 2022/12/02 07:44:00 #20636821 X-KSMG-AntiVirus-Status: Clean, skipped X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221202_025841_587640_48434421 X-CRM114-Status: GOOD ( 24.71 ) X-BeenThere: linux-amlogic@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-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org Hello Krzysztof, Thank you for quick review. Let me explain you why I sent Jian broken patches and applied my fixup patches above. I've found several versions of A1 clkc drivers from Jian Hu Amlogic engineer, the last one version was a couple years ago. I've reworked the last (v7) version and didn't know the correct way to apply all fixes. >From my point of view I thought about two options: 1) Merge all my changes with Jian Hu patches and leave Jian Hu SoB 2) Don't touch Jian Hu v7 patches and apply my changes above. I have chosen the second option, because it looks like respect Jian Hu's effort. I don't want to "stealing" his patches, just want to fix all broken things. So I'm sorry for misunderstanding here and please correct me what's right workflow in such situation. On Fri, Dec 02, 2022 at 11:39:47AM +0100, Krzysztof Kozlowski wrote: > On 02/12/2022 10:49, Dmitry Rokosov wrote: > > On Thu, Dec 01, 2022 at 10:10:04PM -0600, Rob Herring wrote: > >> > >> On Fri, 02 Dec 2022 01:56:55 +0300, Dmitry Rokosov wrote: > >>> From: Jian Hu > >>> > >>> Add the documentation to support Amlogic A1 peripheral clock driver, > >>> and add A1 peripheral clock controller bindings. > >>> > >>> Signed-off-by: Jian Hu > >>> Signed-off-by: Dmitry Rokosov > >>> --- > >>> .../bindings/clock/amlogic,a1-clkc.yaml | 65 ++++++++++++ > >>> include/dt-bindings/clock/a1-clkc.h | 98 +++++++++++++++++++ > >>> 2 files changed, 163 insertions(+) > >>> create mode 100644 Documentation/devicetree/bindings/clock/amlogic,a1-clkc.yaml > >>> create mode 100644 include/dt-bindings/clock/a1-clkc.h > >>> > >> > >> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > >> on your patch (DT_CHECKER_FLAGS is new in v5.13): > >> > >> yamllint warnings/errors: > >> > >> dtschema/dtc warnings/errors: > >> ./Documentation/devicetree/bindings/clock/amlogic,a1-clkc.yaml: $id: relative path/filename doesn't match actual path or filename > >> expected: http://devicetree.org/schemas/clock/amlogic,a1-clkc.yaml# > >> Documentation/devicetree/bindings/clock/amlogic,a1-clkc.example.dts:18.48-30.11: Warning (unit_address_vs_reg): /example-0/periphs-clock-controller: node has a reg or ranges property, but no unit name > >> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/clock/amlogic,a1-clkc.example.dtb: periphs-clock-controller: reg: [[0, 2048], [0, 260]] is too long > >> From schema: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/clock/amlogic,a1-clkc.yaml > >> > >> doc reference errors (make refcheckdocs): > >> > >> See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20221201225703.6507-4-ddrokosov@sberdevices.ru > >> > >> The base for the series is generally the latest rc1. A different dependency > >> should be noted in *this* patch. > >> > >> If you already ran 'make dt_binding_check' and didn't see the above > >> error(s), then make sure 'yamllint' is installed and dt-schema is up to > >> date: > >> > >> pip3 install dtschema --upgrade > >> > >> Please check and re-submit after running the above command yourself. Note > >> that DT_SCHEMA_FILES can be set to your schema file to speed up checking > >> your schema. However, it must be unset to test all examples with your schema. > >> > > > > That's totally right warnings and errors. All of them are fixed in the > > my "fixup" patch of Jian's original dt_binding schema: > > > > https://lore.kernel.org/linux-amlogic/20221201225703.6507-11-ddrokosov@sberdevices.ru/ > > No, this patch must be fixed. It's not correct. > > Best regards, > Krzysztof > -- Thank you, Dmitry _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic