On 2019/1/25 13:41, Julia Lawall wrote: > > > On Fri, 25 Jan 2019, YueHaibing wrote: > >> There is no need to have the 'enum hw_event_mc_err_typetype' variable >> static since new value always be assigned before use it. > > How did you find this? If you used some tool, it would really be kind if > you would reference that tool. People who make tools may need to justify > that the time that they spend doing so is worthwhile. Sure, the tool exists in https://github.com/weiyj/dpatch which run a coccinelle script > > julia > >> >> Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> >> --- >> drivers/edac/debugfs.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/edac/debugfs.c b/drivers/edac/debugfs.c >> index 0a92772..6736543 100644 >> --- a/drivers/edac/debugfs.c >> +++ b/drivers/edac/debugfs.c >> @@ -8,7 +8,7 @@ static ssize_t edac_fake_inject_write(struct file *file, >> { >> struct device *dev = file->private_data; >> struct mem_ctl_info *mci = to_mci(dev); >> - static enum hw_event_mc_err_type type; >> + enum hw_event_mc_err_type type; >> u16 errcount = mci->fake_inject_count; >> >> if (!errcount) >> >> >> >> >> >> > > . >