net: stmmac: do not silently change auxiliary snapshot capture channel
authorJohannes Zink <j.zink@pengutronix.de>
Wed, 18 Oct 2023 07:09:57 +0000 (09:09 +0200)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 19 Oct 2023 11:01:33 +0000 (13:01 +0200)
commit2ddd05d1d5ed2aa868df202b30b0ce7a1db5f14a
treeca685c9db1c218a5685e6c387a7ffabbbc2b6c14
parent7d3077482578fcc6eef6faeef7b8174fd32b7619
net: stmmac: do not silently change auxiliary snapshot capture channel

Even though the hardware theoretically supports up to 4 simultaneous
auxiliary snapshot capture channels, the stmmac driver does support only
a single channel to be active at a time.

Previously in case of a PTP_CLK_REQ_EXTTS request, previously active
auxiliary snapshot capture channels were silently dropped and the new
channel was activated.

Instead of silently changing the state for all consumers, log an error
and return -EBUSY if a channel is already in use in order to signal to
userspace to disable the currently active channel before enabling another one.

Signed-off-by: Johannes Zink <j.zink@pengutronix.de>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c