Michal Suchánek <msucha...@suse.de> writes: > On 2017-07-29 09:24, SZ Lin wrote: >> Fix following checkpatch.pl warning: >> WARNING: Prefer using '"%s...", __func__' to using >> the function's name, in a string >> >> Signed-off-by: SZ Lin <sz....@moxa.com> >> --- >> drivers/char/tpm/tpm_ibmvtpm.c | 12 ++++++------ >> 1 file changed, 6 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/char/tpm/tpm_ibmvtpm.c >> b/drivers/char/tpm/tpm_ibmvtpm.c >> index e75a674b44ac..2d33acc43e25 100644 >> --- a/drivers/char/tpm/tpm_ibmvtpm.c >> +++ b/drivers/char/tpm/tpm_ibmvtpm.c >> @@ -151,7 +151,7 @@ static int tpm_ibmvtpm_send(struct tpm_chip *chip, >> u8 *buf, size_t count) >> rc = ibmvtpm_send_crq(ibmvtpm->vdev, be64_to_cpu(word[0]), >> be64_to_cpu(word[1])); >> if (rc != H_SUCCESS) { >> - dev_err(ibmvtpm->dev, "tpm_ibmvtpm_send failed rc=%d\n", rc); >> + dev_err(ibmvtpm->dev, "%s failed rc=%d\n", __func__, rc); > > Can function name contain a %?
Let's hope not. $ git grep "%s.*__func__" | wc -l 16937 cheers