From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from bhuna.collabora.co.uk ([93.93.135.160]:55416 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752057AbbAOQvZ (ORCPT ); Thu, 15 Jan 2015 11:51:25 -0500 Message-ID: <54B7F008.9060100@collabora.com> Date: Thu, 15 Jan 2015 11:51:20 -0500 From: Nicolas Dufresne MIME-Version: 1.0 To: =?windows-1252?Q?Fr=E9d=E9ric_Sureau?= , Philipp Zabel , Kamil Debski CC: Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Jean-Michel Hautbois Subject: Re: [RFC PATCH] [media] coda: Use S_PARM to set nominal framerate for h.264 encoder References: <1419264000-11761-1-git-send-email-p.zabel@pengutronix.de> <54B7E978.5050505@vodalys.com> In-Reply-To: <54B7E978.5050505@vodalys.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-media-owner@vger.kernel.org List-ID: Le 2015-01-15 11:23, Frédéric Sureau a écrit : > Maybe a->parm.output.capability should be set to > |V4L2_CAP_TIMEPERFRAME| here. > I think it is required by GStreamer V4L2 plugin. Looking at this, I think output device is indeed the right place to set this, and the capability should indeed be updated. Now for the GStreamer side, it will need patching. At the moment, this cap is only checked for capture device. It's not a major change to enabled that for output device with that capability. cheers, Nicolas