build/configs/bk7239n: Add signature and encryption to the resource#7258
Merged
sunghan-chang merged 1 commit intoSamsung:masterfrom Apr 14, 2026
Merged
build/configs/bk7239n: Add signature and encryption to the resource#7258sunghan-chang merged 1 commit intoSamsung:masterfrom
sunghan-chang merged 1 commit intoSamsung:masterfrom
Conversation
jylee9613
reviewed
Apr 11, 2026
Comment on lines
+1070
to
+1072
| CONFIG_FLASH_PART_SIZE="92,8,8,532, 1844,3400,1024,768,1844,3400,1024,768,512,1012,100,8,4,4,24,8," | ||
| CONFIG_FLASH_PART_TYPE="none,none,none,none,kernel,bin,bin,resource,kernel,bin,bin,resource,smartfs,none,none,none,none,none,none,bootparam," | ||
| CONFIG_FLASH_PART_NAME="bl1,sys_its,sys_ps,ss,kernel,common,app1,resource,kernel,common,app1,resource,userfs,reserved,usrconfig,system_data,rf,net,easyflash,bootparam," |
Contributor
There was a problem hiding this comment.
Would you please apply this change to other defconfigs too?
Contributor
There was a problem hiding this comment.
Oh other defconfigs disables CONFIG_RESOURCE_FS.. Please ignore it.
| CONFIG_FLASH_PART_SIZE="92,8,8,532, 1844,3400,1024,1844,3400,1024,512,768,768,1012,100,8,4,4,24,8," | ||
| CONFIG_FLASH_PART_TYPE="none,none,none,none,kernel,bin,bin,kernel,bin,bin,smartfs,resource,resource,none,none,none,none,none,none,bootparam," | ||
| CONFIG_FLASH_PART_NAME="bl1,sys_its,sys_ps,ss,kernel,common,app1,kernel,common,app1,userfs,resource,resource,reserved,usrconfig,system_data,rf,net,easyflash,bootparam," | ||
| CONFIG_FLASH_PART_SIZE="92,8,8,532, 1844,3400,1024,768,1844,3400,1024,768,512,1012,100,8,4,4,24,8," |
Contributor
There was a problem hiding this comment.
Could you check my query below?
- reason of ss size 532
- what is usrconfig for?
- what is easyflash?
- There is the rf partition, then what is net for?
- sys_its, sys_ps, usrconfig, system_data, rf, net, easyflash all are for beken. Right?
Contributor
There was a problem hiding this comment.
Could you check my query below?
- reason of ss size 532
- what is usrconfig for?
- what is easyflash?
- There is the rf partition, then what is net for?
- sys_its, sys_ps, usrconfig, system_data, rf, net, easyflash all are for beken. Right?
Hello @sunghan-chang
These partition details were discussed in the email.
- The sys_its and sys_ps partitions are needed for security
- The usrconfig, system_data partitions are not used; they were only included during the early porting stage as references to other vendors’ configurations,thought it was a partition needed by Samsung
- The rf partition is used to store calibration data, and the net partition is used to store information such as MAC addresses. these two partitions are only used during the R&D phase and can be removed in the final mass-production stage.
- The easyflash partition stores Bluetooth pairing information, whitelists, and related data, so it needs to be retained
| CONFIG_FLASH_PART_SIZE="92,8,8,532, 1844,3400,1024,1844,3400,1024,512,768,768,1012,100,8,4,4,24,8," | ||
| CONFIG_FLASH_PART_TYPE="none,none,none,none,kernel,bin,bin,kernel,bin,bin,smartfs,resource,resource,none,none,none,none,none,none,bootparam," | ||
| CONFIG_FLASH_PART_NAME="bl1,sys_its,sys_ps,ss,kernel,common,app1,kernel,common,app1,userfs,resource,resource,reserved,usrconfig,system_data,rf,net,easyflash,bootparam," | ||
| CONFIG_FLASH_PART_SIZE="92,8,8,532, 1844,3400,1024,768,1844,3400,1024,768,512,1012,100,8,4,4,24,8," |
Contributor
There was a problem hiding this comment.
let's remove a space.
,532, 1844, --> ,532,1844,
Contributor
Author
There was a problem hiding this comment.
Thank you for your feedback, I have modified it
1.Adjust the resource partition 2.Add signature and encryption to the resource Signed-off-by: xiaotao.yuan <xiaotao.yuan@aclsemi.com>
seokhun-eom24
approved these changes
Apr 14, 2026
sunghan-chang
approved these changes
Apr 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1.Adjust the resource partition
2. Add signature and encryption to the resource