Re: [PATCH v1 6/6] arm64: dts: allwinner: h616: Add BigTreeTech Pi support

From: Martin Botka
Date: Thu Aug 03 2023 - 11:46:25 EST




On 8/3/23 2:39 PM, Andre Przywara wrote:
On Thu, 3 Aug 2023 00:02:39 +0200
Martin Botka <martin@xxxxxxxxxx> wrote:

Hi,

The BigTreeTech Pi is an H616 based board based on CB1.
Just in Rpi format board.

It features the same internals as BTT CB1 but adds:
- Fan port
- IR receiver

You would then need to enable the "ir" node, check the X96 Mate .dts.
Got it.

And I think this "ADXL345 SPI" connector justifies enabling the respective
SPI node.
Yea the accelerator. Sadly this is bit hard to actually test fully if not wired up to a printer. Tho i do have a spare board to serve as fake printer. Klipper will not know any better :)

- 24V DC power supply via terminal plugs
- USB to CAN module connector (The actual USB to CAN happens on the external module)

Still confused how this works. So you would need to connect an USB->CAN
adapter to one of the USB ports, but how do you feed the CAN signal back
to the board? Or if this is done via this port, where does it go then? To
the GPIO header?
OK so its not great but an USB port is also connected to 2x2 pin connector and secondary 2x2 pin connector slightly away. The USB to CAN module plugs into these 2x2 pin connectors where one serves as USB and the other one receives the CAN signal from the module and just wires it up to an XH2.54 3 pin connector with CAN+ CAN- and GND.

List of currently working things is the same as BTT CB1.

Signed-off-by: Martin Botka <martin@xxxxxxxxxx>
---
.../allwinner/sun50i-h616-bigtreetech-pi.dts | 44 +++++++++++++++++++

Please wire this up in the Makefile as well, otherwise we will miss out on
the build testing and DTB checking.
YEP very much my bad on that one. Didnt add the file during git add :)

1 file changed, 44 insertions(+)
create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-pi.dts

diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-pi.dts b/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-pi.dts
new file mode 100644
index 000000000000..05f39b3606ba
--- /dev/null
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h616-bigtreetech-pi.dts
@@ -0,0 +1,44 @@
+// SPDX-License-Identifier: (GPL-2.0+ or MIT)
+/*
+ * Copyright (C) 2023 Martin Botka <martin@xxxxxxxxxx>.
+ */
+
+/dts-v1/;
+
+#include "sun50i-h616-bigtreetech-cb1.dtsi"
+
+/ {
+ compatible = "bigtreetech,pi", "allwinner,sun50i-h616";
+};
+
+&ehci0 {
+ status = "okay";
+};
+
+&ohci0 {
+ status = "okay";
+};
+
+&ehci1 {
+ status = "okay";
+};
+
+&ohci1 {
+ status = "okay";
+};
+
+&ehci2 {
+ status = "okay";
+};
+
+&ohci2 {
+ status = "okay";
+};
+
+&ehci3 {
+ status = "okay";
+};
+
+&ohci3 {
+ status = "okay";
+};

Oh wow, it really connects all four USB ports directly to the SoC? Nice.
It indeed does :)
Do you have access to the schematic to check what the power supply
situation is? Are all ports hardwired to a fixed 5V source?
Correct just fixed 5V. Tho with the changes needed in CB1 DTSI file no change will be necessary here for this :)

And you would of course need to adjust this file according to the
changes requested in the previous patch, so that you have the UART,
stdout-path and other moved nodes in here.
Got it will do.

Cheers,
Martin

Cheers,
Andre