generic: permit support of standalone PCS for external kernel module

The current code permits support of the standalone PCS feature only for
in-tree kernel module but doesn't correctly support PCS from external
kernel module.

This is caused by the fact that the FWNODE_PCS config flag is internally
selected by any PCS driver and can't be selected directly. This is
problematic for any external kernel module that wants to use the standalone
PCS feature and needs the OPs provided by the generic PCS code.

Moreover compiling the standalone PCS code as a module is problematic and
would cause link error caused by the late PCS code that introduce a
notifier where phylink code depends on.

To address both problem, permit to select the FWNODE_PCS and change it to a
simple bool preventing it to compile as a module.

Link: https://github.com/openwrt/openwrt/pull/23349
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
This commit is contained in:
Christian Marangi 2026-05-13 18:34:32 +02:00
parent 3cc55ea28c
commit 14beb3408d
WARNING! Although there is a key with this ID in the database it does not verify this commit! This commit is SUSPICIOUS.
GPG Key ID: AC001D09ADBFEAD7
4 changed files with 4 additions and 2 deletions

View File

@ -2175,6 +2175,7 @@ CONFIG_FW_LOADER=y
CONFIG_FW_LOADER_USER_HELPER=y
CONFIG_FW_LOADER_USER_HELPER_FALLBACK=y
# CONFIG_FW_UPLOAD is not set
# CONFIG_FWNODE_PCS is not set
# CONFIG_FXAS21002C is not set
# CONFIG_FXLS8962AF_I2C is not set
# CONFIG_FXLS8962AF_SPI is not set

View File

@ -2272,6 +2272,7 @@ CONFIG_FW_LOADER=y
CONFIG_FW_LOADER_USER_HELPER=y
CONFIG_FW_LOADER_USER_HELPER_FALLBACK=y
# CONFIG_FW_UPLOAD is not set
# CONFIG_FWNODE_PCS is not set
# CONFIG_FXAS21002C is not set
# CONFIG_FXLS8962AF_I2C is not set
# CONFIG_FXLS8962AF_SPI is not set

View File

@ -56,7 +56,7 @@ Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
menu "PCS device drivers"
+config FWNODE_PCS
+ tristate
+ bool "PCS Firmware Node"
+ depends on (ACPI || OF)
+ depends on PHYLINK
+ help

View File

@ -56,7 +56,7 @@ Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
menu "PCS device drivers"
+config FWNODE_PCS
+ tristate
+ bool "PCS Firmware Node"
+ depends on (ACPI || OF)
+ depends on PHYLINK
+ help