Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751429AbdFZGDY (ORCPT ); Mon, 26 Jun 2017 02:03:24 -0400 Received: from mo4-p00-ob.smtp.rzone.de ([81.169.146.219]:32006 "EHLO mo4-p00-ob.smtp.rzone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751190AbdFZGDS (ORCPT ); Mon, 26 Jun 2017 02:03:18 -0400 X-RZG-AUTH: :JGIXVUS7cutRB/49FwqZ7WcecEarQROEYabkiUo6mSAGQ+qKID8yPJWK6w== X-RZG-CLASS-ID: mo00 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 9.3 \(3124\)) Subject: Re: [PATCH v1 0/6] Add support of OV9655 camera From: "H. Nikolaus Schaller" In-Reply-To: <1E453955-0C1A-414B-BBB2-C64B6D0EF378@goldelico.com> Date: Mon, 26 Jun 2017 08:02:31 +0200 Cc: Mark Rutland , devicetree , Benjamin Gaignard , Alexandre Torgue , LKML , Hans Verkuil , Rob Herring , Maxime Coquelin , Yannick Fertre , Sylwester Nawrocki , Mauro Carvalho Chehab , Guennadi Liakhovetski , linux-arm-kernel , linux-media@vger.kernel.org, Discussions about the Letux Kernel Message-Id: References: <1498143942-12682-1-git-send-email-hugues.fruchet@st.com> <385A82AC-CC23-41BD-9F57-0232F713FED9@goldelico.com> <1E453955-0C1A-414B-BBB2-C64B6D0EF378@goldelico.com> To: Hugues Fruchet X-Mailer: Apple Mail (2.3124) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v5Q63T2p015248 Content-Length: 2509 Lines: 58 Hi Hugues, > Am 23.06.2017 um 12:25 schrieb H. Nikolaus Schaller : > > Hi Hugues, > >> Am 22.06.2017 um 17:41 schrieb H. Nikolaus Schaller : >> >> >>> Am 22.06.2017 um 17:05 schrieb Hugues Fruchet : >>> >>> This patchset enables OV9655 camera support. >>> >>> OV9655 support has been tested using STM32F4DIS-CAM extension board >>> plugged on connector P1 of STM32F746G-DISCO board. >>> Due to lack of OV9650/52 hardware support, the modified related code >>> could not have been checked for non-regression. >>> >>> First patches upgrade current support of OV9650/52 to prepare then >>> introduction of OV9655 variant patch. >>> Because of OV9655 register set slightly different from OV9650/9652, >>> not all of the driver features are supported (controls). Supported >>> resolutions are limited to VGA, QVGA, QQVGA. >>> Supported format is limited to RGB565. >>> Controls are limited to color bar test pattern for test purpose. >>> >>> OV9655 initial support is based on a driver written by H. Nikolaus Schaller [1]. >> >> Great! > > Thanks again for picking up or work and trying to get it upstream. > >> >> I will test as soon as possible. Here are some more test results and fixes: > > I have tried and had to fix some issues first: > * gpio properties have a different name than in our approach (but that is something maintainers have to decide and is easy to follow this or that way) > * there is no clock-frequency property which makes the driver request a clock frequency (something our camera interface expects this way) This can indeed be replaced by assigned-clock-rates and no additional driver code. So there is no need to implement anything new here. > * there is no vana-supply regulator and we need that to power on/off the camera on demand (reset and pwdn isn't enough in our hardware) this is something we still need to have added by patch > * for some unknown reason the driver does not load automatically from DT compatibility string and needs to be explicitly modprobed This turned out to be because the i2c device ids are upper case while compatible-strings are lower-case. See comment for patch 6/6. BR and looking forward to v2, Nikolaus