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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E3EB3C7EE22 for ; Fri, 5 May 2023 12:24:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231971AbjEEMYi (ORCPT ); Fri, 5 May 2023 08:24:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39940 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231393AbjEEMYi (ORCPT ); Fri, 5 May 2023 08:24:38 -0400 Received: from mail-ed1-x52e.google.com (mail-ed1-x52e.google.com [IPv6:2a00:1450:4864:20::52e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 004581C0CA for ; Fri, 5 May 2023 05:24:35 -0700 (PDT) Received: by mail-ed1-x52e.google.com with SMTP id 4fb4d7f45d1cf-50bc070c557so3250683a12.0 for ; Fri, 05 May 2023 05:24:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1683289474; x=1685881474; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=R+J3eJ9rAr0Anc7jTVTfZv8HQJ6Km2BEzdl+jDxdnjQ=; b=mpCXaWYjwVF9T60fJELXiXgOGrRUxFskLmWMmGjFsVr9pYVHGh41sHvSANfySraYPq FxjzoDK6hqOED6ZaAVZBINZ4balvN2lT5yS85N+DOg1fr1QfsLciGevJmi6gYk/oie6H LLzcBuyJ/IsYPvH72reKvHe+tR+PJyYjVL9Bcqt9IQsKeKRGCtNI8d9qudPm+zheV4Rv iosiPVoFfbjAgOewjWz8OZRSrXDPXgF7jM09jJULUtf6ZbQWrEkrot6xonH6/5Uo53kX fLXJp+2UGcaIrx3tDLvPOC3Gwj9n1hmnLqk7Jh55a+9Yps+CKv2Ab3NMKwmaVB8dT994 023g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683289474; x=1685881474; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=R+J3eJ9rAr0Anc7jTVTfZv8HQJ6Km2BEzdl+jDxdnjQ=; b=gSPaW1vj6gY/zU3HDJYTmlcCGtElncL58xEwFJxCAiReVSRzoVCrNNi1ERPmXJFWm4 o5CrzXlxu2UHIqOUcrNZgMGEIYkiuaZxQEPf4EO+zWocDf8s4J5m07SLG2yIh1nzUmZz n1cvumAH+iWxP/SnuuUHujZd09TfvFrsKYERbyTXFtwzst4zaebZRK1nD4h7QMGrBi9B BMXZg9wA38uoigxHPShmlgcMJ0nI3vfZru12XkzVYChjDD7iNo7raXNzakhZu6CNsUnt Ff3KtQKB8PSK5asUVjANf0V7snGdURSqv+EK7OSZT7htV0j+MCGy26WxI6wa0uf7vcoy 8vyg== X-Gm-Message-State: AC+VfDzmMi2/8sF/9xVwAKC7LNuBf9P1q/OnSuJKi+rIwNpMLlCF0O2r 5FtzQrzQBCwmDN2j4EAPmEmGVg== X-Google-Smtp-Source: ACHHUZ7XXQzqvBr/9IQJvWOwjqwXpa8SpSp8T8lg2RLLJpcPx3Ukf+D9VGZzyq30KdybZpFJrCr9Rw== X-Received: by 2002:a05:6402:398:b0:50b:c4b7:ee7c with SMTP id o24-20020a056402039800b0050bc4b7ee7cmr1177934edv.36.1683289474481; Fri, 05 May 2023 05:24:34 -0700 (PDT) Received: from ?IPV6:2a02:810d:15c0:828:52e:24ce:bbc1:127d? ([2a02:810d:15c0:828:52e:24ce:bbc1:127d]) by smtp.gmail.com with ESMTPSA id n24-20020a056402061800b0050bd245d39esm2909078edv.6.2023.05.05.05.24.33 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 05 May 2023 05:24:33 -0700 (PDT) Message-ID: Date: Fri, 5 May 2023 14:24:32 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.1 Subject: Re: [PATCH v4 0/8] Add StarFive Camera Subsystem driver Content-Language: en-US To: Jack Zhu , Mauro Carvalho Chehab , Maxime Ripard , Rob Herring , Krzysztof Kozlowski , Robert Foss , Todor Tomov , Philipp Zabel , Laurent Pinchart , Ezequiel Garcia , Hans Verkuil , Eugen Hristev Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, changhuang.liang@starfivetech.com References: <20230413035541.62129-1-jack.zhu@starfivetech.com> <14c06503-621f-2477-7b15-b17f1890ecfe@starfivetech.com> <7bd29805-11e7-68ee-aa47-68bae2a2fb38@starfivetech.com> <925bf170-bb54-b427-976a-87e0dca230da@linaro.org> <817b8919-e9dd-cf2a-41e0-9b50747ab4cf@starfivetech.com> From: Krzysztof Kozlowski In-Reply-To: <817b8919-e9dd-cf2a-41e0-9b50747ab4cf@starfivetech.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 05/05/2023 10:14, Jack Zhu wrote: > > > On 2023/5/5 14:40, Krzysztof Kozlowski wrote: >> On 05/05/2023 07:57, Jack Zhu wrote: >>> >>> >>> On 2023/4/24 19:19, Jack Zhu wrote: >>>> >>>> >>>> On 2023/4/13 11:55, Jack Zhu wrote: >>>>> Hi, >>>>> >>>>> This patch series adds support for the StarFive Camera Subsystem >>>>> found on StarFive JH7110 SoC. >>>>> >>>>> The driver implements V4L2, Media controller and V4L2 subdev interfaces. >>>>> Camera sensor using V4L2 subdev interface in the kernel is supported. >>>>> >>>>> The driver is tested on VisionFive V2 board with IMX219 camera sensor. >>>>> GStreamer 1.18.5 with v4l2src plugin is supported. >>>>> >>>>> Changes since v3: >>>>> Patch 1: >>>>> - Modified port@0 and port@1 properties. >>>>> - Extended the port@0 example with appropriate properties. >>>>> - Added 'port@0' for 'required' >>>>> Patch 2: >>>>> - Modified spelling errors. >>>>> Patch 3: >>>>> - Merged patch 5 into the patch with an explanation for compatible in >>>>> commit msg. >>>>> Patch 6: >>>>> - Asserted pixel_rst[i] reset in the loop after the err_disable_pixclk >>>>> label. >>>>> - Modified Code Style for getting sys_rst and p_rst. >>>>> - Renamed clk_name to name and modified the relevant code. >>>>> Patch 9: >>>>> - Added static for stfcamss_get_mem_res function. >>>>> - Added static for isp_close function. >>>>> - Fixed implicit conversion warning for stf_vin_map_isp_pad function. >>>>> - Dropped unused variables. >>>>> >>>>> v3: https://lore.kernel.org/all/20230331121826.96973-1-jack.zhu@starfivetech.com/ >>>>> >>>> >>>> Hello everyone, >>>> >>>> From the current review status, the patches related to the CSI module >>>> have 'reviewed-by' tags. I would like to know if it is okay to add >>>> patches 1-5 from this series to a PR first. >>>> >>>> Thank you! >>>> >>>> Jack >>>> >>> >>> Hello Mauro, Laurent, Maxime, Rob, Krzysztof, Robert, Todor and Philipp, >>> >>> Can you give me some suggestions and comments on the previous request >>> to commit CSI related patches first? Thank you for your time. >> >> You received very specific feedback, so know you decided to ignore it? >> >> No, implement what you were asked for. >> > > Hi Krzysztof, > > Thank you for your comments. > > I am talking about CSI-related patches 1-5, not including the patches > 6-8. The CSI module is a relatively functionally independent hardware > module. The CSI-related patches 1-5 already have 'reviewed-by' tags, > and there are no unprocessed comments left. So, made the previous > request. Please let me know if I understand something wrong. You pinged also me, so we talk about bindings. You got comments to fix, so if you are not going to fix them, the patches will not get accepted. > I don't want to ignore any comments, I will continue to modify the isp > patches 6-8 in subsequent versions according to your comments. The > ISP-related patches are being prepared. And how are these patches related to me and Rob? Best regards, Krzysztof