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: <109d69d1-bd64-9157-da6b-a099dbe700fe@starfivetech.com>
Date:   Fri, 25 Aug 2023 17:57:58 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.14.0
Subject: Re: [PATCH v8 0/8] Add StarFive Camera Subsystem driver
Content-Language: en-US
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
CC:     Mauro Carvalho Chehab <mchehab@kernel.org>,
        Robert Foss <rfoss@kernel.org>,
        Todor Tomov <todor.too@gmail.com>,
        <bryan.odonoghue@linaro.org>, Rob Herring <robh+dt@kernel.org>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Philipp Zabel <p.zabel@pengutronix.de>,
        "Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
        Hans Verkuil <hverkuil-cisco@xs4all.nl>,
        <linux-media@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <devicetree@vger.kernel.org>, <linux-staging@lists.linux.dev>,
        <changhuang.liang@starfivetech.com>
References: <20230824080109.89613-1-jack.zhu@starfivetech.com>
 <2023082436-spendable-reshuffle-878d@gregkh>
 <a6f60dfc-ffbf-da4d-a81c-06caf92955d6@starfivetech.com>
 <2023082416-flanking-requisite-a8ed@gregkh>
From:   Jack Zhu <jack.zhu@starfivetech.com>
In-Reply-To: <2023082416-flanking-requisite-a8ed@gregkh>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 2023/8/24 21:08, Greg Kroah-Hartman wrote:
> On Thu, Aug 24, 2023 at 08:23:33PM +0800, Jack Zhu wrote:
>> Hi Greg,
>> 
>> On 2023/8/24 19:37, Greg Kroah-Hartman wrote:
>> > On Thu, Aug 24, 2023 at 04:01:01PM +0800, Jack Zhu wrote:
>> >> Hi,
>> >> 
>> >> This series is the v8 series that attempts to support the Camera Subsystem
>> >> found on StarFive JH7110 SoC.
>> > 
>> > I don't see anything here about why this is in drivers/staging/media/
>> > now and not just in drivers/media/.  What is preventing this to be put
>> > into the correct place to start with?  What needs to be done to the code
>> > to get it out of drivers/staging/media/ and who is going to do that
>> > work?
>> > 
>> 
>> The series does not contain 3A interface. According to Laurent's suggestion,
>> we put the driver in the staging directory first. In the next stage, we will
>> continue to submit the 3A interface in the way of incremental development,
>> and finally expect the code to be placed in the drivers/media/.
> 
> Can you please say that in the changelog text for when you are adding
> the driver so that we know this?
> 

Yes, will do it. Thank you for your suggestion!

> thanks,
> 
> greg k-h