ext: debug: segger: Fix SEGGER header inclusion (421a44f5) · Commits · Wenxi XU / Segger

Fix issue caused by 3fc497ac9af7119e0e56d4b7fe2e47d6ae12cd8a This change removes `rtt` and `systemview` from header includes as these are already placed in the path. Also `SEGGER_SYSVIEW_ConfDefaults.h` header included from `SEGGER_SYSVIEW_Int.h` is placed higher to make sure `INLINE` definition is properly visible. Signed-off-by:

Pawel Dunaj <pawel.dunaj@nordicsemi.no>

systemview/SEGGER_SYSVIEW.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -146,7 +146,7 @@ Additional information:
		*/

		#include "SEGGER_SYSVIEW_Int.h"
		#include "rtt/SEGGER_RTT.h"
		#include "SEGGER_RTT.h"
		#include <string.h>
		#include <stdlib.h>
		#include <stdarg.h>

systemview/SEGGER_SYSVIEW_ConfDefaults.h

+1 −1

Original line number	Diff line number	Diff line
		@@ -73,7 +73,7 @@ Revision: $Rev: 3734 $
		*/

		#include "SEGGER_SYSVIEW_Conf.h"
		#include "rtt/SEGGER_RTT_Conf.h"
		#include "SEGGER_RTT_Conf.h"

		#ifdef __cplusplus
		extern "C" {

systemview/SEGGER_SYSVIEW_Int.h

+1 −1

Original line number	Diff line number	Diff line
		@@ -71,9 +71,9 @@ Revision: $Rev: 5626 $
		**********************************************************************
		*/

		#include "SEGGER_SYSVIEW_ConfDefaults.h"
		#include "SEGGER_SYSVIEW.h"
		#include "SEGGER_SYSVIEW_Conf.h"
		#include "SEGGER_SYSVIEW_ConfDefaults.h"

		#ifdef __cplusplus
		extern "C" {

Admin message