Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <925bf170-bb54-b427-976a-87e0dca230da@linaro.org>
Date:   Fri, 5 May 2023 08:40:05 +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 <jack.zhu@starfivetech.com>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Maxime Ripard <mripard@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Robert Foss <rfoss@kernel.org>,
        Todor Tomov <todor.too@gmail.com>,
        Philipp Zabel <p.zabel@pengutronix.de>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>,
        Hans Verkuil <hverkuil-cisco@xs4all.nl>,
        Eugen Hristev <eugen.hristev@collabora.com>
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>
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <7bd29805-11e7-68ee-aa47-68bae2a2fb38@starfivetech.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

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.

Best regards,
Krzysztof