]> git.ipfire.org Git - thirdparty/openwrt.git/commitdiff
generic: permit support of standalone PCS for external kernel module 23349/head
authorChristian Marangi <ansuelsmth@gmail.com>
Wed, 13 May 2026 16:34:32 +0000 (18:34 +0200)
committerChristian Marangi <ansuelsmth@gmail.com>
Fri, 15 May 2026 17:42:16 +0000 (19:42 +0200)
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>
target/linux/generic/config-6.12
target/linux/generic/config-6.18
target/linux/generic/pending-6.12/770-04-net-pcs-implement-Firmware-node-support-for-PCS-driv.patch
target/linux/generic/pending-6.18/737-04-net-pcs-implement-Firmware-node-support-for-PCS-driv.patch

index 51b7dc79d8de20b4bec6ece3f9ed4b886902a364..ccd23a0ca2adb889b5a33e20d77c0ba349d9ab6e 100644 (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
index 44578c1d731f09dd85a858d0f78546d8390432d7..6373a39880446f88784254c756be10892e83914d 100644 (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
index 676e23c00194500ad83aab1492dd7e26bc7163dd..da608c6c21ef6691d93fce5103140a11dfaeea2a 100644 (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
index 676e23c00194500ad83aab1492dd7e26bc7163dd..da608c6c21ef6691d93fce5103140a11dfaeea2a 100644 (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