From: Matthias Brugger <matthias.bgg@gmail.com>
To: Hsin-Yi Wang <hsinyi@chromium.org>, linux-arm-kernel@lists.infradead.org
Cc: Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org,
Nicolas Boichat <drinkcat@chromium.org>,
linux-kernel@vger.kernel.org, Daniel Kurtz <djkurtz@chromium.org>,
Rob Herring <robh+dt@kernel.org>,
linux-mediatek@lists.infradead.org,
Enric Balletbo i Serra <enric.balletbo@collabora.com>
Subject: Re: [PATCH v7 3/5] arm64: dts: mt8173: fix unit name warnings
Date: Thu, 20 Feb 2020 23:17:46 +0100 [thread overview]
Message-ID: <bbc75f19-0581-c902-a455-13157d66d72f@gmail.com> (raw)
In-Reply-To: <20200210063523.133333-4-hsinyi@chromium.org>
On 10/02/2020 07:35, Hsin-Yi Wang wrote:
> Fixing several unit name warnings:
>
> Warning (unit_address_vs_reg): /oscillator@0: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /oscillator@1: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /oscillator@2: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /thermal-zones/cpu_thermal/trips/trip-point@0: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /thermal-zones/cpu_thermal/trips/trip-point@1: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /thermal-zones/cpu_thermal/trips/cpu_crit@0: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /thermal-zones/cpu_thermal/cooling-maps/map@0: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /thermal-zones/cpu_thermal/cooling-maps/map@1: node has a unit name, but no reg property
> Warning (unit_address_vs_reg): /reserved-memory/vpu_dma_mem_region: node has a reg or ranges property, but no unit name
> Warning (simple_bus_reg): /soc/pinctrl@10005000: simple-bus unit address format error, expected "1000b000"
> Warning (simple_bus_reg): /soc/interrupt-controller@10220000: simple-bus unit address format error, expected "10221000"
> Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-'
>
> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
> ---
> arch/arm64/boot/dts/mediatek/mt8173.dtsi | 38 ++++++++++++------------
> 1 file changed, 19 insertions(+), 19 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> index 790cd64aa447..2b7f566fb407 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> @@ -42,14 +42,14 @@ aliases {
> dpi0 = &dpi0;
> dsi0 = &dsi0;
> dsi1 = &dsi1;
> - mdp_rdma0 = &mdp_rdma0;
> - mdp_rdma1 = &mdp_rdma1;
> - mdp_rsz0 = &mdp_rsz0;
> - mdp_rsz1 = &mdp_rsz1;
> - mdp_rsz2 = &mdp_rsz2;
> - mdp_wdma0 = &mdp_wdma0;
> - mdp_wrot0 = &mdp_wrot0;
> - mdp_wrot1 = &mdp_wrot1;
> + mdp-rdma0 = &mdp_rdma0;
> + mdp-rdma1 = &mdp_rdma1;
> + mdp-rsz0 = &mdp_rsz0;
> + mdp-rsz1 = &mdp_rsz1;
> + mdp-rsz2 = &mdp_rsz2;
> + mdp-wdma0 = &mdp_wdma0;
> + mdp-wrot0 = &mdp_wrot0;
> + mdp-wrot1 = &mdp_wrot1;
Won't we need to update the mdp driver as well, as it uses of_alias_get_id()?
> serial0 = &uart0;
> serial1 = &uart1;
> serial2 = &uart2;
> @@ -246,21 +246,21 @@ psci {
> cpu_on = <0x84000003>;
> };
>
> - clk26m: oscillator@0 {
> + clk26m: oscillator0 {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> clock-frequency = <26000000>;
> clock-output-names = "clk26m";
> };
>
> - clk32k: oscillator@1 {
> + clk32k: oscillator1 {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> clock-frequency = <32000>;
> clock-output-names = "clk32k";
> };
>
> - cpum_ck: oscillator@2 {
> + cpum_ck: oscillator2 {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> clock-frequency = <0>;
> @@ -276,19 +276,19 @@ cpu_thermal: cpu_thermal {
> sustainable-power = <1500>; /* milliwatts */
>
> trips {
> - threshold: trip-point@0 {
> + threshold: trip-point0 {
> temperature = <68000>;
> hysteresis = <2000>;
> type = "passive";
> };
>
> - target: trip-point@1 {
> + target: trip-point1 {
> temperature = <85000>;
> hysteresis = <2000>;
> type = "passive";
> };
>
> - cpu_crit: cpu_crit@0 {
> + cpu_crit: cpu_crit0 {
> temperature = <115000>;
> hysteresis = <2000>;
> type = "critical";
> @@ -296,13 +296,13 @@ cpu_crit: cpu_crit@0 {
> };
>
> cooling-maps {
> - map@0 {
> + map0 {
> trip = <&target>;
> cooling-device = <&cpu0 0 0>,
> <&cpu1 0 0>;
> contribution = <3072>;
> };
> - map@1 {
> + map1 {
> trip = <&target>;
> cooling-device = <&cpu2 0 0>,
> <&cpu3 0 0>;
> @@ -316,7 +316,7 @@ reserved-memory {
> #address-cells = <2>;
> #size-cells = <2>;
> ranges;
> - vpu_dma_reserved: vpu_dma_mem_region {
> + vpu_dma_reserved: vpu_dma_mem_region@b7000000 {
> compatible = "shared-dma-pool";
> reg = <0 0xb7000000 0 0x500000>;
> alignment = <0x1000>;
> @@ -368,7 +368,7 @@ syscfg_pctl_a: syscfg_pctl_a@10005000 {
> reg = <0 0x10005000 0 0x1000>;
> };
>
> - pio: pinctrl@10005000 {
> + pio: pinctrl@1000b000 {
> compatible = "mediatek,mt8173-pinctrl";
> reg = <0 0x1000b000 0 0x1000>;
> mediatek,pctl-regmap = <&syscfg_pctl_a>;
> @@ -575,7 +575,7 @@ mipi_tx1: mipi-dphy@10216000 {
> status = "disabled";
> };
>
> - gic: interrupt-controller@10220000 {
> + gic: interrupt-controller@10221000 {
> compatible = "arm,gic-400";
> #interrupt-cells = <3>;
> interrupt-parent = <&gic>;
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-02-20 22:18 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-10 6:35 [PATCH v7 0/5] Add mt8173 elm and hana board Hsin-Yi Wang
2020-02-10 6:35 ` [PATCH v7 1/5] dt-bindings: arm64: dts: mediatek: Add mt8173 elm and hana Hsin-Yi Wang
2020-04-13 17:14 ` Matthias Brugger
2020-02-10 6:35 ` [PATCH v7 2/5] arm64: dts: mt8173: add uart aliases Hsin-Yi Wang
2020-04-13 17:14 ` Matthias Brugger
2020-02-10 6:35 ` [PATCH v7 3/5] arm64: dts: mt8173: fix unit name warnings Hsin-Yi Wang
2020-02-20 22:17 ` Matthias Brugger [this message]
2020-02-21 2:22 ` Hsin-Yi Wang
2020-04-13 17:16 ` Matthias Brugger
2020-02-10 6:35 ` [PATCH v7 4/5] arm64: dts: mediatek: add mt8173 elm and hana board Hsin-Yi Wang
2020-04-13 17:16 ` Matthias Brugger
2020-02-10 6:35 ` [PATCH v7 5/5] media: mtk-mdp: Use correct aliases name Hsin-Yi Wang
2020-04-13 17:20 ` Matthias Brugger
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=bbc75f19-0581-c902-a455-13157d66d72f@gmail.com \
--to=matthias.bgg@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=djkurtz@chromium.org \
--cc=drinkcat@chromium.org \
--cc=enric.balletbo@collabora.com \
--cc=hsinyi@chromium.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).