Not sure about this. I think Controller is reserved for actual routing controllers, this is an entity controller that will soon be renamed to EntityListBuilder. So I kind of like to keep them unter the Entity namespace.

Same for entity forms, they're not standalone forms.

Core isn't doing that yet, it's more of a personal preference that I started to experiment with in my own contrib projects.

Moving the Settings form out of Entity makes a lot of sense, though