Re: [PATCHv2 03/10] Revert "drm: omapdrm: Remove manual update display support"

From: Tony Lindgren
Date: Mon Mar 20 2017 - 13:09:28 EST


* Tomi Valkeinen <tomi.valkeinen@xxxxxx> [170320 09:38]:
> On 20/03/17 18:14, Tony Lindgren wrote:
> > * Tomi Valkeinen <tomi.valkeinen@xxxxxx> [170320 04:10]:
> >> On 05/03/17 02:43, Sebastian Reichel wrote:
> >>> This reverts commit 5a35876e2830511cb8110667fc426c6a6165a593.
> >>>
> >>> Revert the removal of manual update display support in
> >>> preparation for DSI command mode panels.
> >>
> >> I think it's much better to just add the manual update support from
> >> scratch rather than first reverting this old cleanup commit and then
> >> adding more bits and pieces.
> >
> > Do you know if the command mode panels support partial refresh?
> >
> > Just wondering if we want to keep "int x, int y, int w, int h"
> > around for omap_connector_flush() or whatever it ends up being?
>
> Partial update? Yes they do. But setting everything up for partial
> update on the DSS side is horribly complex, and support for that was
> removed from DSS driver long ago.
>
> So I think those parameters can be removed.

OK works for me.

Regards,

Tony