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

Setting default BT_VER to 9. #397

Merged
merged 3 commits into from
Feb 9, 2023
Merged

Setting default BT_VER to 9. #397

merged 3 commits into from
Feb 9, 2023

Conversation

kevin-gillespie
Copy link
Contributor

Nordic device getting hard fault with HCI UART traffic. Works okay with limited traffic, but hard fault occurs with heavy traffic in automated tests.

0x00028982 in SystemDefaultHandler () at ../../../../platform/targets/nordic/build/startup_gcc_nrf52840.c:313
313   while (forever);
(gdb) bt
#0  0x00028982 in SystemDefaultHandler () at ../../../../platform/targets/nordic/build/startup_gcc_nrf52840.c:313
#1  <signal handler called>
#2  0x000227f8 in LctrRxIso () at ../../../../controller/sources/ble/lctr/lctr_main_iso.c:676
#3  0x00023de8 in LlRecvIsoData () at ../../../../controller/sources/ble/ll/ll_main_iso.c:143
#4  0x00023c6e in lhciRecvIso () at ../../../../controller/sources/ble/lhci/lhci_evt_iso.c:131
#5  0x00017c20 in lhciService (pType=0x2003ff59 "", pLen=0x2003ff5a, pBuf=0x2003ff5c) at ../../../../controller/sources/ble/lhci/lhci_main.c:288
#6  0x0001c2fe in ChciTrHandler (event=<optimized out>, pMsg=<optimized out>) at ../../../../controller/sources/common/chci/chci_tr.c:542
#7  ChciTrHandler (event=<optimized out>, pMsg=<optimized out>) at ../../../../controller/sources/common/chci/chci_tr.c:526
#8  0x00017a4e in LhciHandler (event=<optimized out>, pMsg=<optimized out>) at ../../../../controller/sources/ble/lhci/lhci_main.c:145
#9  0x000277a6 in wsfOsDispatcher () at ../../../../wsf/sources/targets/baremetal/wsf_os.c:328
#10 0x000277de in WsfOsEnterMainLoop () at ../../../../wsf/sources/targets/baremetal/wsf_os.c:398
#11 0x000004e0 in main () at ../../../../controller/build/ble5-ctr/main.c:184

Nordic device getting hard fault with HCI UART traffic.
@yc-adi
Copy link
Contributor

yc-adi commented Feb 8, 2023

In the future, this kind of changes should trigger a simple PER CI test after we make the RF-PHY-closed public.

@kevin-gillespie
Copy link
Contributor Author

Well maybe this isn't the golden ticket. I'm seeing a similar problem occur when I restart the test.

@kevin-gillespie kevin-gillespie added the WIP work in progress label Feb 8, 2023
Copy link
Contributor

@yc-adi yc-adi left a comment

Choose a reason for hiding this comment

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

Will the SRAM adjustments change the performance?

@kevin-gillespie
Copy link
Contributor Author

Yes, the problem I was seeing was fixed with this change.

@kevin-gillespie kevin-gillespie merged commit 25155da into main Feb 9, 2023
@kevin-gillespie kevin-gillespie deleted the nordic-ctr-ver branch February 9, 2023 22:35
tim-magnusen added a commit to tim-magnusen/msdk that referenced this pull request Feb 16, 2023
…to dev-dc_delay

* 'main' of https://github.com/Analog-Devices-MSDK/msdk: (27 commits)
  Add SBT vs non-SBT Linkerfile Selection to MAX32672 LP Example (analogdevicesinc#401)
  Fixed script path error and added ME18 WLP PER test. (analogdevicesinc#418)
  Adding BTLE_LDO_TRIM register. (analogdevicesinc#402)
  Add Paths ignore to BLE test (analogdevicesinc#415)
  Not running on draft PR. (analogdevicesinc#414)
  Added a simple PER CI test. (analogdevicesinc#407)
  CAN Library Touch-Up (analogdevicesinc#400)
  [MSDK-976] Removed PUF references from ME18. (analogdevicesinc#318)
  Porting EEPROM_Emulator example (analogdevicesinc#379)
  MAX32520: Fix I2C slave address register offset. (analogdevicesinc#384)
  Add AI87 KWS20 Demo (analogdevicesinc#385)
  MAX32672EVKIT display example update (analogdevicesinc#392)
  Calibration and Initialization tools for CI (analogdevicesinc#398)
  Setting default BT_VER to 9. (analogdevicesinc#397)
  Added mini-circuits rf switch control. (analogdevicesinc#399)
  Updating FreeRTOS (v10.2.0) and FreeRTOS-Plus libraries to latest FreeRTOS (v10.5.1) release. (analogdevicesinc#361)
  Removed some unneeded type  hints to lower Python requirements (analogdevicesinc#387)
  Updated board files for ME18 (analogdevicesinc#394)
  Adding enumeration for btleldoctrl register. (analogdevicesinc#390)
  [MSDK-1042]: Resolving hardfault in MAX32672 LP example. (analogdevicesinc#391)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants