Should the synchronous operation that build the query to DB be wrapped into Task.FromResult()

Question

I have a code and I\'m feeling (but not quite sure) there is a code smell in it:

This method calls repository method GetAllByOrderIdAsync: