From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [-next] EDAC: Drop pointless static qualifier in edac_fake_inject_write() From: YueHaibing Message-Id: Date: Fri, 25 Jan 2019 15:57:10 +0800 To: Julia Lawall Cc: Borislav Petkov , Mauro Carvalho Chehab , James Morse , linux-edac@vger.kernel.org, kernel-janitors@vger.kernel.org List-ID: T24gMjAxOS8xLzI1IDEzOjQxLCBKdWxpYSBMYXdhbGwgd3JvdGU6Cj4gCj4gCj4gT24gRnJpLCAy NSBKYW4gMjAxOSwgWXVlSGFpYmluZyB3cm90ZToKPiAKPj4gVGhlcmUgaXMgbm8gbmVlZCB0byBo YXZlIHRoZSAnZW51bSBod19ldmVudF9tY19lcnJfdHlwZXR5cGUnIHZhcmlhYmxlCj4+IHN0YXRp YyBzaW5jZSBuZXcgdmFsdWUgYWx3YXlzIGJlIGFzc2lnbmVkIGJlZm9yZSB1c2UgaXQuCj4gCj4g SG93IGRpZCB5b3UgZmluZCB0aGlzPyAgSWYgeW91IHVzZWQgc29tZSB0b29sLCBpdCB3b3VsZCBy ZWFsbHkgYmUga2luZCBpZgo+IHlvdSB3b3VsZCByZWZlcmVuY2UgdGhhdCB0b29sLiAgUGVvcGxl IHdobyBtYWtlIHRvb2xzIG1heSBuZWVkIHRvIGp1c3RpZnkKPiB0aGF0IHRoZSB0aW1lIHRoYXQg dGhleSBzcGVuZCBkb2luZyBzbyBpcyB3b3J0aHdoaWxlLgoKClN1cmUsICB0aGUgdG9vbCBleGlz dHMgaW4gaHR0cHM6Ly9naXRodWIuY29tL3dlaXlqL2RwYXRjaAoKd2hpY2ggcnVuIGEgY29jY2lu ZWxsZSBzY3JpcHQKCj4gCj4ganVsaWEKPiAKPj4KPj4gU2lnbmVkLW9mZi1ieTogWXVlSGFpYmlu ZyA8eXVlaGFpYmluZ0BodWF3ZWkuY29tPgo+PiAtLS0KPj4gIGRyaXZlcnMvZWRhYy9kZWJ1Z2Zz LmMgfCAyICstCj4+ICAxIGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24o LSkKPj4KPj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZWRhYy9kZWJ1Z2ZzLmMgYi9kcml2ZXJzL2Vk YWMvZGVidWdmcy5jCj4+IGluZGV4IDBhOTI3NzIuLjY3MzY1NDMgMTAwNjQ0Cj4+IC0tLSBhL2Ry aXZlcnMvZWRhYy9kZWJ1Z2ZzLmMKPj4gKysrIGIvZHJpdmVycy9lZGFjL2RlYnVnZnMuYwo+PiBA QCAtOCw3ICs4LDcgQEAgc3RhdGljIHNzaXplX3QgZWRhY19mYWtlX2luamVjdF93cml0ZShzdHJ1 Y3QgZmlsZSAqZmlsZSwKPj4gIHsKPj4gIAlzdHJ1Y3QgZGV2aWNlICpkZXYgPSBmaWxlLT5wcml2 YXRlX2RhdGE7Cj4+ICAJc3RydWN0IG1lbV9jdGxfaW5mbyAqbWNpID0gdG9fbWNpKGRldik7Cj4+ IC0Jc3RhdGljIGVudW0gaHdfZXZlbnRfbWNfZXJyX3R5cGUgdHlwZTsKPj4gKwllbnVtIGh3X2V2 ZW50X21jX2Vycl90eXBlIHR5cGU7Cj4+ICAJdTE2IGVycmNvdW50ID0gbWNpLT5mYWtlX2luamVj dF9jb3VudDsKPj4KPj4gIAlpZiAoIWVycmNvdW50KQo+Pgo+Pgo+Pgo+Pgo+Pgo+Pgo+IAo+IC4K Pgo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: YueHaibing Date: Fri, 25 Jan 2019 07:57:10 +0000 Subject: Re: [PATCH -next] EDAC: Drop pointless static qualifier in edac_fake_inject_write() Message-Id: List-Id: References: <1548383420-163054-1-git-send-email-yuehaibing@huawei.com> In-Reply-To: <1548383420-163054-1-git-send-email-yuehaibing@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Julia Lawall Cc: Borislav Petkov , Mauro Carvalho Chehab , James Morse , linux-edac@vger.kernel.org, kernel-janitors@vger.kernel.org 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 >> --- >> 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) >> >> >> >> >> >> > > . >