Both Trezor and Ledger Nano S are good choice. It will be no bad choice between them. I'd even say - must have at least one of them, if you are holding some real amount of coins.