diff mbox series

[OpenWrt-Devel,2/5] apm821xx: set uart compatible to ns16750

Message ID 74ba72893a16e4cd80a3f09a0c31771ec0ae5c22.1539626110.git.chunkeey@gmail.com
State Accepted
Delegated to: Hauke Mehrtens
Headers show
Series [OpenWrt-Devel,1/5] build: remove obsolete -rc kernel testing rewrites | expand

Commit Message

Christian Lamparter Oct. 15, 2018, 6 p.m. UTC
The APM821XX/PPC460EX datasheet mentions:
"Register compatibility with 16750 register set"

Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
---
 target/linux/apm821xx/dts/apm82181.dtsi | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)
diff mbox series

Patch

diff --git a/target/linux/apm821xx/dts/apm82181.dtsi b/target/linux/apm821xx/dts/apm82181.dtsi
index be8ace9207..79596da672 100644
--- a/target/linux/apm821xx/dts/apm82181.dtsi
+++ b/target/linux/apm821xx/dts/apm82181.dtsi
@@ -306,24 +306,29 @@ 
 			};
 
 			UART0: serial@ef600300 {
-				device_type = "serial";
-				compatible = "ns16550";
+				/*
+				 * AMCC's BSP u-boot scans for the "ns16550"
+				 * compatible, without it, u-boot wouldn't
+				 * set the "clock-frequency" which is required.
+				 *
+				 * However, the hardware documentation lists:
+				 * "Register compatibility with 16750 register set"
+				 */
+				compatible = "ns16750", "ns16550";
 				reg = <0xef600300 0x00000008>;
 				virtual-reg = <0xef600300>;
 				clock-frequency = <0>; /* Filled in by U-Boot */
-				current-speed = <0>; /* Filled in by U-Boot */
 				interrupt-parent = <&UIC1>;
 				interrupts = <0x01 IRQ_TYPE_LEVEL_HIGH>;
 				status = "disabled";
 			};
 
 			UART1: serial@ef600400 {
-				device_type = "serial";
-				compatible = "ns16550";
+				/* same "ns16750" as with UART0 */
+				compatible = "ns16750", "ns16550";
 				reg = <0xef600400 0x00000008>;
 				virtual-reg = <0xef600400>;
 				clock-frequency = <0>; /* Filled in by U-Boot */
-				current-speed = <0>; /* Filled in by U-Boot */
 				interrupt-parent = <&UIC0>;
 				interrupts = <0x01 IRQ_TYPE_LEVEL_HIGH>;
 				status = "disabled";