From: Matthias Brugger <matthias.bgg@gmail.com>
To: sean.wang@mediatek.com, robh+dt@kernel.org, mark.rutland@arm.com,
devicetree@vger.kernel.org, linux-mediatek@lists.infradead.org
Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 11/19] arm: dts: mt7623: use - instead of _ in DT node name
Date: Mon, 19 Mar 2018 00:50:14 +0100 [thread overview]
Message-ID: <4b7707b0-d2d5-7af2-216a-91015ad85173@gmail.com> (raw)
In-Reply-To: <eb782e65f83246f0747ee21fbfb03ab13bddcc12.1519378872.git.sean.wang@mediatek.com>
On 02/23/2018 11:16 AM, sean.wang@mediatek.com wrote:
> From: Sean Wang <sean.wang@mediatek.com>
>
> It should be good that no use "_" is in DT node name. Consequently,
> those nodes in certain files which have an inappropriate name containing
> "_" are all being replaced with "-".
>
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: devicetree@vger.kernel.org
Pushed to v4.16-next/dts32
Thanks!
> ---
> arch/arm/boot/dts/mt7623.dtsi | 12 +++----
> arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts | 50 +++++++++++++--------------
> arch/arm/boot/dts/mt7623n-rfb-nand.dts | 6 ++--
> 3 files changed, 34 insertions(+), 34 deletions(-)
>
> diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
> index 5cf93a4..4b68448 100644
> --- a/arch/arm/boot/dts/mt7623.dtsi
> +++ b/arch/arm/boot/dts/mt7623.dtsi
> @@ -28,7 +28,7 @@
> compatible = "mediatek,mt7623";
> interrupt-parent = <&sysirq>;
>
> - cpu_opp_table: opp_table {
> + cpu_opp_table: opp-table {
> compatible = "operating-points-v2";
> opp-shared;
>
> @@ -145,32 +145,32 @@
> };
>
> thermal-zones {
> - cpu_thermal: cpu_thermal {
> + cpu_thermal: cpu-thermal {
> polling-delay-passive = <1000>;
> polling-delay = <1000>;
>
> thermal-sensors = <&thermal 0>;
>
> trips {
> - cpu_passive: cpu_passive {
> + cpu_passive: cpu-passive {
> temperature = <47000>;
> hysteresis = <2000>;
> type = "passive";
> };
>
> - cpu_active: cpu_active {
> + cpu_active: cpu-active {
> temperature = <67000>;
> hysteresis = <2000>;
> type = "active";
> };
>
> - cpu_hot: cpu_hot {
> + cpu_hot: cpu-hot {
> temperature = <87000>;
> hysteresis = <2000>;
> type = "hot";
> };
>
> - cpu_crit {
> + cpu-crit {
> temperature = <107000>;
> hysteresis = <2000>;
> type = "critical";
> diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> index f2d3cfb..a6295ef 100644
> --- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> +++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> @@ -66,7 +66,7 @@
> regulator-always-on;
> };
>
> - gpio_keys {
> + gpio-keys {
> compatible = "gpio-keys";
> pinctrl-names = "default";
> pinctrl-0 = <&key_pins_a>;
> @@ -263,14 +263,14 @@
>
> &pio {
> cir_pins_a:cir@0 {
> - pins_cir {
> + pins-cir {
> pinmux = <MT7623_PIN_46_IR_FUNC_IR>;
> bias-disable;
> };
> };
>
> i2c0_pins_a: i2c@0 {
> - pins_i2c0 {
> + pins-i2c0 {
> pinmux = <MT7623_PIN_75_SDA0_FUNC_SDA0>,
> <MT7623_PIN_76_SCL0_FUNC_SCL0>;
> bias-disable;
> @@ -278,7 +278,7 @@
> };
>
> i2c1_pins_a: i2c@1 {
> - pin_i2c1 {
> + pin-i2c1 {
> pinmux = <MT7623_PIN_57_SDA1_FUNC_SDA1>,
> <MT7623_PIN_58_SCL1_FUNC_SCL1>;
> bias-disable;
> @@ -286,7 +286,7 @@
> };
>
> i2s0_pins_a: i2s@0 {
> - pin_i2s0 {
> + pin-i2s0 {
> pinmux = <MT7623_PIN_49_I2S0_DATA_FUNC_I2S0_DATA>,
> <MT7623_PIN_72_I2S0_DATA_IN_FUNC_I2S0_DATA_IN>,
> <MT7623_PIN_73_I2S0_LRCK_FUNC_I2S0_LRCK>,
> @@ -298,7 +298,7 @@
> };
>
> i2s1_pins_a: i2s@1 {
> - pin_i2s1 {
> + pin-i2s1 {
> pinmux = <MT7623_PIN_33_I2S1_DATA_FUNC_I2S1_DATA>,
> <MT7623_PIN_34_I2S1_DATA_IN_FUNC_I2S1_DATA_IN>,
> <MT7623_PIN_35_I2S1_BCK_FUNC_I2S1_BCK>,
> @@ -310,7 +310,7 @@
> };
>
> key_pins_a: keys@0 {
> - pins_keys {
> + pins-keys {
> pinmux = <MT7623_PIN_256_GPIO256_FUNC_GPIO256>,
> <MT7623_PIN_257_GPIO257_FUNC_GPIO257> ;
> input-enable;
> @@ -318,7 +318,7 @@
> };
>
> led_pins_a: leds@0 {
> - pins_leds {
> + pins-leds {
> pinmux = <MT7623_PIN_239_EXT_SDIO0_FUNC_GPIO239>,
> <MT7623_PIN_240_EXT_XCS_FUNC_GPIO240>,
> <MT7623_PIN_241_EXT_SCK_FUNC_GPIO241>;
> @@ -326,7 +326,7 @@
> };
>
> mmc0_pins_default: mmc0default {
> - pins_cmd_dat {
> + pins-cmd-dat {
> pinmux = <MT7623_PIN_111_MSDC0_DAT7_FUNC_MSDC0_DAT7>,
> <MT7623_PIN_112_MSDC0_DAT6_FUNC_MSDC0_DAT6>,
> <MT7623_PIN_113_MSDC0_DAT5_FUNC_MSDC0_DAT5>,
> @@ -340,19 +340,19 @@
> bias-pull-up;
> };
>
> - pins_clk {
> + pins-clk {
> pinmux = <MT7623_PIN_117_MSDC0_CLK_FUNC_MSDC0_CLK>;
> bias-pull-down;
> };
>
> - pins_rst {
> + pins-rst {
> pinmux = <MT7623_PIN_115_MSDC0_RSTB_FUNC_MSDC0_RSTB>;
> bias-pull-up;
> };
> };
>
> mmc0_pins_uhs: mmc0 {
> - pins_cmd_dat {
> + pins-cmd-dat {
> pinmux = <MT7623_PIN_111_MSDC0_DAT7_FUNC_MSDC0_DAT7>,
> <MT7623_PIN_112_MSDC0_DAT6_FUNC_MSDC0_DAT6>,
> <MT7623_PIN_113_MSDC0_DAT5_FUNC_MSDC0_DAT5>,
> @@ -367,20 +367,20 @@
> bias-pull-up = <MTK_PUPD_SET_R1R0_01>;
> };
>
> - pins_clk {
> + pins-clk {
> pinmux = <MT7623_PIN_117_MSDC0_CLK_FUNC_MSDC0_CLK>;
> drive-strength = <MTK_DRIVE_2mA>;
> bias-pull-down = <MTK_PUPD_SET_R1R0_01>;
> };
>
> - pins_rst {
> + pins-rst {
> pinmux = <MT7623_PIN_115_MSDC0_RSTB_FUNC_MSDC0_RSTB>;
> bias-pull-up;
> };
> };
>
> mmc1_pins_default: mmc1default {
> - pins_cmd_dat {
> + pins-cmd-dat {
> pinmux = <MT7623_PIN_107_MSDC1_DAT0_FUNC_MSDC1_DAT0>,
> <MT7623_PIN_108_MSDC1_DAT1_FUNC_MSDC1_DAT1>,
> <MT7623_PIN_109_MSDC1_DAT2_FUNC_MSDC1_DAT2>,
> @@ -391,26 +391,26 @@
> bias-pull-up = <MTK_PUPD_SET_R1R0_10>;
> };
>
> - pins_clk {
> + pins-clk {
> pinmux = <MT7623_PIN_106_MSDC1_CLK_FUNC_MSDC1_CLK>;
> bias-pull-down;
> drive-strength = <MTK_DRIVE_4mA>;
> };
>
> - pins_wp {
> + pins-wp {
> pinmux = <MT7623_PIN_29_EINT7_FUNC_MSDC1_WP>;
> input-enable;
> bias-pull-up;
> };
>
> - pins_insert {
> + pins-insert {
> pinmux = <MT7623_PIN_261_MSDC1_INS_FUNC_GPIO261>;
> bias-pull-up;
> };
> };
>
> mmc1_pins_uhs: mmc1 {
> - pins_cmd_dat {
> + pins-cmd-dat {
> pinmux = <MT7623_PIN_107_MSDC1_DAT0_FUNC_MSDC1_DAT0>,
> <MT7623_PIN_108_MSDC1_DAT1_FUNC_MSDC1_DAT1>,
> <MT7623_PIN_109_MSDC1_DAT2_FUNC_MSDC1_DAT2>,
> @@ -421,7 +421,7 @@
> bias-pull-up = <MTK_PUPD_SET_R1R0_10>;
> };
>
> - pins_clk {
> + pins-clk {
> pinmux = <MT7623_PIN_106_MSDC1_CLK_FUNC_MSDC1_CLK>;
> drive-strength = <MTK_DRIVE_4mA>;
> bias-pull-down = <MTK_PUPD_SET_R1R0_10>;
> @@ -437,7 +437,7 @@
> };
>
> pwm_pins_a: pwm@0 {
> - pins_pwm {
> + pins-pwm {
> pinmux = <MT7623_PIN_203_PWM0_FUNC_PWM0>,
> <MT7623_PIN_204_PWM1_FUNC_PWM1>,
> <MT7623_PIN_205_PWM2_FUNC_PWM2>,
> @@ -447,7 +447,7 @@
> };
>
> spi0_pins_a: spi@0 {
> - pins_spi {
> + pins-spi {
> pinmux = <MT7623_PIN_53_SPI0_CSN_FUNC_SPI0_CS>,
> <MT7623_PIN_54_SPI0_CK_FUNC_SPI0_CK>,
> <MT7623_PIN_55_SPI0_MI_FUNC_SPI0_MI>,
> @@ -457,21 +457,21 @@
> };
>
> uart0_pins_a: uart@0 {
> - pins_dat {
> + pins-dat {
> pinmux = <MT7623_PIN_79_URXD0_FUNC_URXD0>,
> <MT7623_PIN_80_UTXD0_FUNC_UTXD0>;
> };
> };
>
> uart1_pins_a: uart@1 {
> - pins_dat {
> + pins-dat {
> pinmux = <MT7623_PIN_81_URXD1_FUNC_URXD1>,
> <MT7623_PIN_82_UTXD1_FUNC_UTXD1>;
> };
> };
>
> uart2_pins_a: uart@2 {
> - pins_dat {
> + pins-dat {
> pinmux = <MT7623_PIN_14_GPIO14_FUNC_URXD2>,
> <MT7623_PIN_15_GPIO15_FUNC_UTXD2>;
> };
> diff --git a/arch/arm/boot/dts/mt7623n-rfb-nand.dts b/arch/arm/boot/dts/mt7623n-rfb-nand.dts
> index e66de86..f729c71 100644
> --- a/arch/arm/boot/dts/mt7623n-rfb-nand.dts
> +++ b/arch/arm/boot/dts/mt7623n-rfb-nand.dts
> @@ -81,13 +81,13 @@
>
> &pio {
> nand_pins_default: nanddefault {
> - pins_ale {
> + pins-ale {
> pinmux = <MT7623_PIN_116_MSDC0_CMD_FUNC_NALE>;
> drive-strength = <MTK_DRIVE_8mA>;
> bias-pull-down = <MTK_PUPD_SET_R1R0_10>;
> };
>
> - pins_dat {
> + pins-dat {
> pinmux = <MT7623_PIN_111_MSDC0_DAT7_FUNC_NLD7>,
> <MT7623_PIN_112_MSDC0_DAT6_FUNC_NLD6>,
> <MT7623_PIN_114_MSDC0_DAT4_FUNC_NLD4>,
> @@ -102,7 +102,7 @@
> bias-pull-up;
> };
>
> - pins_we {
> + pins-we {
> pinmux = <MT7623_PIN_117_MSDC0_CLK_FUNC_NWEB>;
> drive-strength = <MTK_DRIVE_8mA>;
> bias-pull-up = <MTK_PUPD_SET_R1R0_10>;
>
next prev parent reply other threads:[~2018-03-18 23:50 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-23 10:16 [PATCH v1 00/19] refactor dts and add support for more boards sean.wang
2018-02-23 10:16 ` [PATCH v1 01/19] dt-bindings: pinctrl: mediatek: use - instead of _ in examples sean.wang
2018-03-02 7:43 ` Linus Walleij
2018-02-23 10:16 ` [PATCH v1 02/19] dt-bindings: cpufreq: " sean.wang
2018-02-26 4:13 ` Viresh Kumar
2018-03-19 2:51 ` Sean Wang
2018-03-19 10:35 ` Matthias Brugger
2018-03-02 15:53 ` Rob Herring
2018-02-23 10:16 ` [PATCH v1 03/19] dt-bindings: pinctrl: mediatek: add bindings for I2C2 and SPI2 on MT7623 sean.wang
2018-03-02 7:44 ` Linus Walleij
2018-02-23 10:16 ` [PATCH v1 04/19] dt-bindings: arm: mediatek: add support for more mt7623 reference boards sean.wang
2018-03-02 16:00 ` Rob Herring
2018-03-12 11:03 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 05/19] arm: dts: mt7623: add BTIF, HSDMA and SPI-NOR device nodes sean.wang
2018-03-12 9:40 ` Matthias Brugger
2018-03-12 9:53 ` Sean Wang
2018-02-23 10:16 ` [PATCH v1 06/19] arm: dts: mt7623: fix USB initialization fails on bananapi-r2 sean.wang
2018-03-12 11:03 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 07/19] arm: dts: mt7623: fix the regulators mmc should use " sean.wang
2018-03-18 23:49 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 08/19] arm: dts: mt7623: enable three available UARTs " sean.wang
2018-03-18 23:49 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 09/19] arm: dts: mt7623: add related clock properties to cpu[1-3] nodes sean.wang
2018-02-26 4:13 ` Viresh Kumar
2018-03-18 23:49 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 10/19] arm: dts: mt7623: remove useless property pinctrl-names at node switch@0 sean.wang
2018-03-18 23:50 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 11/19] arm: dts: mt7623: use - instead of _ in DT node name sean.wang
2018-03-18 23:50 ` Matthias Brugger [this message]
2018-02-23 10:16 ` [PATCH v1 12/19] arm: dts: mt7623: fix all Warnings (unit_address_vs_reg) sean.wang
2018-03-18 21:54 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 13/19] arm: dts: mt7623: move node mt6323 leds to mt6323.dtsi sean.wang
2018-03-02 15:40 ` Rob Herring
2018-03-02 22:46 ` Sean Wang
2018-03-18 23:46 ` Matthias Brugger
2018-03-18 22:01 ` Matthias Brugger
2018-02-23 10:16 ` [PATCH v1 14/19] arm: dts: mt7623: extend common file to unify all boards with MT7623 SoCs sean.wang
2018-02-23 10:16 ` [PATCH v1 15/19] arm: dts: mt7623: add SPI[1,2], I2C2 and sound related nodes to reference board sean.wang
2018-02-23 10:16 ` [PATCH v1 16/19] arm: dts: mt7623: fixup available memory size on bananapi-r2 sean.wang
2018-03-02 15:42 ` Rob Herring
2018-03-02 23:27 ` Sean Wang
2018-03-05 14:16 ` Rob Herring
2018-03-05 15:46 ` Sean Wang
2018-02-23 10:16 ` [PATCH v1 17/19] arm: dts: mt7623: add MT7623N reference board with eMMC sean.wang
2018-02-23 10:16 ` [PATCH v1 18/19] arm: dts: mt7623: add MT7623A reference boards sean.wang
2018-03-02 15:45 ` Rob Herring
2018-03-03 2:15 ` Sean Wang
2018-03-05 14:10 ` Rob Herring
2018-03-05 15:10 ` Sean Wang
2018-02-23 10:16 ` [PATCH v1 19/19] arm: dts: mediatek: converted to using SPDX identifiers sean.wang
2018-03-02 15:52 ` Rob Herring
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=4b7707b0-d2d5-7af2-216a-91015ad85173@gmail.com \
--to=matthias.bgg@gmail.com \
--cc=devicetree@vger.kernel.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 \
--cc=sean.wang@mediatek.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: 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
Powered by JetHome