Skip to content

iq-x7181-evk: rename _a/_b partitions to main/backup naming#122

Merged
lumag merged 1 commit into
qualcomm-linux:mainfrom
xueqnie:hamoa-rename-backup-partitions
Jun 6, 2026
Merged

iq-x7181-evk: rename _a/_b partitions to main/backup naming#122
lumag merged 1 commit into
qualcomm-linux:mainfrom
xueqnie:hamoa-rename-backup-partitions

Conversation

@xueqnie

@xueqnie xueqnie commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

The dtb, uefi_dtb and uefisecapp partition pairs use a main/backup model — the primary partition is always active and the backup holds a rollback copy. Rename them from the A/B slot convention (_a/_b) to the main/backup convention used by all other partition pairs on this platform (_BACKUP suffix).

Partition GUIDs are unchanged so firmware that locates partitions by GUID is unaffected by this rename.

@lumag

lumag commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

From: xueqnie xueqnie@qti.qualcomm.com

Please fix your Git setup.

@xueqnie xueqnie force-pushed the hamoa-rename-backup-partitions branch from 0567bfe to 7ae8933 Compare June 3, 2026 05:39
@xueqnie

xueqnie commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

From: xueqnie xueqnie@qti.qualcomm.com

Please fix your Git setup.

done

@lumag lumag requested a review from shoudil June 3, 2026 05:47
@lumag

lumag commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

@shoudil could you please review?

shoudil
shoudil reviewed Jun 3, 2026
View reviewed changes
Comment thread platforms/iq-x7181-evk/spinor/partitions.conf Outdated
@shoudil

shoudil commented Jun 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

@shoudil could you please review?

Actually I am not quite sure if the partition name get referred/hardcoded in boot images or not. I will check once with boot team. Meanwhile, @xueqnie let's verify once the capsule update E2E workflow with this partition change.

@xueqnie xueqnie force-pushed the hamoa-rename-backup-partitions branch from 7ae8933 to d81f437 Compare June 3, 2026 07:28
@xueqnie
iq-x7181-evk: rename _a/_b partitions to main/backup naming
fd2c49e 
The dtb, uefi_dtb and uefisecapp partition pairs use a main/backup
model — the primary partition is always active and the backup holds
a rollback copy. Rename them from the A/B slot convention (_a/_b)
to the main/backup convention used by all other partition pairs on
this platform (_BACKUP suffix).

Partition GUIDs are unchanged so firmware that locates partitions by
GUID is unaffected by this rename.

Signed-off-by: Xueqian Nie <xueqnie@qti.qualcomm.com>
@xueqnie xueqnie force-pushed the hamoa-rename-backup-partitions branch from d81f437 to fd2c49e Compare June 4, 2026 06:36
@xueqnie

xueqnie commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

@shoudil could you please review?

Actually I am not quite sure if the partition name get referred/hardcoded in boot images or not. I will check once with boot team. Meanwhile, @xueqnie let's verify once the capsule update E2E workflow with this partition change.

Boot validation: Reflashed with the new GPT (renamed partitions). System boots normally — UEFI logs confirm partitions are located by GUID (GetPartHandleFromDiskGuid), not by name, so the rename is transparent to firmware.

@shoudil

shoudil commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

@shoudil could you please review?

Actually I am not quite sure if the partition name get referred/hardcoded in boot images or not. I will check once with boot team. Meanwhile, @xueqnie let's verify once the capsule update E2E workflow with this partition change.

Boot validation: Reflashed with the new GPT (renamed partitions). System boots normally — UEFI logs confirm partitions are located by GUID (GetPartHandleFromDiskGuid), not by name, so the rename is transparent to firmware.

Yes, boot team also confirmed only GUID used in UEFI.

@lumag

lumag commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

@shoudil thanks for the confirmations!

@lumag lumag enabled auto-merge June 6, 2026 06:55
@lumag lumag merged commit e5cc153 into qualcomm-linux:main Jun 6, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lumag lumag lumag approved these changes

@vkraleti vkraleti Awaiting requested review from vkraleti vkraleti is a code owner

@ndechesne ndechesne Awaiting requested review from ndechesne ndechesne is a code owner

+1 more reviewer

@shoudil shoudil shoudil left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@xueqnie @lumag @shoudil