Skip to content

Commit

Permalink
phy: samsung: Merge Kconfig for Exynos5420 and Exynos5250
Browse files Browse the repository at this point in the history
Exynos5420 variant of USB2 PHY is handled by the same code as the
Exynos5250 one. Introducing a separate Kconfig symbol for it was an
over-engineering, which turned out to cause build break for certain
configurations:

ERROR: modpost: "exynos5420_usb2_phy_config" [drivers/phy/samsung/phy-exynos-usb2.ko] undefined!

Fix this by removing PHY_EXYNOS5420_USB2 symbol and using
PHY_EXYNOS5250_USB2 also for Exynos5420 SoCs.

Reported-by: Markus Reichl <[email protected]>
Fixes: 81b534f ("phy: samsung: Add support for the Exynos5420 variant of the USB2 PHY")
Signed-off-by: Marek Szyprowski <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
Reviewed-by: Jaehoon Chung <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Vinod Koul <[email protected]>
  • Loading branch information
mszyprow authored and vinodkoul committed Dec 2, 2020
1 parent 8145dcb commit 3eaf2da
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
7 changes: 1 addition & 6 deletions drivers/phy/samsung/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -64,12 +64,7 @@ config PHY_EXYNOS4X12_USB2
config PHY_EXYNOS5250_USB2
bool
depends on PHY_SAMSUNG_USB2
default SOC_EXYNOS5250

config PHY_EXYNOS5420_USB2
bool
depends on PHY_SAMSUNG_USB2
default SOC_EXYNOS5420
default SOC_EXYNOS5250 || SOC_EXYNOS5420

config PHY_S5PV210_USB2
bool "Support for S5PV210"
Expand Down
2 changes: 0 additions & 2 deletions drivers/phy/samsung/phy-samsung-usb2.c
Original file line number Diff line number Diff line change
Expand Up @@ -127,8 +127,6 @@ static const struct of_device_id samsung_usb2_phy_of_match[] = {
.compatible = "samsung,exynos5250-usb2-phy",
.data = &exynos5250_usb2_phy_config,
},
#endif
#ifdef CONFIG_PHY_EXYNOS5420_USB2
{
.compatible = "samsung,exynos5420-usb2-phy",
.data = &exynos5420_usb2_phy_config,
Expand Down

0 comments on commit 3eaf2da

Please sign in to comment.