ledger nano s supports all the erc20 tokens which trezor dont,but these two are secured wallets.
this not true though. i have a trezor and its ethereum wallet attaches to myetherwallet, which most definitely supports all erc20 tokens.
Ledger and trezor are both good. I think trezor is more secure but ledger supports more coins overall.