[libcamera-devel] [PATCH v3 1/3] src: Remove all unused sys/mman.h inclusions

Laurent Pinchart laurent.pinchart at ideasonboard.com
Mon Aug 9 16:15:31 CEST 2021


Hi Kieran,

Thank you for the patch.

On Mon, Aug 09, 2021 at 02:29:27PM +0100, Kieran Bingham wrote:
> Remove leftover inclusions of the sys/mman header file.
> 
> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  src/android/camera_device.cpp                      | 1 -
>  src/libcamera/ipa_module.cpp                       | 1 -
>  src/libcamera/pipeline/raspberrypi/raspberrypi.cpp | 1 -
>  src/libcamera/v4l2_videodevice.cpp                 | 1 -
>  src/v4l2/v4l2_camera_proxy.h                       | 1 -
>  src/v4l2/v4l2_compat_manager.h                     | 1 -
>  6 files changed, 6 deletions(-)
> 
> diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp
> index 3266c495d0ca..59b37bc864b5 100644
> --- a/src/android/camera_device.cpp
> +++ b/src/android/camera_device.cpp
> @@ -12,7 +12,6 @@
>  
>  #include <algorithm>
>  #include <fstream>
> -#include <sys/mman.h>
>  #include <unistd.h>
>  #include <vector>
>  
> diff --git a/src/libcamera/ipa_module.cpp b/src/libcamera/ipa_module.cpp
> index 7c52ad8d5796..908f2e07bd3f 100644
> --- a/src/libcamera/ipa_module.cpp
> +++ b/src/libcamera/ipa_module.cpp
> @@ -16,7 +16,6 @@
>  #include <fcntl.h>
>  #include <link.h>
>  #include <string.h>
> -#include <sys/mman.h>
>  #include <sys/stat.h>
>  #include <sys/types.h>
>  #include <unistd.h>
> diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> index 0bab3bedd402..2e8774ae360d 100644
> --- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> +++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> @@ -10,7 +10,6 @@
>  #include <memory>
>  #include <mutex>
>  #include <queue>
> -#include <sys/mman.h>
>  #include <unordered_set>
>  
>  #include <libcamera/camera.h>
> diff --git a/src/libcamera/v4l2_videodevice.cpp b/src/libcamera/v4l2_videodevice.cpp
> index da2af6a13dfb..cddabad753e1 100644
> --- a/src/libcamera/v4l2_videodevice.cpp
> +++ b/src/libcamera/v4l2_videodevice.cpp
> @@ -13,7 +13,6 @@
>  #include <sstream>
>  #include <string.h>
>  #include <sys/ioctl.h>
> -#include <sys/mman.h>
>  #include <sys/syscall.h>
>  #include <sys/time.h>
>  #include <unistd.h>
> diff --git a/src/v4l2/v4l2_camera_proxy.h b/src/v4l2/v4l2_camera_proxy.h
> index 92a79abb9e21..f1a8b61c5eac 100644
> --- a/src/v4l2/v4l2_camera_proxy.h
> +++ b/src/v4l2/v4l2_camera_proxy.h
> @@ -12,7 +12,6 @@
>  #include <map>
>  #include <memory>
>  #include <set>
> -#include <sys/mman.h>
>  #include <sys/types.h>
>  #include <vector>
>  
> diff --git a/src/v4l2/v4l2_compat_manager.h b/src/v4l2/v4l2_compat_manager.h
> index bc548ab2a8bd..1ec46162211d 100644
> --- a/src/v4l2/v4l2_compat_manager.h
> +++ b/src/v4l2/v4l2_compat_manager.h
> @@ -11,7 +11,6 @@
>  #include <fcntl.h>
>  #include <map>
>  #include <memory>
> -#include <sys/mman.h>
>  #include <sys/types.h>
>  #include <vector>
>  

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list