[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAAyq3SZ91aK=r8S+u0aWNsR3g6zoVWp7Lx51B1ygCFBe-kH5Fw@mail.gmail.com>
Date: Mon, 22 Jul 2024 18:31:26 +0800
From: Cheng Ming Lin <linchengming884@...il.com>
To: Michael Walle <mwalle@...nel.org>
Cc: tudor.ambarus@...aro.org, pratyush@...nel.org, miquel.raynal@...tlin.com,
richard@....at, vigneshr@...com, linux-mtd@...ts.infradead.org,
linux-kernel@...r.kernel.org, leoyu@...c.com.tw, alvinzhou@...c.com.tw,
Cheng Ming Lin <chengminglin@...c.com.tw>
Subject: Re: [PATCH v2] mtd: spi-nor: macronix: Add support for serial NOR flash
Hi,
Michael Walle <mwalle@...nel.org> 於 2024年7月22日 週一 下午5:20寫道:
>
> On Mon Jul 22, 2024 at 10:49 AM CEST, Cheng Ming Lin wrote:
> > From: Cheng Ming Lin <chengminglin@...c.com.tw>
> >
> > MX66U1G45G and MX66L2G45G are Macronix serial NOR flash.
> >
> > These flashes have been tested on Xilinx Zynq-picozed board
> > using MXIC SPI controller.
>
> I've asked for a changelog. Now, what you've sent is a v2 (of what
> seems to be a split off of a split off). Before sending out another
> version, please carefully read:
> https://www.kernel.org/doc/html/latest/process/submitting-patches.html
>
I apologize for the misunderstanding.
The lack of a changelog is my oversight.
> To be clear, please explain why this is a split off of the larger
> series, what has changed (or if nothing has changed then state so).
> Why did you split it even more (before it were two patches).
>
The patch Jaime previously submitted included support
for octal DTR mode and added the new octal NOR flash ID
with RWW feature.
However, these two things should belong to different patch
series, so I split them apart.
The reason for splitting them again this time is because
I think that adding support for the octal NOR flash with
RWW feature should be proposed after Alvinzhou's v9
version, which supports octal DTR mode, is accepted.
So, I independently added support for serial NOR flash
without RWW feature.
> Honestly, back when Jaime submitted these patches, I already spent
> way too much time to review this stuff just to get it dropped or
> handed over to another employee who then comes back very late.
> Please align your workflow at Macronix and get an internal review
> and some training how to submit patches upstream and don't put that
> burden on maintainers.
>
I apologize again for adding burden to the maintainers.
Moving forward, we will ensure everything is internally
reviewed before submitting patches. Thank you for your
patience in addressing this matter.
> Thanks,
> -michael
Best regards,
Cheng Ming Lin
Powered by blists - more mailing lists