You typically use either discard or the fstrim.timer, but not both at the same time.

Using the discard option means that trims are being done on the fly every time blocks are deleted, using fstrim.timer means that trims are being done periodically. The former carries a performance penalty, so it’s usually not recommended unless you need it (for example, if you regularly do huge amounts of writes and deletes on this drive).