From: Krzysztof Kozlowski <krzk@kernel.org> To: Drew Fustini <dfustini@tenstorrent.com>, Jisheng Zhang <jszhang@kernel.org>, Guo Ren <guoren@kernel.org>, Fu Wei <wefu@redhat.com>, Yangtao Li <frank.li@vivo.com>, Thomas Bonnefille <thomas.bonnefille@bootlin.com>, Emil Renner Berthing <emil.renner.berthing@canonical.com>, Michael Turquette <mturquette@baylibre.com>, Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh@kernel.org>, Krzysztof Kozlowski <krzk+dt@kernel.org>, Conor Dooley <conor+dt@kernel.org>, Paul Walmsley <paul.walmsley@sifive.com>, Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu> Cc: linux-riscv@lists.infradead.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH RFC v2 1/4] dt-bindings: clock: Document T-Head TH1520 AP_SUBSYS controller Date: Mon, 29 Apr 2024 07:10:50 +0200 [thread overview] Message-ID: <302e582e-d433-44df-ab7a-65de5bccd419@kernel.org> (raw) In-Reply-To: <20240426-th1520-clk-v2-v2-1-96b829e6fcee@tenstorrent.com> On 27/04/2024 02:10, Drew Fustini wrote: > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/thead,th1520-clk-ap.h> > + soc { > + > + #address-cells = <2>; > + #size-cells = <2>; > + > + clk: clock-controller@ffef010000 { This is a friendly reminder during the review process. It seems my or other reviewer's previous comments were not fully addressed. Maybe the feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. Thank you. > + compatible = "thead,th1520-clk-ap"; > + reg = <0xff 0xef010000 0x0 0x1000>; > + clocks = <&osc>; > + clock-names = "osc"; > + #clock-cells = <1>; > + }; > + }; > diff --git a/MAINTAINERS b/MAINTAINERS > index ebf03f5f0619..51d550cb1329 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -19023,7 +19023,9 @@ M: Guo Ren <guoren@kernel.org> > M: Fu Wei <wefu@redhat.com> > L: linux-riscv@lists.infradead.org > S: Maintained > +F: Documentation/devicetree/bindings/clock/thead,th1520-clk-ap.yaml > F: arch/riscv/boot/dts/thead/ > +F: include/dt-bindings/clock/thead,th1520-clk-ap.h > Best regards, Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org> To: Drew Fustini <dfustini@tenstorrent.com>, Jisheng Zhang <jszhang@kernel.org>, Guo Ren <guoren@kernel.org>, Fu Wei <wefu@redhat.com>, Yangtao Li <frank.li@vivo.com>, Thomas Bonnefille <thomas.bonnefille@bootlin.com>, Emil Renner Berthing <emil.renner.berthing@canonical.com>, Michael Turquette <mturquette@baylibre.com>, Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh@kernel.org>, Krzysztof Kozlowski <krzk+dt@kernel.org>, Conor Dooley <conor+dt@kernel.org>, Paul Walmsley <paul.walmsley@sifive.com>, Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu> Cc: linux-riscv@lists.infradead.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH RFC v2 1/4] dt-bindings: clock: Document T-Head TH1520 AP_SUBSYS controller Date: Mon, 29 Apr 2024 07:10:50 +0200 [thread overview] Message-ID: <302e582e-d433-44df-ab7a-65de5bccd419@kernel.org> (raw) In-Reply-To: <20240426-th1520-clk-v2-v2-1-96b829e6fcee@tenstorrent.com> On 27/04/2024 02:10, Drew Fustini wrote: > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/thead,th1520-clk-ap.h> > + soc { > + > + #address-cells = <2>; > + #size-cells = <2>; > + > + clk: clock-controller@ffef010000 { This is a friendly reminder during the review process. It seems my or other reviewer's previous comments were not fully addressed. Maybe the feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. Thank you. > + compatible = "thead,th1520-clk-ap"; > + reg = <0xff 0xef010000 0x0 0x1000>; > + clocks = <&osc>; > + clock-names = "osc"; > + #clock-cells = <1>; > + }; > + }; > diff --git a/MAINTAINERS b/MAINTAINERS > index ebf03f5f0619..51d550cb1329 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -19023,7 +19023,9 @@ M: Guo Ren <guoren@kernel.org> > M: Fu Wei <wefu@redhat.com> > L: linux-riscv@lists.infradead.org > S: Maintained > +F: Documentation/devicetree/bindings/clock/thead,th1520-clk-ap.yaml > F: arch/riscv/boot/dts/thead/ > +F: include/dt-bindings/clock/thead,th1520-clk-ap.h > Best regards, Krzysztof _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2024-04-29 5:10 UTC|newest] Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-04-27 0:10 [PATCH RFC v2 0/4] clk: thead: Add support for TH1520 AP_SUBSYS clock controller Drew Fustini 2024-04-27 0:10 ` Drew Fustini 2024-04-27 0:10 ` [PATCH RFC v2 1/4] dt-bindings: clock: Document T-Head TH1520 AP_SUBSYS controller Drew Fustini 2024-04-27 0:10 ` Drew Fustini 2024-04-29 5:10 ` Krzysztof Kozlowski [this message] 2024-04-29 5:10 ` Krzysztof Kozlowski 2024-04-27 0:10 ` [PATCH RFC v2 2/4] clk: thead: Add support for T-Head TH1520 AP_SUBSYS clocks Drew Fustini 2024-04-27 0:10 ` Drew Fustini 2024-05-02 7:47 ` Thomas Bonnefille 2024-05-02 7:47 ` Thomas Bonnefille 2024-04-27 0:10 ` [PATCH RFC v2 3/4] riscv: dts: thead: Add TH1520 AP_SUBSYS clock controller Drew Fustini 2024-04-27 0:10 ` Drew Fustini 2024-04-27 0:10 ` [PATCH RFC v2 4/4] riscv: dts: thead: Add clock to TH1520 mmc controllers Drew Fustini 2024-04-27 0:10 ` Drew Fustini 2024-05-02 7:47 ` Thomas Bonnefille 2024-05-02 7:47 ` Thomas Bonnefille 2024-05-02 15:53 ` Drew Fustini 2024-05-02 15:53 ` Drew Fustini 2024-05-03 6:58 ` Thomas Bonnefille 2024-05-03 6:58 ` Thomas Bonnefille 2024-05-02 7:47 ` [PATCH RFC v2 0/4] clk: thead: Add support for TH1520 AP_SUBSYS clock controller Thomas Bonnefille 2024-05-02 7:47 ` Thomas Bonnefille
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=302e582e-d433-44df-ab7a-65de5bccd419@kernel.org \ --to=krzk@kernel.org \ --cc=aou@eecs.berkeley.edu \ --cc=conor+dt@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=dfustini@tenstorrent.com \ --cc=emil.renner.berthing@canonical.com \ --cc=frank.li@vivo.com \ --cc=guoren@kernel.org \ --cc=jszhang@kernel.org \ --cc=krzk+dt@kernel.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=mturquette@baylibre.com \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=robh@kernel.org \ --cc=sboyd@kernel.org \ --cc=thomas.bonnefille@bootlin.com \ --cc=wefu@redhat.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.