On Mon, Jan 18, 2016 at 09:52:27AM +0000, Liviu.Dudau@arm.com wrote:
On Sat, Jan 16, 2016 at 10:17:54PM +0200, Jyri Sarha wrote:
It just makes me wonder if drm_atomic_helper_connector_dpms() should call the legacy callback automatically if DRIVER_ATOMIC is not set or at least bail out gracefully with an error message. Then again it may be overkill if the tda998x is the only driver that need to support both situations.
I would guess that in general it makes sense for the core code to do what you suggest, but the idea is to try to encourage the drivers to convert to atomic ops rather than give them too many crutches. And the tda998x will need this for a short period of time until tilcdc converts as well.
It's not just tilcdc that this driver is used by.
I have no plans at present to convert armada to atomic mode setting, although I occasionally push it (I hope) towards that goal. This isn't going to happen quickly, because I've other areas that need work too.