Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp3339466ybk; Tue, 19 May 2020 02:06:15 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzMTIBcFmiUrceLOh7g2m61ddh8d8SlIdd/75OBlTu56WVL9F1rUsBLJssSzVMoznI43wDE X-Received: by 2002:a05:6402:2058:: with SMTP id bc24mr17586420edb.134.1589879174896; Tue, 19 May 2020 02:06:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589879174; cv=none; d=google.com; s=arc-20160816; b=oJzewTt0Bb9H+Nta/9tsDIVeqGue4bl12QWR+Nw1OiespJBOPvarAAksChjmO4Lwkf wLAA6ndApZ8vmbPe0t8rPyDR+Jpx7EDqmUDhJx8VoFA60b2Cgd9AeeOqYMyM/jBFWVm6 kVuYKwkNwooOnDBm35aacn8OGckyETVdhkeyzV0Hq+/T5x0L53TsU/5aCs314l1ap7bV 26Hc/+ahlkI++aAJMNueubNVT2V2x3RgQOKGT5Gd4ZvMdAmZfh08T0wMnhUA2wg0hMQV OvqjQdEu8KwX7DU1SbGetwb1fcN6cAaiGTPpPUiHnyhOwJtJxy3GJ0gwm+Y+Ek9FRNgZ pyqw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=8CHrF8OyXkxTiEIKT5ENDGSgfKYd3E4Mhatl7vDGVMc=; b=lhJdlpHhoba/8XmakAo8vr1qM3LHNULq4KqBrnK08VG6GEdtT1Eg8oOwdiXMyfxUYj ufPGiSzCWhLyx5u9Kk9W6SR6E8PHjSmI3P79tWDVAf56BTRjzDun70IHyow075Xq7Tb5 yqZeCd6sUokU9iO5nvhSUe7jQND8iNkTFJBYPk53lgCCooqEp3L2gSResSStAKZmVIfc Q2fJTyolpMbViETMrcDGXAhxtgIHpBV8GoVLTzlFt1yHnGc6CSN0QPx8tffLnYHsySsF bbgpCNqWer3jujFyLPruATqwJ3A23ArC1MCHk4xmMsuShO4Qu+6XP57fted0uOTAVKj/ sQdw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id j93si3434590edb.327.2020.05.19.02.05.52; Tue, 19 May 2020 02:06:14 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728514AbgESJEP (ORCPT + 99 others); Tue, 19 May 2020 05:04:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34168 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726991AbgESJEO (ORCPT ); Tue, 19 May 2020 05:04:14 -0400 X-Greylist: delayed 3026 seconds by postgrey-1.37 at lindbergh.monkeyblade.net; Tue, 19 May 2020 02:04:14 PDT Received: from hillosipuli.retiisi.org.uk (hillosipuli.retiisi.org.uk [IPv6:2a01:4f9:c010:4572::81:2]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7CB65C061A0C; Tue, 19 May 2020 02:04:14 -0700 (PDT) Received: from valkosipuli.localdomain (valkosipuli.retiisi.org.uk [IPv6:2a01:4f9:c010:4572::80:2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by hillosipuli.retiisi.org.uk (Postfix) with ESMTPS id 31B02634C87; Tue, 19 May 2020 12:03:57 +0300 (EEST) Received: from sailus by valkosipuli.localdomain with local (Exim 4.92) (envelope-from ) id 1jayAH-00016V-7Z; Tue, 19 May 2020 12:03:57 +0300 Date: Tue, 19 May 2020 12:03:57 +0300 From: Sakari Ailus To: Kieran Bingham Cc: Niklas =?iso-8859-1?Q?S=F6derlund?= , linux-renesas-soc@vger.kernel.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Mauro Carvalho Chehab , Kieran Bingham , Laurent Pinchart , Jacopo Mondi , Niklas =?iso-8859-1?Q?S=F6derlund?= , Hans Verkuil , Hyun Kwon , Manivannan Sadhasivam , Rob Herring , Jacopo Mondi , Laurent Pinchart Subject: Re: [PATCH v9 2/4] media: i2c: Add MAX9286 driver Message-ID: <20200519090357.GD3877@valkosipuli.retiisi.org.uk> References: <20200512155105.1068064-1-kieran.bingham+renesas@ideasonboard.com> <20200512155105.1068064-3-kieran.bingham+renesas@ideasonboard.com> <20200516215103.GA857@valkosipuli.retiisi.org.uk> <930009cd-d887-752a-4f1f-567c795101ee@ideasonboard.com> <20200519081019.GB3877@valkosipuli.retiisi.org.uk> <8932699a-b321-2308-8903-31268af774cb@ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8932699a-b321-2308-8903-31268af774cb@ideasonboard.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Kieran, On Tue, May 19, 2020 at 09:55:24AM +0100, Kieran Bingham wrote: > Hi Sakari, > > On 19/05/2020 09:10, Sakari Ailus wrote: > > Hi Kieran, > > > > On Mon, May 18, 2020 at 12:45:18PM +0100, Kieran Bingham wrote: > >> Hi Sakari, > >> > >> There are only fairly minor comments here, fix ups will be included in a > >> v10. > >> > >> Is there anything major blocking integration? > > > > Not that I can see. But please see my comments below. > > Thanks, > > We might have some more work tidying up the DT validation anyway which > has come too late, and perhaps is going to bump this to v5.9 now anyway. > > I can still try but ... ;-S > > At least hopefully now we /can/ see a path to integration though. > > I probably don't care if it's 5.8 or 5.9 as long as it's not 8.5 ;-) > > >> > >> Regards > >> > >> Kieran > >> > >> > >> > >> On 16/05/2020 22:51, Sakari Ailus wrote: > >>> Hi Kieran, > >>> > >>> Thanks for the update. > >>> > >>> On Tue, May 12, 2020 at 04:51:03PM +0100, Kieran Bingham wrote: > >>> > >>> ... > >>> > >>>> +static int max9286_enum_mbus_code(struct v4l2_subdev *sd, > >>>> + struct v4l2_subdev_pad_config *cfg, > >>>> + struct v4l2_subdev_mbus_code_enum *code) > >>>> +{ > >>>> + if (code->pad || code->index > 0) > >>>> + return -EINVAL; > >>>> + > >>>> + code->code = MEDIA_BUS_FMT_UYVY8_2X8; > >>> > >>> Why UYVY8_2X8 and not UYVY8_1X16? In general, the single sample / pixel > >>> variant of the format is generally used on the serial busses. This choice > >>> was made when serial busses were introduced. > >> > >> Ok - I presume this doesn't really have much effect anyway, they just > >> have to match for the transmitter/receiver? > > > > In this case, yes. But it's harder to change later, so let's indeed do that > > now. > > Yes indeed, I have to change my test scripts for the new configuration > (or we should update the scripts to get the configuration from the > device ;D) > > > >> But it makes sense to me, so I'll update to the 1x16 variant. > > > > ... > > done anyway ;-) > > I see the ADV748x is using the 2x8 variants though ... (all the more > reason for our scripts to /get/ the correct version when propagating > formats). > > Perhaps I should/could add the 1x16 formats to the ADV748x too. (later) It's a driver bug, yes. > > > >>> And as you don't, you also won't know which frequencies are known to be > >>> safe to use. That said, perhaps where this device is used having a random > >>> frequency on that bus could not be an issue. Perhaps. > >> > >> Does this generate a range? or a list of static supported frequencies? > >> > >> We configure the pixel clock based upon the number of cameras connected, > >> and their pixel rates etc ... > >> > >> Are you saying that the frequency of this clock should be validated to > >> be a specific range? or are you talking about a different frequency? > > > > It depends on the system. In general, only frequencies known to be safe > > should be used. If this one has enough shielding to guarantee there won't > > be problems in using a random frequency in the entire range, is there a > > guarantee that will be the case for all systems with this chip? > > I have no idea here... Maybe Niklas knows more having dealt more with > the RCar-VIN/CSI parts. > > It seems like this is something we can add later if necessary, by > extending the descriptions in the DT? Works for me. Niklas, any idea? -- Regards, Sakari Ailus