Is conceptually correct to access DB from Controller?

If it's the Models save() function it's indeed correct. You trigger Models from Controllers, but should not use the query builder itself in Controllers.

You can read more about it in the userguide: https://codeigniter.com/user_guide/concepts/mvc.html