Thread (12 messages) 12 messages, 5 authors, 2026-01-19

Re: [PATCH 2/2] arm64: dts: amlogic: add the type-c controller on Radxa Zero 2

From: Ricardo Pardini <hidden>
Date: 2026-01-15 13:59:50
Also in: linux-amlogic, linux-devicetree, lkml

Hi Neil,

On 15/01/2026 14:00, Neil Armstrong wrote:
quoted
The Radxa Zero2 has an FUSB302 controller on i2c3 at address 0x22 and
INT# wired to GPIOA-13.

+        interrupts = <13 IRQ_TYPE_LEVEL_LOW>;
OK it seems I did read too fast, this should be 74 !

S922X Datasheet (same as A311D for this)

Table 6-32 GPIO Interrupt Sources

---------------------------------------------
| Input Mux Location    |    Description |
---------------------------------------------
| [76:61]        |    gpioA[15:0] |
---------------------------------------------

So gpioA13 ==> 74
Indeed. Sorry for this.
quoted
+
+        vbus-supply = <&ao_5v>;
+
+        status = "okay";
And I get:

   DTC [C] arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-radxa-zero2.dtb: fusb302@22 
(fcs,fusb302): 'connector' is a required property
         from schema $id: http://devicetree.org/schemas/usb/ 
fcs,fusb302.yaml#

Please add a minimal connector, like:

+               connector {
+                       compatible = "usb-c-connector";
+                       label = "USB-C";
+                       data-role = "host";
+                       power-role = "source";
+                       source-pdos = <PDO_FIXED(5000, 2000, 
PDO_FIXED_USB_COMM)>;
+               };

I'll drop this change from my tree.
Again, my bad. My intention here was a "minimal" description, mostly to 
reserve 0x22 so end-users can run `i2cdetect` or such on the 40-pin 
exposed i2c3 and not be surprised. Guess I didn't run dtbs_check hard 
enough and missed the required connector.
And you may also add support for the superspeed mux to switch the USB3 
polarity.

I have a prototype at https://gitlab.com/superna9999/linux/-/tree/topic/ 
amlogic/radxa-zero2/fusb302?ref_type=heads
I'll try your prototype. I had no idea it actually had those capabilities.

Do you think it is worth to resend this with just the fixed 74 pin & the 
minimal connector as you suggested? Now that I've seen your work, a "add 
FUSB302" that does nothing (while being capable) feels frivolous and 
confusing.

Thanks,
Ricardo



Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help