Re: [PATCH] phy: spacemit: enable K1 USB 2.0 PHY by default on ARCH_SPACEMIT

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:

Pre-existing issues:
- [High] Missing `select REGMAP_MMIO` in `PHY_SPACEMIT_K1_USB2` Kconfig entry.
- [Medium] Missing `depends on HAS_IOMEM` in `PHY_SPACEMIT_K1_USB2` Kconfig entry.
--

--- Patch [1]: [PATCH] phy: spacemit: enable K1 USB 2.0 PHY by default on ARCH_SPACEMIT ---
commit f756c3cf5b8ff136e8d286e16f1e25289aa9e095
Author: Bruno Banelli <bbanelli@gmail.com>

phy: spacemit: enable K1 USB 2.0 PHY by default on ARCH_SPACEMIT

This patch adds a default ARCH_SPACEMIT configuration for the K1 USB 2.0 PHY
driver. This matches the PCIe PHY configuration and ensures the USB driver is
enabled by default on SpacemiT K1 boards.

> diff --git a/drivers/phy/spacemit/Kconfig b/drivers/phy/spacemit/Kconfig
> index 5fdf18fce4995..dd74a3faf01fd 100644
> --- a/drivers/phy/spacemit/Kconfig
> +++ b/drivers/phy/spacemit/Kconfig
> @@ -20,6 +20,7 @@ config PHY_SPACEMIT_K1_USB2
>  	depends on COMMON_CLK
>  	depends on USB_COMMON
>  	select GENERIC_PHY

[Severity: High]
This is a pre-existing issue, but does this config entry need to select
REGMAP_MMIO?

If a randconfig build has PHY_SPACEMIT_K1_USB2 enabled and CONFIG_REGMAP
enabled, but CONFIG_REGMAP_MMIO is disabled, would this result in a linkage
error? The driver uses devm_regmap_init_mmio() during probe:

drivers/phy/spacemit/phy-k1-usb2.c:spacemit_usb2phy_probe() {
    ...
    sphy->regmap_base = devm_regmap_init_mmio(dev, base, &phy_regmap_config);
    ...
}

[Severity: Medium]
This is also a pre-existing issue, but should this config depend on HAS_IOMEM?

The driver maps I/O memory during probe:

drivers/phy/spacemit/phy-k1-usb2.c:spacemit_usb2phy_probe() {
    ...
    base = devm_platform_ioremap_resource(pdev, 0);
    ...
}

Could this cause build or runtime failures on architectures lacking I/O memory
support (such as UML) when COMPILE_TEST is enabled?

> +	default ARCH_SPACEMIT
>  	help
>  	  Enable this to support K1 USB 2.0 PHY driver. This driver takes care of
>  	  enabling and clock setup and will be used by K1 udc/ehci/otg/xhci driver.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260825052158.66367-1-bbanelli@gmail.com?part=1

-- 
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy