net: dsa: tag_sja1105: be dsa_loop-safe
authorVladimir Oltean <vladimir.oltean@nxp.com>
Tue, 17 Aug 2021 14:58:47 +0000 (17:58 +0300)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Aug 2021 09:33:15 +0000 (10:33 +0100)
commit994d2cbb08ca05e3c1af954ec63a3ae32a862ac5
tree8a15be27717739c6f4809478f7d6563a04605671
parent93e271632ccf0c65395900a9c12720652f769d2a
net: dsa: tag_sja1105: be dsa_loop-safe

Add support for tag_sja1105 running on non-sja1105 DSA ports, by making
sure that every time we dereference dp->priv, we check the switch's
dsa_switch_ops (otherwise we access a struct sja1105_port structure that
is in fact something else).

This adds an unconditional build-time dependency between sja1105 being
built as module => tag_sja1105 must also be built as module. This was
there only for PTP before.

Some sane defaults must also take place when not running on sja1105
hardware. These are:

- sja1105_xmit_tpid: the sja1105 driver uses different VLAN protocols
  depending on VLAN awareness and switch revision (when an encapsulated
  VLAN must be sent). Default to 0x8100.

- sja1105_rcv_meta_state_machine: this aggregates PTP frames with their
  metadata timestamp frames. When running on non-sja1105 hardware, don't
  do that and accept all frames unmodified.

- sja1105_defer_xmit: calls sja1105_port_deferred_xmit in sja1105_main.c
  which writes a management route over SPI. When not running on sja1105
  hardware, bypass the SPI write and send the frame as-is.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/sja1105/sja1105_main.c
include/linux/dsa/sja1105.h
net/dsa/Kconfig
net/dsa/tag_sja1105.c