Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1604739 > unrolled thread

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

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  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

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

* 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?

Regards,

Tony

[toc] | [next] | [standalone]

#1604768

[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

[toc] | [prev] | [next] | [standalone]

#1604809

* Tomi Valkeinen <tomi.valkeinen@ti.com> [170320 09:38]:
> 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.

OK works for me.

Regards,

Tony

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web