> Hi, > > Here is the updated RFC incorporating comments received on last one. > > Following are the changes:- > > 1) Removed V4L2_DV_CUSTOM since we have a capability flag for it in > v4l2_input/output structure > > 2) Replaced V4L2_DV_CUSTOMS with V4L2_DV_INVALID (value 0x00000000) > > 3) Renamed capability flags with V4L2_IN_CAP/V4L2_OUT_CAP prefixes > > ============================================================================ > RFC (v1.2): V4L - Support for video timings at the input/output interface > > Version : 1.2 > > Background > ----------- > > Currently v4l2 specification supports capturing video frames from TV > signals using tuners (input type V4L2_INPUT_TYPE_TUNER) and baseband TV > signals (V4L2_INPUT_TYPE_CAMERA) and sensors. Similarly on the output > side, the signals could be TV signal (V4L2_OUTPUT_TYPE_MODULATOR), > baseband TV signal (V4L2_OUTPUT_TYPE_ANALOG) or hybrid analog VGA overlay > (V4L2_OUTPUT_TYPE_ANALOGVGAOVERLAY) which output from a graphics card and > then use chromakeying to replace part of the picture with the video. > V4L2_OUTPUT_TYPE_ANALOG & V4L2_INPUT_TYPE_CAMERA are for analog interfaces > that includes composite, S-Video and VGA (for output only). Note that even > though VGA is a supported output, we don't have anyway to set the standard > or timing on the output. Standard ids are only defined for TVs using > v4l2_std_id and a set of bit masks defined for analog TV standards. > > Today we have a wide variety of different interfaces available to > transmit/receive video or graphics content between source device and > destination device. Following are some of the interfaces used in addition > to the ones described in the v4l2 specification. > > Component analog input/output interface - ED/HD video > DVI - Digital only, ANALOG only, DVI integrated that support Digital and > Analog; > Dual Link - Where second data link is used for higher bandwidth > SDI - Serial digital interface standardized by SMPTE > HDMI - HD video and Audio > DisplayPort - digital audio/video interconnect by VESA > > V4L2 specification currently defined NTSC/PAL/SECAM (all variants) > standards for describing the timing of the signal transmitted over these > interfaces. Even though the specification defined ANALOG output type for > VGA, there are no ways to set the timings used for output to VGA or LCD > display monitors. Some of the proprietary implementations used existing > standards IOCTL, VIDIOC_S_STD, to set these timings over these interfaces. > For example, TI SoCs have Video Processing Back End (VPBE) on various > media SOCs (Eg, DM6446, DM355 etc) that can output signal for Analog TV > and VGA interfaces (using Digital LCD port) and support timings for > displaying SD and HD videos (1080i, 1080p and 720p) as well as over VGA > interface to a CRT or LCD display monitor. So we need to enhance the v4l2 > specification to allow applications to set these timings in the capture or > output devices. This RFC proposes to add new IOCTLs for setting/getting > timings over the different interfaces described above and freeze the the > use of existing standards IOCTL and standards IDs for analog TVs only. > > Timings > ------- > > The timings at the analog or digital interface that are not covered by the > v4l2_std_id can be defined using a set of preset values that are used by > the hardware where the timings are predefined or by a set of timing values > which can be configured at the hardware to generate the signal expected at > the interface. The former will be used for hardware like TVP7002/THS8200 > which specifies preset timing required for output HD video such > 1080i50/60, 720p50/60 etc. The latter can be used for hardware that > requires configuration of frame timing such as front porch, hsync length, > vsync length, pixel clock etc. For example the earlier mentioned TI SOCs > have a Digital LCD port that can be configured to output different timing > values expected by LCD Display monitors. > > Preset timings (defined by VESA, SMPTE or BT.656/BT.1120 or others) can be > defined by the following structure:- > > struct v4l2_dv_preset { > __u32 preset; > __u32 reserved[4]; > }; > > Where preset is one of the following values:- > > #define V4L2_DV_INVALID 0x00000000 > #define V4L2_DV_480I59_94 0x00000001 > #define V4L2_DV_480I60 0x00000002 > #define V4L2_DV_480P23_976 0x00000003 > #define V4L2_DV_480P24 0x00000004 > #define V4L2_DV_480P29_97 0x00000005 > #define V4L2_DV_480P30 0x00000006 > #define V4L2_DV_576I50 0x00000007 > #define V4L2_DV_576P25 0x00000008 > #define V4L2_DV_576P50 0x00000009 > #define V4L2_DV_720P23_976 0x0000000A > #define V4L2_DV_720P24 0x0000000B > #define V4L2_DV_720P25 0x0000000C > #define V4L2_DV_720P29_97 0x0000000D > #define V4L2_DV_720P30 0x0000000E > #define V4L2_DV_720P50 0x0000000F > #define V4L2_DV_720P59_94 0x00000010 > #define V4L2_DV_720P60 0x00000011 > #define V4L2_DV_1080I50 0x00000012 > #define V4L2_DV_1080I59_94 0x00000013 > #define V4L2_DV_1080I60 0x00000014 > #define V4L2_DV_1080P23_976 0x00000015 > #define V4L2_DV_1080P24 0x00000016 > #define V4L2_DV_1080P25 0x00000017 > #define V4L2_DV_1080P29_97 0x00000018 > #define V4L2_DV_1080P30 0x00000019 > #define V4L2_DV_1080P60 0x00000020 Note: when defining this in the videodev2 header we also need to document the timings for each preset and where we got this data from (e.g. CEA861-E if I recall correctly). > > > Driver sets preset to V4L2_DV_INVALID to indicate there is no signal > detected or could not lock to the signal for a VIDIOC_QUERY_DV_PRESET > IOCTL > > This list is expected grow over time. So a bit mask is not used for this > (Unlike analog TV standards) so that many presets can be defined as needed > in the future. > > To enumerate the DV preset timings available, applications call > VIDIOC_ENUM_DV_PRESETS using the following structure:- > > struct v4l2_dv_enum_presets { > __u32 index; > __u32 preset; > __u8 name[32]; /* Name of the preset timing */ > __u32 width; > __u32 height; > __u32 reserved[4]; > }; > > Application set/get the preset by calling VIDIOC_S/G_DV_PRESET using > v4l2_dv_preset structure. > > Also add a capabilities field in the input and output structure to support > presets > > /* > * V I D E O I N P U T S > */ > struct v4l2_input { > __u32 index; /* Which input */ > __u8 name[32]; /* Label */ > __u32 type; /* Type of input */ > __u32 audioset; /* Associated audios (bitfield) > */ > __u32 tuner; /* Associated tuner */ > v4l2_std_id std; > __u32 status; > __u32 capabilities; > __u32 reserved[3]; > }; > > where capabilities can be one or more of the following:- > > #define V4L2_IN_CAP_PRESETS 0x00000001 > #define V4L2_IN_CAP_CUSTOM_TIMINGS 0x00000002 > #define V4L2_IN_CAP_STD 0x00000004 > > /* > * V I D E O O U T P U T S > */ > struct v4l2_output { > __u32 index; /* Which output */ > __u8 name[32]; /* Label */ > __u32 type; /* Type of output */ > __u32 audioset; /* Associated audios (bitfield) > */ > __u32 modulator; /* Associated modulator */ > v4l2_std_id std; > __u32 capabilities; > __u32 reserved[3]; > }; > > where capabilities can be one or more of the following:- > > #define V4L2_OUT_CAP_PRESETS 0x00000001 > #define V4L2_OUT_CAP_CUSTOM_TIMINGS 0x00000002 > #define V4L2_OUT_CAP_STD 0x00000004 > > For setting custom timing at the device, following structure is used which > defines the complete set of timing values required at the input and output > interface:- > > /* timing data values specified by various standards such as BT.1120, > BT.656 etc. */ > > /* bt.656/bt.1120 timing data */ > struct v4l2_bt_timings { > __u32 interlaced; > __u64 pixelclock; > __u32 width, height; > __u32 polarities; > __u32 hfrontporch, hsync, htotal; > __u32 vfrontporch, vsync, vtotal; > /* timings for bottom frame for interlaced formats */ > __u32 il_vtotal; > __u32 reserved[16]; > }; I think that we should be a change here: instead of specifying htotal and vtotal it is more symmetrical to specify hbackporch and vbackporch. And instead of il_vtotal I think it is better to specify il_vfrontporch, il_vsync, il_vbackporch. Since any of these three parameters can be the one that is 1 longer than the top field. > > > interlaced - Interlaced or progressive. use following values:- > > #define V4L2_DV_INTERLACED 0 > #define V4L2_DV_PROGRESSIVE 1 Make PROGRESSIVE 0 instead of 1. It fits the meaning of the field better. For the interlaced formats we might end up with two: INTERLACED_TOP_FIRST and INTERLACED_BOTTOM_FIRST: depending on the interlaced format the top field or the bottom field is sent first. That's something that can differ depending on the format (I know it is different between PAL and NTSC). I'm not sure how the CEA standard defines this. Otherwise it looks good. Regards, Hans > > pixelclock - expressed in HZ. So for 74.25MHz, use 74250000. > width - number of pixels in horizontal direction > height - number of lines in vertical direction > polarities - Bit mask for following polarities to begin with > (if polarity bit is not set, corresponding polarity is assumed to be > negative) > > #define V4L2_DV_VSYNC_POS_POL 0x00000001 > #define V4L2_DV_HSYNC_POS_POL 0x00000002 > > hfrontporch,hsync, and htotal for horizontal direction and vfrontporch, > vsync, and vtotal for vertical direction. il_vtotal is the number of > vertical lines for interlaced video for bottom field. > > To define a particular timing type, following enum is used:- > > enum v4l2_dv_timings_type { > V4L2_DV_BT_656_1120, > }; > > This will allow adding new timing types in the future. > > If the driver supports a set of custom timing, it can be set/get using > VIDIOC_S/G_DV_TIMINGS IOCTL and specifying timings using the structure > > struct v4l2_dv_timings { > enum v4l2_dv_timings_type type; > union { > struct v4l2_bt_timings bt; > __u32 reserved[32]; > }; > }; > > > If the driver supports custom timing as well as Presets, it will return > V4L2_DV_CUSTOM along with other preset timings for the > VIDIOC_ENUM_DV_PRESETS IOCTL call. Application can then call > VIDIOC_S/G_TIMING to get/set custom timings at the driver. > > To detect a preset timing at the input application calls > VIDIOC_QUERY_DV_PRESET which returns the preset using the v4l2_dv_preset > structure. > > Following summarize the new ioctls added by this RFC > > #define VIDIOC_ENUM_DV_PRESETS _IOWR('V', 79, struct > v4l2_dv_enum_presets) > #define VIDIOC_S_DV_PRESET _IOWR('V', 80, struct v4l2_dv_preset) > #define VIDIOC_G_DV_PRESET _IOWR('V', 81, struct v4l2_dv_preset) > #define VIDIOC_QUERY_DV_PRESET _IOR('V', 82, struct v4l2_dv_preset) > #define VIDIOC_S_DV_TIMINGS _IOWR('V', 83, struct v4l2_dv_timings) > #define VIDIOC_G_DV_TIMINGS _IOWR('V', 84, struct v4l2_dv_timings) > > Open issues > ----------- > > 1.How to handle an HDMI transmitter? It can be put in two different modes: > DVI compatible > or HDMI compatible. Some of the choices are > a) enumerate them as two different outputs when enumerating. > b) adding a status bit on the input. > c) change it using a control > > 2. Detecting whether there is an analog or digital signal on an DVI-I > input: > a) add new status field value for v4l2_input ? > #define V4L2_IN_ST_DVI_ANALOG_DETECTED 0x10000000 > #define V4L2_IN_ST_DVI_DIGITAL_DETECTED 0x20000000 > > 3. Detecting an EDID. > a) adding a status field in v4l2_output and two new ioctls that can set > the EDID for an input or retrieve it for an output. It should also be > added as an input/output capability. > > 4. ATSC bits in v4l2_std_id: how are they used? Are they used at all for > that matter? > 5. There are a lot of status bits defined in v4l2_input, but only a few > are actually used. What are we going to do with that? > > 6. HDMI requires additional investigation. HDMI defines a whole bunch of > infoframe fields. Most of these can probably be exported as controls?? Is > HDMI audio handled by alsa? > > 7. Can sensor driver use the same API for setting frame size and frame > rate? It was discussed and the general agreement was to use this API only > for video timing. > > Murali Karicheri > Software Design Engineer > Texas Instruments Inc. > Germantown, MD 20874 > email: m-karicheri2@xxxxxx > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html