arm64: dts: imx8mp: Improve bluetooth UART on DH electronics i.MX8M Plus DHCOM

Use PLL1_80M instead of PLL3 to drive UART2 clock divided down to 80 MHz
instead of 64 MHz to obtain suitable block clock for exact 4 Mbdps, which
is the maximum supported baud rate by the muRata 2AE BT UART.

The difference here is that at 64 MHz UART block clock, the clock with are
divided by 16 (due to oversampling) to 4 MHz and the baud rate generator
then needs to be set to UBIR+1/UBMR+1 = 1/1 to yield 4 Mbdps . In case of
80 MHz UART block clock divided by 16 to 5 MHz, the baud rate generator
needs to be set to UBIR+1/UBMR+1 = 4/5 to yield 4 Mbdps .

Both options are valid and yield the same result, except using the PLL1_80M
output requires fewer clock tree changes, since the PLL1 already generates
the 80 MHz usable for UART, which frees the PLL3 for other uses.

Suggested-by: Christoph Niedermaier <cniedermaier@dh-electronics.com>
Signed-off-by: Marek Vasut <marex@denx.de>
Reviewed-by: Fabio Estevam <festevam@denx.de>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi
index 21b1d75a9a1cfa103af7d9bc439a93ba9d28b992..35394b611f6723a0d0c1ebd16aeab8cedba2e014 100644 (file)
--- a/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-dhcom-som.dtsi
@@ -420,14 +420,15 @@
        status = "okay";
 
        /*
-        * PLL3 at 320 MHz supplies UART2 root with 64 MHz clock,
-        * which with 16x oversampling yields 4 Mbdps baud base,
+        * PLL1 at 80 MHz supplies UART2 root with 80 MHz clock,
+        * which with 16x oversampling yields 5 Mbdps baud base,
+        * which can be well divided by 5/4 to achieve 4 Mbdps,
         * which is exactly the maximum rate supported by muRata
         * 2AE bluetooth UART.
         */
-       assigned-clocks = <&clk IMX8MP_SYS_PLL3>, <&clk IMX8MP_CLK_UART2>;
-       assigned-clock-parents = <0>, <&clk IMX8MP_SYS_PLL3_OUT>;
-       assigned-clock-rates = <320000000>, <64000000>;
+       assigned-clocks = <&clk IMX8MP_CLK_UART2>;
+       assigned-clock-parents = <&clk IMX8MP_SYS_PLL1_80M>;
+       assigned-clock-rates = <80000000>;
 
        bluetooth {
                pinctrl-names = "default";