Skip to content

os/* : Adjust partition size for bk7239n#7267

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

os/* : Adjust partition size for bk7239n#7267
Poly-J wants to merge 1 commit intoSamsung:masterfrom
bekencorp:BK7239N_dev

Conversation

@Poly-J
Copy link
Copy Markdown
Contributor

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

-Adjust the partition size in the following way
ss partition: 532K –> 404K
remove the redundant usrconfig and system_data partitions
-Due to the modification of the security partition, the first address
of the non-secure partition is referenced, and ld needs to be adjusted accordingly.
-Added a new script for automatically obtaining insecure starting addresses.
-Due to security partition adjustments, it is necessary to synchronously update the TFM closed-source library

Copy link
Copy Markdown
Contributor

@jylee9613 jylee9613 left a comment

Choose a reason for hiding this comment

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

SS partition size meets with our requirement

import sys


DEFAULT_PRIMARY_TFM_S_BYTES = 0x30000
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.

The TFM flash region is fixed? 0x3000?
is it include resolved size also?
bk7239n will not change TFM size?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, TFM has already reserved space.

#define CONFIG_SECONDARY_ALL_VIRTUAL_CODE_SIZE 0x193000

#define CONFIG_COMMON_B_PHY_PARTITION_OFFSET 0x54c000
#define CONFIG_COMMON_B_PHY_PARTITION_OFFSET 0x52c000
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.

if we change common or app1 partition size, do we have to change here?

if this file is generated from build, Please remove this file in git

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No need, these are automatically generated by scripts.However, for the closed-source version, we uploaded this file for the convenience of subsequent debugging.

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.

I didn't get this point, For what purpose is this debugging intended?

When auto-generated files are uploaded to Git, it becomes more inconvenient because many developers manually ignore them.

If it is madantory for bk7239n, Could you please remove and exclude it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay, I will delete this file.

@Poly-J Poly-J force-pushed the BK7239N_dev branch 2 times, most recently from 96e92c3 to 87bd56f Compare April 16, 2026 09:30
-Adjust the partition size in the following way
  ss partition: 532K –> 404K
  remove the redundant usrconfig and system_data partitions
-Due to the modification of the security partition, the first address
 of the non-secure partition is referenced, and ld needs to be adjusted accordingly.
-Added a new script for automatically obtaining insecure starting addresses.
-Due to security partition adjustments, it is necessary to synchronously update
 the TFM closed-source library
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