On 04/05/2016 01:26 PM, Dan Murphy wrote: > On 04/05/2016 12:40 PM, Michal Simek wrote: >> On 5.4.2016 19:23, 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 <dmur...@ti.com> >>> --- >>> >>> v2 - Align the documentation with the actual bindings - >>> https://patchwork.ozlabs.org/patch/606502/ >>> >>> doc/device-tree-bindings/net/ti,dp83867.txt | 28 +++++++++++++++++++++++ >>> include/dt-bindings/net/ti-dp83867.h | 35 >>> +++++++++++++++++++++++++++++ >>> 2 files changed, 63 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..34b1c35 >>> --- /dev/null >>> +++ b/doc/device-tree-bindings/net/ti,dp83867.txt >>> @@ -0,0 +1,28 @@ >>> +* Texas Instruments - dp83867 Giga bit ethernet phy >>> + >>> +Required properties: >>> + - ti,rx_int_delay - RGMII Recieve Clock Delay - see >>> dt-bindings/net/ti-dp83867.h >>> + for applicable values >>> + - ti,tx_int_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_int_delay = <DP83867_RGMIIDCTL_1_50_NS>; >>> + ti,tx_int_delay = <DP83867_RGMIIDCTL_2_50_NS>; >>> + ti,fifo_depth = <DP83867_PHYCR_FIFO_DEPTH_8_B_NIB>; >>> + >>> +}; >>> + >>> +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 >>> >> Nack from me. Just follow Linux binding instead of create new one. > Why a nack? > > This comes from the Linux kernel mainline code I submited. I am not creating > anything new. > This technically should have come in with the original port of the driver I > pushed to the Linux kernel. > > But I defer to the Tom for the resolution.
Well boo. I know what you are referring to now. I did not use ti,tx-internal-delay and ti,rx-internal-delay. I used ti,tx-int-delay and ti,rx-int-delay which is wrong. Here come v3 with the update Dan > > Dan > >> Thanks, >> Michal >> > -- ------------------ Dan Murphy _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot