From: Conor Dooley <conor@kernel.org> To: Mason Huo <mason.huo@starfivetech.com> Cc: Conor Dooley <conor.dooley@microchip.com>, "Rafael J. Wysocki" <rafael@kernel.org>, Viresh Kumar <viresh.kumar@linaro.org>, Emil Renner Berthing <kernel@esmil.dk>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Paul Walmsley <paul.walmsley@sifive.com>, Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu>, Shengyu Qu <wiagn233@outlook.com>, "linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-riscv@lists.infradead.org" <linux-riscv@lists.infradead.org> Subject: Re: [PATCH v3 0/3] Add JH7110 cpufreq support Date: Mon, 5 Jun 2023 10:54:47 +0100 [thread overview] Message-ID: <20230605-strive-caddie-e570c6c4f958@spud> (raw) In-Reply-To: <c5bde0d92d514250a21120ed33e0f6b2@EXMBX067.cuchost.com> [-- Attachment #1: Type: text/plain, Size: 2087 bytes --] On Mon, Jun 05, 2023 at 09:36:51AM +0000, Mason Huo wrote: > > On Fri, May 05, 2023 at 09:38:38AM +0800, Mason Huo wrote: > >> Thanks for your review, and is there any comments about these v3 patches? > > > Firstly there appears to have been some mess-up with the driver/bindings > > for 1/3, so I am waiting to see if the binding gets reverted before > > doing anything and secondly it's the merge window so I can't do anything > > about 3/3 until next week. > Could you help to check if this patch can be moved on? The dependencies are in, I guess when I went checking through things on patchwork after the merge window I spotted some issue & didn't reply to the patch. So sorry about that. The issue that I must've spotted is that patch 1/3 doesn't pass dtbs_check: arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.2a.dtb: pmic@36: 'interrupts' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.2a.dtb: pmic@36: '#interrupt-cells' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.2a.dtb: pmic@36: 'interrupt-controller' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.3b.dtb: pmic@36: 'interrupts' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.3b.dtb: pmic@36: '#interrupt-cells' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.3b.dtb: pmic@36: 'interrupt-controller' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml Could you please fix that up & resend the 2 unapplied patches? Cheers, Conor. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Conor Dooley <conor@kernel.org> To: Mason Huo <mason.huo@starfivetech.com> Cc: Conor Dooley <conor.dooley@microchip.com>, "Rafael J. Wysocki" <rafael@kernel.org>, Viresh Kumar <viresh.kumar@linaro.org>, Emil Renner Berthing <kernel@esmil.dk>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Paul Walmsley <paul.walmsley@sifive.com>, Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu>, Shengyu Qu <wiagn233@outlook.com>, "linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-riscv@lists.infradead.org" <linux-riscv@lists.infradead.org> Subject: Re: [PATCH v3 0/3] Add JH7110 cpufreq support Date: Mon, 5 Jun 2023 10:54:47 +0100 [thread overview] Message-ID: <20230605-strive-caddie-e570c6c4f958@spud> (raw) In-Reply-To: <c5bde0d92d514250a21120ed33e0f6b2@EXMBX067.cuchost.com> [-- Attachment #1.1: Type: text/plain, Size: 2087 bytes --] On Mon, Jun 05, 2023 at 09:36:51AM +0000, Mason Huo wrote: > > On Fri, May 05, 2023 at 09:38:38AM +0800, Mason Huo wrote: > >> Thanks for your review, and is there any comments about these v3 patches? > > > Firstly there appears to have been some mess-up with the driver/bindings > > for 1/3, so I am waiting to see if the binding gets reverted before > > doing anything and secondly it's the merge window so I can't do anything > > about 3/3 until next week. > Could you help to check if this patch can be moved on? The dependencies are in, I guess when I went checking through things on patchwork after the merge window I spotted some issue & didn't reply to the patch. So sorry about that. The issue that I must've spotted is that patch 1/3 doesn't pass dtbs_check: arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.2a.dtb: pmic@36: 'interrupts' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.2a.dtb: pmic@36: '#interrupt-cells' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.2a.dtb: pmic@36: 'interrupt-controller' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.3b.dtb: pmic@36: 'interrupts' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.3b.dtb: pmic@36: '#interrupt-cells' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml arch/riscv/boot/dts/starfive/jh7110-starfive-visionfive-2-v1.3b.dtb: pmic@36: 'interrupt-controller' is a required property From schema: Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml Could you please fix that up & resend the 2 unapplied patches? Cheers, Conor. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 161 bytes --] _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2023-06-05 9:55 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-04-21 3:14 [PATCH v3 0/3] Add JH7110 cpufreq support Mason Huo 2023-04-21 3:14 ` Mason Huo 2023-04-21 3:14 ` [PATCH v3 1/3] riscv: dts: starfive: Enable axp15060 pmic for cpufreq Mason Huo 2023-04-21 3:14 ` Mason Huo 2023-04-21 3:14 ` [PATCH v3 2/3] cpufreq: dt-platdev: Add JH7110 SOC to the allowlist Mason Huo 2023-04-21 3:14 ` Mason Huo 2023-04-21 7:37 ` Viresh Kumar 2023-04-21 7:37 ` Viresh Kumar 2023-04-21 3:14 ` [PATCH v3 3/3] riscv: dts: starfive: Add cpu scaling for JH7110 SoC Mason Huo 2023-04-21 3:14 ` Mason Huo 2023-05-05 1:38 ` [PATCH v3 0/3] Add JH7110 cpufreq support Mason Huo 2023-05-05 1:38 ` Mason Huo 2023-05-05 6:29 ` Conor Dooley 2023-05-05 6:29 ` Conor Dooley 2023-06-05 9:36 ` Mason Huo 2023-06-05 9:36 ` Mason Huo 2023-06-05 9:54 ` Conor Dooley [this message] 2023-06-05 9:54 ` Conor Dooley
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=20230605-strive-caddie-e570c6c4f958@spud \ --to=conor@kernel.org \ --cc=aou@eecs.berkeley.edu \ --cc=conor.dooley@microchip.com \ --cc=devicetree@vger.kernel.org \ --cc=kernel@esmil.dk \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=mason.huo@starfivetech.com \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=rafael@kernel.org \ --cc=robh+dt@kernel.org \ --cc=viresh.kumar@linaro.org \ --cc=wiagn233@outlook.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.