* [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
@ 2023-11-23 5:47 Srikanth Boyapally
2023-11-24 14:47 ` Pratyush Yadav
0 siblings, 1 reply; 6+ messages in thread
From: Srikanth Boyapally @ 2023-11-23 5:47 UTC (permalink / raw)
To: tudor.ambarus, pratyush, michael, miquel.raynal, richard,
vigneshr
Cc: linux-mtd, linux-kernel, git, Srikanth Boyapally
Add support for issi is25lp02g.
Verified on AMD-Xilinx versal platform and executed
mtd_debug read/write test.
Signed-off-by: Srikanth Boyapally <srikanth.boyapally@amd.com>
---
BRANCH: spi-nor/next
---
drivers/mtd/spi-nor/issi.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/mtd/spi-nor/issi.c b/drivers/mtd/spi-nor/issi.c
index 18d9a00aa22e..98df64caf436 100644
--- a/drivers/mtd/spi-nor/issi.c
+++ b/drivers/mtd/spi-nor/issi.c
@@ -126,6 +126,12 @@ static const struct flash_info issi_nor_parts[] = {
.flags = SPI_NOR_QUAD_PP,
.fixups = &is25lp256_fixups,
.fixup_flags = SPI_NOR_4B_OPCODES,
+ }, {
+ .id = SNOR_ID(0x9d, 0x60, 0x22),
+ .name = "is25lp02",
+ .size = SZ_2G,
+ .fixup_flags = SPI_NOR_4B_OPCODES,
+ .no_sfdp_flags = SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ,
}
};
--
2.25.1
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
2023-11-23 5:47 [PATCH] mtd: spi-nor: issi: Add support for is25lp02g Srikanth Boyapally
@ 2023-11-24 14:47 ` Pratyush Yadav
2023-11-28 6:02 ` Boyapally, Srikanth
0 siblings, 1 reply; 6+ messages in thread
From: Pratyush Yadav @ 2023-11-24 14:47 UTC (permalink / raw)
To: Srikanth Boyapally
Cc: tudor.ambarus, pratyush, michael, miquel.raynal, richard,
vigneshr, linux-mtd, linux-kernel, git
Hi,
On Thu, Nov 23 2023, Srikanth Boyapally wrote:
> Add support for issi is25lp02g.
>
> Verified on AMD-Xilinx versal platform and executed
> mtd_debug read/write test.
>
> Signed-off-by: Srikanth Boyapally <srikanth.boyapally@amd.com>
Based on a datasheet [0] I found online, this flash seems to have SFDP.
And since you do not add any flash specific fixups, it likely does not
need an entry and would work fine with the generic flash driver. Did you
try using that? See [1] for more info on contribution guidelines.
[0] https://www.issi.com/WW/pdf/25LP-WP02GG.pdf
[1] https://lore.kernel.org/linux-mtd/20231123160721.64561-2-tudor.ambarus@linaro.org/
--
Regards,
Pratyush Yadav
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
2023-11-24 14:47 ` Pratyush Yadav
@ 2023-11-28 6:02 ` Boyapally, Srikanth
2023-11-28 8:36 ` Michael Walle
0 siblings, 1 reply; 6+ messages in thread
From: Boyapally, Srikanth @ 2023-11-28 6:02 UTC (permalink / raw)
To: Pratyush Yadav
Cc: tudor.ambarus@linaro.org, michael@walle.cc,
miquel.raynal@bootlin.com, richard@nod.at, vigneshr@ti.com,
linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org,
git (AMD-Xilinx)
Hello Pratyush,
> -----Original Message-----
> From: Pratyush Yadav <pratyush@kernel.org>
> Sent: Friday, November 24, 2023 8:17 PM
> To: Boyapally, Srikanth <Srikanth.Boyapally@amd.com>
> Cc: tudor.ambarus@linaro.org; pratyush@kernel.org; michael@walle.cc;
> miquel.raynal@bootlin.com; richard@nod.at; vigneshr@ti.com; linux-
> mtd@lists.infradead.org; linux-kernel@vger.kernel.org; git (AMD-Xilinx)
> <git@amd.com>
> Subject: Re: [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Hi,
>
> On Thu, Nov 23 2023, Srikanth Boyapally wrote:
>
> > Add support for issi is25lp02g.
> >
> > Verified on AMD-Xilinx versal platform and executed mtd_debug
> > read/write test.
> >
> > Signed-off-by: Srikanth Boyapally <srikanth.boyapally@amd.com>
>
> Based on a datasheet [0] I found online, this flash seems to have SFDP.
> And since you do not add any flash specific fixups, it likely does not need an entry
> and would work fine with the generic flash driver. Did you try using that? See [1]
> for more info on contribution guidelines.
>
> [0] https://www.issi.com/WW/pdf/25LP-WP02GG.pdf
> [1] https://lore.kernel.org/linux-mtd/20231123160721.64561-2-
> tudor.ambarus@linaro.org/
>
I ran mtd_debug read/write tests on this flash, without adding flash
entry, it worked fine for me. Please ignore this patch.
Regards,
Srikanth B
> --
> Regards,
> Pratyush Yadav
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
2023-11-28 6:02 ` Boyapally, Srikanth
@ 2023-11-28 8:36 ` Michael Walle
2023-11-29 12:34 ` Boyapally, Srikanth
0 siblings, 1 reply; 6+ messages in thread
From: Michael Walle @ 2023-11-28 8:36 UTC (permalink / raw)
To: Boyapally, Srikanth
Cc: Pratyush Yadav, tudor.ambarus, miquel.raynal, richard, vigneshr,
linux-mtd, linux-kernel, git (AMD-Xilinx)
Hi,
>> > Add support for issi is25lp02g.
>> >
>> > Verified on AMD-Xilinx versal platform and executed mtd_debug
>> > read/write test.
>> >
>> > Signed-off-by: Srikanth Boyapally <srikanth.boyapally@amd.com>
>>
>> Based on a datasheet [0] I found online, this flash seems to have
>> SFDP.
>> And since you do not add any flash specific fixups, it likely does not
>> need an entry
>> and would work fine with the generic flash driver. Did you try using
>> that? See [1]
>> for more info on contribution guidelines.
>>
>> [0] https://www.issi.com/WW/pdf/25LP-WP02GG.pdf
>> [1] https://lore.kernel.org/linux-mtd/20231123160721.64561-2-
>> tudor.ambarus@linaro.org/
>>
> I ran mtd_debug read/write tests on this flash, without adding flash
> entry, it worked fine for me. Please ignore this patch.
Great! Could you please still dump and post your SFDP just for us to
have more SFDP of diferent flashes? See [1].
-michael
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
2023-11-28 8:36 ` Michael Walle
@ 2023-11-29 12:34 ` Boyapally, Srikanth
2023-11-29 13:05 ` Michael Walle
0 siblings, 1 reply; 6+ messages in thread
From: Boyapally, Srikanth @ 2023-11-29 12:34 UTC (permalink / raw)
To: Michael Walle
Cc: Pratyush Yadav, tudor.ambarus@linaro.org,
miquel.raynal@bootlin.com, richard@nod.at, vigneshr@ti.com,
linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org,
git (AMD-Xilinx)
Hi Michael,
> -----Original Message-----
> From: Michael Walle <michael@walle.cc>
> Sent: Tuesday, November 28, 2023 2:06 PM
> To: Boyapally, Srikanth <Srikanth.Boyapally@amd.com>
> Cc: Pratyush Yadav <pratyush@kernel.org>; tudor.ambarus@linaro.org;
> miquel.raynal@bootlin.com; richard@nod.at; vigneshr@ti.com; linux-
> mtd@lists.infradead.org; linux-kernel@vger.kernel.org; git (AMD-Xilinx)
> <git@amd.com>
> Subject: Re: [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Hi,
>
> >> > Add support for issi is25lp02g.
> >> >
> >> > Verified on AMD-Xilinx versal platform and executed mtd_debug
> >> > read/write test.
> >> >
> >> > Signed-off-by: Srikanth Boyapally <srikanth.boyapally@amd.com>
> >>
> >> Based on a datasheet [0] I found online, this flash seems to have
> >> SFDP.
> >> And since you do not add any flash specific fixups, it likely does
> >> not need an entry and would work fine with the generic flash driver.
> >> Did you try using that? See [1] for more info on contribution
> >> guidelines.
> >>
> >> [0] https://www.issi.com/WW/pdf/25LP-WP02GG.pdf
> >> [1] https://lore.kernel.org/linux-mtd/20231123160721.64561-2-
> >> tudor.ambarus@linaro.org/
> >>
> > I ran mtd_debug read/write tests on this flash, without adding flash
> > entry, it worked fine for me. Please ignore this patch.
>
> Great! Could you please still dump and post your SFDP just for us to have more
> SFDP of diferent flashes? See [1].
>
SFDP dump:
versal-generic:/home/petalinux# hexdump -C /sys/bus/spi/devices/spi0.0/spi-nor/sfdp
00000000 53 46 44 50 06 01 01 ff 00 06 01 10 30 00 00 ff |SFDP........0...|
00000010 84 00 01 02 80 00 00 ff ff ff ff ff ff ff ff ff |................|
00000020 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |................|
00000030 e5 20 fb ff ff ff ff 7f 44 eb 08 6b 08 3b 80 bb |. ......D..k.;..|
00000040 fe ff ff ff ff ff 00 ff ff ff 44 eb 0c 20 0f 52 |..........D.. .R|
00000050 10 d8 00 ff 62 42 a9 00 82 64 02 e2 ec 8d 69 4c |....bB...d....iL|
00000060 7a 75 7a 75 f7 a2 d5 5c 4a c2 2c ff e8 30 fa a9 |zuzu...\J.,..0..|
00000070 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |................|
00000080 ff ee ff ff 21 5c dc ff |....!\..|
00000088
regards,
Srikanth B
>
> -michael
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mtd: spi-nor: issi: Add support for is25lp02g
2023-11-29 12:34 ` Boyapally, Srikanth
@ 2023-11-29 13:05 ` Michael Walle
0 siblings, 0 replies; 6+ messages in thread
From: Michael Walle @ 2023-11-29 13:05 UTC (permalink / raw)
To: Boyapally, Srikanth
Cc: Pratyush Yadav, tudor.ambarus, miquel.raynal, richard, vigneshr,
linux-mtd, linux-kernel, git (AMD-Xilinx)
Hi,
>> >> > Add support for issi is25lp02g.
>> >> >
>> >> > Verified on AMD-Xilinx versal platform and executed mtd_debug
>> >> > read/write test.
>> >> >
>> >> > Signed-off-by: Srikanth Boyapally <srikanth.boyapally@amd.com>
>> >>
>> >> Based on a datasheet [0] I found online, this flash seems to have
>> >> SFDP.
>> >> And since you do not add any flash specific fixups, it likely does
>> >> not need an entry and would work fine with the generic flash driver.
>> >> Did you try using that? See [1] for more info on contribution
>> >> guidelines.
>> >>
>> >> [0] https://www.issi.com/WW/pdf/25LP-WP02GG.pdf
>> >> [1] https://lore.kernel.org/linux-mtd/20231123160721.64561-2-
>> >> tudor.ambarus@linaro.org/
>> >>
>> > I ran mtd_debug read/write tests on this flash, without adding flash
>> > entry, it worked fine for me. Please ignore this patch.
>>
>> Great! Could you please still dump and post your SFDP just for us to
>> have more
>> SFDP of diferent flashes? See [1].
>>
> SFDP dump:
> versal-generic:/home/petalinux# hexdump -C
> /sys/bus/spi/devices/spi0.0/spi-nor/sfdp
> 00000000 53 46 44 50 06 01 01 ff 00 06 01 10 30 00 00 ff
> |SFDP........0...|
> 00000010 84 00 01 02 80 00 00 ff ff ff ff ff ff ff ff ff
> |................|
> 00000020 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> |................|
> 00000030 e5 20 fb ff ff ff ff 7f 44 eb 08 6b 08 3b 80 bb |.
> ......D..k.;..|
> 00000040 fe ff ff ff ff ff 00 ff ff ff 44 eb 0c 20 0f 52
> |..........D.. .R|
> 00000050 10 d8 00 ff 62 42 a9 00 82 64 02 e2 ec 8d 69 4c
> |....bB...d....iL|
> 00000060 7a 75 7a 75 f7 a2 d5 5c 4a c2 2c ff e8 30 fa a9
> |zuzu...\J.,..0..|
> 00000070 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> |................|
> 00000080 ff ee ff ff 21 5c dc ff |....!\..|
> 00000088
Thanks!
-michael
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-11-29 13:05 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-23 5:47 [PATCH] mtd: spi-nor: issi: Add support for is25lp02g Srikanth Boyapally
2023-11-24 14:47 ` Pratyush Yadav
2023-11-28 6:02 ` Boyapally, Srikanth
2023-11-28 8:36 ` Michael Walle
2023-11-29 12:34 ` Boyapally, Srikanth
2023-11-29 13:05 ` Michael Walle
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).