* [PATCH 1/2] dt-bindings: phy: amlogic,g12a-usb2-phy: add S4 compatible
2026-10-25 19:22 [PATCH 0/2] phy: amlogic: meson-g12a-usb2: add S4 support christian.koever-draxl
@ 2026-10-25 19:22 ` christian.koever-draxl
2026-10-30 11:10 ` Krzysztof Kozlowski
2026-10-25 19:22 ` [PATCH 2/2] phy: amlogic: meson-g12a-usb2: add S4 support christian.koever-draxl
2026-11-03 8:56 ` [PATCH 0/2] " neil.armstrong
2 siblings, 1 reply; 6+ messages in thread
From: christian.koever-draxl @ 2026-10-25 19:22 UTC (permalink / raw)
To: neil.armstrong, khilman, jbrunet, martin.blumenstingl, vkoul,
mani, robh, krzk+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, linux-phy, devicetree,
linux-kernel, Christian Stefan Kövér-Draxl
From: Christian Stefan Kövér-Draxl <ckoever@outlook.de>
The S4 USB2 PHY is the G12A/A1 design with a few different analog
settings, so it needs its own compatible.
Signed-off-by: Christian Stefan Kövér-Draxl <ckoever@outlook.de>
---
Documentation/devicetree/bindings/phy/amlogic,g12a-usb2-phy.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb2-phy.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb2-phy.yaml
index 1a0c436b87a0..54058d2e6053 100644
--- a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb2-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb2-phy.yaml
@@ -15,6 +15,7 @@ properties:
enum:
- amlogic,g12a-usb2-phy
- amlogic,a1-usb2-phy
+ - amlogic,s4-usb2-phy
reg:
maxItems: 1
--
2.56.0
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 2/2] phy: amlogic: meson-g12a-usb2: add S4 support
2026-10-25 19:22 [PATCH 0/2] phy: amlogic: meson-g12a-usb2: add S4 support christian.koever-draxl
2026-10-25 19:22 ` [PATCH 1/2] dt-bindings: phy: amlogic,g12a-usb2-phy: add S4 compatible christian.koever-draxl
@ 2026-10-25 19:22 ` christian.koever-draxl
2026-11-01 11:08 ` Manivannan Sadhasivam
2026-11-03 8:56 ` [PATCH 0/2] " neil.armstrong
2 siblings, 1 reply; 6+ messages in thread
From: christian.koever-draxl @ 2026-10-25 19:22 UTC (permalink / raw)
To: neil.armstrong, khilman, jbrunet, martin.blumenstingl, vkoul,
mani, robh, krzk+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, linux-phy, devicetree,
linux-kernel, Christian Stefan Kövér-Draxl
From: Christian Stefan Kövér-Draxl <ckoever@outlook.de>
The S4 USB2 PHY follows the A1 programming (MPLL DCO clock select, R21
calibration, R13 analog settings) with a few differences taken from the
vendor driver:
- edge drive trim 2 instead of 3,
- HS disconnect reference 3 instead of 1,
- a disconnect threshold in R14, and the R13 PMA signal update bit
set to latch the analog settings.
Without the PMA update the host controller sees no root ports and
nothing enumerates.
Tested on an S905W2 board: devices enumerate and work behind a hub on
the host ports.
Signed-off-by: Christian Stefan Kövér-Draxl <ckoever@outlook.de>
---
drivers/phy/amlogic/phy-meson-g12a-usb2.c | 65 +++++++++++++++++------
1 file changed, 48 insertions(+), 17 deletions(-)
diff --git a/drivers/phy/amlogic/phy-meson-g12a-usb2.c b/drivers/phy/amlogic/phy-meson-g12a-usb2.c
index 66bf0b7ef8ed..18d40f7e48fb 100644
--- a/drivers/phy/amlogic/phy-meson-g12a-usb2.c
+++ b/drivers/phy/amlogic/phy-meson-g12a-usb2.c
@@ -72,6 +72,7 @@
#define PHY_CTRL_R14_I_C2L_ASSERT_SINGLE_EN_ZERO BIT(6)
#define PHY_CTRL_R14_BYPASS_CTRL_7_0 GENMASK(15, 8)
#define PHY_CTRL_R14_BYPASS_CTRL_15_8 GENMASK(23, 16)
+ #define PHY_CTRL_R14_DISCONNECT_THRESHOLD_S4 GENMASK(27, 26)
#define PHY_CTRL_R15 0x3c
#define PHY_CTRL_R16 0x40
@@ -149,6 +150,7 @@
enum meson_soc_id {
MESON_SOC_G12A = 0,
MESON_SOC_A1,
+ MESON_SOC_S4,
};
struct phy_meson_g12a_usb2_priv {
@@ -217,7 +219,12 @@ static int phy_meson_g12a_usb2_init(struct phy *phy)
FIELD_PREP(PHY_CTRL_R18_MPLL_ADJ_LDO, 1) |
PHY_CTRL_R18_MPLL_ACG_RANGE;
- if (priv->soc_id == MESON_SOC_A1)
+ /*
+ * The S4 shares the A1's MPLL programming: its stock "pll-setting-3"
+ * (0xac5f69e5) is the G12A R18 value with MPLL_DCO_CLK_SEL (bit 13)
+ * set, which is exactly what the A1 path adds here.
+ */
+ if (priv->soc_id == MESON_SOC_A1 || priv->soc_id == MESON_SOC_S4)
value |= PHY_CTRL_R18_MPLL_DCO_CLK_SEL;
regmap_write(priv->regmap, PHY_CTRL_R18, value);
@@ -234,15 +241,19 @@ static int phy_meson_g12a_usb2_init(struct phy *phy)
PHY_CTRL_R16_MPLL_EN);
/* PHY Tuning */
- regmap_write(priv->regmap, PHY_CTRL_R20,
- FIELD_PREP(PHY_CTRL_R20_USB2_OTG_VBUS_TRIM_2_0, 4) |
- PHY_CTRL_R20_USB2_OTG_VBUSDET_EN |
- FIELD_PREP(PHY_CTRL_R20_USB2_DMON_SEL_3_0, 15) |
- PHY_CTRL_R20_USB2_EDGE_DRV_EN |
- FIELD_PREP(PHY_CTRL_R20_USB2_EDGE_DRV_TRIM_1_0, 3) |
- FIELD_PREP(PHY_CTRL_R20_USB2_BGR_ADJ_4_0, 0) |
- FIELD_PREP(PHY_CTRL_R20_USB2_BGR_VREF_4_0, 0) |
- FIELD_PREP(PHY_CTRL_R20_USB2_BGR_DBG_1_0, 0));
+ value = FIELD_PREP(PHY_CTRL_R20_USB2_OTG_VBUS_TRIM_2_0, 4) |
+ PHY_CTRL_R20_USB2_OTG_VBUSDET_EN |
+ FIELD_PREP(PHY_CTRL_R20_USB2_DMON_SEL_3_0, 15) |
+ PHY_CTRL_R20_USB2_EDGE_DRV_EN |
+ FIELD_PREP(PHY_CTRL_R20_USB2_BGR_ADJ_4_0, 0) |
+ FIELD_PREP(PHY_CTRL_R20_USB2_BGR_VREF_4_0, 0) |
+ FIELD_PREP(PHY_CTRL_R20_USB2_BGR_DBG_1_0, 0);
+ /* S4 uses edge-drive trim 2; other SoCs use 3 */
+ if (priv->soc_id == MESON_SOC_S4)
+ value |= FIELD_PREP(PHY_CTRL_R20_USB2_EDGE_DRV_TRIM_1_0, 2);
+ else
+ value |= FIELD_PREP(PHY_CTRL_R20_USB2_EDGE_DRV_TRIM_1_0, 3);
+ regmap_write(priv->regmap, PHY_CTRL_R20, value);
if (priv->soc_id == MESON_SOC_G12A)
regmap_write(priv->regmap, PHY_CTRL_R4,
@@ -252,22 +263,38 @@ static int phy_meson_g12a_usb2_init(struct phy *phy)
PHY_CTRL_R4_TEST_BYPASS_MODE_EN |
FIELD_PREP(PHY_CTRL_R4_I_C2L_BIAS_TRIM_1_0, 0) |
FIELD_PREP(PHY_CTRL_R4_I_C2L_BIAS_TRIM_3_2, 0));
- else if (priv->soc_id == MESON_SOC_A1) {
+ else if (priv->soc_id == MESON_SOC_A1 ||
+ priv->soc_id == MESON_SOC_S4) {
regmap_write(priv->regmap, PHY_CTRL_R21,
PHY_CTRL_R21_USB2_CAL_ACK_EN |
PHY_CTRL_R21_USB2_TX_STRG_PD |
FIELD_PREP(PHY_CTRL_R21_USB2_OTG_ACA_TRIM_1_0, 2));
/* Analog Settings */
- regmap_write(priv->regmap, PHY_CTRL_R13,
- FIELD_PREP(PHY_CTRL_R13_MIN_COUNT_FOR_SYNC_DET, 7));
+ value = FIELD_PREP(PHY_CTRL_R13_MIN_COUNT_FOR_SYNC_DET, 7);
+ if (priv->soc_id == MESON_SOC_S4) {
+ /*
+ * S4: set the disconnect threshold and latch the PMA
+ * signals so the host controller detects its root ports.
+ */
+ regmap_update_bits(priv->regmap, PHY_CTRL_R14,
+ PHY_CTRL_R14_DISCONNECT_THRESHOLD_S4,
+ FIELD_PREP(PHY_CTRL_R14_DISCONNECT_THRESHOLD_S4,
+ 2));
+ value |= PHY_CTRL_R13_UPDATE_PMA_SIGNALS;
+ }
+ regmap_write(priv->regmap, PHY_CTRL_R13, value);
}
/* Tuning Disconnect Threshold */
- regmap_write(priv->regmap, PHY_CTRL_R3,
- FIELD_PREP(PHY_CTRL_R3_SQUELCH_REF, 0) |
- FIELD_PREP(PHY_CTRL_R3_HSDIC_REF, 1) |
- FIELD_PREP(PHY_CTRL_R3_DISC_THRESH, 3));
+ value = FIELD_PREP(PHY_CTRL_R3_SQUELCH_REF, 0) |
+ FIELD_PREP(PHY_CTRL_R3_DISC_THRESH, 3);
+ /* S4 wants a higher HS disconnect reference than the other SoCs */
+ if (priv->soc_id == MESON_SOC_S4)
+ value |= FIELD_PREP(PHY_CTRL_R3_HSDIC_REF, 3);
+ else
+ value |= FIELD_PREP(PHY_CTRL_R3_HSDIC_REF, 1);
+ regmap_write(priv->regmap, PHY_CTRL_R3, value);
if (priv->soc_id == MESON_SOC_G12A) {
/* Analog Settings */
@@ -360,6 +387,10 @@ static const struct of_device_id phy_meson_g12a_usb2_of_match[] = {
.compatible = "amlogic,a1-usb2-phy",
.data = (void *)MESON_SOC_A1,
},
+ {
+ .compatible = "amlogic,s4-usb2-phy",
+ .data = (void *)MESON_SOC_S4,
+ },
{ /* Sentinel */ }
};
MODULE_DEVICE_TABLE(of, phy_meson_g12a_usb2_of_match);
--
2.56.0
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 0/2] phy: amlogic: meson-g12a-usb2: add S4 support
2026-10-25 19:22 [PATCH 0/2] phy: amlogic: meson-g12a-usb2: add S4 support christian.koever-draxl
2026-10-25 19:22 ` [PATCH 1/2] dt-bindings: phy: amlogic,g12a-usb2-phy: add S4 compatible christian.koever-draxl
2026-10-25 19:22 ` [PATCH 2/2] phy: amlogic: meson-g12a-usb2: add S4 support christian.koever-draxl
@ 2026-11-03 8:56 ` neil.armstrong
2 siblings, 0 replies; 6+ messages in thread
From: neil.armstrong @ 2026-11-03 8:56 UTC (permalink / raw)
To: christian.koever-draxl, khilman, jbrunet, martin.blumenstingl,
vkoul, mani, robh, krzk+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, linux-phy, devicetree,
linux-kernel, Christian Stefan Kövér-Draxl
On 10/25/26 20:22, christian.koever-draxl@student.uibk.ac.at wrote:
> From: Christian Stefan Kövér-Draxl <ckoever@outlook.de>
>
> The S4 USB2 PHY is the A1 design with different analog trims and an
> extra disconnect threshold; without its PMA update bit the host sees no
> root ports. Add a compatible and the S4 settings from the vendor driver.
>
> Tested: both host ports enumerate keyboards, mice, hubs and storage.
>
> Tool use (Documentation/process/generated-content.rst):
> This work was done with the help of an AI coding assistant, Claude
> (Anthropic), used through Claude Code over many sessions. It did a large
> part of the hands-on work: it read the vendor kernel and the firmware
> interfaces, probed registers on the board, drafted the code and the
> changelogs, and ran the builds, checkpatch, dt_binding_check and the
> tests on the hardware. I directed the work, reviewed every change and
> the final patches, and take responsibility for them. There is no single
> prompt to quote; the sessions were of the form "find out why X fails on
> this board, fix it, test it on the box".
>
> Tested on an Amediatech X98Q TV box (Amlogic S905W2, S4 family, 2 GiB),
> vendor U-Boot and BL31/OP-TEE, Linux 7.3 plus this work.
>
> Christian Stefan Kövér-Draxl (2):
> dt-bindings: phy: amlogic,g12a-usb2-phy: add S4 compatible
> phy: amlogic: meson-g12a-usb2: add S4 support
>
> .../bindings/phy/amlogic,g12a-usb2-phy.yaml | 1 +
> drivers/phy/amlogic/phy-meson-g12a-usb2.c | 65 ++++++++++++++-----
> 2 files changed, 49 insertions(+), 17 deletions(-)
>
>
> base-commit: 7be4bc52fd9e0e55dc9a2e0232b89427d2c4b614
Acked-by: Neil Armstrong <neil.armstrong@linaro.org>
Thanks,
Neil
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 6+ messages in thread