| Message ID | 20220920063202.215088-1-ye.xingchen@zte.com.cn (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [linux-next] selftests: timers: Remove the unneeded result variable | expand |
On Tue, Sep 20, 2022 at 06:32:02AM +0000, cgel.zte@gmail.com wrote: > From: ye xingchen <ye.xingchen@zte.com.cn> > > Return the value adjtimex() directly instead of storing it in another > redundant variable. > > Reported-by: Zeal Robot <zealci@zte.com.cn> > Signed-off-by: ye xingchen <ye.xingchen@zte.com.cn> Once again, as this seems to keep being forgotten, you are sending multiple patches all from the same gmail.com account, yet are claiming they are from different people from a company that no one from that company has agreed is actually at that company. Then there's the whole "ignoring Documentation/process/researcher-guidelines.rst" issues, which would disqualify the changes anyway. So we can not take these patches at all. Please fix up your corporate email structures so that you can properly submit changes to the kernel that can be verified to actually come from your company. Otherwise they can not be accepted at all for obvious reasons (ZTE would not want to have people claiming to work for them that are not actually working for them, right?) thanks, greg k-h
diff --git a/tools/testing/selftests/timers/valid-adjtimex.c b/tools/testing/selftests/timers/valid-adjtimex.c index 48b9a803235a..4bca39e0bd16 100644 --- a/tools/testing/selftests/timers/valid-adjtimex.c +++ b/tools/testing/selftests/timers/valid-adjtimex.c @@ -50,12 +50,10 @@ int clock_adjtime(clockid_t id, struct timex *tx) int clear_time_state(void) { struct timex tx; - int ret; tx.modes = ADJ_STATUS; tx.status = 0; - ret = adjtimex(&tx); - return ret; + return adjtimex(&tx); } #define NUM_FREQ_VALID 32