Commit e8ab2317 authored by Vadim Fedorenko's avatar Vadim Fedorenko Committed by Jakub Kicinski
Browse files

net: ethtool: tsconfig: set command must provide a reply 



Timestamping configuration through ethtool has inconsistent behavior of
skipping the reply for set command if configuration was not changed. Fix
it be providing reply in any case.

Fixes: 6e9e2eed ("net: ethtool: Add support for tsconfig command to get/set hwtstamp config")
Signed-off-by: default avatarVadim Fedorenko <vadfed@meta.com>
Reviewed-by: default avatarKory Maincent <kory.maincent@bootlin.com>
Link: https://patch.msgid.link/20250922231924.2769571-1-vadfed@meta.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent f67e9ae7

net/ethtool/tsconfig.c

+5 −7
Original line number Diff line number Diff line
@@ -423,14 +423,12 @@ static int ethnl_set_tsconfig(struct ethnl_req_info *req_base, 
			return ret;

	}



	if (hwprov_mod || config_mod) {

	ret = tsconfig_send_reply(dev, info);

	if (ret && ret != -EOPNOTSUPP) {

		NL_SET_ERR_MSG(info->extack,

			       "error while reading the new configuration set");

		return ret;

	}

	}



	/* tsconfig has no notification */

	return 0;