I’m not saying you can’t have all the doctor and patient information shared in the user profile sheet. It’s just that they are unrelated pieces of information. In my opinion, good database design keeps that information separate. Otherwise for each new component you add on a profile detail screen, you will have to determine if it can be visible only for doctors, for patients, or for both.
Yes this affectively still doubles the row count, but ultimately you do what works for you. There’s no right or wrong way to do it. Just think about future maintenance.
Good idea with the separate visits sheet.