From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.9 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3D399C2D0A7 for ; Tue, 8 Sep 2020 20:05:12 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0C17621919 for ; Tue, 8 Sep 2020 20:05:12 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="sBPNOifh" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731434AbgIHUFG (ORCPT ); Tue, 8 Sep 2020 16:05:06 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:51488 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730161AbgIHPWF (ORCPT ); Tue, 8 Sep 2020 11:22:05 -0400 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 838AF3B; Tue, 8 Sep 2020 17:21:17 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1599578477; bh=LMM+o53Loonyo3K7NsY/3nOFPFD/RwsRCBZOk1MmTyI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=sBPNOifhWcZEx8UJjl51ursiwoqD6krJiG6VQGLDMjSN9gUwmMK5iTFoXJKOpvf2t pAMZ2zgEnzAtfIJgPttDVvxELcDi213fX9QOd/T6JTaSXir50iuRgmMvWxZ0YGTcL7 Dno37GnwVU0Ro8tDpMtcB0Nia5jn8trdB5GvWR/k= Date: Tue, 8 Sep 2020 18:20:52 +0300 From: Laurent Pinchart To: Jacopo Mondi Cc: Jacopo Mondi , robh+dt@kernel.org, devicetree@vger.kernel.org, linux-media@vger.kernel.org, mchehab@kernel.org, sakari.ailus@linux.intel.com, hverkuil-cisco@xs4all.nl, linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH v4 0/3] dt-bindings: media: i2c: Convert 3 sensor bindings Message-ID: <20200908152052.GC11405@pendragon.ideasonboard.com> References: <20200903115143.13717-1-jacopo+renesas@jmondi.org> <20200903211420.GF6492@pendragon.ideasonboard.com> <20200904080302.47yshw6z6n33xh7k@uno.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20200904080302.47yshw6z6n33xh7k@uno.localdomain> Sender: linux-renesas-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org Hi Jacopo, On Fri, Sep 04, 2020 at 10:03:02AM +0200, Jacopo Mondi wrote: > On Fri, Sep 04, 2020 at 12:14:20AM +0300, Laurent Pinchart wrote: > > On Thu, Sep 03, 2020 at 01:51:40PM +0200, Jacopo Mondi wrote: > > > I know I've sent the previous version first as part of a longer list of patches, > > > later individually, and now grouped together again. Sorry for the fuss, hope > > > it's not too confusing. > > > > > > These three sensor bindings conversions go together as they all happen in a > > > single patch, so I deemed it was easier to just group them. > > > > > > In v4 after finalizing the discussion with Laurent and Rob on how to handle > > > of-graph endpoint, I have dropped them from mt9v111 and imx274. For imx214 as I > > > have endpoint properties to document I have also documented 'endpoint' (not > > > mandatory) and 'remote-endpoint' (mandatory). Hope I got the outcome of the > > > discussion right. For imx214 I also took in Laurent's suggestion on how to > > > document the 'data-lanes' supported values. > > > > I think you can drop remote-endpoint, it will be defined in > > of-graph.yaml. Apart from that, it sounds good to me. > > Will I ever get this right ? Should we record the policy to document > endpoints until we don't have an of-graph.yaml to avoid other to have > go through 5 iterations to do it right ? I'll give graph.yaml a go :-) > On this patch: > I owe an updated to the imx274 patch but I'm waiting for this > discussion to get somewhere (you're welcome to step-in :) > https://patchwork.linuxtv.org/project/linux-media/patch/1599012278-10203-3-git-send-email-skomatineni@nvidia.com/ > > Then I'll re-send a v5 > > > > Individual maintainers cc-ed per-patch. > > > > > > Jacopo Mondi (3): > > > dt-bindings: media: mt9v111: Convert to json-schema > > > dt-bindings: media: imx274: Convert to json-schema > > > dt-bindings: media: imx214: Convert to json-schema > > > > > > .../bindings/media/i2c/aptina,mt9v111.txt | 46 ------ > > > .../bindings/media/i2c/aptina,mt9v111.yaml | 75 ++++++++++ > > > .../devicetree/bindings/media/i2c/imx274.txt | 38 ----- > > > .../bindings/media/i2c/sony,imx214.txt | 53 ------- > > > .../bindings/media/i2c/sony,imx214.yaml | 137 ++++++++++++++++++ > > > .../bindings/media/i2c/sony,imx274.yaml | 59 ++++++++ > > > MAINTAINERS | 6 +- > > > 7 files changed, 274 insertions(+), 140 deletions(-) > > > delete mode 100644 Documentation/devicetree/bindings/media/i2c/aptina,mt9v111.txt > > > create mode 100644 Documentation/devicetree/bindings/media/i2c/aptina,mt9v111.yaml > > > delete mode 100644 Documentation/devicetree/bindings/media/i2c/imx274.txt > > > delete mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx214.txt > > > create mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml > > > create mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml -- Regards, Laurent Pinchart