Skip to content

Commit

Permalink
samples: benchmarks: coremark: unify format in different logging modes
Browse files Browse the repository at this point in the history
Applied the same logging format in the standard logging mode as is
currently used in the multi-domain logging mode in the CoreMark sample.

Ref: NCSDK-27517

Signed-off-by: Kamil Piszczek <Kamil.Piszczek@nordicsemi.no>
  • Loading branch information
kapi-no committed Nov 13, 2024
1 parent 297385a commit ba59f44
Show file tree
Hide file tree
Showing 8 changed files with 21 additions and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -381,7 +381,10 @@ Other samples

* :ref:`coremark_sample` sample:

* Updated configuration for the :ref:`zephyr:nrf54h20dk_nrf54h20` board to support multi-domain logging using the ARM Coresight STM.
* Updated:

* Configuration for the :ref:`zephyr:nrf54h20dk_nrf54h20` board to support multi-domain logging using the ARM Coresight STM.
* The logging format in the standard logging mode to align it with the format used in the multi-domain logging mode.

Drivers
=======
Expand Down
22 changes: 11 additions & 11 deletions samples/benchmarks/coremark/README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -232,21 +232,21 @@ After programming the sample to your development kit, complete the following ste

.. code-block:: console
*** Booting nRF Connect SDK v2.8.99-bd215d43d184 ***
*** Booting nRF Connect SDK v2.8.99-bd4a30a3a758 ***
*** Using Zephyr OS v3.7.99-02718211f9a9 ***
I: Standard logging mode
[00:00:00.261,383] <inf> app: Standard logging mode
I: CoreMark sample for nrf52840dk/nrf52840
I: Press Push button switch 0 to start the test ...
[00:00:00.266,967] <inf> app: CoreMark sample for nrf52840dk/nrf52840
[00:00:00.274,139] <inf> app: Press Push button switch 0 to start the test ...
I: Push button switch 0 pressed!
I: CoreMark started! CPU FREQ: 64000000 Hz, threads: 1, data size: 2000; iterations: 2000
[00:00:01.267,608] <inf> app: Push button switch 0 pressed!
[00:00:01.273,864] <inf> app: CoreMark started! CPU FREQ: 64000000 Hz, threads: 1, data size: 2000; iterations: 2000
2K performance run parameters for coremark.
CoreMark Size : 666
Total ticks : 399362
Total time (secs): 12.187000
Iterations/Sec : 164.109297
Total ticks : 401215
Total time (secs): 12.244000
Iterations/Sec : 163.345312
Iterations : 2000
Compiler version : GCC12.2.0
Compiler flags : -O3 + see compiler flags added by Zephyr
Expand All @@ -257,8 +257,8 @@ After programming the sample to your development kit, complete the following ste
[0]crcstate : 0x8e3a
[0]crcfinal : 0x4983
Correct operation validated. See README.md for run and reporting rules.
CoreMark 1.0 : 164.109297 / GCC12.2.0 -O3 + see compiler flags added by Zephyr / STACK
I: CoreMark finished! Press Push button switch 0 to restart ...
CoreMark 1.0 : 163.345312 / GCC12.2.0 -O3 + see compiler flags added by Zephyr / STACK
[00:00:13.597,778] <inf> app: CoreMark finished! Press Push button switch 0 to restart ...
.. group-tab:: Multi-domain logging

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
#
# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause

CONFIG_LOG_MODE_MINIMAL=y
CONFIG_LOG_MODE_IMMEDIATE=y
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
#
# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause

CONFIG_LOG_MODE_MINIMAL=y
CONFIG_LOG_MODE_IMMEDIATE=y
2 changes: 1 addition & 1 deletion samples/benchmarks/coremark/boards/nrf52dk_nrf52832.conf
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
#
# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause

CONFIG_LOG_MODE_MINIMAL=y
CONFIG_LOG_MODE_IMMEDIATE=y
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@

CONFIG_SOC_NRF53_CPUNET_ENABLE=y

CONFIG_LOG_MODE_MINIMAL=y
CONFIG_LOG_MODE_IMMEDIATE=y
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
#
# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause

CONFIG_LOG_MODE_MINIMAL=y
CONFIG_LOG_MODE_IMMEDIATE=y
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@

CONFIG_COREMARK_ITERATIONS=4000

CONFIG_LOG_MODE_MINIMAL=y
CONFIG_LOG_MODE_IMMEDIATE=y

0 comments on commit ba59f44

Please sign in to comment.