From: Mugunthan V N <mugunthanvnm@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [uboot][PATCH v3 1/2] net: phy: dp83867: Add device tree bindings and documentation
Date: Thu, 7 Apr 2016 10:15:14 +0530 [thread overview]
Message-ID: <5705E5DA.1050208@ti.com> (raw)
In-Reply-To: <1459942680-18696-1-git-send-email-dmurphy@ti.com>
On Wednesday 06 April 2016 05:07 PM, Dan Murphy wrote:
> Add the device tree bindings and the accompanying documentation
> for the TI DP83867 Giga bit ethernet phy driver.
>
> The original document was from:
> [commit 2a10154abcb75ad0d7b6bfea6210ac743ec60897 from the Linux kernel]
>
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
> ---
>
> v3 - Modify the binding to the kernel changed int -> internal for the delay - https://patchwork.ozlabs.org/patch/606595/
>
> doc/device-tree-bindings/net/ti,dp83867.txt | 29 ++++++++++++++++++++++++
> include/dt-bindings/net/ti-dp83867.h | 35 +++++++++++++++++++++++++++++
> 2 files changed, 64 insertions(+)
> create mode 100644 doc/device-tree-bindings/net/ti,dp83867.txt
> create mode 100644 include/dt-bindings/net/ti-dp83867.h
>
> diff --git a/doc/device-tree-bindings/net/ti,dp83867.txt b/doc/device-tree-bindings/net/ti,dp83867.txt
> new file mode 100644
> index 0000000..0ec009c
> --- /dev/null
> +++ b/doc/device-tree-bindings/net/ti,dp83867.txt
> @@ -0,0 +1,29 @@
> +* Texas Instruments - dp83867 Giga bit ethernet phy
> +
> +Required properties:
> + - ti,rx-internal-delay - RGMII Recieve Clock Delay - see dt-bindings/net/ti-dp83867.h
> + for applicable values
> + - ti,tx-internal-delay - RGMII Transmit Clock Delay - see dt-bindings/net/ti-dp83867.h
> + for applicable values
> + - ti,fifo-depth - Transmitt FIFO depth- see dt-bindings/net/ti-dp83867.h
> + for applicable values
> +
> +Default child nodes are standard Ethernet PHY device
> +nodes as described in doc/devicetree/bindings/net/ethernet.txt
> +
> +Example:
> +
> +&mac {
> + pinctrl-0 = <&davinci_mdio_default>;
> + pinctrl-1 = <&davinci_mdio_sleep>;
> + status = "okay";
> +
> + ti,rx_internal_delay = <DP83867_RGMIIDCTL_1_50_NS>;
> + ti,tx_internal_delay = <DP83867_RGMIIDCTL_2_50_NS>;
> + ti,fifo_depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>;
> +
> +};
This example should in phy node and not in mac node as per the linux
commmit mentioned above?
Regards
Mugunthan V N
> +
> +
> +Datasheet can be found:
> +http://www.ti.com/product/DP83867IR/datasheet
> diff --git a/include/dt-bindings/net/ti-dp83867.h b/include/dt-bindings/net/ti-dp83867.h
> new file mode 100644
> index 0000000..5c592fb
> --- /dev/null
> +++ b/include/dt-bindings/net/ti-dp83867.h
> @@ -0,0 +1,35 @@
> +/*
> + * TI DP83867 PHY drivers
> + *
> + * SPDX-License-Identifier: GPL-2.0
> + *
> + */
> +
> +#ifndef _DT_BINDINGS_TI_DP83867_H
> +#define _DT_BINDINGS_TI_DP83867_H
> +
> +/* PHY CTRL bits */
> +#define DP83867_PHYCR_FIFO_DEPTH_3_B_NIB 0x00
> +#define DP83867_PHYCR_FIFO_DEPTH_4_B_NIB 0x01
> +#define DP83867_PHYCR_FIFO_DEPTH_6_B_NIB 0x02
> +#define DP83867_PHYCR_FIFO_DEPTH_8_B_NIB 0x03
> +
> +/* RGMIIDCTL internal delay for rx and tx */
> +#define DP83867_RGMIIDCTL_250_PS 0x0
> +#define DP83867_RGMIIDCTL_500_PS 0x1
> +#define DP83867_RGMIIDCTL_750_PS 0x2
> +#define DP83867_RGMIIDCTL_1_NS 0x3
> +#define DP83867_RGMIIDCTL_1_25_NS 0x4
> +#define DP83867_RGMIIDCTL_1_50_NS 0x5
> +#define DP83867_RGMIIDCTL_1_75_NS 0x6
> +#define DP83867_RGMIIDCTL_2_00_NS 0x7
> +#define DP83867_RGMIIDCTL_2_25_NS 0x8
> +#define DP83867_RGMIIDCTL_2_50_NS 0x9
> +#define DP83867_RGMIIDCTL_2_75_NS 0xa
> +#define DP83867_RGMIIDCTL_3_00_NS 0xb
> +#define DP83867_RGMIIDCTL_3_25_NS 0xc
> +#define DP83867_RGMIIDCTL_3_50_NS 0xd
> +#define DP83867_RGMIIDCTL_3_75_NS 0xe
> +#define DP83867_RGMIIDCTL_4_00_NS 0xf
> +
> +#endif
>
next prev parent reply other threads:[~2016-04-07 4:45 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-06 11:37 [U-Boot] [uboot][PATCH v3 1/2] net: phy: dp83867: Add device tree bindings and documentation Dan Murphy
2016-04-06 11:38 ` [U-Boot] [uboot][PATCH v3 2/2] net: phy: ti: Allow the driver to be more configurable Dan Murphy
2016-04-07 4:45 ` Mugunthan V N [this message]
2016-04-07 16:02 ` [U-Boot] [uboot][PATCH v3 1/2] net: phy: dp83867: Add device tree bindings and documentation Dan Murphy
2016-04-08 9:25 ` Michal Simek
2016-04-08 12:05 ` Dan Murphy
2016-04-08 12:29 ` Michal Simek
2016-04-08 15:26 ` Dan Murphy
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=5705E5DA.1050208@ti.com \
--to=mugunthanvnm@ti.com \
--cc=u-boot@lists.denx.de \
/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