From mboxrd@z Thu Jan  1 00:00:00 1970
From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
Date: Mon, 22 Apr 2013 21:00:49 +0000
Subject: Re: [PATCH v2 1/5] V4L2: I2C: ML86V7667 video decoder driver
Message-Id: <5175A501.2060009@cogentembedded.com>
List-Id: <linux-sh.vger.kernel.org>
References: <201304212240.30949.sergei.shtylyov@cogentembedded.com> <201304220848.04870.hverkuil@xs4all.nl> <5174F74E.9030707@cogentembedded.com> <201304221106.20598.hverkuil@xs4all.nl> <5175A3C9.4050204@cogentembedded.com>
In-Reply-To: <5175A3C9.4050204@cogentembedded.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>, mchehab@redhat.com, linux-media@vger.kernel.org, linux-sh@vger.kernel.org, matsu@igel.co.jp

Vladimir Barinov wrote:
> Hi Hans,
>
> Hans Verkuil wrote:
>>>>> +     */
>>>>> +    val = i2c_smbus_read_byte_data(client, STATUS_REG);
>>>>> +    if (val < 0)
>>>>> +        return val;
>>>>> +
>>>>> +    priv->std = val & STATUS_NTSCPAL ? V4L2_STD_PAL : V4L2_STD_NTSC;
>>>>>             
>>>> Shouldn't this be 50 Hz vs 60 Hz formats? There are 60 Hz PAL 
>>>> standards
>>>> and usually these devices detect 50 Hz vs 60 Hz, not NTSC vs PAL.
>>>>         
>>> In the reference manual it is not mentioned about 50/60Hz input 
>>> format selection/detection but it mentioned just PAL/NTSC.
>>> The 50hz formats can be ether PAL and NTSC formats variants. The 
>>> same is applied to 60Hz.
>>>
>>> In the ML86V7667 datasheet the description for STATUS register 
>>> detection bit is just PAL/NTSC:
>>> " $2C/STATUS [2] NTSC/PAL identification 0: NTSC /1: PAL "
>>>
>>> If you assure me that I must judge their description as 50 vs 60Hz 
>>> formats and not PAL/NTSC then I will make the change.
>>>     
>>
>> I can't judge that. Are there no status bits anywhere that tell you 
>> something
>> about the number of lines per frame or the framerate?
>>   
> You are right. I've found a relationship table with description of 
> number of total H/V pixels vs Video Modes mentioned in datasheet.
> It's  "NTSC" has Odd/263 and Even/262 vertical lines. The "PAL" has 
> Odd/312 and Even/313.
A little clarification: it's "NTSC" relates to 485 active lines (after 
rejection of blank lines) and "PAL" relates to  578 active lines.