realtek: Remove extern from function declarations
authorSven Eckelmann <[email protected]>
Sun, 23 Nov 2025 11:49:24 +0000 (12:49 +0100)
committerHauke Mehrtens <[email protected]>
Mon, 24 Nov 2025 23:28:49 +0000 (00:28 +0100)
In contrast to variables, functions don't need extern declarations. It is
also preferred in the kernel not to use extern in front of function
declarations.

The extern+static parts in clk-rtl83xx.c were skipped because they are a
little bit unexpected ("extern *_dram_set_rate" are never used, "static
_sram_set_rate" are used but should from the C code perspective always be
NULL). This is left for an interested reader with the correct test HW and
some interests to dig in the code from commit 4850bd887c3a ("realtek: add
RTL83XX clock driver") for the *_dram_set_rate -> *_sram_set_rate
relocation and how these SRAM function pointers are set in this translation
unit.

Signed-off-by: Sven Eckelmann <[email protected]>
Link: https://github.com/openwrt/openwrt/pull/20906
Signed-off-by: Hauke Mehrtens <[email protected]>
target/linux/realtek/files-6.12/drivers/net/dsa/rtl83xx/common.c
target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.c

index 8c4880d9e45e75886e1f0783d8769142500c2502..ded16a480e5a67432acb83ce854077ab06417689 100644 (file)
@@ -26,7 +26,7 @@ extern const struct rtl838x_reg rtl931x_reg;
 extern const struct dsa_switch_ops rtl83xx_switch_ops;
 extern const struct dsa_switch_ops rtl93xx_switch_ops;
 
-extern struct phylink_pcs *rtpcs_create(struct device *dev, struct device_node *np, int port);
+struct phylink_pcs *rtpcs_create(struct device *dev, struct device_node *np, int port);
 
 int rtl83xx_port_get_stp_state(struct rtl838x_switch_priv *priv, int port)
 {
index 2cdb726f9a2451c8b4dbb8b738032a0fab12e0e2..f937be226c12c4e6d54d3783bcc6ba820a830c3e 100644 (file)
@@ -25,7 +25,7 @@
 
 extern struct rtl83xx_soc_info soc_info;
 
-extern int rtl83xx_setup_tc(struct net_device *dev, enum tc_setup_type type, void *type_data);
+int rtl83xx_setup_tc(struct net_device *dev, enum tc_setup_type type, void *type_data);
 
 /* Maximum number of RX rings is 8 on RTL83XX and 32 on the 93XX
  * The ring is assigned by switch based on packet/port priortity