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

boards/frdm-k64f: hack disable periph_hwrng as it crashes #11454

Merged
merged 1 commit into from
Apr 29, 2019

Conversation

cladmi
Copy link
Contributor

@cladmi cladmi commented Apr 26, 2019

Contribution description

Disable the frdm-k64f as is it misconfigured since 925a908

Use the same disable as in frdm-kw41z.

It fixes breakage with all firmwares using rng.

This is not the fix to make hwrng work but only disable it until fixed.

Testing procedure

tests/periph_hwrng

Run tests/periph_hwrng crashes with the frdm-k64f now it is said to be unsupported (still crashes).

BOARD=frdm-k64f make -C tests/periph_hwrng/
make: Entering directory '/home/harter/work/git/RIOT/tests/periph_hwrng'
There are unsatisfied feature requirements: periph_hwrng


EXPECT ERRORS!

Firmwares using rng:

examples/gnrc_networking for example, does not crash anymore as it does not use hwrng anymore.

examples/gnrc_networking for frdm-k64f with master:

Type '/exit' to exit.
2019-04-26 12:17:19,136 - INFO # 00
2019-04-26 12:17:19,138 - INFO #  HFSR: 0x40000000
2019-04-26 12:17:19,138 - INFO #  DFSR: 0x00000000
2019-04-26 12:17:19,139 - INFO #  AFSR: 0x00000000
2019-04-26 12:17:19,140 - INFO #  BFAR: 0x40029004
2019-04-26 12:17:19,141 - INFO # Misc
2019-04-26 12:17:19,141 - INFO # EXC_RET: 0xfffffffd
2019-04-26 12:17:19,143 - INFO # Attempting to reconstruct state for debugging...
2019-04-26 12:17:19,144 - INFO # In GDB:
2019-04-26 12:17:19,144 - INFO #   set $pc=0x7fc2
2019-04-26 12:17:19,145 - INFO #   frame 0
2019-04-26 12:17:19,145 - INFO #   bt
2019-04-26 12:17:19,146 - INFO #
2019-04-26 12:17:19,147 - INFO # ISR stack overflowed by at least 16 bytes.
2019-04-26 12:17:19,148 - INFO #
2019-04-26 12:17:19,149 - INFO # Context before hardfault:
2019-04-26 12:17:19,149 - INFO #    r0: 0x1fff0c04
2019-04-26 12:17:19,150 - INFO #    r1: 0x00000004
2019-04-26 12:17:19,151 - INFO #    r2: 0x00000002
2019-04-26 12:17:19,151 - INFO #    r3: 0x40029000
2019-04-26 12:17:19,152 - INFO #   r12: 0x00000000
2019-04-26 12:17:19,153 - INFO #    lr: 0x00008019
2019-04-26 12:17:19,154 - INFO #    pc: 0x00007fc2
2019-04-26 12:17:19,154 - INFO #   psr: 0x01000200
2019-04-26 12:17:19,155 - INFO #
2019-04-26 12:17:19,155 - INFO # FSR/FAR:
2019-04-26 12:17:19,156 - INFO #  CFSR: 0x00008600
2019-04-26 12:17:19,156 - INFO #  HFSR: 0x40000000

Issues/PRs references

Issue: #11447
It does not closes the issue as it is not a fix to make the 'hwrng' but disable it until fixed.

The test automation for testing tests/periph_hwrng #11448

Use the same disable as in `frdm-kw41z`.
@cladmi cladmi added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch labels Apr 26, 2019
@cladmi cladmi added this to the Release 2019.04 milestone Apr 26, 2019
@cladmi
Copy link
Contributor Author

cladmi commented Apr 26, 2019

This is fixing all the following tests:

--- 2019.04-RC1/frdm-k64f/failuresummary.md     2019-04-24 19:17:40.000000000 +0200
+++ 2019.04-RC1_periph_hwrng_fix/frdm-k64f/failuresummary.md    2019-04-26 16:50:14.000000000 +0200
@@ -1,29 +1,8 @@
 Failures during test:
 - [tests/gnrc_ipv6_ext](tests/gnrc_ipv6_ext/test.failed)
-- [tests/gnrc_ipv6_nib](tests/gnrc_ipv6_nib/test.failed)
-- [tests/gnrc_ipv6_nib_6ln](tests/gnrc_ipv6_nib_6ln/test.failed)
-- [tests/gnrc_ndp](tests/gnrc_ndp/test.failed)
-- [tests/gnrc_netif](tests/gnrc_netif/test.failed)
 - [tests/gnrc_rpl_srh](tests/gnrc_rpl_srh/test.failed)
-- [tests/gnrc_sixlowpan](tests/gnrc_sixlowpan/test.failed)
 - [tests/gnrc_sock_dns](tests/gnrc_sock_dns/test.failed)
-- [tests/gnrc_sock_ip](tests/gnrc_sock_ip/test.failed)
-- [tests/gnrc_sock_udp](tests/gnrc_sock_udp/test.failed)
-- [tests/nhdp](tests/nhdp/test.failed)
 - [tests/periph_rtt](tests/periph_rtt/test.failed)
 - [tests/periph_timer](tests/periph_timer/test.failed)
-- [tests/pkg_c25519](tests/pkg_c25519/test.failed)
 - [tests/pkg_fatfs_vfs](tests/pkg_fatfs_vfs/test.failed)
-- [tests/pkg_hacl](tests/pkg_hacl/test.failed)
-- [tests/pkg_libcoap](tests/pkg_libcoap/test.failed)
-- [tests/pkg_libcose](tests/pkg_libcose/test.failed)
-- [tests/pkg_libhydrogen](tests/pkg_libhydrogen/test.failed)
-- [tests/pkg_micro-ecc-with-hwrng](tests/pkg_micro-ecc-with-hwrng/test.failed)
-- [tests/pkg_monocypher](tests/pkg_monocypher/test.failed)
-- [tests/pkg_qdsa](tests/pkg_qdsa/test.failed)
-- [tests/pkg_tweetnacl](tests/pkg_tweetnacl/test.failed)
 - [tests/ps_schedstatistics](tests/ps_schedstatistics/test.failed)
-- [tests/pthread_barrier](tests/pthread_barrier/test.failed)
-- [tests/pthread_rwlock](tests/pthread_rwlock/test.failed)
-- [tests/rng](tests/rng/test.failed)
-- [tests/trickle](tests/trickle/test.failed)

@cladmi cladmi requested a review from jcarrano April 26, 2019 17:10
@fjmolinas
Copy link
Contributor

@cladmi looks good but I don't have kw64f, only kw41z so I can't really test.

Copy link
Contributor

@jcarrano jcarrano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on a frdm-k64f. It solves the crash.

@jcarrano jcarrano merged commit d4edbea into RIOT-OS:master Apr 29, 2019
@cladmi
Copy link
Contributor Author

cladmi commented Apr 29, 2019

Thanks for the review.

@cladmi cladmi deleted the pr/frdm-k64f/hack/hwrng branch April 29, 2019 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants