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=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS autolearn=ham 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 5BD57C43382 for ; Fri, 28 Sep 2018 11:11:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 0F78D215F0 for ; Fri, 28 Sep 2018 11:11:16 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0F78D215F0 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=csie.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729371AbeI1Re3 (ORCPT ); Fri, 28 Sep 2018 13:34:29 -0400 Received: from mail-ed1-f66.google.com ([209.85.208.66]:35649 "EHLO mail-ed1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729089AbeI1Re3 (ORCPT ); Fri, 28 Sep 2018 13:34:29 -0400 Received: by mail-ed1-f66.google.com with SMTP id y21-v6so7838364edr.2; Fri, 28 Sep 2018 04:11:11 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=lNLw2EDB/0ZGOEwR/b6SNIhp1EW1z+JSRiWPuEI8Vwg=; b=YXu/LhfIbLAnTPuFl8rOaPg1RsPXKSKbabT4/xNanxEsB2aS2WkToXkBx7LCPYbXFc 4ZHjgV+YDB2LgqIWtIfzfu4xjl6fFF7JjHXqNxmf19DWoVBPXqa+NVI/XimYbwFHjn2a L352RIXGfB3I7BTYXzLaSrrgwNiRDPovysl4+RQ4F4K6mJPr12W1M7Fyo+4mbkZAjN3+ qM7Da+N6Dr4NUGkkuKMfExi+E4F7kk6wz1eyD0tjzdGNKch3yOxBCQLLMEYoZpne+Ai1 QLCs0ofmaZvF+DoEmS4+DUX6XgGbb67RaU+PDohiSFvWdcQ7a05dAtTHaY9v2CaPVmg/ ELUg== X-Gm-Message-State: ABuFfohozVjW8tRU8v7jXZK9QmZIuee7AVx/HJD0nBPHcsN7pT28TALJ 020owzD4iUIV8SjYIdyd5Z+pUMCosYM= X-Google-Smtp-Source: ACcGV630H5jGkquqlkpY9sptCR/fFHxi9tFYHwwHhghFcMdzMHZjniRidfSFVOU/TURTu60r5En5+w== X-Received: by 2002:a50:9306:: with SMTP id m6-v6mr2251643eda.206.1538133070905; Fri, 28 Sep 2018 04:11:10 -0700 (PDT) Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com. [209.85.128.42]) by smtp.gmail.com with ESMTPSA id 30-v6sm2101158edt.24.2018.09.28.04.11.10 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 28 Sep 2018 04:11:10 -0700 (PDT) Received: by mail-wm1-f42.google.com with SMTP id 189-v6so1492251wmw.2; Fri, 28 Sep 2018 04:11:10 -0700 (PDT) X-Received: by 2002:a1c:1d6:: with SMTP id 205-v6mr1393154wmb.151.1538133069789; Fri, 28 Sep 2018 04:11:09 -0700 (PDT) MIME-Version: 1.0 References: <1537951204-24672-1-git-send-email-yong.deng@magewell.com> <20180928093833.gwmskm2jvby6x4s6@paasikivi.fi.intel.com> <14114604.4rraf0qJLU@avalon> <20180928102345.r2g342tg5mgcwfw6@paasikivi.fi.intel.com> In-Reply-To: <20180928102345.r2g342tg5mgcwfw6@paasikivi.fi.intel.com> From: Chen-Yu Tsai Date: Fri, 28 Sep 2018 19:10:58 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v11 1/2] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) To: Sakari Ailus Cc: Laurent Pinchart , Yong Deng , Maxime Ripard , Mauro Carvalho Chehab , Rob Herring , Mark Rutland , David Miller , Greg Kroah-Hartman , Andrew Morton , Arnd Bergmann , Hans Verkuil , Geert Uytterhoeven , Jacob Chen , Neil Armstrong , Thierry Reding , Philipp Zabel , Todor Tomov , Linux Media Mailing List , devicetree , linux-arm-kernel , linux-kernel , linux-sunxi Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 28, 2018 at 6:23 PM Sakari Ailus wrote: > > Hi Laurent, > > On Fri, Sep 28, 2018 at 12:45:12PM +0300, Laurent Pinchart wrote: > > Hi Sakari, > > > > On Friday, 28 September 2018 12:38:33 EEST Sakari Ailus wrote: > > > On Wed, Sep 26, 2018 at 04:40:04PM +0800, Yong Deng wrote: > > > > Add binding documentation for Allwinner V3s CSI. > > > > > > > > Acked-by: Maxime Ripard > > > > Acked-by: Sakari Ailus > > > > > > I know... but I have a few more comments. > > > > > > > Reviewed-by: Rob Herring > > > > Signed-off-by: Yong Deng > > > > --- > > > > > > > > .../devicetree/bindings/media/sun6i-csi.txt | 59 +++++++++++++++++ > > > > 1 file changed, 59 insertions(+) > > > > create mode 100644 Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > > > > > diff --git a/Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > b/Documentation/devicetree/bindings/media/sun6i-csi.txt new file mode > > > > 100644 > > > > index 000000000000..2ff47a9507a6 > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > @@ -0,0 +1,59 @@ > > > > +Allwinner V3s Camera Sensor Interface > > > > +------------------------------------- > > > > + > > > > +Allwinner V3s SoC features two CSI module. CSI0 is used for MIPI CSI-2 > > > > +interface and CSI1 is used for parallel interface. > > > > + > > > > +Required properties: > > > > + - compatible: value must be "allwinner,sun8i-v3s-csi" > > > > + - reg: base address and size of the memory-mapped region. > > > > + - interrupts: interrupt associated to this IP > > > > + - clocks: phandles to the clocks feeding the CSI > > > > + * bus: the CSI interface clock > > > > + * mod: the CSI module clock > > > > + * ram: the CSI DRAM clock > > > > + - clock-names: the clock names mentioned above > > > > + - resets: phandles to the reset line driving the CSI > > > > + > > > > +Each CSI node should contain one 'port' child node with one child > > > > 'endpoint' +node, according to the bindings defined in > > > > +Documentation/devicetree/bindings/media/video-interfaces.txt. As > > > > mentioned > > > > +above, the endpoint's bus type should be MIPI CSI-2 for CSI0 and parallel > > > > or +Bt656 for CSI1. > > > > > > Which port represents CSI0 and which one is CSI1? That needs to be > > > documented. > > > > There are two CSI devices, named CSI0 and CSI1, with one port each. The CSI0 > > device supports CSI-2 only, and the CSI1 device parallel (BT.601 or BT.656) > > only. > > > > > > + > > > > +Endpoint node properties for CSI1 > > > > > > How about CSI0? I'd expect at least data-lanes, and clock-lanes as well if > > > the hardware supports lane mapping. > > > > I enquired about that too. As far as I understand, CSI0 isn't supported yet in > > the driver due to lack of documentation and lack of open-source vendor- > > provided source code. While DT bindings are not tied to driver > > implementations, it's not the best idea to design DT bindings without at least > > one working implementation to test them. I thus proposed just listing CSI0 as > > being unsupported for now. > > Ack. > > We should still define which receiver corresponds to a given port. Probably > 1 for CSI1 would make sense, in order to avoid changing the order the > hardware already uses. 0 doesn't need to be documented no IMO. > > What do you think? AFAICT it would be a completely seperate node, since they have different address spaces, clocks and reset controls. So there's no possibility of confusion. According to Yong, CSI0 is tied internally to some unknown MIPI CSI2-receiver, which is the undocumented part. CSI1 has its parallel data pins exposed to the outside. ChenYu > > > > > > +--------------------------------- > > > > + > > > > +- remote-endpoint : (required) a phandle to the bus receiver's endpoint > > > > + node > > > > +- bus-width: : (required) must be 8, 10, 12 or 16 > > > > +- pclk-sample : (optional) (default: sample on falling edge) > > > > > > Could you add that video-interfaces.txt contains documentation of these > > > properties as well? There's a reference above but only discusses port and > > > endpoint nodes. > > > > > > > +- hsync-active : (only required for parallel) > > > > +- vsync-active : (only required for parallel) > > > > > > As you support both Bt656 and parallel (with sync signals), you can detect > > > the interface type from the presence of these properties. I think you > > > should also say that these properties are not allowed on Bt656. So I'd > > > change this to e.g. > > > > > > (required; parallel-only) > > > > > > > + > > > > +Example: > > > > + > > > > +csi1: csi@1cb4000 { > > > > + compatible = "allwinner,sun8i-v3s-csi"; > > > > + reg = <0x01cb4000 0x1000>; > > > > + interrupts = ; > > > > + clocks = <&ccu CLK_BUS_CSI>, > > > > + <&ccu CLK_CSI1_SCLK>, > > > > + <&ccu CLK_DRAM_CSI>; > > > > + clock-names = "bus", "mod", "ram"; > > > > + resets = <&ccu RST_BUS_CSI>; > > > > + > > > > + port { > > > > + /* Parallel bus endpoint */ > > > > + csi1_ep: endpoint { > > > > + remote-endpoint = <&adv7611_ep>; > > > > + bus-width = <16>; > > > > + > > > > + /* If hsync-active/vsync-active are missing, > > > > + embedded BT.656 sync is used */ > > > > + hsync-active = <0>; /* Active low */ > > > > + vsync-active = <0>; /* Active low */ > > > > + pclk-sample = <1>; /* Rising */ > > > > + }; > > > > + }; > > > > +}; > > -- > Terveisin, > > Sakari Ailus > sakari.ailus@linux.intel.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen-Yu Tsai Subject: Re: [PATCH v11 1/2] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) Date: Fri, 28 Sep 2018 19:10:58 +0800 Message-ID: References: <1537951204-24672-1-git-send-email-yong.deng@magewell.com> <20180928093833.gwmskm2jvby6x4s6@paasikivi.fi.intel.com> <14114604.4rraf0qJLU@avalon> <20180928102345.r2g342tg5mgcwfw6@paasikivi.fi.intel.com> Reply-To: wens-jdAy2FN1RRM@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org In-Reply-To: <20180928102345.r2g342tg5mgcwfw6-z7MJbOB4PBP+e+fPlCVrcFDQ4js95KgL@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Sakari Ailus Cc: Laurent Pinchart , Yong Deng , Maxime Ripard , Mauro Carvalho Chehab , Rob Herring , Mark Rutland , David Miller , Greg Kroah-Hartman , Andrew Morton , Arnd Bergmann , Hans Verkuil , Geert Uytterhoeven , Jacob Chen , Neil Armstrong , Thierry Reding , Philipp Zabel , Todor Tomov , Linux Media Mailing List , devicetree , linux-arm-kernel , linux-kernel List-Id: devicetree@vger.kernel.org On Fri, Sep 28, 2018 at 6:23 PM Sakari Ailus wrote: > > Hi Laurent, > > On Fri, Sep 28, 2018 at 12:45:12PM +0300, Laurent Pinchart wrote: > > Hi Sakari, > > > > On Friday, 28 September 2018 12:38:33 EEST Sakari Ailus wrote: > > > On Wed, Sep 26, 2018 at 04:40:04PM +0800, Yong Deng wrote: > > > > Add binding documentation for Allwinner V3s CSI. > > > > > > > > Acked-by: Maxime Ripard > > > > Acked-by: Sakari Ailus > > > > > > I know... but I have a few more comments. > > > > > > > Reviewed-by: Rob Herring > > > > Signed-off-by: Yong Deng > > > > --- > > > > > > > > .../devicetree/bindings/media/sun6i-csi.txt | 59 +++++++++++++++++ > > > > 1 file changed, 59 insertions(+) > > > > create mode 100644 Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > > > > > diff --git a/Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > b/Documentation/devicetree/bindings/media/sun6i-csi.txt new file mode > > > > 100644 > > > > index 000000000000..2ff47a9507a6 > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > @@ -0,0 +1,59 @@ > > > > +Allwinner V3s Camera Sensor Interface > > > > +------------------------------------- > > > > + > > > > +Allwinner V3s SoC features two CSI module. CSI0 is used for MIPI CSI-2 > > > > +interface and CSI1 is used for parallel interface. > > > > + > > > > +Required properties: > > > > + - compatible: value must be "allwinner,sun8i-v3s-csi" > > > > + - reg: base address and size of the memory-mapped region. > > > > + - interrupts: interrupt associated to this IP > > > > + - clocks: phandles to the clocks feeding the CSI > > > > + * bus: the CSI interface clock > > > > + * mod: the CSI module clock > > > > + * ram: the CSI DRAM clock > > > > + - clock-names: the clock names mentioned above > > > > + - resets: phandles to the reset line driving the CSI > > > > + > > > > +Each CSI node should contain one 'port' child node with one child > > > > 'endpoint' +node, according to the bindings defined in > > > > +Documentation/devicetree/bindings/media/video-interfaces.txt. As > > > > mentioned > > > > +above, the endpoint's bus type should be MIPI CSI-2 for CSI0 and parallel > > > > or +Bt656 for CSI1. > > > > > > Which port represents CSI0 and which one is CSI1? That needs to be > > > documented. > > > > There are two CSI devices, named CSI0 and CSI1, with one port each. The CSI0 > > device supports CSI-2 only, and the CSI1 device parallel (BT.601 or BT.656) > > only. > > > > > > + > > > > +Endpoint node properties for CSI1 > > > > > > How about CSI0? I'd expect at least data-lanes, and clock-lanes as well if > > > the hardware supports lane mapping. > > > > I enquired about that too. As far as I understand, CSI0 isn't supported yet in > > the driver due to lack of documentation and lack of open-source vendor- > > provided source code. While DT bindings are not tied to driver > > implementations, it's not the best idea to design DT bindings without at least > > one working implementation to test them. I thus proposed just listing CSI0 as > > being unsupported for now. > > Ack. > > We should still define which receiver corresponds to a given port. Probably > 1 for CSI1 would make sense, in order to avoid changing the order the > hardware already uses. 0 doesn't need to be documented no IMO. > > What do you think? AFAICT it would be a completely seperate node, since they have different address spaces, clocks and reset controls. So there's no possibility of confusion. According to Yong, CSI0 is tied internally to some unknown MIPI CSI2-receiver, which is the undocumented part. CSI1 has its parallel data pins exposed to the outside. ChenYu > > > > > > +--------------------------------- > > > > + > > > > +- remote-endpoint : (required) a phandle to the bus receiver's endpoint > > > > + node > > > > +- bus-width: : (required) must be 8, 10, 12 or 16 > > > > +- pclk-sample : (optional) (default: sample on falling edge) > > > > > > Could you add that video-interfaces.txt contains documentation of these > > > properties as well? There's a reference above but only discusses port and > > > endpoint nodes. > > > > > > > +- hsync-active : (only required for parallel) > > > > +- vsync-active : (only required for parallel) > > > > > > As you support both Bt656 and parallel (with sync signals), you can detect > > > the interface type from the presence of these properties. I think you > > > should also say that these properties are not allowed on Bt656. So I'd > > > change this to e.g. > > > > > > (required; parallel-only) > > > > > > > + > > > > +Example: > > > > + > > > > +csi1: csi@1cb4000 { > > > > + compatible = "allwinner,sun8i-v3s-csi"; > > > > + reg = <0x01cb4000 0x1000>; > > > > + interrupts = ; > > > > + clocks = <&ccu CLK_BUS_CSI>, > > > > + <&ccu CLK_CSI1_SCLK>, > > > > + <&ccu CLK_DRAM_CSI>; > > > > + clock-names = "bus", "mod", "ram"; > > > > + resets = <&ccu RST_BUS_CSI>; > > > > + > > > > + port { > > > > + /* Parallel bus endpoint */ > > > > + csi1_ep: endpoint { > > > > + remote-endpoint = <&adv7611_ep>; > > > > + bus-width = <16>; > > > > + > > > > + /* If hsync-active/vsync-active are missing, > > > > + embedded BT.656 sync is used */ > > > > + hsync-active = <0>; /* Active low */ > > > > + vsync-active = <0>; /* Active low */ > > > > + pclk-sample = <1>; /* Rising */ > > > > + }; > > > > + }; > > > > +}; > > -- > Terveisin, > > Sakari Ailus > sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org From mboxrd@z Thu Jan 1 00:00:00 1970 From: wens@csie.org (Chen-Yu Tsai) Date: Fri, 28 Sep 2018 19:10:58 +0800 Subject: [PATCH v11 1/2] dt-bindings: media: Add Allwinner V3s Camera Sensor Interface (CSI) In-Reply-To: <20180928102345.r2g342tg5mgcwfw6@paasikivi.fi.intel.com> References: <1537951204-24672-1-git-send-email-yong.deng@magewell.com> <20180928093833.gwmskm2jvby6x4s6@paasikivi.fi.intel.com> <14114604.4rraf0qJLU@avalon> <20180928102345.r2g342tg5mgcwfw6@paasikivi.fi.intel.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Sep 28, 2018 at 6:23 PM Sakari Ailus wrote: > > Hi Laurent, > > On Fri, Sep 28, 2018 at 12:45:12PM +0300, Laurent Pinchart wrote: > > Hi Sakari, > > > > On Friday, 28 September 2018 12:38:33 EEST Sakari Ailus wrote: > > > On Wed, Sep 26, 2018 at 04:40:04PM +0800, Yong Deng wrote: > > > > Add binding documentation for Allwinner V3s CSI. > > > > > > > > Acked-by: Maxime Ripard > > > > Acked-by: Sakari Ailus > > > > > > I know... but I have a few more comments. > > > > > > > Reviewed-by: Rob Herring > > > > Signed-off-by: Yong Deng > > > > --- > > > > > > > > .../devicetree/bindings/media/sun6i-csi.txt | 59 +++++++++++++++++ > > > > 1 file changed, 59 insertions(+) > > > > create mode 100644 Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > > > > > diff --git a/Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > b/Documentation/devicetree/bindings/media/sun6i-csi.txt new file mode > > > > 100644 > > > > index 000000000000..2ff47a9507a6 > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/media/sun6i-csi.txt > > > > @@ -0,0 +1,59 @@ > > > > +Allwinner V3s Camera Sensor Interface > > > > +------------------------------------- > > > > + > > > > +Allwinner V3s SoC features two CSI module. CSI0 is used for MIPI CSI-2 > > > > +interface and CSI1 is used for parallel interface. > > > > + > > > > +Required properties: > > > > + - compatible: value must be "allwinner,sun8i-v3s-csi" > > > > + - reg: base address and size of the memory-mapped region. > > > > + - interrupts: interrupt associated to this IP > > > > + - clocks: phandles to the clocks feeding the CSI > > > > + * bus: the CSI interface clock > > > > + * mod: the CSI module clock > > > > + * ram: the CSI DRAM clock > > > > + - clock-names: the clock names mentioned above > > > > + - resets: phandles to the reset line driving the CSI > > > > + > > > > +Each CSI node should contain one 'port' child node with one child > > > > 'endpoint' +node, according to the bindings defined in > > > > +Documentation/devicetree/bindings/media/video-interfaces.txt. As > > > > mentioned > > > > +above, the endpoint's bus type should be MIPI CSI-2 for CSI0 and parallel > > > > or +Bt656 for CSI1. > > > > > > Which port represents CSI0 and which one is CSI1? That needs to be > > > documented. > > > > There are two CSI devices, named CSI0 and CSI1, with one port each. The CSI0 > > device supports CSI-2 only, and the CSI1 device parallel (BT.601 or BT.656) > > only. > > > > > > + > > > > +Endpoint node properties for CSI1 > > > > > > How about CSI0? I'd expect at least data-lanes, and clock-lanes as well if > > > the hardware supports lane mapping. > > > > I enquired about that too. As far as I understand, CSI0 isn't supported yet in > > the driver due to lack of documentation and lack of open-source vendor- > > provided source code. While DT bindings are not tied to driver > > implementations, it's not the best idea to design DT bindings without at least > > one working implementation to test them. I thus proposed just listing CSI0 as > > being unsupported for now. > > Ack. > > We should still define which receiver corresponds to a given port. Probably > 1 for CSI1 would make sense, in order to avoid changing the order the > hardware already uses. 0 doesn't need to be documented no IMO. > > What do you think? AFAICT it would be a completely seperate node, since they have different address spaces, clocks and reset controls. So there's no possibility of confusion. According to Yong, CSI0 is tied internally to some unknown MIPI CSI2-receiver, which is the undocumented part. CSI1 has its parallel data pins exposed to the outside. ChenYu > > > > > > +--------------------------------- > > > > + > > > > +- remote-endpoint : (required) a phandle to the bus receiver's endpoint > > > > + node > > > > +- bus-width: : (required) must be 8, 10, 12 or 16 > > > > +- pclk-sample : (optional) (default: sample on falling edge) > > > > > > Could you add that video-interfaces.txt contains documentation of these > > > properties as well? There's a reference above but only discusses port and > > > endpoint nodes. > > > > > > > +- hsync-active : (only required for parallel) > > > > +- vsync-active : (only required for parallel) > > > > > > As you support both Bt656 and parallel (with sync signals), you can detect > > > the interface type from the presence of these properties. I think you > > > should also say that these properties are not allowed on Bt656. So I'd > > > change this to e.g. > > > > > > (required; parallel-only) > > > > > > > + > > > > +Example: > > > > + > > > > +csi1: csi at 1cb4000 { > > > > + compatible = "allwinner,sun8i-v3s-csi"; > > > > + reg = <0x01cb4000 0x1000>; > > > > + interrupts = ; > > > > + clocks = <&ccu CLK_BUS_CSI>, > > > > + <&ccu CLK_CSI1_SCLK>, > > > > + <&ccu CLK_DRAM_CSI>; > > > > + clock-names = "bus", "mod", "ram"; > > > > + resets = <&ccu RST_BUS_CSI>; > > > > + > > > > + port { > > > > + /* Parallel bus endpoint */ > > > > + csi1_ep: endpoint { > > > > + remote-endpoint = <&adv7611_ep>; > > > > + bus-width = <16>; > > > > + > > > > + /* If hsync-active/vsync-active are missing, > > > > + embedded BT.656 sync is used */ > > > > + hsync-active = <0>; /* Active low */ > > > > + vsync-active = <0>; /* Active low */ > > > > + pclk-sample = <1>; /* Rising */ > > > > + }; > > > > + }; > > > > +}; > > -- > Terveisin, > > Sakari Ailus > sakari.ailus at linux.intel.com