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.

Sign up for GitHub

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

Jump to bottom

tests/drivers/watchdog/wdt_basic_api failed on mec15xxevb_assy6853 board. #20301

Closed
chen-png opened this issue Nov 4, 2019 · 8 comments · Fixed by #23534
Closed

tests/drivers/watchdog/wdt_basic_api failed on mec15xxevb_assy6853 board. #20301

chen-png opened this issue Nov 4, 2019 · 8 comments · Fixed by #23534
Assignees
@enjiamai @albertofloyd @dcpleung @mengxianglinx @scottwcpg
Labels
bug The issue is a bug, or the PR is fixing a bug platform: Microchip MEC Microchip MEC Platform priority: low Low impact/importance bug

Comments

@chen-png
Copy link
Collaborator

chen-png commented Nov 4, 2019 •
edited
Loading

Describe the bug
Running tests/drivers/watchdog/wdt_basic_api test case on mec15xxevb_assy6853 board, it will loop to report the message below.

To Reproduce
Steps to reproduce the behavior:

  1. west build -b mec15xxevb_assy6853 tests/drivers/watchdog/wdt_basic_api
  2. west flash
  3. See error

Screenshots or console output
Running test suite wdt_basic_test
starting test - test_wdt
Testcase: test_wdt_no_callback
Waiting to restart MCU

Running test suite wdt_basic_test
starting test - test_wdt
Testcase: test_wdt_no_callback
Waiting to restart MCU

Running test suite wdt_basic_test
starting test - test_wdt
Testcase: test_wdt_no_callback
Waiting to restart MCU

Running test suite wdt_basic_test
starting test - test_wdt
Testcase: test_wdt_no_callback
Waiting to restart MCU

Environment (please complete the following information):

  • OS: Fedora28
  • Toolchain: zephyr-sdk-0.11.1
  • Commit ID: 9c6a6c4
@chen-png chen-png added the bug The issue is a bug, or the PR is fixing a bug label Nov 4, 2019
@aescolar aescolar added the platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) label Nov 4, 2019
@aurel32
Copy link
Collaborator

aurel32 commented Nov 4, 2019

The problem is likely that the RAM is reset when the watchdog is triggered on the SAM70 MCU. IIRC there is/was a bug opened about that, but I can't find it anymore.

@chen-png
Copy link
Collaborator Author

chen-png commented Nov 5, 2019

The problem is likely that the RAM is reset when the watchdog is triggered on the SAM70 MCU. IIRC there is/was a bug opened about that, but I can't find it anymore.

do you mean the #13468 and #14968, it seems the same issue.

@ioannisg
Copy link
Member

ioannisg commented Nov 5, 2019

Looks like duplicate; might want to close it. It's anyways, a LOW priority bug.

@ioannisg ioannisg added the priority: low Low impact/importance bug label Nov 5, 2019
@ashaannajoy
Copy link

This issue is observed on microchip mec15xxevb_assy6853 also

@jocelyn-li
Copy link
Collaborator

@albertofloyd is this issue caused by limited RAM resource on microchip mec15xxevb_assy6853?

@albertofloyd
Copy link
Collaborator

albertofloyd commented Feb 27, 2020 •
edited
Loading

@jocelyn-li there is no limited RAM resource.
In other reset conditions the bootloader fetches the code again from SPI flash and loads into RAM, this is most likely what is happening here.
Need confirmation from @scottwcpg to see RAM is preserved during WDT_EVENT since that's what the test code is expecting. If RAM is not preserved in this case the test have to skipped.

@chen-png
Copy link
Collaborator Author

chen-png commented Feb 28, 2020 •
edited
Loading

this test has been skipped on sam_e70_xplained.

@chen-png chen-png changed the title tests/drivers/watchdog/wdt_basic_api failed on sam_e70_xplained board. tests/drivers/watchdog/wdt_basic_api failed on mec15xxevb_assy6853 board. Feb 28, 2020
@chen-png chen-png mentioned this issue Feb 28, 2020
Closed
@chen-png chen-png reopened this Feb 28, 2020
@scottwcpg
Copy link
Collaborator

Hi,
MEC150x Boot-ROM clears all SRAM per security requirements of its secure boot loader feature.
You could write a test flag into VBAT memory which retains state if the VTR or VBAT power rails stay up.

dcpleung added a commit to dcpleung/zephyr that referenced this issue Mar 17, 2020
@dcpleung
tests: wdt_basic_api: make mec15xxevb_assy6853 build only
2bfb355 
The bootloader on mec15xxevb_assy6853 clears the memory on reboot,
which would fail the wdt_basic_api test. Instead of simply exclude
the platform, add a build only test for this board so the watchdog
is being built to avoid build breakage.

Fixes zephyrproject-rtos#20301

Signed-off-by: Daniel Leung <daniel.leung@intel.com>
@dcpleung dcpleung added platform: Microchip MEC Microchip MEC Platform and removed platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) labels Mar 17, 2020
@dcpleung dcpleung mentioned this issue Mar 17, 2020
Merged
nashif pushed a commit that referenced this issue Mar 17, 2020
@dcpleung @nashif
tests: wdt_basic_api: make mec15xxevb_assy6853 build only
4b19273 
The bootloader on mec15xxevb_assy6853 clears the memory on reboot,
which would fail the wdt_basic_api test. Instead of simply exclude
the platform, add a build only test for this board so the watchdog
is being built to avoid build breakage.

Fixes #20301

Signed-off-by: Daniel Leung <daniel.leung@intel.com>
hakehuang pushed a commit to hakehuang/zephyr that referenced this issue Mar 18, 2020
@dcpleung @hakehuang
tests: wdt_basic_api: make mec15xxevb_assy6853 build only
1941cdd 
The bootloader on mec15xxevb_assy6853 clears the memory on reboot,
which would fail the wdt_basic_api test. Instead of simply exclude
the platform, add a build only test for this board so the watchdog
is being built to avoid build breakage.

Fixes zephyrproject-rtos#20301

Signed-off-by: Daniel Leung <daniel.leung@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@enjiamai enjiamai

@albertofloyd albertofloyd

@dcpleung dcpleung

@mengxianglinx mengxianglinx

@scottwcpg scottwcpg

Labels
bug The issue is a bug, or the PR is fixing a bug platform: Microchip MEC Microchip MEC Platform priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests: wdt_basic_api: make mec15xxevb_assy6853 build only dcpleung/zephyr