Re: [PATCH 08/65] OMAPDSS: remove partial update from panel-taal

Archit Taneja
2011-11-23T05:50:21Z

On Tuesday 22 November 2011 06:02 PM, Tomi Valkeinen wrote:
> On Tue, 2011-11-22 at 17:23 +0530, Archit Taneja wrote:
>>> - r = taal_set_update_window(td, x, y, w, h);
>>> + /* XXX no need to send this every frame, but dsi break if not
>> done */
>>> + r = taal_set_update_window(td, 0, 0,
>>> + td->panel_config->timings.x_res,
>>> + td->panel_config->timings.y_res);
>>
>> How about sending a null short packet, and a BTA after that. This
>> will
>> keep automatic TE mode in place, and we'll need to send 1 short
>> packet
>> instead of 2 long packets every frame.
>
> To be honest, I didn't spend any time with this. True, sending a null
> packet and BTA instead of the 2 long packets is possible. But probably
> even better would be to track the TE status, and send a BTA only when
> needed.

Right, if we maintain the TE status, we may not need to send a packet at
all, so I guess we could stick with this for now, and remove it once we
start maintaining the number of BTAs we have sent to the panel.

Archit

>
> Tomi
>