Please clarify for me this doubt I have about Always On - Active/Passive.
There is no such things as Active/Passive. It's an x number of instance and x number of node setup. If you had 4 instances on 5 nodes, what would you say? Active/Passive/Active+Active+Active/Passive/Passive? That doesn't make any sense.
I have set it up so that the node 1 is primary, and node 2 is secondary. Not Readable.
Pretty normal setup, nothing wrong with that.
Is this correct for Active/Passive ?
See above. This is correct if you're looking to have a primary with a non-readable secondary.