Date: Mon, 27 Oct 1997 16:25:25 +0100 (MET) From: Eivind Eklund <perhaps@yes.no> To: Joao Carlos Mendes Luis <jonny@coppe.ufrj.br> Cc: multimedia@FreeBSD.ORG Subject: Re: Bt848 driver patches Message-ID: <199710271525.QAA02082@bitbox.follo.net> In-Reply-To: Joao Carlos Mendes Luis's message of Mon, 27 Oct 1997 11:55:23 -0200 (EDT) References: <199710270202.DAA00345@bitbox.follo.net> <199710271355.LAA22646@gaia.coppe.ufrj.br>
next in thread | previous in thread | raw e-mail | index | archive | help
>
> #define quoting(Eivind Eklund)
> ...
> // +/* # define BT848_IFORM_F_NTSCM (0x1) */
> // + { 525, 22, 480, 910, 135, 754, 640, 780, 30, 0x68, 0x5d, BT848_IFORM_X_XT0 },
> ...
> // +/* # define BT848_IFORM_F_PALM (0x4) */
> // + { 625, 32, 576, 910, 186, 922, 640, 780, 25, 0x68, 0x5d, BT848_IFORM_X_XT0 },
>
> I have not yet tested your code, but PAL/M is just like NTSC when dealing
> with B/W format. In other words: 525x60 and not 625x50...
OK, is
{ 525, 22, 480, 910, 186, 754, 640, 780, 30, 0x68, 0x5d, BT848_IFORM_X_XT0 },
correct then? Can you test it?
> // case BT848_IFORM_F_NTSCM:
> // case BT848_IFORM_F_NTSCJ:
> // - case BT848_IFORM_F_PALM:
>
> That's why this line was here...
>
> // case BT848_IFORM_F_PALBDGHI:
> // case BT848_IFORM_F_PALN:
> // case BT848_IFORM_F_SECAM:
> // case BT848_IFORM_F_RSVD:
> // + case BT848_IFORM_F_PALM:
>
> ...and not here. And this part of code works. I use it everyday. :)
This part of the code did not work for me; it might have worked for
PAL/M, but not for e.g. PAL/N mode.
However, that part of the code is now changed to only to only set the
Meteor flags to match PAL/NTSC, and calling PAL/M PAL instead of NTSC
seemed reasonable to me.
Eivind.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199710271525.QAA02082>