Re: [vdr] [VGA sync field] question to fb-radeon-intel.patch

2009-03-12 Thread Paul Menzel
Am Donnerstag, den 12.03.2009, 18:31 +0100 schrieb Thomas Hilber:
> On Wed, Mar 11, 2009 at 06:15:04PM +0100, Paul Menzel wrote:
> > 1. In your patchset the patch fb-radeon-intel.patch [1] does include
> > changes to drivers/video/intelfb/intelfbhw.c although in the install
> > instructions [2] it is stated that intel drivers do not need to be
> > modified.
> 
> sorry, install instructions are not up to date for the Intel version
> of the patch.
> 
> > 
> > The changelog in 0.11 includes these two items.
> > 
> > - patch against intelfb (kernel 2.6.26) to allow for PAL/SCART
> >   video timings. You now can use a regular SCART CRT as display
> >   for linux console.
> > - fixed a bug in intelfb initialization which sporadically
> >   setup video timing with weirdous values
> > 
> > I guess the installation instructions need to be updated for release
> > 0.11.
> 
> right. as said installation instructions have not been touched since
> a while. I just included the plain Intel patches into the package.
> There was not yet too much interest in the patches anyway. So I 
> did not want to waste my time for documentation:-) 
> 
> > If this is correct I would update the instructions.
> 
> thank you for that.

Done (commit 89ddb9af67be949be3be01f795f964926728f45a). Not much, but I
hope alright.


Thanks,

Paul


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


Re: [vdr] [VGA sync field] question to fb-radeon-intel.patch

2009-03-12 Thread Thomas Hilber
On Thu, Mar 12, 2009 at 07:36:53PM +0100, Torgeir Veimo wrote:
> Did you look into making your approach work with DirectFB yet?

I don't think it's a big deal to port the Intel version of the
FRC patch to DirectFB. 

But sorry, I don't use DirectFB. I can't see much advantage over a plain
Xserver unless running something like an embedded system.

- Thomas


___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


Re: [vdr] [VGA sync field] question to fb-radeon-intel.patch

2009-03-12 Thread Thomas Hilber
On Thu, Mar 12, 2009 at 07:36:01PM +0100, Paul Menzel wrote:
> I hope, you have seen my message to linux-fbdev-devel [1]. I will try to
> do what Krzysztof suggested, but will have to read more about that.
> Maybe we get it into Linux kernel 2.6.30.

yeah, I've seen that. 

> Should we continue discussion about this on linux-fbdev-devel, that
> means, can subscribe there?

I subscribed to linux-fbdev-devel. But haven't had the time to adapt the
patch as requested.

- Thomas


___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


Re: [vdr] [VGA sync field] question to fb-radeon-intel.patch

2009-03-12 Thread Torgeir Veimo

On 12 Mar 2009, at 19:08, Thomas Hilber wrote:

> I forgot to say patching the intelfb is only needed if you even want  
> to
> run the linux console in VGA2SCART mode.


Did you look into making your approach work with DirectFB yet?

-- 
Torgeir Veimo
torg...@pobox.com





___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


Re: [vdr] [VGA sync field] question to fb-radeon-intel.patch

2009-03-12 Thread Paul Menzel
Am Donnerstag, den 12.03.2009, 19:08 +0100 schrieb Thomas Hilber:
> On Thu, Mar 12, 2009 at 06:31:47PM +0100, Thomas Hilber wrote:
> > > a) Why is MIN_CLOCK set to 25000 in intelfbhw.h [3]? What would be the
> > > downside of setting it to 1?
> > 
> > MIN_CLOCK is set to 12000 by the patch. What allows for SCART suitable
> > dotclocks. I don't know why the original driver denies such low
> > clock frequencies.
> 
> I forgot to say patching the intelfb is only needed if you even want to
> run the linux console in VGA2SCART mode.
> 
> For the Xserver in VGA2SCART+FRC mode it's not needed.

Thanks. I figured that after reading a lot of threads. (And thanks for
your other answer.)

I hope, you have seen my message to linux-fbdev-devel [1]. I will try to
do what Krzysztof suggested, but will have to read more about that.
Maybe we get it into Linux kernel 2.6.30.

Should we continue discussion about this on linux-fbdev-devel, that
means, can subscribe there?


Thanks,

Paul


[1] 
http://sourceforge.net/mailarchive/forum.php?thread_name=1236701836.4068.85.camel%40mattotaupa.wohnung.familie-menzel.net&forum_name=linux-fbdev-devel


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


Re: [vdr] [VGA sync field] question to fb-radeon-intel.patch

2009-03-12 Thread Thomas Hilber
On Thu, Mar 12, 2009 at 06:31:47PM +0100, Thomas Hilber wrote:
> > a) Why is MIN_CLOCK set to 25000 in intelfbhw.h [3]? What would be the
> > downside of setting it to 1?
> 
> MIN_CLOCK is set to 12000 by the patch. What allows for SCART suitable
> dotclocks. I don't know why the original driver denies such low
> clock frequencies.

I forgot to say patching the intelfb is only needed if you even want to
run the linux console in VGA2SCART mode.

For the Xserver in VGA2SCART+FRC mode it's not needed.

- Thomas

___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


Re: [vdr] [VGA sync field] question to fb-radeon-intel.patch

2009-03-12 Thread Thomas Hilber
On Wed, Mar 11, 2009 at 06:15:04PM +0100, Paul Menzel wrote:
> 1. In your patchset the patch fb-radeon-intel.patch [1] does include
> changes to drivers/video/intelfb/intelfbhw.c although in the install
> instructions [2] it is stated that intel drivers do not need to be
> modified.

sorry, install instructions are not up to date for the Intel version
of the patch.

> 
> The changelog in 0.11 includes these two items.
> 
> - patch against intelfb (kernel 2.6.26) to allow for PAL/SCART
>   video timings. You now can use a regular SCART CRT as display
>   for linux console.
> - fixed a bug in intelfb initialization which sporadically
>   setup video timing with weirdous values
> 
> I guess the installation instructions need to be updated for release
> 0.11.

right. as said installation instructions have not been touched since
a while. I just included the plain Intel patches into the package.
There was not yet too much interest in the patches anyway. So I 
did not want to waste my time for documentation:-) 

> If this is correct I would update the instructions.

thank you for that.

> a) Why is MIN_CLOCK set to 25000 in intelfbhw.h [3]? What would be the
> downside of setting it to 1?

MIN_CLOCK is set to 12000 by the patch. What allows for SCART suitable
dotclocks. I don't know why the original driver denies such low
clock frequencies.

> b) Where do I get information about the registers, like what DPLL_A and
> DPLL_VCO_ENABLE is? In the header file some values are assigned to them,
> but where is it documented what they mean?

DPLL_A (DPLLA_CTRL-DPLL A Control Register) is documented here:

IntelĀ® 965 Express Chipset Family, Volume Three
Section: Display Clock Control Registers

The doc can be found here:
http://intellinuxgraphics.org/documentation.html

Cheers
  Thomas


___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr