* [GIT PULL] mtd: Changes for 6.17
@ 2025-07-31 16:59 Miquel Raynal
2025-07-31 20:51 ` pr-tracker-bot
2025-08-01 14:47 ` Pratyush Yadav
0 siblings, 2 replies; 7+ messages in thread
From: Miquel Raynal @ 2025-07-31 16:59 UTC (permalink / raw)
To: Linus Torvalds
Cc: Richard Weinberger, Tudor Ambarus, Vignesh Raghavendra,
Frieder Schrempf, Michael Walle, Pratyush Yadav, linux-mtd,
Miquel Raynal
Hello Linus,
I was on my side on vacation the past weeks, I've tried to make this up
as fast as possible for you to merge before your trip, so here is the
MTD PR for 6.17.
Thanks,
Miquèl
The following changes since commit d0b3b7b22dfa1f4b515fd3a295b3fd958f9e81af:
Linux 6.16-rc4 (2025-06-29 13:09:04 -0700)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git tags/mtd/for-6.17
for you to fetch changes up to 9cf9db888f387844e063efc6296e9fa5c042995e:
Merge tag 'nand/for-6.17' into mtd/next (2025-07-31 18:52:16 +0200)
----------------------------------------------------------------
* MTD changes:
Apart from a binding conversion to yaml, only minor changes/small fixes
have been merged.
* Raw NAND changes:
Various controller drivers received minor fixes like DMA mapping checks,
better timing derivations or bitflip statistics.
It has also been discovered that some Hynix NAND flashes were not
supporting read-retries, which is not properly supported.
* SPI NAND changes:
In order to support high-speed modes, certain chips need extra
configuration like adding more dummy cycles. This is now possible,
especially on Winbond chips.
Aside from that, Gigadevice gets support for a new chip (GD5F1GM9).
* SPI NOR changes:
A notable changes is the fix for exiting 4-byte addressing on Infineon
SEMPER flashes. These flashes do not support the standard EX4B
opcode (E9h), and use a vendor-specific opcode (B8h) instead.
There is also a fix for unlocking flashes that are write-protected at
power-on. This was caused by using an uninitialized mtd_info in
spi_nor_try_unlock_all().
----------------------------------------------------------------
Andy Shevchenko (1):
mtd: map: Don't use "proxy" headers
Arnd Bergmann (1):
mtd: nftl: reduce stack usage in NFTL_movebuf()
Balamanikandan Gunasundar (1):
mtd: rawnand: atmel: set pmecc data setup time
Christophe JAILLET (1):
mtd: spi-nor: Constify struct spi_nor_fixups
David Regan (1):
mtd: nand: brcmnand: fix mtd corrected bits stat
Frank Li (2):
dt-bindings: mtd: convert nxp-spifi.txt to yaml format
dt-bindings: mtd: jedec,spi-nor: Add atmel,at26* compatible string
Gabor Juhos (1):
mtd: spinand: propagate spinand_wait() errors from spinand_write_page()
Hector Palacios (1):
mtd: rawnand: hynix: don't try read-retry on SLC NANDs
Ivan Stepchenko (1):
mtd: fix possible integer overflow in erase_xfer()
Michael Walle (1):
mtd: spi-nor: Fix spi_nor_try_unlock_all()
Miquel Raynal (10):
spi: spi-mem: Use picoseconds for calculating the op durations
spi: spi-mem: Take into account the actual maximum frequency
mtd: spinand: Fix macro alignment
mtd: spinand: Add a frequency field to all READ_FROM_CACHE variants
mtd: spinand: Add a ->configure_chip() hook
mtd: spinand: winbond: Enable high-speed modes on w25n0xjw
mtd: spinand: winbond: Enable high-speed modes on w35n0xjw
mtd: spinand: winbond: Add comment about the maximum frequency
Merge tag 'spi-nor/for-6.17' into mtd/next
Merge tag 'nand/for-6.17' into mtd/next
Takahiro Kuwano (1):
mtd: spi-nor: spansion: Fixup params->set_4byte_addr_mode for SEMPER
Teng Wu (1):
mtd: spinand: gigadevice: Add support for GD5F1GM9 chips
Thomas Fourier (4):
mtd: rawnand: renesas: Add missing check after DMA map
mtd: rawnand: atmel: Fix dma_mapping_error() address
mtd: rawnand: rockchip: Add missing check after DMA map
mtd: rawnand: fsmc: Add missing check after DMA map
Yuesong Li (1):
mtd: nand: brcmnand: replace manual string choices with standard helpers
.../devicetree/bindings/mtd/jedec,spi-nor.yaml | 2 +-
.../devicetree/bindings/mtd/nxp,lpc1773-spifi.yaml | 74 ++++++++++
.../devicetree/bindings/mtd/nxp-spifi.txt | 58 --------
drivers/mtd/ftl.c | 2 +-
drivers/mtd/nand/raw/atmel/nand-controller.c | 2 +-
drivers/mtd/nand/raw/atmel/pmecc.c | 6 +
drivers/mtd/nand/raw/brcmnand/brcmnand.c | 62 +++++---
drivers/mtd/nand/raw/fsmc_nand.c | 2 +
drivers/mtd/nand/raw/nand_hynix.c | 4 +-
drivers/mtd/nand/raw/renesas-nand-controller.c | 6 +
drivers/mtd/nand/raw/rockchip-nand-controller.c | 15 ++
drivers/mtd/nand/spi/alliancememory.c | 12 +-
drivers/mtd/nand/spi/ato.c | 6 +-
drivers/mtd/nand/spi/core.c | 27 +++-
drivers/mtd/nand/spi/esmt.c | 8 +-
drivers/mtd/nand/spi/foresee.c | 8 +-
drivers/mtd/nand/spi/gigadevice.c | 68 ++++++---
drivers/mtd/nand/spi/macronix.c | 8 +-
drivers/mtd/nand/spi/micron.c | 20 +--
drivers/mtd/nand/spi/paragon.c | 12 +-
drivers/mtd/nand/spi/skyhigh.c | 12 +-
drivers/mtd/nand/spi/toshiba.c | 8 +-
drivers/mtd/nand/spi/winbond.c | 163 +++++++++++++++++++--
drivers/mtd/nand/spi/xtx.c | 12 +-
drivers/mtd/nftlcore.c | 43 +++---
drivers/mtd/spi-nor/micron-st.c | 8 +-
drivers/mtd/spi-nor/spansion.c | 35 ++++-
drivers/mtd/spi-nor/swp.c | 19 +--
drivers/spi/spi-mem.c | 27 +++-
include/linux/mtd/map.h | 13 +-
include/linux/mtd/spinand.h | 70 +++++----
include/linux/spi/spi-mem.h | 2 +-
32 files changed, 556 insertions(+), 258 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mtd/nxp,lpc1773-spifi.yaml
delete mode 100644 Documentation/devicetree/bindings/mtd/nxp-spifi.txt
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [GIT PULL] mtd: Changes for 6.17
2025-07-31 16:59 [GIT PULL] mtd: Changes for 6.17 Miquel Raynal
@ 2025-07-31 20:51 ` pr-tracker-bot
2025-08-01 14:47 ` Pratyush Yadav
1 sibling, 0 replies; 7+ messages in thread
From: pr-tracker-bot @ 2025-07-31 20:51 UTC (permalink / raw)
To: Miquel Raynal
Cc: Linus Torvalds, Richard Weinberger, Tudor Ambarus,
Vignesh Raghavendra, Frieder Schrempf, Michael Walle,
Pratyush Yadav, linux-mtd, Miquel Raynal
The pull request you sent on Thu, 31 Jul 2025 18:59:44 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git tags/mtd/for-6.17
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/cbbf0a759ff96c80dfc32192a2cc427b79447f74
Thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [GIT PULL] mtd: Changes for 6.17
2025-07-31 16:59 [GIT PULL] mtd: Changes for 6.17 Miquel Raynal
2025-07-31 20:51 ` pr-tracker-bot
@ 2025-08-01 14:47 ` Pratyush Yadav
2025-08-01 15:01 ` Miquel Raynal
1 sibling, 1 reply; 7+ messages in thread
From: Pratyush Yadav @ 2025-08-01 14:47 UTC (permalink / raw)
To: Miquel Raynal
Cc: Linus Torvalds, Richard Weinberger, Tudor Ambarus,
Vignesh Raghavendra, Frieder Schrempf, Michael Walle,
Pratyush Yadav, linux-mtd
Hi Miquel,
On Thu, Jul 31 2025, Miquel Raynal wrote:
[...]
> * MTD changes:
>
> Apart from a binding conversion to yaml, only minor changes/small fixes
> have been merged.
>
> * Raw NAND changes:
>
> Various controller drivers received minor fixes like DMA mapping checks,
> better timing derivations or bitflip statistics.
> It has also been discovered that some Hynix NAND flashes were not
> supporting read-retries, which is not properly supported.
>
> * SPI NAND changes:
>
> In order to support high-speed modes, certain chips need extra
> configuration like adding more dummy cycles. This is now possible,
> especially on Winbond chips.
>
> Aside from that, Gigadevice gets support for a new chip (GD5F1GM9).
>
> * SPI NOR changes:
>
> A notable changes is the fix for exiting 4-byte addressing on Infineon
> SEMPER flashes. These flashes do not support the standard EX4B
> opcode (E9h), and use a vendor-specific opcode (B8h) instead.
>
> There is also a fix for unlocking flashes that are write-protected at
> power-on. This was caused by using an uninitialized mtd_info in
> spi_nor_try_unlock_all().
I personally think bullet points are a bit easier to read for changelogs
compared to paragraphs since there is less filler text needed. For
example, my original merge message didn't the "there is also a fix for".
It just said "Fix unlocking of...".
Anyway, if you still prefer paragraphs then do let me know and I'll use
those next time around and save you some editing.
[...]
--
Regards,
Pratyush Yadav
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [GIT PULL] mtd: Changes for 6.17
2025-08-01 14:47 ` Pratyush Yadav
@ 2025-08-01 15:01 ` Miquel Raynal
2025-08-04 14:40 ` Pratyush Yadav
0 siblings, 1 reply; 7+ messages in thread
From: Miquel Raynal @ 2025-08-01 15:01 UTC (permalink / raw)
To: Pratyush Yadav
Cc: Linus Torvalds, Richard Weinberger, Tudor Ambarus,
Vignesh Raghavendra, Frieder Schrempf, Michael Walle, linux-mtd
Hello Pratyush,
>> * SPI NOR changes:
>>
>> A notable changes is the fix for exiting 4-byte addressing on Infineon
>> SEMPER flashes. These flashes do not support the standard EX4B
>> opcode (E9h), and use a vendor-specific opcode (B8h) instead.
>>
>> There is also a fix for unlocking flashes that are write-protected at
>> power-on. This was caused by using an uninitialized mtd_info in
>> spi_nor_try_unlock_all().
>
> I personally think bullet points are a bit easier to read for changelogs
> compared to paragraphs since there is less filler text needed. For
> example, my original merge message didn't the "there is also a fix for".
> It just said "Fix unlocking of...".
>
> Anyway, if you still prefer paragraphs then do let me know and I'll use
> those next time around and save you some editing.
Some time ago Linus asked me to change the content of the PR description
(I was mostly listing the patches) and pointed to examples which were
mostly plain English, not bullet points. So I changed both the content
(hinting what are the major changes and roughly describing the small
ones) and the form (no bullet points). I've kept splitting the
subsystems with '*' though, as the changes in each subsystem are
mostly unrelated.
I'm fine either ways, but I assumed until now that the preferred form
was plain text descriptions.
Cheers,
Miquèl
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [GIT PULL] mtd: Changes for 6.17
2025-08-01 15:01 ` Miquel Raynal
@ 2025-08-04 14:40 ` Pratyush Yadav
2025-08-04 15:53 ` Linus Torvalds
0 siblings, 1 reply; 7+ messages in thread
From: Pratyush Yadav @ 2025-08-04 14:40 UTC (permalink / raw)
To: Miquel Raynal
Cc: Pratyush Yadav, Linus Torvalds, Richard Weinberger, Tudor Ambarus,
Vignesh Raghavendra, Frieder Schrempf, Michael Walle, linux-mtd
On Fri, Aug 01 2025, Miquel Raynal wrote:
> Hello Pratyush,
>
>>> * SPI NOR changes:
>>>
>>> A notable changes is the fix for exiting 4-byte addressing on Infineon
>>> SEMPER flashes. These flashes do not support the standard EX4B
>>> opcode (E9h), and use a vendor-specific opcode (B8h) instead.
>>>
>>> There is also a fix for unlocking flashes that are write-protected at
>>> power-on. This was caused by using an uninitialized mtd_info in
>>> spi_nor_try_unlock_all().
>>
>> I personally think bullet points are a bit easier to read for changelogs
>> compared to paragraphs since there is less filler text needed. For
>> example, my original merge message didn't the "there is also a fix for".
>> It just said "Fix unlocking of...".
>>
>> Anyway, if you still prefer paragraphs then do let me know and I'll use
>> those next time around and save you some editing.
>
> Some time ago Linus asked me to change the content of the PR description
> (I was mostly listing the patches) and pointed to examples which were
> mostly plain English, not bullet points. So I changed both the content
> (hinting what are the major changes and roughly describing the small
> ones) and the form (no bullet points). I've kept splitting the
> subsystems with '*' though, as the changes in each subsystem are
> mostly unrelated.
>
> I'm fine either ways, but I assumed until now that the preferred form
> was plain text descriptions.
Okay, fair enough. I can do plain text next time around.
--
Regards,
Pratyush Yadav
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [GIT PULL] mtd: Changes for 6.17
2025-08-04 14:40 ` Pratyush Yadav
@ 2025-08-04 15:53 ` Linus Torvalds
2025-08-05 7:47 ` Miquel Raynal
0 siblings, 1 reply; 7+ messages in thread
From: Linus Torvalds @ 2025-08-04 15:53 UTC (permalink / raw)
To: Pratyush Yadav
Cc: Miquel Raynal, Richard Weinberger, Tudor Ambarus,
Vignesh Raghavendra, Frieder Schrempf, Michael Walle, linux-mtd
On Mon, 4 Aug 2025 at 07:40, Pratyush Yadav <pratyush@kernel.org> wrote:
>
> >
> > I'm fine either ways, but I assumed until now that the preferred form
> > was plain text descriptions.
>
> Okay, fair enough. I can do plain text next time around.
I'm actually fine either way - what I care about is that it's a
human-readable explanation of what is going on, and what I get when I
pull.
It should also strive to not only be relevant to me when I pull, but
also hopefully be relevant to others if they end up having to look at
the resulting merge - perhaps due to bugs that got introduced.
So in a perfect world, think about how to explain to me what I'm
getting in the pull, but also explaining to somebody who is in the
middle of a bisect, and notices that the problem they are trying to
figure out was introduced by merging that tree/
Or maybe the bisect is down to two or three merges, and the person
bisecting - who does *NOT* necessarily know anything about those
subsystems - is trying to judge what is the most likely culprit to
speed up bisection. A good merge message - from the pull request -
gives outsiders a feel for what hs going on and whether that merge is
likely to be relevant to them.
(And yes, I often find myself to be that person: I bisect some issue
pretty much every single merge window, and while I usually have a good
idea of what is going on, and just trust "git bisect" to do the right
thing, I *do* end up looking back at merges to see where I am in the
bisection process)
It can be bullet points when that makes sense, or it can be more of a
plain text explanation of the background to a change.
What I don't want is some cryptic "this is just the shortlog". I can
see that in git. The difference between a shortlog and a commit
message is that the commit message should *explain* what is going on.
Sometimes the two can be the same. Sometimes things are just so
obvious that they don't need further explanation than a one-liner. If
a commit fixes a NULL pointer dereference due to a missing check, it's
not like you need to say more than that.
Also note that sometimes the explanation can be *shorter* than the
shortlog. If there are a hundred small independent details that got
fixed, listing them one by one doesn't make it human-readable.
So being human-readable can mean expanding on *why* something
happened. But it can also be about giving a top-level view of lots of
individually irrelevant details.
Linus
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [GIT PULL] mtd: Changes for 6.17
2025-08-04 15:53 ` Linus Torvalds
@ 2025-08-05 7:47 ` Miquel Raynal
0 siblings, 0 replies; 7+ messages in thread
From: Miquel Raynal @ 2025-08-05 7:47 UTC (permalink / raw)
To: Linus Torvalds
Cc: Pratyush Yadav, Richard Weinberger, Tudor Ambarus,
Vignesh Raghavendra, Frieder Schrempf, Michael Walle, linux-mtd
Hello,
On 04/08/2025 at 08:53:14 -07, Linus Torvalds <torvalds@linux-foundation.org> wrote:
> On Mon, 4 Aug 2025 at 07:40, Pratyush Yadav <pratyush@kernel.org> wrote:
>>
>> >
>> > I'm fine either ways, but I assumed until now that the preferred form
>> > was plain text descriptions.
>>
>> Okay, fair enough. I can do plain text next time around.
>
> I'm actually fine either way - what I care about is that it's a
> human-readable explanation of what is going on, and what I get when I
> pull.
[...]
> (And yes, I often find myself to be that person: I bisect some issue
> pretty much every single merge window, and while I usually have a good
> idea of what is going on, and just trust "git bisect" to do the right
> thing, I *do* end up looking back at merges to see where I am in the
> bisection process)
:)
> It can be bullet points when that makes sense, or it can be more of a
> plain text explanation of the background to a change.
Thanks Linus for the detailed answer.
Pratyush, I guess I'll switch back to bullet points to highlight what
are the important changes, so you can stick to what you were doing. If I
feel like plain text would be sometimes better I might mix both
occasionally.
Thanks,
Miquèl
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2025-08-05 7:51 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-31 16:59 [GIT PULL] mtd: Changes for 6.17 Miquel Raynal
2025-07-31 20:51 ` pr-tracker-bot
2025-08-01 14:47 ` Pratyush Yadav
2025-08-01 15:01 ` Miquel Raynal
2025-08-04 14:40 ` Pratyush Yadav
2025-08-04 15:53 ` Linus Torvalds
2025-08-05 7:47 ` Miquel Raynal
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).