Skip to content

platforms/shikra-evk: remove keymint.mbn and tools.fv entries#125

Merged
lumag merged 1 commit into
qualcomm-linux:mainfrom
vivpuar:shikra_conf
Jun 12, 2026
Merged

platforms/shikra-evk: remove keymint.mbn and tools.fv entries#125
lumag merged 1 commit into
qualcomm-linux:mainfrom
vivpuar:shikra_conf

Conversation

@vivpuar

@vivpuar vivpuar commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

keymint.mbn and tools.fv are for internal usage and not required to be flashed on Shikra device.

@vivpuar vivpuar marked this pull request as draft June 11, 2026 10:19
@vivpuar

vivpuar commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

There is some issue seen recently, will make it ready once resolved

@vivpuar vivpuar marked this pull request as ready for review June 11, 2026 14:33

@lumag lumag left a comment

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.

Should they be zeroed instead?

@vivpuar

vivpuar commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Should they be zeroed instead?

You mean remove both the partition completely from file?

@lumag

lumag commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Should they be zeroed instead?

You mean remove both the partition completely from file?

No, I mean filling them with zeroes using the zeroes*bin files.

keymint.mbn and tools.fv are for internal usage and not required
to be flashed on Shikra device, filling them with zeros.

Signed-off-by: Vivek Puar <vpuar@qti.qualcomm.com>
@vivpuar

vivpuar commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Should they be zeroed instead?

You mean remove both the partition completely from file?

No, I mean filling them with zeroes using the zeroes*bin files.

Updated

@lumag lumag left a comment

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.

Does the same apply to other platforms too?

@vivpuar

vivpuar commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Does the same apply to other platforms too?

I need to check for other platforms with target team, will confirm and raise PR for all if it's applicable.

@lumag lumag enabled auto-merge June 12, 2026 08:46
@lumag lumag merged commit f457022 into qualcomm-linux:main Jun 12, 2026
3 checks passed
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.

2 participants