From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753828AbcGSNcZ (ORCPT ); Tue, 19 Jul 2016 09:32:25 -0400 Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:41192 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753735AbcGSNcV (ORCPT ); Tue, 19 Jul 2016 09:32:21 -0400 Message-ID: <1468935136.13622.68.camel@pengutronix.de> Subject: Re: [PATCH 12/16] gpu: ipu-v3: Fix CSI0 blur in NTSC format From: Philipp Zabel To: Steve Longerbeam Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Hans Verkuil Date: Tue, 19 Jul 2016 15:32:16 +0200 In-Reply-To: <578A97E2.1090308@mentor.com> References: <1467932621-358-1-git-send-email-steve_longerbeam@mentor.com> <1467932621-358-13-git-send-email-steve_longerbeam@mentor.com> <1467999258.2365.71.camel@pengutronix.de> <578278C7.8020503@mentor.com> <5786C876.50300@mentor.com> <1468587534.7213.41.camel@pengutronix.de> <57896D1F.3000002@mentor.com> <578A97E2.1090308@mentor.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.9-1+b1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 2001:67c:670:100:96de:80ff:fec2:9969 X-SA-Exim-Mail-From: p.zabel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Samstag, den 16.07.2016, 13:24 -0700 schrieb Steve Longerbeam: [...] > > Hmm, do you mean define something like a V4L2_MBUS_BT656_NEWAVMODE, > > and then add a new "newavmode" boolean DT binding parsed by > > v4l2_of_parse_endpoint()? > > > > I don't know if that would make sense given that this NEWAVMODE is a kind > > of hack of the BT.656 standard, only used by Analog Devices > > encoders/decoders. > > > > Although there a _lot_ of AD encoder/decoder chips (and subdev drivers > > written for > > them), so maybe it would make sense to do this. > > I don't think a "newavmode" boolean property would necessitate a whole new > mbus type, but perhaps just a new parallel bus flag. I will propose a > patch at linux-media that adds this. Sounds good to me, the V4L2_MBUS_BT656 documentation comment in include/media/v4l2-mediabus.h should be extended to include the new non-standard mode then. regards Philipp