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