linux-kernel - Re: [PATCH v2] mtd: micron-st: enable lock/unlock for mt25qu512a

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <084ed945-7674-280f-5866-9238473a294d@leica-geosystems.com>
Date:   Tue, 22 Aug 2023 12:16:42 +0000
From:   SHUKLA Mamta Ramendra <mamta.shukla@...ca-geosystems.com>
To:     Tudor Ambarus <tudor.ambarus@...aro.org>,
        "pratyush@...nel.org" <pratyush@...nel.org>,
        "michael@...le.cc" <michael@...le.cc>,
        "miquel.raynal@...tlin.com" <miquel.raynal@...tlin.com>,
        "richard@....at" <richard@....at>,
        "vigneshr@...com" <vigneshr@...com>,
        "linux-mtd@...ts.infradead.org" <linux-mtd@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC:     GEO-CHHER-bsp-development 
        <bsp-development.geo@...ca-geosystems.com>
Subject: Re: [PATCH v2] mtd: micron-st: enable lock/unlock for mt25qu512a


On 14.07.23 10:15, SHUKLA Mamta Ramendra wrote:
> Hello Tudor,
> 
> On 13.07.23 05:43, Tudor Ambarus wrote:
>> This email is not from Hexagon’s Office 365 instance. Please be careful while clicking links, opening attachments, or replying to this email.
>>
>>
>> On 05.07.2023 18:49, Mamta Shukla wrote:
>>> mt25qu512a[1] supports locking/unlocking through BP bits in SR.
>>>
>>> Tested using mtd-utils- flash_lock/flash_unlock for MT25QU512ABB8E12.
>>>
>>> Link: https://media-www.micron.com/-/media/client/global/documents/products/data-sheet/nor-flash/serial-nor/mt25q/die-rev-b/mt25q_qlkt_u_512_abb_0.pdf?rev=b259aadc3bea49ea8210a41c9ad58211
>>> Signed-off-by: Mamta Shukla <mamta.shukla@...ca-geosystems.com>
>>> ---
>>>    Changes in v2:
>>>    - add Link tag
>>>    - fix chip part number mt25ql512a->mt25qu512a
>>>
>>>    drivers/mtd/spi-nor/micron-st.c | 2 ++
>>>    1 file changed, 2 insertions(+)
>>>
>>> diff --git a/drivers/mtd/spi-nor/micron-st.c b/drivers/mtd/spi-nor/micron-st.c
>>> index 4b919756a205..08e94340ebaa 100644
>>> --- a/drivers/mtd/spi-nor/micron-st.c
>>> +++ b/drivers/mtd/spi-nor/micron-st.c
>>> @@ -229,6 +229,8 @@ static const struct flash_info st_nor_parts[] = {
>>>                 MFR_FLAGS(USE_FSR)
>>>         },
>>>         { "mt25qu512a",  INFO6(0x20bb20, 0x104400, 64 * 1024, 1024)
>>> +             FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP |
>>> +                   SPI_NOR_BP3_SR_BIT6)
>>>                 NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ)
>>>                 FIXUP_FLAGS(SPI_NOR_4B_OPCODES)
>>>                 MFR_FLAGS(USE_FSR)
>>
>> Can you try the following instead? We try to use SFDP parsing whenever
>> possible.
>>           { "mt25qu512a",  INFO6(0x20bb20, 0x104400, 0, 0)
>>                   FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP |
>>                   SPI_NOR_BP3_SR_BIT6)
>>                   PARSE_SFDP
>>                   MFR_FLAGS(USE_FSR)
> 
> 
> I tested with SFDP Parsing Flag. It works fine.
> 
> ---------------------------------------------------------------------------------
> [  214.726090] ACPI: Host-directed Dynamic ACPI Table Load:
> [  214.731482] ACPI: SSDT 0xFFFF892882D89800 0000EC (v02 ALASKA MT25QU
> 00001000 INTL 20190509)
> [  214.766082] spi-nor spi-PRP0001:00: mt25qu512a (65536 Kbytes)
> 
> 
> cat /sys/bus/spi/devices/spi-PRP0001:00/spi-nor/jedec_id
> 20bb20104400
> 
> cat /sys/bus/spi/devices/spi-PRP0001:00/spi-nor/manufacturer
> st
> 
> cat /sys/bus/spi/devices/spi-PRP0001:00/spi-nor/partname
> mt25qu512a
> 
> xxd -p  /sys/bus/spi/devices/spi-PRP0001:00/spi-nor/sfdp
> 53464450060101ff00060110300000ff84000102800000ffffffffffffff
> ffffffffffffffffffffffffffffffffffffe520fbffffffff1f29eb276b
> 273b27bbffffffffffff27bbffff29eb0c2010d80f520000244a99008b8e
> 03e1ac0127387a757a75fbbdd55c4a0f82ff81bd3d36ffffffffffffffff
> ffffffffffffffffffe7ffff21dcffff
> 
> md5sum  /sys/bus/spi/devices/spi-PRP0001:00/spi-nor/sfdp
> 610efba1647e00ac6db18beb11e84c04
> /sys/bus/spi/devices/spi-PRP0001:00/spi-nor/sfdp
> 
> dd if=/dev/urandom of=/tmp/qspi_write  bs=1M count=1
> 1+0 records in
> 1+0 records out
> 1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.0221391 s, 47.4 MB/s
> 
> mtd_debug write /dev/mtd0 0 1048576 /tmp/qspi_write
> Copied 1048576 bytes from /tmp/qspi_write to address 0x00000000 in flash
> 
> mtd_debug erase /dev/mtd0 0 1048576
> Erased 1048576 bytes from address 0x00000000 in flash
> 
> mtd_debug read /dev/mtd0 0 1048576 /tmp/qspi_read
> Copied 1048576 bytes from address 0x00000000 in flash to /tmp/qspi_read
> 
> hexdump /tmp/qspi_read
> 0000000 ffff ffff ffff ffff ffff ffff ffff ffff
> *
> 0100000
> 
> mtd_debug write /dev/mtd0 0 1048576 /tmp/qspi_write
> Copied 1048576 bytes from /tmp/qspi_write to address 0x00000000 in flash
> 
> mtd_debug read /dev/mtd0 0 1048576 /tmp/qspi_read
> Copied 1048576 bytes from address 0x00000000 in flash to /tmp/qspi_read
> 
> sha1sum /tmp/qspi_write  /tmp/qspi_read
> 4fe4e71b11f44e9672bd49e2e32c0fd94da4feb6  /tmp/qspi_write
> 4fe4e71b11f44e9672bd49e2e32c0fd94da4feb6  /tmp/qspi_read
> 
> ----------------------------------------------------------------------------------
> 
> Shall I write a new commit for PARSE_SFDP Flag or update this commit as-
> "Modify mt25qu512a Flags" ?


Ping! Just to follow up on this.

>>
>> You then have to dump the SFDP tables and do a sanity check to make sure
>> everything's fine after your changes. You can an example on how to do
>> that on the commit message from
>> https://lore.kernel.org/linux-mtd/8b7d2420-26b6-405b-8ef7-75e0a0201402@linaro.org/T/#m3550973e0884ec4a288d344fabd4a9c3b64af46e
> 
> Thanks,
> Mamta