From: sergei.shtylyov@cogentembedded.com (Sergei Shtylyov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 2/4] Documentation: dts: Add bindings for APM X-Gene SoC ethernet driver
Date: Fri, 18 Apr 2014 23:51:07 +0400 [thread overview]
Message-ID: <5351822B.7010402@cogentembedded.com> (raw)
In-Reply-To: <1397271984-23405-3-git-send-email-isubramanian@apm.com>
Hello.
On 04/12/2014 07:06 AM, Iyappan Subramanian wrote:
> This patch adds documentation for APM X-Gene SoC ethernet DTS binding.
> Signed-off-by: Iyappan Subramanian <isubramanian@apm.com>
> Signed-off-by: Ravi Patel <rapatel@apm.com>
> Signed-off-by: Keyur Chudgar <kchudgar@apm.com>
> ---
> .../devicetree/bindings/net/apm-xgene-enet.txt | 46 ++++++++++++++++++++
> 1 file changed, 46 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> diff --git a/Documentation/devicetree/bindings/net/apm-xgene-enet.txt b/Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> new file mode 100644
> index 0000000..0c983e8
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> @@ -0,0 +1,46 @@
> +APM X-Gene SoC Ethernet nodes
> +
> +Ethernet nodes are defined to describe on-chip ethernet interfaces in
> +APM X-Gene SoC.
> +
> +Required properties:
> +- compatible : Should be "apm,xgene-enet"
> +- reg : First resource is the ethernet base register set
> + Second resource is the ring base register set
> + Third resource is the ring command register set
> +- interrupts : Ethernet main interrupt
> +- clocks : Reference to the clock entry.
> +- local-mac-address : Ethernet MAC address.
This property is described in
Documentation/devicetree/bindings/net/ethernet.txt, please refer to it.
> +- phy-addr : Ethernet MII phy address.
No, thus won't do. Please use "phy-handle" property to refer to a PHY subnode.
> +- phy-mode : Ethernet MII mode.
This one is described in
Documentation/devicetree/bindings/net/ethernet.txt as well.
WBR, Sergei
WARNING: multiple messages have this Message-ID (diff)
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
To: Iyappan Subramanian <isubramanian@apm.com>,
davem@davemloft.net, netdev@vger.kernel.org,
devicetree@vger.kernel.org
Cc: gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, jcm@redhat.com,
patches@apm.com, Ravi Patel <rapatel@apm.com>,
Keyur Chudgar <kchudgar@apm.com>
Subject: Re: [PATCH v2 2/4] Documentation: dts: Add bindings for APM X-Gene SoC ethernet driver
Date: Fri, 18 Apr 2014 23:51:07 +0400 [thread overview]
Message-ID: <5351822B.7010402@cogentembedded.com> (raw)
In-Reply-To: <1397271984-23405-3-git-send-email-isubramanian@apm.com>
Hello.
On 04/12/2014 07:06 AM, Iyappan Subramanian wrote:
> This patch adds documentation for APM X-Gene SoC ethernet DTS binding.
> Signed-off-by: Iyappan Subramanian <isubramanian@apm.com>
> Signed-off-by: Ravi Patel <rapatel@apm.com>
> Signed-off-by: Keyur Chudgar <kchudgar@apm.com>
> ---
> .../devicetree/bindings/net/apm-xgene-enet.txt | 46 ++++++++++++++++++++
> 1 file changed, 46 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> diff --git a/Documentation/devicetree/bindings/net/apm-xgene-enet.txt b/Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> new file mode 100644
> index 0000000..0c983e8
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> @@ -0,0 +1,46 @@
> +APM X-Gene SoC Ethernet nodes
> +
> +Ethernet nodes are defined to describe on-chip ethernet interfaces in
> +APM X-Gene SoC.
> +
> +Required properties:
> +- compatible : Should be "apm,xgene-enet"
> +- reg : First resource is the ethernet base register set
> + Second resource is the ring base register set
> + Third resource is the ring command register set
> +- interrupts : Ethernet main interrupt
> +- clocks : Reference to the clock entry.
> +- local-mac-address : Ethernet MAC address.
This property is described in
Documentation/devicetree/bindings/net/ethernet.txt, please refer to it.
> +- phy-addr : Ethernet MII phy address.
No, thus won't do. Please use "phy-handle" property to refer to a PHY subnode.
> +- phy-mode : Ethernet MII mode.
This one is described in
Documentation/devicetree/bindings/net/ethernet.txt as well.
WBR, Sergei
next prev parent reply other threads:[~2014-04-18 19:51 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-12 3:06 [PATCH v2 0/4] net: Add APM X-Gene SoC Ethernet driver support Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 3:06 ` [PATCH v2 1/4] MAINTAINERS: Add entry for APM X-Gene SoC ethernet driver Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 3:06 ` [PATCH v2 2/4] Documentation: dts: Add bindings " Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 18:55 ` Florian Fainelli
2014-04-12 18:55 ` Florian Fainelli
2014-04-14 22:11 ` Iyappan Subramanian
2014-04-14 22:11 ` Iyappan Subramanian
2014-04-14 22:11 ` Iyappan Subramanian
2014-04-14 13:06 ` Ben Dooks
2014-04-14 13:06 ` Ben Dooks
2014-04-14 22:15 ` Iyappan Subramanian
2014-04-14 22:15 ` Iyappan Subramanian
2014-04-14 22:15 ` Iyappan Subramanian
2014-04-18 19:51 ` Sergei Shtylyov [this message]
2014-04-18 19:51 ` Sergei Shtylyov
2014-04-12 3:06 ` [PATCH v2 3/4] " Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 3:06 ` [PATCH v2 4/4] drivers: net: Add APM X-Gene SoC ethernet driver support Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 3:06 ` Iyappan Subramanian
2014-04-12 20:55 ` David Miller
2014-04-12 20:55 ` David Miller
2014-04-14 22:09 ` Iyappan Subramanian
2014-04-14 22:09 ` Iyappan Subramanian
2014-04-14 22:09 ` Iyappan Subramanian
2014-04-14 14:05 ` Ben Dooks
2014-04-14 14:05 ` Ben Dooks
2014-04-17 2:06 ` Iyappan Subramanian
2014-04-17 2:06 ` Iyappan Subramanian
2014-04-17 2:06 ` Iyappan Subramanian
2014-04-14 13:07 ` [PATCH v2 0/4] net: Add APM X-Gene SoC Ethernet " Ben Dooks
2014-04-14 13:07 ` Ben Dooks
2014-04-14 13:12 ` Ben Dooks
2014-04-14 13:12 ` Ben Dooks
2014-04-14 13:12 ` Ben Dooks
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=5351822B.7010402@cogentembedded.com \
--to=sergei.shtylyov@cogentembedded.com \
--cc=linux-arm-kernel@lists.infradead.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 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.