[flashrom] Re: Support for GD25LR512ME flash chip.

[Munduru, Avinash]

[AMD Official Use Only - AMD Internal Distribution Only]

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 
Sent: Friday, June 14, 2024 8:11:00 PM
To: Munduru, Avinash 
Cc: Vlim ; [e-mail] 
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  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.