Skip to content

Commit

Permalink
boards: nordic: nrf54h20dk: Remove etr_buffer
Browse files Browse the repository at this point in the history
Move the etr_buffer definition into an overlay so the region is
unallocated by default.

Signed-off-by: Karsten Koenig <[email protected]>
  • Loading branch information
karstenkoenig committed Nov 14, 2024
1 parent 9d06a3a commit 35c9815
Show file tree
Hide file tree
Showing 4 changed files with 37 additions and 19 deletions.
15 changes: 0 additions & 15 deletions boards/nordic/nrf54h20dk/nrf54h20dk_nrf54h20-memory_map.dtsi
Original file line number Diff line number Diff line change
Expand Up @@ -47,21 +47,6 @@
};
};

etr_buf_ram0x_region: memory@2f0be000 {
compatible = "nordic,owned-memory";
reg = <0x2f0be000 DT_SIZE_K(4)>;
status = "disabled";
nordic,access = <NRF_OWNER_ID_APPLICATION NRF_PERM_RWS>;
#address-cells = <1>;
#size-cells = <1>;
ranges = <0x0 0x2f0be000 0x1000>;

/* TODO In future move this region to cpuapp_ram0x_region. */
etr_buffer: memory@0 {
reg = <0x0 DT_SIZE_K(4)>;
};
};

cpuapp_cpurad_ram0x_region: memory@2f0bf000 {
compatible = "nordic,owned-memory";
reg = <0x2f0bf000 DT_SIZE_K(4)>;
Expand Down
4 changes: 0 additions & 4 deletions boards/nordic/nrf54h20dk/nrf54h20dk_nrf54h20_cpuapp.dts
Original file line number Diff line number Diff line change
Expand Up @@ -112,10 +112,6 @@
status = "okay";
};

&etr_buf_ram0x_region {
status = "okay";
};

&ram21_region {
status = "okay";
};
Expand Down
18 changes: 18 additions & 0 deletions snippets/nordic-log-stm-dict/boards/nrf54h20_cpuapp.overlay
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,24 @@
* SPDX-License-Identifier: Apache-2.0
*/

/ {

Check warning on line 6 in snippets/nordic-log-stm-dict/boards/nrf54h20_cpuapp.overlay

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

LEADING_SPACE

snippets/nordic-log-stm-dict/boards/nrf54h20_cpuapp.overlay:6 please, no spaces at the start of a line
reserved-memory {
etr_buf_ram0x_region: memory@2f0be000 {
compatible = "nordic,owned-memory";
reg = <0x2f0be000 DT_SIZE_K(4)>;
status = "okay";
nordic,access = <NRF_OWNER_ID_APPLICATION NRF_PERM_RWS>;
#address-cells = <1>;
#size-cells = <1>;
ranges = <0x0 0x2f0be000 0x1000>;

etr_buffer: memory@0 {
reg = <0x0 DT_SIZE_K(4)>;
};
};
};
};

&tbm {
status = "okay";
};
Expand Down
19 changes: 19 additions & 0 deletions snippets/nordic-log-stm/boards/nrf54h20_cpuapp.overlay
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,25 @@
* SPDX-License-Identifier: Apache-2.0
*/

/ {

Check warning on line 6 in snippets/nordic-log-stm/boards/nrf54h20_cpuapp.overlay

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

LEADING_SPACE

snippets/nordic-log-stm/boards/nrf54h20_cpuapp.overlay:6 please, no spaces at the start of a line
reserved-memory {
etr_buf_ram0x_region: memory@2f0be000 {
compatible = "nordic,owned-memory";
reg = <0x2f0be000 DT_SIZE_K(4)>;
status = "okay";
nordic,access = <NRF_OWNER_ID_APPLICATION NRF_PERM_RWS>;
#address-cells = <1>;
#size-cells = <1>;
ranges = <0x0 0x2f0be000 0x1000>;

etr_buffer: memory@0 {
reg = <0x0 DT_SIZE_K(4)>;
};
};
};
};


&tbm {
status = "okay";
};
Expand Down

0 comments on commit 35c9815

Please sign in to comment.