Re: [U-Boot] [PATCH 2/4] board: ti: am57xx-idk: Configure the CDCE913 clock synthesizer

2019-10-12 Thread Tom Rini
On Fri, Sep 27, 2019 at 07:14:27PM +0300, Tero Kristo wrote:

> AM57xx-IDK 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 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH 2/4] board: ti: am57xx-idk: Configure the CDCE913 clock synthesizer

2019-09-27 Thread Tero Kristo
AM57xx-IDK 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 
---
 arch/arm/dts/am571x-idk-u-boot.dtsi|  5 +
 arch/arm/dts/am572x-idk-u-boot.dtsi|  5 +
 arch/arm/dts/am574x-idk-u-boot.dtsi|  5 +
 arch/arm/dts/am57xx-idk-common-u-boot.dtsi | 23 ++
 board/ti/am57xx/board.c|  5 +
 configs/am57xx_evm_defconfig   |  2 ++
 6 files changed, 45 insertions(+)
 create mode 100644 arch/arm/dts/am571x-idk-u-boot.dtsi
 create mode 100644 arch/arm/dts/am572x-idk-u-boot.dtsi
 create mode 100644 arch/arm/dts/am574x-idk-u-boot.dtsi
 create mode 100644 arch/arm/dts/am57xx-idk-common-u-boot.dtsi

diff --git a/arch/arm/dts/am571x-idk-u-boot.dtsi 
b/arch/arm/dts/am571x-idk-u-boot.dtsi
new file mode 100644
index 00..65199200ed
--- /dev/null
+++ b/arch/arm/dts/am571x-idk-u-boot.dtsi
@@ -0,0 +1,5 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2019 Texas Instruments Incorporated - http://www.ti.com/
+ */
+#include "am57xx-idk-common-u-boot.dtsi"
diff --git a/arch/arm/dts/am572x-idk-u-boot.dtsi 
b/arch/arm/dts/am572x-idk-u-boot.dtsi
new file mode 100644
index 00..65199200ed
--- /dev/null
+++ b/arch/arm/dts/am572x-idk-u-boot.dtsi
@@ -0,0 +1,5 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2019 Texas Instruments Incorporated - http://www.ti.com/
+ */
+#include "am57xx-idk-common-u-boot.dtsi"
diff --git a/arch/arm/dts/am574x-idk-u-boot.dtsi 
b/arch/arm/dts/am574x-idk-u-boot.dtsi
new file mode 100644
index 00..65199200ed
--- /dev/null
+++ b/arch/arm/dts/am574x-idk-u-boot.dtsi
@@ -0,0 +1,5 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2019 Texas Instruments Incorporated - http://www.ti.com/
+ */
+#include "am57xx-idk-common-u-boot.dtsi"
diff --git a/arch/arm/dts/am57xx-idk-common-u-boot.dtsi 
b/arch/arm/dts/am57xx-idk-common-u-boot.dtsi
new file mode 100644
index 00..b07aea0048
--- /dev/null
+++ b/arch/arm/dts/am57xx-idk-common-u-boot.dtsi
@@ -0,0 +1,23 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2019 Texas Instruments Incorporated - http://www.ti.com/
+ */
+#include "omap5-u-boot.dtsi"
+
+/ {
+   xtal25mhz: xtal25mhz {
+   compatible = "fixed-clock";
+   #clock-cells = <0>;
+   clock-frequency = <2500>;
+   };
+};
+
+&i2c1 {
+   cdce913: cdce913@65 {
+   compatible = "ti,cdce913";
+   reg = <0x65>;
+   clocks = <&xtal25mhz>;
+   #clock-cells = <1>;
+   xtal-load-pf = <0>;
+   };
+};
diff --git a/board/ti/am57xx/board.c b/board/ti/am57xx/board.c
index f78e6c2e1f..237a834c53 100644
--- a/board/ti/am57xx/board.c
+++ b/board/ti/am57xx/board.c
@@ -30,6 +30,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "../common/board_detect.h"
 #include "mux_data.h"
@@ -689,6 +690,7 @@ int board_late_init(void)
 {
setup_board_eeprom_env();
u8 val;
+   struct udevice *dev;
 
/*
 * DEV_CTRL.DEV_ON = 1 please - else palmas switches off in 8 seconds
@@ -720,6 +722,9 @@ int board_late_init(void)
 
am57x_idk_lcd_detect();
 
+   /* Just probe the potentially supported cdce913 device */
+   uclass_get_device(UCLASS_CLK, 0, &dev);
+
 #if !defined(CONFIG_SPL_BUILD)
board_ti_set_ethaddr(2);
 #endif
diff --git a/configs/am57xx_evm_defconfig b/configs/am57xx_evm_defconfig
index 3b613e135c..e2f558cde7 100644
--- a/configs/am57xx_evm_defconfig
+++ b/configs/am57xx_evm_defconfig
@@ -98,3 +98,5 @@ CONFIG_USB_GADGET=y
 CONFIG_USB_GADGET_MANUFACTURER="Texas Instruments"
 CONFIG_USB_GADGET_VENDOR_NUM=0x0451
 CONFIG_USB_GADGET_PRODUCT_NUM=0xd022
+CONFIG_CLK=y
+CONFIG_CLK_CDCE9XX=y
-- 
2.17.1

--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. 
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot