[AMD Official Use Only - AMD Internal Distribution Only]
+Ritul
Regards, Avinash. Get Outlook for iOShttps://aka.ms/o0ukef ________________________________ From: Munduru, Avinash Avinash.Munduru@amd.com Sent: Saturday, June 15, 2024 2:14 AM To: Anastasia Klimchuk aklm@chromium.org Cc: Vlim vlim@gigadevice.com; flashrom@flashrom.org flashrom@flashrom.org Subject: Re: [flashrom] Re: Support for GD25LR512ME flash chip.
Hi Anastasia,
Thanks for your input I will try looking into this possible solution.
I already tried the padding solution but it did not work.
Once again, thank you for mainlining the patch.
Regards, Avinash.
Get Outlook for iOShttps://aka.ms/o0ukef ________________________________ From: Anastasia Klimchuk aklm@chromium.org Sent: Friday, June 14, 2024 8:11:00 PM To: Munduru, Avinash Avinash.Munduru@amd.com Cc: Vlim vlim@gigadevice.com; flashrom@flashrom.org flashrom@flashrom.org Subject: Re: [flashrom] Re: Support for GD25LR512ME flash chip.
Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
As a quick addition to my previous message:
What you also can try, is to create a layout file with the region of size 16MB, and then set the option to only write this region. The area outside the region will not be modified. If you decide to go this way, please read manpage, especially -l and -i options https://flashrom.org/classic_cli_manpage.html
On Fri, Jun 14, 2024 at 12:22 PM Anastasia Klimchuk aklm@chromium.org wrote:
Hello Avinash,
Sorry for some delay in response. I was also thinking, what's useful that I could tell you.
I understand your idea to shrink the image to 16MB, so that work around the bug which affects images >16MB. However, flashrom checks whether the size of the image is the same as the size of the chip (should throw an error if not). So to write 16MB onto GD25LR512ME which is 64MB, you would need to pad the file, with zeros usually, to make the file 64MB. I don't know whether it would work. Is it possible that you would try (if you have tools to recover your device from failure of course)? Or perhaps you have tried already?
On the other hand, some good news: support for chips GD25LR256E, GD251R512ME has been submitted meanwhile. So no need for patching anymore, the patch linked earlier in the thread is merged into main.
-- Anastasia.
-- Anastasia.