diff mbox series

[U-Boot,4/4] board: ti: am335x-ice: Configure the CDCE913 clock synthesizer

Message ID 20190927161429.20832-5-t-kristo@ti.com
State Accepted
Commit 752a45a153c778bb8b2cdcd0933180b311858b86
Delegated to: Tom Rini
Headers show
Series clk: add support for cdce9xx clock synthesizer | expand

Commit Message

Tero Kristo Sept. 27, 2019, 4:14 p.m. UTC
AM335x-ICE boards contain the CDCE913 clock synthesizer, and their
reset crystal capacitance load value of 10pF is wrong leading into
lost packets in certain networking tests. Add DT data for this
device, and probe it from the board file to program the crystal
capacitance load value to 0pF to avoid any problems.

Signed-off-by: Tero Kristo <t-kristo@ti.com>
---
 arch/arm/dts/am335x-icev2-u-boot.dtsi | 21 +++++++++++++++++++++
 board/ti/am335x/board.c               |  4 ++++
 configs/am335x_evm_defconfig          |  2 ++
 3 files changed, 27 insertions(+)
 create mode 100644 arch/arm/dts/am335x-icev2-u-boot.dtsi

Comments

Tom Rini Oct. 12, 2019, 8:28 p.m. UTC | #1
On Fri, Sep 27, 2019 at 07:14:29PM +0300, Tero Kristo wrote:

> AM335x-ICE boards contain the CDCE913 clock synthesizer, and their
> reset crystal capacitance load value of 10pF is wrong leading into
> lost packets in certain networking tests. Add DT data for this
> device, and probe it from the board file to program the crystal
> capacitance load value to 0pF to avoid any problems.
> 
> Signed-off-by: Tero Kristo <t-kristo@ti.com>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/arch/arm/dts/am335x-icev2-u-boot.dtsi b/arch/arm/dts/am335x-icev2-u-boot.dtsi
new file mode 100644
index 0000000000..cc9569af03
--- /dev/null
+++ b/arch/arm/dts/am335x-icev2-u-boot.dtsi
@@ -0,0 +1,21 @@ 
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2019 Texas Instruments Incorporated - http://www.ti.com/
+ */
+/ {
+	xtal25mhz: xtal25mhz {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <25000000>;
+	};
+};
+
+&i2c0 {
+	cdce913: cdce913@65 {
+		compatible = "ti,cdce913";
+		reg = <0x65>;
+		clocks = <&xtal25mhz>;
+		#clock-cells = <1>;
+		xtal-load-pf = <0>;
+	};
+};
diff --git a/board/ti/am335x/board.c b/board/ti/am335x/board.c
index 7eaa6cd96d..499c872227 100644
--- a/board/ti/am335x/board.c
+++ b/board/ti/am335x/board.c
@@ -791,6 +791,7 @@  int board_init(void)
 #ifdef CONFIG_BOARD_LATE_INIT
 int board_late_init(void)
 {
+	struct udevice *dev;
 #if !defined(CONFIG_SPL_BUILD)
 	uint8_t mac_addr[6];
 	uint32_t mac_hi, mac_lo;
@@ -871,6 +872,9 @@  int board_late_init(void)
 			env_set("serial#", board_serial);
 	}
 
+	/* Just probe the potentially supported cdce913 device */
+	uclass_get_device(UCLASS_CLK, 0, &dev);
+
 	return 0;
 }
 #endif
diff --git a/configs/am335x_evm_defconfig b/configs/am335x_evm_defconfig
index 2aa9b65caf..93a28b290b 100644
--- a/configs/am335x_evm_defconfig
+++ b/configs/am335x_evm_defconfig
@@ -77,3 +77,5 @@  CONFIG_DYNAMIC_CRC_TABLE=y
 CONFIG_RSA=y
 CONFIG_LZO=y
 # CONFIG_OF_LIBFDT_OVERLAY is not set
+CONFIG_CLK=y
+CONFIG_CLK_CDCE9XX=y