[U-Boot] [PATCH v3 0/8] sf: improve support of (Q)SPI flash memories
jagannadh.teki at gmail.com
Sat Aug 26 06:14:03 UTC 2017
On Sat, Aug 26, 2017 at 4:35 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
> On Sat, Aug 26, 2017 at 12:45 AM, Marek Vasut <marex at denx.de> wrote:
>> On 08/25/2017 06:28 PM, Jagan Teki wrote:
>>> On Fri, Aug 25, 2017 at 9:43 PM, Marek Vasut <marex at denx.de> wrote:
>>>> On 08/25/2017 06:07 PM, Jagan Teki wrote:
>>>>> On Fri, Aug 25, 2017 at 6:47 AM, Yang, Wenyou <Wenyou.Yang at microchip.com> wrote:
>>>>>> This patch set has been here for a long time, could you have a look and take
>>>>> Yeah, I'm holding this because of my current spi-nor work. But anyway
>>>>> I will try to merge on coming MW If all OK.
>>>> Is your work posted somewhere or available in some git repository ?
>> You did not answer this question.
>>>> I don't see any reason why you should not perform your maintainer duties
>>>> by reviewing/replying to an incoming patch, no matter what work you do
>>>> to the subsystem though ...
>>> I didn't write "this series holding spi-nor work" since it has some
>>> new features I'm taking time to review.
>> I never implied this. Rather the opposite, you claim you do some work on
>> the SPI NOR core, yet you let this patchset rot in the list for over a
>> month now and gave the author zero feedback.
>> Notifying the author about the core changes early could've prevented a
>> lot of wasted effort on his side. Reviewing early could've prevented a
>> lot of frustration from patches being ignored.
>>> There is nothing wrong with
>>> maintainer duties here, you must need to understand.
>> I disagree.
> I agree with Marek here. I found it's really hard to get feedback for
> SF patches in time and what I have to do is to ping again and again.
I agree that the SF side patches have got some delay recently, but ie
something not intentional. SF stack is not stable as of now, so I'm
working on SPI-NOR (v10) the new patches for SF need to wait -
please be patient.
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
More information about the U-Boot