devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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 10/19] arm: dts: mt7623: remove useless property pinctrl-names at node switch@0
Date: Mon, 19 Mar 2018 00:50:01 +0100	[thread overview]
Message-ID: <06ea08cd-0b31-49e4-0008-df9933c709cb@gmail.com> (raw)
In-Reply-To: <9c8c23657d9812e24c6f07b7477a6ac0ebdd1c68.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>
> 
> The property pinctrl-names is totally superfluous. It would be good to
> remove the property to keep the node neatness. There is actually
> unnecessary to set up any pins for data path TRGMII between main SoC and
> MT7530. Furthermore, it's more reasonable for the pin setup of control
> path MDIO bus is being placed inside the node of ethernet controller.
> 
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>

Pushed to v4.16-next/dts32

Thanks!

> ---
>  arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> index 6b760be..f2d3cfb 100644
> --- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> +++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> @@ -151,7 +151,6 @@
>  			#address-cells = <1>;
>  			#size-cells = <0>;
>  			reg = <0>;
> -			pinctrl-names = "default";
>  			reset-gpios = <&pio 33 0>;
>  			core-supply = <&mt6323_vpa_reg>;
>  			io-supply = <&mt6323_vemc3v3_reg>;
> 

  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 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
     [not found] ` <cover.1519378871.git.sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2018-02-23 10:16   ` [PATCH v1 01/19] dt-bindings: pinctrl: mediatek: use - instead of _ in examples sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-02  7:43     ` Linus Walleij
2018-02-23 10:16   ` [PATCH v1 02/19] dt-bindings: cpufreq: " sean.wang-NuS5LvNUpcJWk0Htik3J/w
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-NuS5LvNUpcJWk0Htik3J/w
2018-03-02  7:44     ` Linus Walleij
2018-02-23 10:16   ` [PATCH v1 06/19] arm: dts: mt7623: fix USB initialization fails on bananapi-r2 sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-12 11:03     ` Matthias Brugger
2018-02-23 10:16   ` [PATCH v1 19/19] arm: dts: mediatek: converted to using SPDX identifiers sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-02 15:52     ` Rob Herring
2018-02-23 10:16 ` [PATCH v1 07/19] arm: dts: mt7623: fix the regulators mmc should use on bananapi-r2 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 [this message]
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
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

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=06ea08cd-0b31-49e4-0008-df9933c709cb@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;
as well as URLs for NNTP newsgroup(s).