CI: enable clang-tidy readability-duplicate-include (#26554)

Signed-off-by: kuralme <kuralme@protonmail.com>

.clang-tidy

@@ -167,7 +167,6 @@ Checks:          '*,
                   -readability-avoid-return-with-void-value,
                   -readability-avoid-unconditional-preprocessor-if,
                   -readability-delete-null-pointer,
-                  -readability-duplicate-include,
                   -readability-redundant-casting,
                   -readability-redundant-member-init,
                   -readability-reference-to-constructed-temporary,

platforms/common/work_queue/hrt_queue.c

@@ -48,7 +48,6 @@
 #include <stdio.h>
 #include <semaphore.h>
 #include <drivers/drv_hrt.h>
-#include <px4_platform_common/workqueue.h>
 #include <px4_platform_common/posix.h>
 #include "hrt_work.h"
 

platforms/posix/src/px4/common/px4_posix_impl.cpp

@@ -39,7 +39,6 @@
 
 #include <px4_platform_common/defines.h>
 #include <px4_platform_common/workqueue.h>
-#include <px4_platform_common/defines.h>
 #include <px4_platform_common/time.h>
 #include <stdint.h>
 #include <stdio.h>

platforms/posix/src/px4/common/tasks.cpp

@@ -46,7 +46,6 @@
 #include <signal.h>
 #include <fcntl.h>
 #include <sched.h>
-#include <unistd.h>
 #include <string.h>
 #include <pthread.h>
 #include <limits.h>

src/modules/navigator/geofence.cpp

@@ -51,8 +51,6 @@
 #include <systemlib/mavlink_log.h>
 #include <px4_platform_common/events.h>
 
-#include "navigator.h"
-
 static uint32_t crc32_for_fence_point(const mission_fence_point_s &fence_point, uint32_t prev_crc32)
 {
 	union {

src/modules/navigator/mission.cpp

@@ -57,7 +57,6 @@
 #include <uORB/uORB.h>
 #include <uORB/topics/mission.h>
 #include <uORB/topics/mission_result.h>
-#include <drivers/drv_hrt.h>
 #include <px4_platform_common/events.h>
 
 using namespace time_literals;

src/modules/simulation/simulator_mavlink/SimulatorMavlink.cpp

@@ -54,7 +54,6 @@
 #include <pthread.h>
 #include <sys/socket.h>
 #include <termios.h>
-#include <arpa/inet.h>
 
 #include <limits>