adding part #

[Vlim]

Hi, FlashRom team,

I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
Can you please provide some guidance?

Regards,

Victor Lim
FAE Director
GigaDevice Semiconductor
4088833856

[Peter Marheine]

Our "how to add a new chip" documentation should point you in the right
direction: https://flashrom.org/contrib_howtos/how_to_add_new_chip.html

But feel free to ask further questions if anything there doesn't seem
sufficient for your needs.

On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
> Hi, FlashRom team,
>
> I am the FAE director in Gigadevice and I would like to add some part #s
> to the flashchip.c and flashchip.h.
> Can you please provide some guidance?
>
> Regards,
>
> Victor Lim
> FAE Director
> GigaDevice Semiconductor
> 4088833856 <(408)%20883-3856>
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[Vlim]

Thanks, Peter,

Where can I find the definition of terms like,

.feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
SPI_DISABLE_BLOCKPROTECT_BP4_SRWD

Regards,

Victor

________________________________
From: Peter Marheine 
Sent: Tuesday, March 19, 2024 15:42
To: Vlim 
Cc: [e-mail] 
Subject: Re: [flashrom] adding part #

You don't often get email from [e-mail]. Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
Our "how to add a new chip" documentation should point you in the right direction: https://flashrom.org/contrib_howtos/how_to_add_new_chip.html

But feel free to ask further questions if anything there doesn't seem sufficient for your needs.

On Wed, Mar 20, 2024 at 6:31 AM Vlim <[e-mail]> wrote:
Hi, FlashRom team,

I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
Can you please provide some guidance?

Regards,

Victor Lim
FAE Director
GigaDevice Semiconductor
4088833856tel:(408)%20883-3856

_______________________________________________
flashrom mailing list -- [e-mail]
To unsubscribe send an email to [e-mail]

[Anastasia Klimchuk]

Hello Victor,

Various feature bits are defined as macros in include/flash.h , some of
them have comments explaining what they do. If the feature name is not
descriptive, and there is no comment, but you think you might need this
feature: feel free to ask. I would try to add comments later.

SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as
in spi25_statusreg.c. Those functions have comments, hopefully this can
help!

On Thu, Mar 21, 2024 at 2:48 PM Vlim  wrote:
> Thanks, Peter,
>
> Where can I find the definition of terms like,
>
> .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
>
> Regards,
>
> Victor
>
> *From:* Peter Marheine 
> *Sent:* Tuesday, March 19, 2024 15:42
> *To:* Vlim 
> *Cc:* [e-mail] 
> *Subject:* Re: [flashrom] adding part #
>
> You don't often get email from [e-mail]. Learn why this is
> important https://aka.ms/LearnAboutSenderIdentification
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close
> attention to whether the email contains sensitive information
> Our "how to add a new chip" documentation should point you in the right
> direction: https://flashrom.org/contrib_howtos/how_to_add_new_chip.html
>
> But feel free to ask further questions if anything there doesn't seem
> sufficient for your needs.
>
> On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
>
> Hi, FlashRom team,
>
> I am the FAE director in Gigadevice and I would like to add some part #s
> to the flashchip.c and flashchip.h.
> Can you please provide some guidance?
>
> Regards,
>
> Victor Lim
> FAE Director
> GigaDevice Semiconductor
> 4088833856 <(408)%20883-3856>
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[Vlim]

Thanks, Anastasia,

Can you please explain this statement,

.printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
.unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,

Looks like this is to support memory array protection. But it needs BP0-BP4.
Do I need to specify BP0 to BP3 as well?
Or I specify the setting when I initiate the flashrom programming command?

Regards,

Victor

________________________________
From: Anastasia Klimchuk 
Sent: Thursday, March 21, 2024 05:32
To: Vlim ; [e-mail] 
Cc: Peter Marheine 
Subject: [flashrom] Re: adding part #

You don't often get email from [e-mail]. Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
Hello Victor,

Various feature bits are defined as macros in include/flash.h , some of them have comments explaining what they do. If the feature name is not descriptive, and there is no comment, but you think you might need this feature: feel free to ask. I would try to add comments later.

SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in spi25_statusreg.c. Those functions have comments, hopefully this can help!

On Thu, Mar 21, 2024 at 2:48 PM Vlim <[e-mail]> wrote:
Thanks, Peter,

Where can I find the definition of terms like,

.feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
SPI_DISABLE_BLOCKPROTECT_BP4_SRWD

Regards,

Victor

________________________________
From: Peter Marheine <[e-mail]>
Sent: Tuesday, March 19, 2024 15:42
To: Vlim <[e-mail]>
Cc: [e-mail] <[e-mail]>
Subject: Re: [flashrom] adding part #

You don't often get email from [e-mail]. Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
Our "how to add a new chip" documentation should point you in the right direction: https://flashrom.org/contrib_howtos/how_to_add_new_chip.html

But feel free to ask further questions if anything there doesn't seem sufficient for your needs.

On Wed, Mar 20, 2024 at 6:31 AM Vlim <[e-mail]> wrote:
Hi, FlashRom team,

I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
Can you please provide some guidance?

Regards,

Victor Lim
FAE Director
GigaDevice Semiconductor
4088833856tel:(408)%20883-3856

_______________________________________________
flashrom mailing list -- [e-mail]
To unsubscribe send an email to [e-mail]
_______________________________________________
flashrom mailing list -- [e-mail]
To unsubscribe send an email to [e-mail]

[Anastasia Klimchuk]

For these two (printlock and unlock), you need to pick the function
which handles the highest BP bit that chip supports. In your example,
if a chip supports bits BP0-BP4 the correct functions are those with
BP4 in the name. So you did it all right.

If the chip supports write protection, you also need to set .reg_bits
in chip definition. For this you need to have a look at struct
reg_bit_map in include/flash.h, it has comments on what each bit in
the struct means.

On Fri, Mar 22, 2024 at 7:22 AM Vlim  wrote:
> Thanks, Anastasia,
>
> Can you please explain this statement,
>
> .printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> .unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
>
> Looks like this is to support memory array protection. But it needs BP0-BP4.
> Do I need to specify BP0 to BP3 as well?
> Or I specify the setting when I initiate the flashrom programming command?
>
> Regards,
>
> Victor
>
> From: Anastasia Klimchuk 
> Sent: Thursday, March 21, 2024 05:32
> To: Vlim ; [e-mail] 
> Cc: Peter Marheine 
> Subject: [flashrom] Re: adding part #
>
> You don't often get email from [e-mail]. Learn why this is important
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> Hello Victor,
>
> Various feature bits are defined as macros in include/flash.h , some of them have comments explaining what they do. If the feature name is not descriptive, and there is no comment, but you think you might need this feature: feel free to ask. I would try to add comments later.
>
> SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in spi25_statusreg.c. Those functions have comments, hopefully this can help!
>
> On Thu, Mar 21, 2024 at 2:48 PM Vlim  wrote:
>
> Thanks, Peter,
>
> Where can I find the definition of terms like,
>
> .feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
>
> Regards,
>
> Victor
>
> From: Peter Marheine 
> Sent: Tuesday, March 19, 2024 15:42
> To: Vlim 
> Cc: [e-mail] 
> Subject: Re: [flashrom] adding part #
>
> You don't often get email from [e-mail]. Learn why this is important
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> Our "how to add a new chip" documentation should point you in the right direction: https://flashrom.org/contrib_howtos/how_to_add_new_chip.html
>
> But feel free to ask further questions if anything there doesn't seem sufficient for your needs.
>
> On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
>
> Hi, FlashRom team,
>
> I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
> Can you please provide some guidance?
>
> Regards,
>
> Victor Lim
> FAE Director
> GigaDevice Semiconductor
> 4088833856
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[Vlim]

Thanks.

How is Dual IO and Quad IO supported here?
I have the following cases.

1.
parts that need to configure QE bit before Quad IO can be used.
  2.
parts that have default QE bit set for quad.
  3.
Has a dedicated command for quad IO read, which is similar to other vendors.

Regards,

Victor

________________________________
From: Anastasia Klimchuk 
Sent: Thursday, March 21, 2024 23:32
To: Vlim ; [e-mail] 
Cc: Peter Marheine 
Subject: Re: [flashrom] Re: adding part #

[You don't often get email from [e-mail]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]

此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information

For these two (printlock and unlock), you need to pick the function
which handles the highest BP bit that chip supports. In your example,
if a chip supports bits BP0-BP4 the correct functions are those with
BP4 in the name. So you did it all right.

If the chip supports write protection, you also need to set .reg_bits
in chip definition. For this you need to have a look at struct
reg_bit_map in include/flash.h, it has comments on what each bit in
the struct means.

On Fri, Mar 22, 2024 at 7:22 AM Vlim  wrote:
> Thanks, Anastasia,
>
> Can you please explain this statement,
>
> .printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> .unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
>
> Looks like this is to support memory array protection. But it needs BP0-BP4.
> Do I need to specify BP0 to BP3 as well?
> Or I specify the setting when I initiate the flashrom programming command?
>
> Regards,
>
> Victor
>
> From: Anastasia Klimchuk 
> Sent: Thursday, March 21, 2024 05:32
> To: Vlim ; [e-mail] 
> Cc: Peter Marheine 
> Subject: [flashrom] Re: adding part #
>
> You don't often get email from [e-mail]. Learn why this is important
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> Hello Victor,
>
> Various feature bits are defined as macros in include/flash.h , some of them have comments explaining what they do. If the feature name is not descriptive, and there is no comment, but you think you might need this feature: feel free to ask. I would try to add comments later.
>
> SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in spi25_statusreg.c. Those functions have comments, hopefully this can help!
>
> On Thu, Mar 21, 2024 at 2:48 PM Vlim  wrote:
>
> Thanks, Peter,
>
> Where can I find the definition of terms like,
>
> .feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
>
> Regards,
>
> Victor
>
> From: Peter Marheine 
> Sent: Tuesday, March 19, 2024 15:42
> To: Vlim 
> Cc: [e-mail] 
> Subject: Re: [flashrom] adding part #
>
> You don't often get email from [e-mail]. Learn why this is important
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> Our "how to add a new chip" documentation should point you in the right direction: https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fflashrom.o...https://flashrom.org/contrib_howtos/how_to_add_new_chip.html
>
> But feel free to ask further questions if anything there doesn't seem sufficient for your needs.
>
> On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
>
> Hi, FlashRom team,
>
> I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
> Can you please provide some guidance?
>
> Regards,
>
> Victor Lim
> FAE Director
> GigaDevice Semiconductor
> 4088833856
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[Vlim]

Hi,

Can you point me to the file that defines the commands?
Like SPI_CHIP_READ being 0x0b.

Thanks,

Victor

________________________________
From: Anastasia Klimchuk 
Sent: Thursday, March 21, 2024 23:32
To: Vlim ; [e-mail] 
Cc: Peter Marheine 
Subject: Re: [flashrom] Re: adding part #

[You don't often get email from [e-mail]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]

此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information

For these two (printlock and unlock), you need to pick the function
which handles the highest BP bit that chip supports. In your example,
if a chip supports bits BP0-BP4 the correct functions are those with
BP4 in the name. So you did it all right.

If the chip supports write protection, you also need to set .reg_bits
in chip definition. For this you need to have a look at struct
reg_bit_map in include/flash.h, it has comments on what each bit in
the struct means.

On Fri, Mar 22, 2024 at 7:22 AM Vlim  wrote:
> Thanks, Anastasia,
>
> Can you please explain this statement,
>
> .printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> .unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
>
> Looks like this is to support memory array protection. But it needs BP0-BP4.
> Do I need to specify BP0 to BP3 as well?
> Or I specify the setting when I initiate the flashrom programming command?
>
> Regards,
>
> Victor
>
> From: Anastasia Klimchuk 
> Sent: Thursday, March 21, 2024 05:32
> To: Vlim ; [e-mail] 
> Cc: Peter Marheine 
> Subject: [flashrom] Re: adding part #
>
> You don't often get email from [e-mail]. Learn why this is important
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> Hello Victor,
>
> Various feature bits are defined as macros in include/flash.h , some of them have comments explaining what they do. If the feature name is not descriptive, and there is no comment, but you think you might need this feature: feel free to ask. I would try to add comments later.
>
> SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in spi25_statusreg.c. Those functions have comments, hopefully this can help!
>
> On Thu, Mar 21, 2024 at 2:48 PM Vlim  wrote:
>
> Thanks, Peter,
>
> Where can I find the definition of terms like,
>
> .feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
>
> Regards,
>
> Victor
>
> From: Peter Marheine 
> Sent: Tuesday, March 19, 2024 15:42
> To: Vlim 
> Cc: [e-mail] 
> Subject: Re: [flashrom] adding part #
>
> You don't often get email from [e-mail]. Learn why this is important
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> Our "how to add a new chip" documentation should point you in the right direction: https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fflashrom.o...https://flashrom.org/contrib_howtos/how_to_add_new_chip.html
>
> But feel free to ask further questions if anything there doesn't seem sufficient for your needs.
>
> On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
>
> Hi, FlashRom team,
>
> I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
> Can you please provide some guidance?
>
> Regards,
>
> Victor Lim
> FAE Director
> GigaDevice Semiconductor
> 4088833856
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
>
> flashrom mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]

[Anastasia Klimchuk]

I will try to answer to everything in one message, but feel free to
ask more questions
> Can you point me to the file that defines the commands?

Most of them will be in include/spi.h

It is a longer way if you want to trace implementation from, for
example SPI_CHIP_READ, to the place where command is sent, but in case
you would need it:

you would need to go through the sequence of function calls, starting
from `spi_chip_read` in spi.c and then look into the programmer code
(for whichever is the programmer you are using) and see how that
programmer implements spi read and send commands. Specifically look
for `static const struct spi_master` in the programmer source code
file.

Default functions are in spi.c and spi25.c files.
> How is Dual IO and Quad IO supported here?

Currently not implemented, however it would be really great to
implement it in the future (contributions are very welcome).
We do have those aspirational comments in flashchips.c , for example
(I am just copying from some chip)

.write = SPI_CHIP_WRITE256, /* Dual I/O  (0xA2) supported */
.read = SPI_CHIP_READ, /* Fast read (0x0B), dual I/O  (0x3B) supported */

So maybe you can add similar comments for the chip definition for now.

On Tue, Mar 26, 2024 at 5:27 AM Vlim  wrote:
> Hi,
>
> Can you point me to the file that defines the commands?
> Like SPI_CHIP_READ being 0x0b.
>
> Thanks,
>
> Victor
>
> From: Anastasia Klimchuk 
> Sent: Thursday, March 21, 2024 23:32
> To: Vlim ; [e-mail] 
> Cc: Peter Marheine 
> Subject: Re: [flashrom] Re: adding part #
>
> [You don't often get email from [e-mail]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
>
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
>
> For these two (printlock and unlock), you need to pick the function
> which handles the highest BP bit that chip supports. In your example,
> if a chip supports bits BP0-BP4 the correct functions are those with
> BP4 in the name. So you did it all right.
>
> If the chip supports write protection, you also need to set .reg_bits
> in chip definition. For this you need to have a look at struct
> reg_bit_map in include/flash.h, it has comments on what each bit in
> the struct means.
>
> On Fri, Mar 22, 2024 at 7:22 AM Vlim  wrote:
> > Thanks, Anastasia,
> >
> > Can you please explain this statement,
> >
> > .printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> > .unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
> >
> > Looks like this is to support memory array protection. But it needs BP0-BP4.
> > Do I need to specify BP0 to BP3 as well?
> > Or I specify the setting when I initiate the flashrom programming command?
> >
> > Regards,
> >
> > Victor
> >
> > From: Anastasia Klimchuk 
> > Sent: Thursday, March 21, 2024 05:32
> > To: Vlim ; [e-mail] 
> > Cc: Peter Marheine 
> > Subject: [flashrom] Re: adding part #
> >
> > You don't often get email from [e-mail]. Learn why this is important
> > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> > Hello Victor,
> >
> > Various feature bits are defined as macros in include/flash.h , some of them have comments explaining what they do. If the feature name is not descriptive, and there is no comment, but you think you might need this feature: feel free to ask. I would try to add comments later.
> >
> > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in spi25_statusreg.c. Those functions have comments, hopefully this can help!
> >
> > On Thu, Mar 21, 2024 at 2:48 PM Vlim  wrote:
> >
> > Thanks, Peter,
> >
> > Where can I find the definition of terms like,
> >
> > .feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
> >
> > Regards,
> >
> > Victor
> >
> > From: Peter Marheine 
> > Sent: Tuesday, March 19, 2024 15:42
> > To: Vlim 
> > Cc: [e-mail] 
> > Subject: Re: [flashrom] adding part #
> >
> > You don't often get email from [e-mail]. Learn why this is important
> > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> > Our "how to add a new chip" documentation should point you in the right direction: https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fflashrom.o...
> >
> > But feel free to ask further questions if anything there doesn't seem sufficient for your needs.
> >
> > On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
> >
> > Hi, FlashRom team,
> >
> > I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
> > Can you please provide some guidance?
> >
> > Regards,
> >
> > Victor Lim
> > FAE Director
> > GigaDevice Semiconductor
> > 4088833856
> >
> > flashrom mailing list -- [e-mail]
> > To unsubscribe send an email to [e-mail]
> >
> > flashrom mailing list -- [e-mail]
> > To unsubscribe send an email to [e-mail]

[Nikolai Artemiev]

Hi Victor,

One thing to note with the printlock and unlock functions you mentioned
earlier, i.e.:

.printlock = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
.unlock = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,

These functions basically just allow flashrom to remove memory protection
before writing to the chip. They can (and should) be omitted if you define
reg_bits and decode_range in the chip entry, because that will enable full
writeprotect support for the chip.

The reason we still have unlock and printlock functions is for older chips
that don't have reg_bits and decode_range defined.

Cheers,
Nik

On Tue, Mar 26, 2024 at 4:32 PM Anastasia Klimchuk 
wrote:
> I will try to answer to everything in one message, but feel free to
> ask more questions
> > Can you point me to the file that defines the commands?
>
> Most of them will be in include/spi.h
>
> It is a longer way if you want to trace implementation from, for
> example SPI_CHIP_READ, to the place where command is sent, but in case
> you would need it:
>
> you would need to go through the sequence of function calls, starting
> from `spi_chip_read` in spi.c and then look into the programmer code
> (for whichever is the programmer you are using) and see how that
> programmer implements spi read and send commands. Specifically look
> for `static const struct spi_master` in the programmer source code
> file.
>
> Default functions are in spi.c and spi25.c files.
> > How is Dual IO and Quad IO supported here?
>
> Currently not implemented, however it would be really great to
> implement it in the future (contributions are very welcome).
> We do have those aspirational comments in flashchips.c , for example
> (I am just copying from some chip)
>
> .write = SPI_CHIP_WRITE256, /* Dual I/O  (0xA2) supported */
> .read = SPI_CHIP_READ, /* Fast read (0x0B), dual I/O  (0x3B) supported */
>
> So maybe you can add similar comments for the chip definition for now.
>
> On Tue, Mar 26, 2024 at 5:27 AM Vlim  wrote:
> > Hi,
> >
> > Can you point me to the file that defines the commands?
> > Like SPI_CHIP_READ being 0x0b.
> >
> > Thanks,
> >
> > Victor
> >
> > From: Anastasia Klimchuk 
> > Sent: Thursday, March 21, 2024 23:32
> > To: Vlim ; [e-mail] <
>
> [e-mail]>
> > Cc: Peter Marheine 
> > Subject: Re: [flashrom] Re: adding part #
> >
> > [You don't often get email from [e-mail]. Learn why this is
>
> important at https://aka.ms/LearnAboutSenderIdentification ]
> > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > This is an external email, beware of phishing emails. Please pay close
>
> attention to whether the email contains sensitive information
> > For these two (printlock and unlock), you need to pick the function
> > which handles the highest BP bit that chip supports. In your example,
> > if a chip supports bits BP0-BP4 the correct functions are those with
> > BP4 in the name. So you did it all right.
> >
> > If the chip supports write protection, you also need to set .reg_bits
> > in chip definition. For this you need to have a look at struct
> > reg_bit_map in include/flash.h, it has comments on what each bit in
> > the struct means.
> >
> > On Fri, Mar 22, 2024 at 7:22 AM Vlim  wrote:
> > > Thanks, Anastasia,
> > >
> > > Can you please explain this statement,
> > >
> > > .printlock = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> > > .unlock = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
> > >
> > > Looks like this is to support memory array protection. But it needs
>
> BP0-BP4.
> > > Do I need to specify BP0 to BP3 as well?
> > > Or I specify the setting when I initiate the flashrom programming
>
> command?
> > > Regards,
> > >
> > > Victor
> > >
> > > From: Anastasia Klimchuk 
> > > Sent: Thursday, March 21, 2024 05:32
> > > To: Vlim ; [e-mail] <
>
> [e-mail]>
> > > Cc: Peter Marheine 
> > > Subject: [flashrom] Re: adding part #
> > >
> > > You don't often get email from [e-mail]. Learn why this is
>
> important
> > > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > > This is an external email, beware of phishing emails. Please pay close
>
> attention to whether the email contains sensitive information
> > > Hello Victor,
> > >
> > > Various feature bits are defined as macros in include/flash.h , some
>
> of them have comments explaining what they do. If the feature name is not
> descriptive, and there is no comment, but you think you might need this
> feature: feel free to ask. I would try to add comments later.
> > > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in
>
> spi25_statusreg.c. Those functions have comments, hopefully this can help!
> > > On Thu, Mar 21, 2024 at 2:48 PM Vlim  wrote:
> > >
> > > Thanks, Peter,
> > >
> > > Where can I find the definition of terms like,
> > >
> > > .feature_bits = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> > > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
> > >
> > > Regards,
> > >
> > > Victor
> > >
> > > From: Peter Marheine 
> > > Sent: Tuesday, March 19, 2024 15:42
> > > To: Vlim 
> > > Cc: [e-mail] 
> > > Subject: Re: [flashrom] adding part #
> > >
> > > You don't often get email from [e-mail]. Learn why this
>
> is important
> > > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > > This is an external email, beware of phishing emails. Please pay close
>
> attention to whether the email contains sensitive information
> > > Our "how to add a new chip" documentation should point you in the
>
> right direction:
> https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fflashrom.o...
> > > But feel free to ask further questions if anything there doesn't seem
>
> sufficient for your needs.
> > > On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
> > >
> > > Hi, FlashRom team,
> > >
> > > I am the FAE director in Gigadevice and I would like to add some part
>
> #s to the flashchip.c and flashchip.h.
> > > Can you please provide some guidance?
> > >
> > > Regards,
> > >
> > > Victor Lim
> > > FAE Director
> > > GigaDevice Semiconductor
> > > 4088833856 <(408)%20883-3856>
> > >
> > > flashrom mailing list -- [e-mail]
> > > To unsubscribe send an email to [e-mail]
> > >
> > > flashrom mailing list -- [e-mail]
> > > To unsubscribe send an email to [e-mail]

[Vlim]

Thanks.

I believe there is a limit for total_size of 32768. When I increase it to 65536, I am getting test error. The compilation did go through.

.total_size = 32768,

Regards,

Victor

________________________________
From: Nikolai Artemiev 
Sent: Tuesday, March 26, 2024 18:54
To: Anastasia Klimchuk 
Cc: Vlim ; [e-mail] ; Peter Marheine 
Subject: Re: [flashrom] Re: adding part #

You don't often get email from [e-mail]. Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
Hi Victor,

One thing to note with the printlock and unlock functions you mentioned earlier, i.e.:

.printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
.unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,

These functions basically just allow flashrom to remove memory protection before writing to the chip. They can (and should) be omitted if you define reg_bits and decode_range in the chip entry, because that will enable full writeprotect support for the chip.

The reason we still have unlock and printlock functions is for older chips that don't have reg_bits and decode_range defined.

Cheers,
Nik

On Tue, Mar 26, 2024 at 4:32 PM Anastasia Klimchuk <[e-mail]> wrote:
I will try to answer to everything in one message, but feel free to
ask more questions
> Can you point me to the file that defines the commands?

Most of them will be in include/spi.h

It is a longer way if you want to trace implementation from, for
example SPI_CHIP_READ, to the place where command is sent, but in case
you would need it:

you would need to go through the sequence of function calls, starting
from `spi_chip_read` in spi.c and then look into the programmer code
(for whichever is the programmer you are using) and see how that
programmer implements spi read and send commands. Specifically look
for `static const struct spi_master` in the programmer source code
file.

Default functions are in spi.c and spi25.c files.
> How is Dual IO and Quad IO supported here?

Currently not implemented, however it would be really great to
implement it in the future (contributions are very welcome).
We do have those aspirational comments in flashchips.c , for example
(I am just copying from some chip)

.write = SPI_CHIP_WRITE256, /* Dual I/O  (0xA2) supported */
.read = SPI_CHIP_READ, /* Fast read (0x0B), dual I/O  (0x3B) supported */

So maybe you can add similar comments for the chip definition for now.

On Tue, Mar 26, 2024 at 5:27 AM Vlim <[e-mail]> wrote:
> Hi,
>
> Can you point me to the file that defines the commands?
> Like SPI_CHIP_READ being 0x0b.
>
> Thanks,
>
> Victor
>
> From: Anastasia Klimchuk <[e-mail]>
> Sent: Thursday, March 21, 2024 23:32
> To: Vlim <[e-mail]>; [e-mail] <[e-mail]>
> Cc: Peter Marheine <[e-mail]>
> Subject: Re: [flashrom] Re: adding part #
>
> [You don't often get email from [e-mail]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
>
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
>
> For these two (printlock and unlock), you need to pick the function
> which handles the highest BP bit that chip supports. In your example,
> if a chip supports bits BP0-BP4 the correct functions are those with
> BP4 in the name. So you did it all right.
>
> If the chip supports write protection, you also need to set .reg_bits
> in chip definition. For this you need to have a look at struct
> reg_bit_map in include/flash.h, it has comments on what each bit in
> the struct means.
>
> On Fri, Mar 22, 2024 at 7:22 AM Vlim <[e-mail]> wrote:
> > Thanks, Anastasia,
> >
> > Can you please explain this statement,
> >
> > .printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> > .unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
> >
> > Looks like this is to support memory array protection. But it needs BP0-BP4.
> > Do I need to specify BP0 to BP3 as well?
> > Or I specify the setting when I initiate the flashrom programming command?
> >
> > Regards,
> >
> > Victor
> >
> > From: Anastasia Klimchuk <[e-mail]>
> > Sent: Thursday, March 21, 2024 05:32
> > To: Vlim <[e-mail]>; [e-mail] <[e-mail]>
> > Cc: Peter Marheine <[e-mail]>
> > Subject: [flashrom] Re: adding part #
> >
> > You don't often get email from [e-mail]. Learn why this is important
> > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> > Hello Victor,
> >
> > Various feature bits are defined as macros in include/flash.h , some of them have comments explaining what they do. If the feature name is not descriptive, and there is no comment, but you think you might need this feature: feel free to ask. I would try to add comments later.
> >
> > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in spi25_statusreg.c. Those functions have comments, hopefully this can help!
> >
> > On Thu, Mar 21, 2024 at 2:48 PM Vlim <[e-mail]> wrote:
> >
> > Thanks, Peter,
> >
> > Where can I find the definition of terms like,
> >
> > .feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
> >
> > Regards,
> >
> > Victor
> >
> > From: Peter Marheine <[e-mail]>
> > Sent: Tuesday, March 19, 2024 15:42
> > To: Vlim <[e-mail]>
> > Cc: [e-mail] <[e-mail]>
> > Subject: Re: [flashrom] adding part #
> >
> > You don't often get email from [e-mail]. Learn why this is important
> > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> > Our "how to add a new chip" documentation should point you in the right direction: https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fflashrom.o...https://flashrom.org/contrib_howtos/how_to_add_new_chip.html
> >
> > But feel free to ask further questions if anything there doesn't seem sufficient for your needs.
> >
> > On Wed, Mar 20, 2024 at 6:31 AM Vlim <[e-mail]> wrote:
> >
> > Hi, FlashRom team,
> >
> > I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
> > Can you please provide some guidance?
> >
> > Regards,
> >
> > Victor Lim
> > FAE Director
> > GigaDevice Semiconductor
> > 4088833856tel:(408)%20883-3856
> >
> > flashrom mailing list -- [e-mail]
> > To unsubscribe send an email to [e-mail]
> >
> > flashrom mailing list -- [e-mail]
> > To unsubscribe send an email to [e-mail]

[Anastasia Klimchuk]

There is no limit to total_size of chip, and there are chips defined
with size even larger than 65536.

I think you might get `selfcheck_eraseblocks` test failing, is that
what you are getting? If configuration of eraseblocks does not match
total size, this test will fail. You can see exactly what the test is
doing in `tests/selfcheck.c`.

If that's some other test failing, what to do entirely depends on
which test and what is the error. So in any case, you need to re-run
unit tests and inspect the logs.

On Sun, Mar 31, 2024 at 9:24 AM Vlim  wrote:
> Thanks.
>
> I believe there is a limit for total_size of 32768. When I increase it to 65536, I am getting test error. The compilation did go through.
>
> .total_size = 32768,
>
> Regards,
>
> Victor
>
> From: Nikolai Artemiev 
> Sent: Tuesday, March 26, 2024 18:54
> To: Anastasia Klimchuk 
> Cc: Vlim ; [e-mail] ; Peter Marheine 
> Subject: Re: [flashrom] Re: adding part #
>
> You don't often get email from [e-mail]. Learn why this is important
> 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> Hi Victor,
>
> One thing to note with the printlock and unlock functions you mentioned earlier, i.e.:
>
> .printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> .unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
>
> These functions basically just allow flashrom to remove memory protection before writing to the chip. They can (and should) be omitted if you define reg_bits and decode_range in the chip entry, because that will enable full writeprotect support for the chip.
>
> The reason we still have unlock and printlock functions is for older chips that don't have reg_bits and decode_range defined.
>
> Cheers,
> Nik
>
> On Tue, Mar 26, 2024 at 4:32 PM Anastasia Klimchuk  wrote:
>
> I will try to answer to everything in one message, but feel free to
> ask more questions
> > Can you point me to the file that defines the commands?
>
> Most of them will be in include/spi.h
>
> It is a longer way if you want to trace implementation from, for
> example SPI_CHIP_READ, to the place where command is sent, but in case
> you would need it:
>
> you would need to go through the sequence of function calls, starting
> from `spi_chip_read` in spi.c and then look into the programmer code
> (for whichever is the programmer you are using) and see how that
> programmer implements spi read and send commands. Specifically look
> for `static const struct spi_master` in the programmer source code
> file.
>
> Default functions are in spi.c and spi25.c files.
> > How is Dual IO and Quad IO supported here?
>
> Currently not implemented, however it would be really great to
> implement it in the future (contributions are very welcome).
> We do have those aspirational comments in flashchips.c , for example
> (I am just copying from some chip)
>
> .write = SPI_CHIP_WRITE256, /* Dual I/O  (0xA2) supported */
> .read = SPI_CHIP_READ, /* Fast read (0x0B), dual I/O  (0x3B) supported */
>
> So maybe you can add similar comments for the chip definition for now.
>
> On Tue, Mar 26, 2024 at 5:27 AM Vlim  wrote:
> > Hi,
> >
> > Can you point me to the file that defines the commands?
> > Like SPI_CHIP_READ being 0x0b.
> >
> > Thanks,
> >
> > Victor
> >
> > From: Anastasia Klimchuk 
> > Sent: Thursday, March 21, 2024 23:32
> > To: Vlim ; [e-mail] 
> > Cc: Peter Marheine 
> > Subject: Re: [flashrom] Re: adding part #
> >
> > [You don't often get email from [e-mail]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> >
> > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> >
> > For these two (printlock and unlock), you need to pick the function
> > which handles the highest BP bit that chip supports. In your example,
> > if a chip supports bits BP0-BP4 the correct functions are those with
> > BP4 in the name. So you did it all right.
> >
> > If the chip supports write protection, you also need to set .reg_bits
> > in chip definition. For this you need to have a look at struct
> > reg_bit_map in include/flash.h, it has comments on what each bit in
> > the struct means.
> >
> > On Fri, Mar 22, 2024 at 7:22 AM Vlim  wrote:
> > > Thanks, Anastasia,
> > >
> > > Can you please explain this statement,
> > >
> > > .printlock  = SPI_PRETTYPRINT_STATUS_REGISTER_BP4_SRWD,
> > > .unlock     = SPI_DISABLE_BLOCKPROTECT_BP4_SRWD,
> > >
> > > Looks like this is to support memory array protection. But it needs BP0-BP4.
> > > Do I need to specify BP0 to BP3 as well?
> > > Or I specify the setting when I initiate the flashrom programming command?
> > >
> > > Regards,
> > >
> > > Victor
> > >
> > > From: Anastasia Klimchuk 
> > > Sent: Thursday, March 21, 2024 05:32
> > > To: Vlim ; [e-mail] 
> > > Cc: Peter Marheine 
> > > Subject: [flashrom] Re: adding part #
> > >
> > > You don't often get email from [e-mail]. Learn why this is important
> > > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > > This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> > > Hello Victor,
> > >
> > > Various feature bits are defined as macros in include/flash.h , some of them have comments explaining what they do. If the feature name is not descriptive, and there is no comment, but you think you might need this feature: feel free to ask. I would try to add comments later.
> > >
> > > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD and other unlock functions as in spi25_statusreg.c. Those functions have comments, hopefully this can help!
> > >
> > > On Thu, Mar 21, 2024 at 2:48 PM Vlim  wrote:
> > >
> > > Thanks, Peter,
> > >
> > > Where can I find the definition of terms like,
> > >
> > > .feature_bits     = FEATURE_WRSR_WREN | FEATURE_OTP | FEATURE_WRSR_EXT2,
> > > SPI_DISABLE_BLOCKPROTECT_BP4_SRWD
> > >
> > > Regards,
> > >
> > > Victor
> > >
> > > From: Peter Marheine 
> > > Sent: Tuesday, March 19, 2024 15:42
> > > To: Vlim 
> > > Cc: [e-mail] 
> > > Subject: Re: [flashrom] adding part #
> > >
> > > You don't often get email from [e-mail]. Learn why this is important
> > > 此为外部邮件，谨防钓鱼邮件，请注意邮件是否涉及敏感信息
> > > This is an external email, beware of phishing emails. Please pay close attention to whether the email contains sensitive information
> > > Our "how to add a new chip" documentation should point you in the right direction: https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fflashrom.o...
> > >
> > > But feel free to ask further questions if anything there doesn't seem sufficient for your needs.
> > >
> > > On Wed, Mar 20, 2024 at 6:31 AM Vlim  wrote:
> > >
> > > Hi, FlashRom team,
> > >
> > > I am the FAE director in Gigadevice and I would like to add some part #s to the flashchip.c and flashchip.h.
> > > Can you please provide some guidance?
> > >
> > > Regards,
> > >
> > > Victor Lim
> > > FAE Director
> > > GigaDevice Semiconductor
> > > 4088833856
> > >
> > > flashrom mailing list -- [e-mail]
> > > To unsubscribe send an email to [e-mail]
> > >
> > > flashrom mailing list -- [e-mail]
> > > To unsubscribe send an email to [e-mail]