* [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support
@ 2026-10-07 14:44 Jon Hunter
2026-10-07 14:44 ` [PATCH v3 1/3] dt-bindings: phy: tegra194: Add compatible for Tegra238 Jon Hunter
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: Jon Hunter @ 2026-10-07 14:44 UTC (permalink / raw)
To: Vinod Koul, Neil Armstrong, Manivannan Sadhasivam, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Thierry Reding, JC Kuo
Cc: Thierry Reding, Wayne Chang, linux-phy, devicetree, linux-tegra,
Jon Hunter, Krzysztof Kozlowski
Add support for the Tegra238 device to the Tegra XUSB PHY driver and
add the necessary device-tree binding documentation for this device.
This series also includes a trivial clean-up patch to use the
dev_err_probe() function in the Tegra XUSB PHY driver instead of open
coding this function.
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
---
Changes in v3:
- Fixed build issue reported by sashiko.
- Rebased on top of next-20261007 which now includes the dependent
changes [0] and so applies cleanly.
- Link to v2: https://lore.kernel.org/r/20261001-tegra-phy-v2-0-3e453b1fb271@nvidia.com
Changes in v2:
- Dropped extra 'err' variable from patch that migrates to dev_err_probe
- Added Krzysztof's Reviewed-by for device-tree binding.
- Link to v1: https://patch.msgid.link/20260921-tegra-phy-v1-0-83466bf9a7cd@nvidia.com
[0] https://lore.kernel.org/linux-tegra/20260903071937.533646-1-waynec@nvidia.com/
---
Jon Hunter (3):
dt-bindings: phy: tegra194: Add compatible for Tegra238
phy: tegra: xusb: Use dev_err_probe()
phy: tegra: Add support for Tegra238 XUSB pad controller
.../bindings/phy/nvidia,tegra194-xusb-padctl.yaml | 38 ++++++++++++++++++++++
drivers/phy/tegra/Makefile | 1 +
drivers/phy/tegra/xusb-tegra186.c | 29 +++++++++++++----
drivers/phy/tegra/xusb.c | 14 ++++----
drivers/phy/tegra/xusb.h | 3 ++
5 files changed, 72 insertions(+), 13 deletions(-)
---
base-commit: b018686719706a781c45a874ed51374a2dc4b767
change-id: 20260918-tegra-phy-1611f1d54afa
Best regards,
--
Jon Hunter <jonathanh@nvidia.com>
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3 1/3] dt-bindings: phy: tegra194: Add compatible for Tegra238
2026-10-07 14:44 [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Jon Hunter
@ 2026-10-07 14:44 ` Jon Hunter
2026-10-07 14:44 ` [PATCH v3 2/3] phy: tegra: xusb: Use dev_err_probe() Jon Hunter
` (2 subsequent siblings)
3 siblings, 0 replies; 9+ messages in thread
From: Jon Hunter @ 2026-10-07 14:44 UTC (permalink / raw)
To: Vinod Koul, Neil Armstrong, Manivannan Sadhasivam, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Thierry Reding, JC Kuo
Cc: Thierry Reding, Wayne Chang, linux-phy, devicetree, linux-tegra,
Jon Hunter, Krzysztof Kozlowski
The Tegra238 XUSB pad controller is similar to that found on the
Tegra194 and Tegra234, but only has three USB2 ports and three USB3
ports.
Add the Tegra238 compatible string for the XUSB pad controller and
constrain the binding so that a Tegra238 device tree can only have three
USB2 and three USB3 ports.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
---
.../bindings/phy/nvidia,tegra194-xusb-padctl.yaml | 38 ++++++++++++++++++++++
1 file changed, 38 insertions(+)
diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml b/Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml
index f6fe3c90b5b0..fbafe8eb150b 100644
--- a/Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml
+++ b/Documentation/devicetree/bindings/phy/nvidia,tegra194-xusb-padctl.yaml
@@ -45,6 +45,7 @@ properties:
enum:
- nvidia,tegra194-xusb-padctl
- nvidia,tegra234-xusb-padctl
+ - nvidia,tegra238-xusb-padctl
- nvidia,tegra264-xusb-padctl
reg:
@@ -542,6 +543,43 @@ properties:
description: connection to a USB Type C controller
$ref: /schemas/graph.yaml#/properties/port
+allOf:
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: nvidia,tegra238-xusb-padctl
+ then:
+ properties:
+ pads:
+ properties:
+ usb2:
+ properties:
+ lanes:
+ properties:
+ usb2-3: false
+ usb3:
+ properties:
+ lanes:
+ properties:
+ usb3-3: false
+ ports:
+ properties:
+ usb2-3: false
+ usb3-0:
+ properties:
+ nvidia,usb2-companion:
+ enum: [ 0, 1, 2 ]
+ usb3-1:
+ properties:
+ nvidia,usb2-companion:
+ enum: [ 0, 1, 2 ]
+ usb3-2:
+ properties:
+ nvidia,usb2-companion:
+ enum: [ 0, 1, 2 ]
+ usb3-3: false
+
additionalProperties: false
required:
--
2.43.0
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 2/3] phy: tegra: xusb: Use dev_err_probe()
2026-10-07 14:44 [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Jon Hunter
2026-10-07 14:44 ` [PATCH v3 1/3] dt-bindings: phy: tegra194: Add compatible for Tegra238 Jon Hunter
@ 2026-10-07 14:44 ` Jon Hunter
2026-10-07 14:44 ` [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller Jon Hunter
2026-10-07 15:15 ` [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Manivannan Sadhasivam
3 siblings, 0 replies; 9+ messages in thread
From: Jon Hunter @ 2026-10-07 14:44 UTC (permalink / raw)
To: Vinod Koul, Neil Armstrong, Manivannan Sadhasivam, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Thierry Reding, JC Kuo
Cc: Thierry Reding, Wayne Chang, linux-phy, devicetree, linux-tegra,
Jon Hunter
Use the function dev_err_probe() instead of open coding the same logic.
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
---
drivers/phy/tegra/xusb.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/drivers/phy/tegra/xusb.c b/drivers/phy/tegra/xusb.c
index d0d0b252a205..0635f0d82c96 100644
--- a/drivers/phy/tegra/xusb.c
+++ b/drivers/phy/tegra/xusb.c
@@ -1257,13 +1257,7 @@ static int tegra_xusb_padctl_probe(struct platform_device *pdev)
err = tegra_xusb_setup_ports(padctl);
if (err) {
- const char *level = KERN_ERR;
-
- if (err == -EPROBE_DEFER)
- level = KERN_DEBUG;
-
- dev_printk(level, &pdev->dev,
- dev_fmt("failed to setup XUSB ports: %d\n"), err);
+ dev_err_probe(&pdev->dev, err, "failed to setup XUSB ports\n");
goto remove_pads;
}
--
2.43.0
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller
2026-10-07 14:44 [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Jon Hunter
2026-10-07 14:44 ` [PATCH v3 1/3] dt-bindings: phy: tegra194: Add compatible for Tegra238 Jon Hunter
2026-10-07 14:44 ` [PATCH v3 2/3] phy: tegra: xusb: Use dev_err_probe() Jon Hunter
@ 2026-10-07 14:44 ` Jon Hunter
2026-10-07 14:52 ` sashiko-bot
2026-10-07 15:20 ` Manivannan Sadhasivam
2026-10-07 15:15 ` [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Manivannan Sadhasivam
3 siblings, 2 replies; 9+ messages in thread
From: Jon Hunter @ 2026-10-07 14:44 UTC (permalink / raw)
To: Vinod Koul, Neil Armstrong, Manivannan Sadhasivam, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Thierry Reding, JC Kuo
Cc: Thierry Reding, Wayne Chang, linux-phy, devicetree, linux-tegra,
Jon Hunter
Add support for the Tegra238 XUSB pad controller by populating the
Tegra238 specific SoC data for the controller. The Tegra238 has three
USB2 and USB3 ports, uses the same port ops as Tegra186 and has the
same supplies as Tegra194. Due to the dependencies the Tegra238 device
has on the Tegra186 port ops and Tegra194 supplies, drop the compile
time guards around these structures.
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
---
drivers/phy/tegra/Makefile | 1 +
drivers/phy/tegra/xusb-tegra186.c | 29 +++++++++++++++++++++++------
drivers/phy/tegra/xusb.c | 6 ++++++
drivers/phy/tegra/xusb.h | 3 +++
4 files changed, 33 insertions(+), 6 deletions(-)
diff --git a/drivers/phy/tegra/Makefile b/drivers/phy/tegra/Makefile
index f5ed9582d80b..018a4cab2b40 100644
--- a/drivers/phy/tegra/Makefile
+++ b/drivers/phy/tegra/Makefile
@@ -8,6 +8,7 @@ phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_210_SOC) += xusb-tegra210.o
phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_186_SOC) += xusb-tegra186.o
phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_194_SOC) += xusb-tegra186.o
phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_234_SOC) += xusb-tegra186.o
+phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_238_SOC) += xusb-tegra186.o
phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_264_SOC) += xusb-tegra186.o
obj-$(CONFIG_PHY_TEGRA194_P2U) += phy-tegra194-p2u.o
obj-$(CONFIG_PHY_TEGRA_MPHY) += phy-tegra-mphy.o
diff --git a/drivers/phy/tegra/xusb-tegra186.c b/drivers/phy/tegra/xusb-tegra186.c
index 322efdea0d80..aa8ab94fad4c 100644
--- a/drivers/phy/tegra/xusb-tegra186.c
+++ b/drivers/phy/tegra/xusb-tegra186.c
@@ -1599,7 +1599,6 @@ static const struct tegra_xusb_padctl_ops tegra186_xusb_padctl_ops = {
.utmi_pad_power_down = tegra186_utmi_pad_power_down,
};
-#if IS_ENABLED(CONFIG_ARCH_TEGRA_186_SOC)
static const char * const tegra186_xusb_padctl_supply_names[] = {
"avdd-pll-erefeut",
"avdd-usb",
@@ -1665,11 +1664,7 @@ const struct tegra_xusb_padctl_soc tegra186_xusb_padctl_soc = {
.num_supplies = ARRAY_SIZE(tegra186_xusb_padctl_supply_names),
};
EXPORT_SYMBOL_GPL(tegra186_xusb_padctl_soc);
-#endif
-#if IS_ENABLED(CONFIG_ARCH_TEGRA_194_SOC) || \
- IS_ENABLED(CONFIG_ARCH_TEGRA_234_SOC) || \
- IS_ENABLED(CONFIG_ARCH_TEGRA_264_SOC)
static const char * const tegra194_xusb_padctl_supply_names[] = {
"avdd-usb",
"vclamp-usb",
@@ -1755,6 +1750,29 @@ const struct tegra_xusb_padctl_soc tegra234_xusb_padctl_soc = {
};
EXPORT_SYMBOL_GPL(tegra234_xusb_padctl_soc);
+const struct tegra_xusb_padctl_soc tegra238_xusb_padctl_soc = {
+ .num_pads = ARRAY_SIZE(tegra186_pads),
+ .pads = tegra186_pads,
+ .ports = {
+ .usb2 = {
+ .ops = &tegra186_usb2_port_ops,
+ .count = 3,
+ },
+ .usb3 = {
+ .ops = &tegra186_usb3_port_ops,
+ .count = 3,
+ },
+ },
+ .ops = &tegra186_xusb_padctl_ops,
+ .supply_names = tegra194_xusb_padctl_supply_names,
+ .num_supplies = ARRAY_SIZE(tegra194_xusb_padctl_supply_names),
+ .supports_gen2 = true,
+ .poll_trk_completed = true,
+ .trk_hw_mode = false,
+ .supports_lp_cfg_en = true,
+};
+EXPORT_SYMBOL_GPL(tegra238_xusb_padctl_soc);
+
const struct tegra_xusb_padctl_soc tegra264_xusb_padctl_soc = {
.num_pads = ARRAY_SIZE(tegra194_pads),
.pads = tegra194_pads,
@@ -1778,7 +1796,6 @@ const struct tegra_xusb_padctl_soc tegra264_xusb_padctl_soc = {
.has_per_pad_term = true,
};
EXPORT_SYMBOL_GPL(tegra264_xusb_padctl_soc);
-#endif
MODULE_AUTHOR("JC Kuo <jckuo@nvidia.com>");
MODULE_DESCRIPTION("NVIDIA Tegra186 XUSB Pad Controller driver");
diff --git a/drivers/phy/tegra/xusb.c b/drivers/phy/tegra/xusb.c
index 0635f0d82c96..89ce9ac87300 100644
--- a/drivers/phy/tegra/xusb.c
+++ b/drivers/phy/tegra/xusb.c
@@ -78,6 +78,12 @@ static const struct of_device_id tegra_xusb_padctl_of_match[] = {
.data = &tegra234_xusb_padctl_soc,
},
#endif
+#if defined(CONFIG_ARCH_TEGRA_238_SOC)
+ {
+ .compatible = "nvidia,tegra238-xusb-padctl",
+ .data = &tegra238_xusb_padctl_soc,
+ },
+#endif
#if defined(CONFIG_ARCH_TEGRA_264_SOC)
{
.compatible = "nvidia,tegra264-xusb-padctl",
diff --git a/drivers/phy/tegra/xusb.h b/drivers/phy/tegra/xusb.h
index d8d799d9e752..b9de4f3b5781 100644
--- a/drivers/phy/tegra/xusb.h
+++ b/drivers/phy/tegra/xusb.h
@@ -514,6 +514,9 @@ extern const struct tegra_xusb_padctl_soc tegra194_xusb_padctl_soc;
#if defined(CONFIG_ARCH_TEGRA_234_SOC)
extern const struct tegra_xusb_padctl_soc tegra234_xusb_padctl_soc;
#endif
+#if defined(CONFIG_ARCH_TEGRA_238_SOC)
+extern const struct tegra_xusb_padctl_soc tegra238_xusb_padctl_soc;
+#endif
#if defined(CONFIG_ARCH_TEGRA_264_SOC)
extern const struct tegra_xusb_padctl_soc tegra264_xusb_padctl_soc;
#endif
--
2.43.0
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller
2026-10-07 14:44 ` [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller Jon Hunter
@ 2026-10-07 14:52 ` sashiko-bot
2026-10-07 15:20 ` Manivannan Sadhasivam
1 sibling, 0 replies; 9+ messages in thread
From: sashiko-bot @ 2026-10-07 14:52 UTC (permalink / raw)
To: Jon Hunter
Cc: robh, neil.armstrong, vkoul, conor+dt, devicetree, olteanv,
linux-phy
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] phy: tegra: missing declarations for exported SoC structs in xusb.h
--
commit 8acf4872f17465f59f32558aefacb7be8e2c679f
Author: Jon Hunter <jonathanh@nvidia.com>
phy: tegra: Add support for Tegra238 XUSB pad controller
This commit adds support for the Tegra238 XUSB pad controller. It populates
the Tegra238 specific SoC data, reusing structures from Tegra186 and
Tegra194, and drops the compile time guards around these shared structures in
the source file.
> diff --git a/drivers/phy/tegra/xusb-tegra186.c b/drivers/phy/tegra/xusb-tegra186.c
> index 322efdea0d807..aa8ab94fad4cf 100644
> --- a/drivers/phy/tegra/xusb-tegra186.c
> +++ b/drivers/phy/tegra/xusb-tegra186.c
[ ... ]
> @@ -1665,11 +1664,7 @@ const struct tegra_xusb_padctl_soc tegra186_xusb_padctl_soc = {
> .num_supplies = ARRAY_SIZE(tegra186_xusb_padctl_supply_names),
> };
> EXPORT_SYMBOL_GPL(tegra186_xusb_padctl_soc);
> -#endif
[Severity: Low]
Since the guards are removed here, tegra186_xusb_padctl_soc is now defined
and exported unconditionally. However, its extern declaration in
drivers/phy/tegra/xusb.h remains conditionally guarded:
#if defined(CONFIG_ARCH_TEGRA_186_SOC)
extern const struct tegra_xusb_padctl_soc tegra186_xusb_padctl_soc;
#endif
Does this cause missing declaration warnings (-Wmissing-declarations or
-Wmissing-prototypes) during the build if the kernel is configured with
CONFIG_ARCH_TEGRA_238_SOC=y but CONFIG_ARCH_TEGRA_186_SOC=n? Should the
corresponding guards in xusb.h be removed as well?
>
> -#if IS_ENABLED(CONFIG_ARCH_TEGRA_194_SOC) || \
> - IS_ENABLED(CONFIG_ARCH_TEGRA_234_SOC) || \
> - IS_ENABLED(CONFIG_ARCH_TEGRA_264_SOC)
> static const char * const tegra194_xusb_padctl_supply_names[] = {
> "avdd-usb",
> "vclamp-usb",
[Severity: Low]
A similar mismatch occurs here for tegra194_xusb_padctl_soc. The definition
is now unconditional, but the extern declaration in xusb.h is still guarded
by #if defined(CONFIG_ARCH_TEGRA_194_SOC). Could this also trigger missing
prototype warnings when building with certain configs disabled?
--
Sashiko AI review · https://sashiko.dev/#/patchset/20261007-tegra-phy-v3-0-d501de414710@nvidia.com?part=3
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support
2026-10-07 14:44 [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Jon Hunter
` (2 preceding siblings ...)
2026-10-07 14:44 ` [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller Jon Hunter
@ 2026-10-07 15:15 ` Manivannan Sadhasivam
3 siblings, 0 replies; 9+ messages in thread
From: Manivannan Sadhasivam @ 2026-10-07 15:15 UTC (permalink / raw)
To: Jon Hunter
Cc: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Thierry Reding, JC Kuo, Thierry Reding, Wayne Chang,
linux-phy, devicetree, linux-tegra, Krzysztof Kozlowski
On Wed, Oct 07, 2026 at 03:44:40PM +0100, Jon Hunter wrote:
> Add support for the Tegra238 device to the Tegra XUSB PHY driver and
> add the necessary device-tree binding documentation for this device.
>
> This series also includes a trivial clean-up patch to use the
> dev_err_probe() function in the Tegra XUSB PHY driver instead of open
> coding this function.
>
> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@oss.qualcomm.com>
- Mani
--
மணிவண்ணன் சதாசிவம்
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller
2026-10-07 14:44 ` [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller Jon Hunter
2026-10-07 14:52 ` sashiko-bot
@ 2026-10-07 15:20 ` Manivannan Sadhasivam
2026-10-07 16:27 ` Jon Hunter
1 sibling, 1 reply; 9+ messages in thread
From: Manivannan Sadhasivam @ 2026-10-07 15:20 UTC (permalink / raw)
To: Jon Hunter
Cc: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Thierry Reding, JC Kuo, Thierry Reding, Wayne Chang,
linux-phy, devicetree, linux-tegra
On Wed, Oct 07, 2026 at 03:44:43PM +0100, Jon Hunter wrote:
> Add support for the Tegra238 XUSB pad controller by populating the
> Tegra238 specific SoC data for the controller. The Tegra238 has three
> USB2 and USB3 ports, uses the same port ops as Tegra186 and has the
> same supplies as Tegra194. Due to the dependencies the Tegra238 device
> has on the Tegra186 port ops and Tegra194 supplies, drop the compile
> time guards around these structures.
>
> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
> ---
> drivers/phy/tegra/Makefile | 1 +
> drivers/phy/tegra/xusb-tegra186.c | 29 +++++++++++++++++++++++------
> drivers/phy/tegra/xusb.c | 6 ++++++
> drivers/phy/tegra/xusb.h | 3 +++
> 4 files changed, 33 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/phy/tegra/Makefile b/drivers/phy/tegra/Makefile
> index f5ed9582d80b..018a4cab2b40 100644
> --- a/drivers/phy/tegra/Makefile
> +++ b/drivers/phy/tegra/Makefile
> @@ -8,6 +8,7 @@ phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_210_SOC) += xusb-tegra210.o
> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_186_SOC) += xusb-tegra186.o
> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_194_SOC) += xusb-tegra186.o
> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_234_SOC) += xusb-tegra186.o
> +phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_238_SOC) += xusb-tegra186.o
> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_264_SOC) += xusb-tegra186.o
This is just a mess. As a followup patch, could you please please switch to
generic ARCH_TEGRA Kconfig option as done in this commit, 6b5e2f70a95c
("PCI: dwc: tegra194: Broaden architecture dependency")?
- Mani
--
மணிவண்ணன் சதாசிவம்
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller
2026-10-07 15:20 ` Manivannan Sadhasivam
@ 2026-10-07 16:27 ` Jon Hunter
2026-10-08 12:09 ` Jon Hunter
0 siblings, 1 reply; 9+ messages in thread
From: Jon Hunter @ 2026-10-07 16:27 UTC (permalink / raw)
To: Manivannan Sadhasivam
Cc: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Thierry Reding, JC Kuo, Thierry Reding, Wayne Chang,
linux-phy, devicetree, linux-tegra
On 07/10/2026 16:20, Manivannan Sadhasivam wrote:
> On Wed, Oct 07, 2026 at 03:44:43PM +0100, Jon Hunter wrote:
>> Add support for the Tegra238 XUSB pad controller by populating the
>> Tegra238 specific SoC data for the controller. The Tegra238 has three
>> USB2 and USB3 ports, uses the same port ops as Tegra186 and has the
>> same supplies as Tegra194. Due to the dependencies the Tegra238 device
>> has on the Tegra186 port ops and Tegra194 supplies, drop the compile
>> time guards around these structures.
>>
>> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
>> ---
>> drivers/phy/tegra/Makefile | 1 +
>> drivers/phy/tegra/xusb-tegra186.c | 29 +++++++++++++++++++++++------
>> drivers/phy/tegra/xusb.c | 6 ++++++
>> drivers/phy/tegra/xusb.h | 3 +++
>> 4 files changed, 33 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/phy/tegra/Makefile b/drivers/phy/tegra/Makefile
>> index f5ed9582d80b..018a4cab2b40 100644
>> --- a/drivers/phy/tegra/Makefile
>> +++ b/drivers/phy/tegra/Makefile
>> @@ -8,6 +8,7 @@ phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_210_SOC) += xusb-tegra210.o
>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_186_SOC) += xusb-tegra186.o
>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_194_SOC) += xusb-tegra186.o
>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_234_SOC) += xusb-tegra186.o
>> +phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_238_SOC) += xusb-tegra186.o
>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_264_SOC) += xusb-tegra186.o
>
> This is just a mess. As a followup patch, could you please please switch to
> generic ARCH_TEGRA Kconfig option as done in this commit, 6b5e2f70a95c
> ("PCI: dwc: tegra194: Broaden architecture dependency")?
Yes that would be a good idea and yes I can do that as a follow-up.
Jon
--
nvpublic
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller
2026-10-07 16:27 ` Jon Hunter
@ 2026-10-08 12:09 ` Jon Hunter
0 siblings, 0 replies; 9+ messages in thread
From: Jon Hunter @ 2026-10-08 12:09 UTC (permalink / raw)
To: Manivannan Sadhasivam
Cc: Vinod Koul, Neil Armstrong, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Thierry Reding, JC Kuo, Thierry Reding, Wayne Chang,
linux-phy, devicetree, linux-tegra
On 07/10/2026 17:27, Jon Hunter wrote:
>
> On 07/10/2026 16:20, Manivannan Sadhasivam wrote:
>> On Wed, Oct 07, 2026 at 03:44:43PM +0100, Jon Hunter wrote:
>>> Add support for the Tegra238 XUSB pad controller by populating the
>>> Tegra238 specific SoC data for the controller. The Tegra238 has three
>>> USB2 and USB3 ports, uses the same port ops as Tegra186 and has the
>>> same supplies as Tegra194. Due to the dependencies the Tegra238 device
>>> has on the Tegra186 port ops and Tegra194 supplies, drop the compile
>>> time guards around these structures.
>>>
>>> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
>>> ---
>>> drivers/phy/tegra/Makefile | 1 +
>>> drivers/phy/tegra/xusb-tegra186.c | 29 +++++++++++++++++++++++------
>>> drivers/phy/tegra/xusb.c | 6 ++++++
>>> drivers/phy/tegra/xusb.h | 3 +++
>>> 4 files changed, 33 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/drivers/phy/tegra/Makefile b/drivers/phy/tegra/Makefile
>>> index f5ed9582d80b..018a4cab2b40 100644
>>> --- a/drivers/phy/tegra/Makefile
>>> +++ b/drivers/phy/tegra/Makefile
>>> @@ -8,6 +8,7 @@ phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_210_SOC) += xusb-tegra210.o
>>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_186_SOC) += xusb-tegra186.o
>>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_194_SOC) += xusb-tegra186.o
>>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_234_SOC) += xusb-tegra186.o
>>> +phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_238_SOC) += xusb-tegra186.o
>>> phy-tegra-xusb-$(CONFIG_ARCH_TEGRA_264_SOC) += xusb-tegra186.o
>>
>> This is just a mess. As a followup patch, could you please please switch to
>> generic ARCH_TEGRA Kconfig option as done in this commit, 6b5e2f70a95c
>> ("PCI: dwc: tegra194: Broaden architecture dependency")?
>
> Yes that would be a good idea and yes I can do that as a follow-up.
I did see the latest sashiko warning:
"Does this cause missing declaration warnings (-Wmissing-declarations or
-Wmissing-prototypes)".
I am not seeing any warnings but I can also address that in the
follow-up.
Jon
--
nvpublic
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2026-10-08 12:09 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-07 14:44 [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Jon Hunter
2026-10-07 14:44 ` [PATCH v3 1/3] dt-bindings: phy: tegra194: Add compatible for Tegra238 Jon Hunter
2026-10-07 14:44 ` [PATCH v3 2/3] phy: tegra: xusb: Use dev_err_probe() Jon Hunter
2026-10-07 14:44 ` [PATCH v3 3/3] phy: tegra: Add support for Tegra238 XUSB pad controller Jon Hunter
2026-10-07 14:52 ` sashiko-bot
2026-10-07 15:20 ` Manivannan Sadhasivam
2026-10-07 16:27 ` Jon Hunter
2026-10-08 12:09 ` Jon Hunter
2026-10-07 15:15 ` [PATCH v3 0/3] phy: tegra: xusb: Add Tegra238 support Manivannan Sadhasivam
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox