On 01/19/2016 06:12 AM, Ralf Ramsauer wrote:
This patch enables the APB DMA high speed UARTs of the Jetson TK1.
diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
+ /* First high speed UART */
+ serial@0,70006000 {
+ compatible = "nvidia,tegra124-hsuart", "nvidia,tegra30-hsuart";
+ status = "okay";
+ };
It would be nice if the comment described the HW connectivity, i.e.
which signals the UART was connected to on the board, just like the
comments for other IO controllers already enabled in the DT file. I'd
suggest replacing the comment above with:
/* Expansion BR_UART1_RXD/_TXD */
+ /* Second high speed UART */
+ serial@0,70006040 {
+ compatible = "nvidia,tegra124-hsuart", "nvidia,tegra30-hsuart";
+ status = "okay";
+ };
... and that commetn with:
/* Expansion UART2_RXD/_TXD/_RTS/_CTS */
+
+ /* Third high speed UART */
+ serial@0,70006200 {
+ compatible = "nvidia,tegra124-hsuart", "nvidia,tegra30-hsuart";
+ status = "okay";
+ };
That UART doesn't seem to be used at all according to the schematics and
pinmux spreadsheet. Do you have any reference to the contrary aside from
the L4T DT file? I believe it shouldn't be enabled.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
