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=-10.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,NICE_REPLY_A, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 D2C4FC433E6 for ; Mon, 31 Aug 2020 10:35:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A5C442098B for ; Mon, 31 Aug 2020 10:35:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598870146; bh=V1ZLMFYN+hd025pVnh+Yww7lnqsvYV7nIlFHgJ6wVF0=; h=Subject:To:Cc:References:From:Date:In-Reply-To:List-ID:From; b=HZLxtr/1EOZ+yNuBjbA3OkwjvMrtIwjvMyt0eJUfVNCTCB4SBvxHpkjrNH41vPvIT rR3FUFuaNRtHWzMplCLPTp+24IpTfzQhL3/mFNS/T0c/Qv5//CUUqG4Ff5IBUkQdw4 UlfezAHPLGB1A8hNMatAocGl1HHwZISCtY0irxlk= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726248AbgHaKfp (ORCPT ); Mon, 31 Aug 2020 06:35:45 -0400 Received: from mail.kernel.org ([198.145.29.99]:50762 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726446AbgHaKfg (ORCPT ); Mon, 31 Aug 2020 06:35:36 -0400 Received: from [192.168.0.50] (89-70-52-201.dynamic.chello.pl [89.70.52.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 35ED020DD4; Mon, 31 Aug 2020 10:35:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598870136; bh=V1ZLMFYN+hd025pVnh+Yww7lnqsvYV7nIlFHgJ6wVF0=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=QBKfUCQSPkEfS/h6P3Sqp+d62D6FYKrtw5pPhnOmSzIKEUfeuFhiIDB7AW4vjh4cl 7OC2WKi1VGNRtIZY/MIO9uQ5UVV9+IrnFEhW1bY/gSc9BMruuCvNktAMcyExGDJP6P dQ5dT6pubfosL4soPm3ebl8MPyVf0sYhA9g9qbok= Subject: Re: [PATCH 23/33] ARM: dts: exynos: Remove empty camera pinctrl configuration in Odroid X/U3 To: Krzysztof Kozlowski , Marek Szyprowski Cc: Rob Herring , Kukjin Kim , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, "linux-samsung-soc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Denis GNUtoo Carikli , Simon Shields , Bartlomiej Zolnierkiewicz References: <20200830135200.24304-1-krzk@kernel.org> <20200830135200.24304-23-krzk@kernel.org> From: Sylwester Nawrocki Message-ID: <96651e85-c27f-3167-7b41-2ea051c25c65@kernel.org> Date: Mon, 31 Aug 2020 12:35:31 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 8/31/20 10:38, Krzysztof Kozlowski wrote: > On Mon, 31 Aug 2020 at 10:31, Marek Szyprowski wrote: >> On 30.08.2020 15:51, Krzysztof Kozlowski wrote: >>> The camera's pinctrl configuration is simply empty and not effective. >>> Remove it to fix dtbs_check warning: >>> >>> arch/arm/boot/dts/exynos4412-odroidx.dt.yaml: camera: pinctrl-0: True is not of type 'array' >>> >>> Signed-off-by: Krzysztof Kozlowski >> >> I think that this was intentional to properly enable support for >> mem-2-mem mode in Exynos4-IS (FIMC), but I'm not sure what are the >> default values if no pinctrl properties are provided. Sylwester, could >> you comment? > > Indeed it could be intentional... I see now errors: > [ 33.752203] s5p-fimc-md soc:camera: Failed to get pinctrl: -19 > > I wonder why getting an empty pinctrl is needed... maybe the driver > should accept missing pinctrl? It might have been better to have the pinctrl properties optional, as there might be boards without the image sensor attached and FIMC could still be used in memory-to-memory mode, as Marek pointed out. But it seems too late now to change that. The binding defines the pinctrl properties as required (Documentation/devicetree/bindings/media/samsung-fimc.txt) and we need to keep them in dtses. -- Regards, Sylwester 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=-10.6 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,NICE_REPLY_A,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 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 6B48CC433E7 for ; Mon, 31 Aug 2020 10:37:05 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 3C4222083E for ; Mon, 31 Aug 2020 10:37:05 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="0k6gvEnp"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="QBKfUCQS" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3C4222083E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=MSdX5aYtdwGIN+Q494RdCXe32aJBN1L5XNXYYHjr1DM=; b=0k6gvEnpm880xnnHBnDZc6m5R J7rDsk+0GqvhepHtf1K7lNDQyv/+ap2P/fLIFJqf100mMFpPUzikroDr+GMm4NwtY1wqtUnWPaV/Z 6LwJJ5hEVfn1Q5ays6BDwL6ywdcBGhti5Kfp75OeQLbRJU9LA4MWYVIw2cnrPD7JirSar4tTvD+ce X+xBjbaL+AJl/aDdV56SeLwzKP9Q/ygBotX/PSkRlIRGXGXpbrDWE5RISK6FzgU625GWR4Hk31PbE KG0f7Ks4SgMFYQzddBiwlY3OJpSkVBaHRNNQiTfXccZyglCbmWGg8cZkQJMwU4JxmQmI3HEJMaw9a KUbozrE3A==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kChA4-0007Sh-GE; Mon, 31 Aug 2020 10:35:40 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kChA1-0007SJ-Nb for linux-arm-kernel@lists.infradead.org; Mon, 31 Aug 2020 10:35:38 +0000 Received: from [192.168.0.50] (89-70-52-201.dynamic.chello.pl [89.70.52.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 35ED020DD4; Mon, 31 Aug 2020 10:35:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598870136; bh=V1ZLMFYN+hd025pVnh+Yww7lnqsvYV7nIlFHgJ6wVF0=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=QBKfUCQSPkEfS/h6P3Sqp+d62D6FYKrtw5pPhnOmSzIKEUfeuFhiIDB7AW4vjh4cl 7OC2WKi1VGNRtIZY/MIO9uQ5UVV9+IrnFEhW1bY/gSc9BMruuCvNktAMcyExGDJP6P dQ5dT6pubfosL4soPm3ebl8MPyVf0sYhA9g9qbok= Subject: Re: [PATCH 23/33] ARM: dts: exynos: Remove empty camera pinctrl configuration in Odroid X/U3 To: Krzysztof Kozlowski , Marek Szyprowski References: <20200830135200.24304-1-krzk@kernel.org> <20200830135200.24304-23-krzk@kernel.org> From: Sylwester Nawrocki Message-ID: <96651e85-c27f-3167-7b41-2ea051c25c65@kernel.org> Date: Mon, 31 Aug 2020 12:35:31 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200831_063537_914604_FCD0AFF4 X-CRM114-Status: GOOD ( 19.68 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Simon Shields , devicetree@vger.kernel.org, "linux-samsung-soc@vger.kernel.org" , Bartlomiej Zolnierkiewicz , "linux-kernel@vger.kernel.org" , Rob Herring , Kukjin Kim , linux-arm-kernel@lists.infradead.org, Denis GNUtoo Carikli Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, On 8/31/20 10:38, Krzysztof Kozlowski wrote: > On Mon, 31 Aug 2020 at 10:31, Marek Szyprowski wrote: >> On 30.08.2020 15:51, Krzysztof Kozlowski wrote: >>> The camera's pinctrl configuration is simply empty and not effective. >>> Remove it to fix dtbs_check warning: >>> >>> arch/arm/boot/dts/exynos4412-odroidx.dt.yaml: camera: pinctrl-0: True is not of type 'array' >>> >>> Signed-off-by: Krzysztof Kozlowski >> >> I think that this was intentional to properly enable support for >> mem-2-mem mode in Exynos4-IS (FIMC), but I'm not sure what are the >> default values if no pinctrl properties are provided. Sylwester, could >> you comment? > > Indeed it could be intentional... I see now errors: > [ 33.752203] s5p-fimc-md soc:camera: Failed to get pinctrl: -19 > > I wonder why getting an empty pinctrl is needed... maybe the driver > should accept missing pinctrl? It might have been better to have the pinctrl properties optional, as there might be boards without the image sensor attached and FIMC could still be used in memory-to-memory mode, as Marek pointed out. But it seems too late now to change that. The binding defines the pinctrl properties as required (Documentation/devicetree/bindings/media/samsung-fimc.txt) and we need to keep them in dtses. -- Regards, Sylwester _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel