Skip to content

os/board/bk7239n: Adapt to the new flash id#7276

Open
Poly-J wants to merge 1 commit intoSamsung:masterfrom
bekencorp:adapt_flash_id
Open

os/board/bk7239n: Adapt to the new flash id#7276
Poly-J wants to merge 1 commit intoSamsung:masterfrom
bekencorp:adapt_flash_id

Conversation

@Poly-J
Copy link
Copy Markdown
Contributor

@Poly-J Poly-J commented Apr 16, 2026

The bk7239n module needs to be compatible with multiple types of flash, and the software needs to adapt to the new flash ID.

@Taejun-Kwon
Copy link
Copy Markdown
Contributor

can you please let me know which flash you added?
as I know,

  1. GD25Q128E
  2. W25Q128JV
  3. BY25Q1238ES
  4. XM25QH128D
  5. XT25F128W -> already added as a default

#if CONFIG_FLASH_QUAD_ENABLE
{0x684018, 2, FLASH_SIZE_16M, FLASH_LINE_MODE_FOUR, 14, 2, 0x1F, 0x1F, 0x00, 0x0E, 0x00E, 9, 1, 0xA0, 0x02}, //BY25Q128ES
#else
{0x684018, 1, FLASH_SIZE_16M, FLASH_LINE_MODE_TWO, 0, 2, 0x1F, 0x1F, 0x00, 0x0E, 0x00E, 0, 0, 0xA0, 0x01}, //BY25Q128ES
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add some description related to 'BY25Q128ES'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, Kwon. The BY25Q128 is manufactured by BYT Semiconductor.The BY25Q128ES is 128M-bit Serial Peripheral Interface (SPI) Flash memory, and supports the Dual/Quad SPI: Serial Clock, Chip Select, Serial Data I/O0 (SI), I/O1 (SO), I/O2 (/WP), and I/O3(/HOLD). The Dual I/O data is transferred with speed of 216Mbits/s and the Quad I/O & Quad output data is transferred with speed of 432Mbits/s. The device uses a single low voltage power supply, ranging from 2.7 Volt to 3.6 Volt.

{0xCD6017, 1, FLASH_SIZE_8M, FLASH_LINE_MODE_TWO, 0, 2, 0x1F, 0x1F, 0x00, 0x0E, 0x00E, 0, 0, 0xA0, 0x01}, //th_25q64ha
#endif
#if defined(CONFIG_FLASH_QUAD_ENABLE)
{0xC86518, 2, FLASH_SIZE_16M, FLASH_LINE_MODE_FOUR, 14, 2, 0x1F, 0x1F, 0x00, 0x0E, 0x00E, 9, 1, 0xA0, 0x02}, //gd_25w128e
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmm it's not GD25Q128E?? as I know value is 0xc84018 can you please double check this?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My apologies, I should double-check the ID in case a mistake was made. It’s possible that it refers to a different series within the GD line.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,Kwon. I double-checked the manual against the code. It turns out there are two GD models: GD25WQ128E (0xC86518) and GD25Q128E (0xC84018). I’ll update the IDs in the code soon to make sure the mapping is accurate.

@uleemos
Copy link
Copy Markdown

uleemos commented Apr 16, 2026

The XT25F128W is already supported in the previous codebase. Below are the new Flash ID models for which we are now providing support:

BY25Q128ES
GT25Q128EZ
GD25W128E
TH25Q64HA
PY25Q64HA
W25Q128JV"

The bk7239n module needs to be compatible with multiple types of flash,
and the software needs to adapt to the new flash ID.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants