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

From	Tomi Valkeinen <tomi.valkeinen@ti.com>
Newsgroups	linux.kernel
Subject	Re: [PATCHv2 03/10] Revert "drm: omapdrm: Remove manual update display support"
Date	2017-03-20 17:40 +0100
Message-ID	<tn8vF-7pZ-55@gated-at.bofh.it> (permalink)
References	<thrKF-8kZ-5@gated-at.bofh.it> <thsx3-rY-3@gated-at.bofh.it> <thsx3-rY-1@gated-at.bofh.it> <tn8lX-7kZ-9@gated-at.bofh.it> <tn8lX-7kZ-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On 20/03/17 18:14, Tony Lindgren wrote:
> * Tomi Valkeinen <tomi.valkeinen@ti.com> [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.

 Tomi

Thread

Re: [PATCHv2 03/10] Revert "drm: omapdrm: Remove manual update  display support" Tony Lindgren <tony@atomide.com> - 2017-03-20 17:30 +0100
  Re: [PATCHv2 03/10] Revert "drm: omapdrm: Remove manual update  display support" Tomi Valkeinen <tomi.valkeinen@ti.com> - 2017-03-20 17:40 +0100
    Re: [PATCHv2 03/10] Revert "drm: omapdrm: Remove manual update  display support" Tony Lindgren <tony@atomide.com> - 2017-03-20 18:10 +0100

csiph-web