summaryrefslogtreecommitdiffstats
path: root/drivers/net/hyperv
diff options
context:
space:
mode:
authorjustinstitt@google.com <justinstitt@google.com>2023-12-06 23:16:11 +0000
committerDavid S. Miller <davem@davemloft.net>2023-12-08 10:56:25 +0000
commit9b5f621cea6eff2f75b851c6c62cefbdb1673c44 (patch)
treed835f307d5e658665dfe69499b90e19e80cd04a2 /drivers/net/hyperv
parent2a48c635fd9a48699805bbfeee1e4b94b8fe819d (diff)
downloadlinux-9b5f621cea6eff2f75b851c6c62cefbdb1673c44.tar.gz
linux-9b5f621cea6eff2f75b851c6c62cefbdb1673c44.tar.bz2
linux-9b5f621cea6eff2f75b851c6c62cefbdb1673c44.zip
checkpatch: add ethtool_sprintf rules
Add some warnings for using ethtool_sprintf() where a simple ethtool_puts() would suffice. The two cases are: 1) Use ethtool_sprintf() with just two arguments: | ethtool_sprintf(&data, driver[i].name); or 2) Use ethtool_sprintf() with a standalone "%s" fmt string: | ethtool_sprintf(&data, "%s", driver[i].name); The former may cause -Wformat-security warnings while the latter is just not preferred. Both are safely in the category of warnings, not errors. Signed-off-by: Justin Stitt <justinstitt@google.com> Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/hyperv')
0 files changed, 0 insertions, 0 deletions