Skip to content

Conversation

@johnnylinwiwynn
Copy link
Contributor

The vpd.sh can do your own customization such as setting VPD variables during build. For example, project folder is osf-builder/projects/deltalake, you can place vpd.sh under deltalake with a vpd.sh script like: #!/bin/bash
VPD="../../tools/vpd"
ROM="build/deltalake/coreboot/build/coreboot.rom"

#Or don't add '-O' that reformats the RO_VPD created by osf-builder/Makefile.inc $VPD -f $ROM -O -i RO_VPD -s fsp_log_enable=0
$VPD -f $ROM -i RO_VPD -s test_key=123
$VPD -f $ROM -O -i RW_VPD

Signed-off-by: Johnny Lin johnny_lin@wiwynn.com

The vpd.sh can do your own customization such as setting VPD variables
during build. For example, project folder is osf-builder/projects/deltalake,
you can place vpd.sh under deltalake with a vpd.sh script like:
#!/bin/bash
VPD="../../tools/vpd"
ROM="build/deltalake/coreboot/build/coreboot.rom"

#Or don't add '-O' that reformats the RO_VPD created by osf-builder/Makefile.inc
$VPD -f $ROM -O -i RO_VPD -s fsp_log_enable=0
$VPD -f $ROM -i RO_VPD -s test_key=123
$VPD -f $ROM -O -i RW_VPD

Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
@johnnylinwiwynn
Copy link
Contributor Author

johnnylinwiwynn commented Nov 24, 2022

@jonzhang-fb @ed-sharma

Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
@jonzhang-fb
Copy link
Contributor

Great to have this. Two comments:
a. This adds a dependency for the build environment. Please mention it in the readme.
b. It would be good to add a sample vpd.sh.

Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
@johnnylinwiwynn
Copy link
Contributor Author

Great to have this. Two comments: a. This adds a dependency for the build environment. Please mention it in the readme. b. It would be good to add a sample vpd.sh.

Done.

@jonzhang-fb
Copy link
Contributor

LGTM

@rminnich rminnich merged commit e1e74a9 into linuxboot:main Dec 7, 2022
@johnnylinwiwynn johnnylinwiwynn deleted the vpd branch January 12, 2023 03:04
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