Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

yosemite4n: u-boot: Add patch to set memory test result in environment #244

Open
wants to merge 1 commit into
base: helium
Choose a base branch
from

Conversation

linkunfa
Copy link

Description

Add u-boot patch to set memory test result in environment.

Motivation

Add patch to set memory test result in environment after doing mtest in u-boot so that user can check the result in user space. This patch is needed by Wiwynn factory test.

Test Plan

Set mtest command and reboot to check the memory test result.

root@bmc:# fw_setenv preboot 'mtest 0x80000000 0x9A500000 0 1'
root@bmc:
# reboot
root@bmc:~# fw_printenv memtest_result
memtest_result=pass

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Summary:
Add patch to set memory test result in environment after doing mtest
in u-boot so that user can check the result in user space. This patch
is needed by Wiwynn factory test.

Test Plan:
Set mtest command and reboot to check the memory test result.

root@bmc:~# fw_setenv preboot 'mtest 0x80000000 0x9A500000 0 1'
root@bmc:~# reboot
root@bmc:~# fw_printenv memtest_result
memtest_result=pass

Signed-off-by: Marvin Lin <[email protected]>
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants