Commit f8b17a0b authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller

net: dsa: tag_sja1105: optionally build as module when switch driver is module if PTP is enabled

TX timestamps are sent by SJA1110 as Ethernet packets containing
metadata, so they are received by the tagging driver but must be
processed by the switch driver - the one that is stateful since it
keeps the TX timestamp queue.

This means that there is an sja1110_process_meta_tstamp() symbol
exported by the switch driver which is called by the tagging driver.

There is a shim definition for that function when the switch driver is
not compiled, which does nothing, but that shim is not effective when
the tagging protocol driver is built-in and the switch driver is a
module, because built-in code cannot call symbols exported by modules.

So add an optional dependency between the tagger and the switch driver,
if PTP support is enabled in the switch driver. If PTP is not enabled,
sja1110_process_meta_tstamp() will translate into the shim "do nothing
with these meta frames" function.

Fixes: 566b18c8 ("net: dsa: sja1105: implement TX timestamping for SJA1110")
Reported-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent b37a4668
...@@ -138,6 +138,7 @@ config NET_DSA_TAG_LAN9303 ...@@ -138,6 +138,7 @@ config NET_DSA_TAG_LAN9303
config NET_DSA_TAG_SJA1105 config NET_DSA_TAG_SJA1105
tristate "Tag driver for NXP SJA1105 switches" tristate "Tag driver for NXP SJA1105 switches"
depends on (NET_DSA_SJA1105 && NET_DSA_SJA1105_PTP) || !NET_DSA_SJA1105 || !NET_DSA_SJA1105_PTP
select PACKING select PACKING
help help
Say Y or M if you want to enable support for tagging frames with the Say Y or M if you want to enable support for tagging frames with the
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment