On Thu, 8 Feb 2007, Benjamin Herrenschmidt wrote:

On Wed, 2007-02-07 at 17:51 +0100, Geert Uytterhoeven wrote:

quoted

On Wed, 7 Feb 2007, Geert Uytterhoeven wrote:

quoted

On Wed, 7 Feb 2007, Benjamin Herrenschmidt wrote:

quoted

quoted

ps3av: Use a kernel thread to handle the actual video mode setting, as this
involves some quite big delays.

The ps3 fbdev already has it's own kernel thread no ? Why use a second
one ? We could just have the ps3av code keep the delays but be called by
a thread ?

That's another possibility.  And since the screen is blanked during mode
change, there's no need to refresh it, and I may even get rid of the call to
ps3fb_flip_ctl() in ps3av.

I'll look into it...

Below's a new patch (take 3).

But it doesn't work well: after each mode change, the old contents of the
screen are visible for a (noticable) fraction of a second.
Ps3fbd doesn't refresh the screen contents until ps3av_set_video_mode()
returns, which is longer than my display needs to sync to the new video mode
(this was actually the reason behind the msleep() calls).

I also could not get rid of the calls to ps3fb_flip_ctl() in ps3av, as screen
updates have to be disabled while sending other AV packets, too.

So I prefer the previous solution, using a new ps3av kernel thread.
Unless anyone has a better solution, of course :-)

Moving the flip to ps3av ? :-)

Using the same kthread as used for mode setting? Besides moving half of ps3fb
to ps3av, that will give the same problems.

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium