My attention was caught by the explicit call of string_escape_str() which is
not required in this case and saves a lot of memory on the stack (see patch 1).
But then I noticed more to improve, hence this mini-series.
In v2:
- addressed Sashiko conserns
- added patches 2-5
v1: 20260929082722.3368741-1-andriy.shevchenko@linux.intel.com
Andy Shevchenko (5):
eea: Drop temporary buffer by using %*pEhp directly
eea: Remove unneeded assignment
eea: Refactor error handling in eea_adminq_config_host_info()
eea: unify return code variable name
eea: Use 2-argument strscpy()
drivers/net/ethernet/alibaba/eea/eea_adminq.c | 65 +++++++++----------
.../net/ethernet/alibaba/eea/eea_ethtool.c | 4 +-
2 files changed, 32 insertions(+), 37 deletions(-)
--
2.50.1