This should be more than low-hanging. ;)

Is it possible to remove the FATFS_FFCONF_OPT_FS_NORTC on line 353 if the RTC is required anyway?

@bergzand the RTC is no mandatory feature for FatFs to work. You can still use it without RTC. Of course you won't get proper timestamps on your files then.

@MichelRottleuthner Yes, that's what I understood from the documentation. I'm asking the question because @kaspar030 add the periph_rtc to the required modules in this PR for the fatfs test.

If the RTC is considered a requirement for the test in the makefile, why then make it optional in the code.

Makes sense. I would argue that you don't really gain a benefit by adding the RTC as hard dependency. The test doesn't check timestamp relevant stuft at the moment. With adding the RTC as hard dependency you would disable testing for platforms that come without RTC support. If I remeber correctly I just wanted to enable the RTC feature for Boards that actually come with an RTC.

looking at pkg/fatfs/Makefile.dep: I think it sould be FEATURES_OPTIONAL here, right?

looking at pkg/fatfs/Makefile.dep: I think it sould be FEATURES_OPTIONAL here, right?

Yes, done!

&go.