<div dir="ltr"><div>Hello Cyrill,</div><div><br></div><div>Thank you for your response!<br></div><div><br></div><div>I didn't take a look at the file testcases/kernel/device-drivers/rtc/rtc01.c when writing rtc ioctl tests. <br></div><div>I see that it runs tests for certain ioctls that are covered in the patches I sent (i.e. RTC_RD_TIME, RTC_SET_TIME, RTC_AIE_ON,...).</div><div><br></div><div>As I see, this file runs tests for certain ioctls more thoroughly than my test since it is meant to test the functionality of the rtc driver. My tests are only meant to test the functionalities of rtc ioctls without testing whether the driver works correctly. For instance, when running tests for ioctls used for interrupts (RTC_AIE_ON, RTC_AIE_OFF, RTC_PIE_ON,...), I only check whether these ioctls work to turn on/off rtc interrupts (alarm, periodic, update, watchdog). I don't check whether the interrupts work on the driver correctly.</div><div><br></div><div>Anyway, I am new to LTP so I am not that familiar with the scope of functionalities ioctl tests should cover. Any feedback you can give me on this would be greatly appreciated.</div><div><br></div><div>I am looking forward to your review of my patches.</div><div><br></div><div>Best regards,</div><div>Filip<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 24, 2020 at 4:58 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
Looking at these I guess that we should also delete the<br>
testcases/kernel/device-drivers/rtc/ as a last patch in this series<br>
since these tests seems to cover a bit more than the original test.<br>
<br>
I will have a closer look at the patchset next week.<br>
<br>
-- <br>
Cyril Hrubis<br>
<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a><br>
</blockquote></div>