Use hal-api-sensor instead of legacy interface

* This change is for Tizen Next-HAL
* Now, all implementations of sensor device inherit sensor device interface of the hal-api-sensor
* sensor_{device}_create replaces create, see hal-backend-sensor.cpp
* The hal-hackend implementation must define hal_backend_sensor_data

Change-Id: Ib6c9968526f5c4514b078faadc024c325b1e5f38
Signed-off-by: Boram Bae <boram21.bae@samsung.com>

diff --git a/CMakeLists.txt b/CMakeLists.txt
index b89920085c909fb356bc8d8dfcfd4d08870c6b3b..bde390f1f598afd14512d92ba7063aeb7c9239b8 100755 (executable)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1,5 +1,5 @@
 CMAKE_MINIMUM_REQUIRED(VERSION 2.6)
-PROJECT(sensor-hal-tw3 CXX)
+PROJECT(hal-backend-sensor CXX)
 INCLUDE(GNUInstallDirs)
 SET(PREFIX ${CMAKE_INSTALL_PREFIX})
 SET(BINDIR "${PREFIX}/bin")
@@ -136,6 +136,6 @@ MESSAGE("Sources: ${SRCS}")
 ADD_LIBRARY(${PROJECT_NAME} SHARED ${SRCS})
 TARGET_LINK_LIBRARIES(${PROJECT_NAME} ${HAL_PKGS_LDFLAGS})
 
-INSTALL(TARGETS ${PROJECT_NAME} DESTINATION ${CMAKE_INSTALL_LIBDIR}/sensor)
+INSTALL(TARGETS ${PROJECT_NAME} DESTINATION /hal/lib COMPONENT RuntimeLibraries)
 
 ADD_SUBDIRECTORY(testcase)

diff --git a/packaging/sensor-hal-tw3.spec b/packaging/sensor-hal-tw3.spec
index e7c290594b24beb3b46055527be4397d86802661..377620985fac970034b57520583e9546ad2fcbc5 100755 (executable)
--- a/packaging/sensor-hal-tw3.spec
+++ b/packaging/sensor-hal-tw3.spec
@@ -16,7 +16,8 @@ BuildRequires:  pkgconfig(glib-2.0)
 BuildRequires:  pkgconfig(gio-2.0)
 BuildRequires: pkgconfig(gmock)
 BuildRequires:  sensor-hal-devel
-
+BuildRequires:  pkgconfig(hal-api-common)
+BuildRequires:  pkgconfig(hal-api-sensor)
 
 %description
 TW3 Sensor HAL
@@ -54,7 +55,7 @@ install -m 0644 %SOURCE2 %{buildroot}%{_libdir}/udev/rules.d
 %manifest packaging/%{name}.manifest
 %{_libdir}/udev/rules.d/99-sensor.rules
 %{_libdir}/udev/rules.d/99-sensorhub.rules
-%{_libdir}/sensor/*.so
+/hal/lib/*.so*
 %license LICENSE.APLv2
 
 %files haltests

diff --git a/src/accel/accel_device.h b/src/accel/accel_device.h
index 34770be368f83a5082d6bf64fff08000a3880633..7b7ab2550ca6d83027c4cc1f8d7c7526bd5040d8 100755 (executable)
--- a/src/accel/accel_device.h
+++ b/src/accel/accel_device.h
@@ -18,7 +18,7 @@
 #ifndef __ACCEL_DEVICE_H__
 #define __ACCEL_DEVICE_H__
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <string>
 #include <vector>
 

diff --git a/src/create.cpp b/src/create.cpp
deleted file mode 100644 (file)
index 9246692..0000000
--- a/src/create.cpp
+++ /dev/null
@@ -1,61 +0,0 @@
-/*
- * Copyright (c) 2017 Samsung Electronics Co., Ltd.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- */
-
-#include <sensor/sensor_hal.h>
-#include <sensor_log.h>
-#include <vector>
-
-#include "accel/accel_device.h"
-#include "gyro/gyro_device.h"
-#include "pressure/pressure_device.h"
-#include "light/light_device.h"
-#include "hrm_raw/hrm_raw_device.h"
-#include "hrm/hrm_device.h"
-#include "sensorhub/sensorhub.h"
-
-static std::vector<sensor_device_t> devs;
-
-template<typename _sensor>
-void create_sensor(const char *name)
-{
-       sensor_device *instance = NULL;
-       try {
-               instance = new _sensor;
-       } catch (std::exception &e) {
-               ERR("Failed to create %s sensor device, exception: %s", name, e.what());
-               return;
-       } catch (int err) {
-               _ERRNO(err, _E, "Failed to create %s sensor device", name);
-               return;
-       }
-
-       devs.push_back(instance);
-}
-
-extern "C" int create(sensor_device_t **devices)
-{
-       create_sensor<accel_device>("Accelerometer");
-       create_sensor<gyro_device>("Gyroscope");
-       create_sensor<pressure_device>("Pressure");
-       create_sensor<light_device>("Light");
-       create_sensor<hrm_raw_device>("HRM Raw");
-       create_sensor<hrm_device>("HRM");
-       create_sensor<sensorhub_device>("Sensorhub");
-
-       *devices = &devs[0];
-       return devs.size();
-}

diff --git a/src/gyro/gyro_device.h b/src/gyro/gyro_device.h
index 9c8649936557dfa099828da0d2a4160ab7e91823..d0ea33873a0c13f218a7957008efaefd94f8c662 100644 (file)
--- a/src/gyro/gyro_device.h
+++ b/src/gyro/gyro_device.h
@@ -18,7 +18,7 @@
 #ifndef __GYRO_DEVICE_H__
 #define __GYRO_DEVICE_H__
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <string>
 #include <vector>
 

diff --git a/src/hal-backend-sensor.cpp b/src/hal-backend-sensor.cpp
new file mode 100644 (file)
index 0000000..ad0a238
--- /dev/null
+++ b/src/hal-backend-sensor.cpp
@@ -0,0 +1,91 @@
+/*
+ * Copyright (c) 2021 Samsung Electronics Co., Ltd.
+ *
+ * Licensed under the Apache License, Version 2.0 (the License);
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <errno.h>
+#include <hal/hal-sensor-interface.h>
+#include <sensor_log.h>
+#include <stdlib.h>
+
+#include <vector>
+
+#include "accel/accel_device.h"
+#include "gyro/gyro_device.h"
+#include "hrm/hrm_device.h"
+#include "hrm_raw/hrm_raw_device.h"
+#include "light/light_device.h"
+#include "pressure/pressure_device.h"
+#include "sensorhub/sensorhub.h"
+
+static std::vector<sensor_device_t> devs;
+
+template <typename _sensor>
+void create_sensor(const char *name) {
+  sensor_device *instance = NULL;
+  try {
+    instance = new _sensor;
+  } catch (std::exception &e) {
+    ERR("Failed to create %s sensor device, exception: %s", name, e.what());
+    return;
+  } catch (int err) {
+    _ERRNO(err, _E, "Failed to create %s sensor device", name);
+    return;
+  }
+
+  devs.push_back(instance);
+}
+
+static int sensor_tw3_create(sensor_device_t **devices) {
+  create_sensor<accel_device>("Accelerometer");
+  create_sensor<gyro_device>("Gyroscope");
+  create_sensor<pressure_device>("Pressure");
+  create_sensor<light_device>("Light");
+  create_sensor<hrm_raw_device>("HRM Raw");
+  create_sensor<hrm_device>("HRM");
+  create_sensor<sensorhub_device>("Sensorhub");
+
+  *devices = &devs[0];
+  return devs.size();
+}
+
+static int sensor_tw3_init(void **data) {
+  _I("init hal backend sensor");
+  hal_backend_sensor_funcs *funcs;
+
+  funcs =
+      (hal_backend_sensor_funcs *)calloc(1, sizeof(hal_backend_sensor_funcs));
+  if (!funcs) return -ENOMEM;
+
+  funcs->create = sensor_tw3_create;
+
+  *data = (void *)funcs;
+
+  return 0;
+}
+
+static int sensor_tw3_exit(void *data) {
+  if (!data) return -EINVAL;
+  free(data);
+
+  return 0;
+}
+
+extern "C" hal_backend hal_backend_sensor_data = {
+    .name = "sensor-tw3",
+    .vendor = "Tizen",
+    .abi_version = HAL_ABI_VERSION_TIZEN_6_5,
+    .init = sensor_tw3_init,
+    .exit = sensor_tw3_exit,
+};

diff --git a/src/hrm/hrm_device.h b/src/hrm/hrm_device.h
index d3067f19f9b1db2f6f2129b483ead5d8fc9323fe..04bc907f6c2eaaf7dee4f77d4f90dc8718bf1f79 100644 (file)
--- a/src/hrm/hrm_device.h
+++ b/src/hrm/hrm_device.h
@@ -18,7 +18,7 @@
 #ifndef __HRM_DEVICE_H__
 #define __HRM_DEVICE_H__
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <string>
 #include <vector>
 

diff --git a/src/hrm_raw/hrm_raw_device.h b/src/hrm_raw/hrm_raw_device.h
index bb105300e57a02bcf5f59b63c69df0547b129c7a..b32cbc8a76648784591941c47ad1b077eed7a982 100755 (executable)
--- a/src/hrm_raw/hrm_raw_device.h
+++ b/src/hrm_raw/hrm_raw_device.h
@@ -18,7 +18,7 @@
 #ifndef __HRM_RAW_DEVICE_H__
 #define __HRM_RAW_DEVICE_H__
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <string>
 #include <vector>
 

diff --git a/src/light/light_device.h b/src/light/light_device.h
index 2a29dd752ea22119578c627202194c9f862dd595..4b65d52e43e2bec558f970dddcece769ff35adeb 100644 (file)
--- a/src/light/light_device.h
+++ b/src/light/light_device.h
@@ -18,7 +18,7 @@
 #ifndef __LIGHT_DEVICE_H__
 #define __LIGHT_DEVICE_H__
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <string>
 #include <vector>
 

diff --git a/src/pressure/pressure_device.h b/src/pressure/pressure_device.h
index 114523daeaf265a01be00dd25e488fdd4f5040d7..f3ae65f69bde3d37d16cd6225e4cae205f207f02 100644 (file)
--- a/src/pressure/pressure_device.h
+++ b/src/pressure/pressure_device.h
@@ -18,7 +18,7 @@
 #ifndef __PRESSURE_DEVICE_H__
 #define __PRESSURE_DEVICE_H__
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <string>
 #include <vector>
 

diff --git a/src/sensorhub/hrm_batch.cpp b/src/sensorhub/hrm_batch.cpp
index b4d9fa95b4e4165d1116bbfd5441916819f898da..b9feb73f6a5e7ac93ba31841fc742e069dcfcdf7 100644 (file)
--- a/src/sensorhub/hrm_batch.cpp
+++ b/src/sensorhub/hrm_batch.cpp
@@ -17,7 +17,7 @@
 
 #include <stdlib.h>
 #include <sensor_common.h>
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <sensor_log.h>
 #include <deque>
 #include <util.h>

diff --git a/src/sensorhub/hrm_led_green_batch.cpp b/src/sensorhub/hrm_led_green_batch.cpp
index 41d63aa927f2830df042d590e76ff8cbd04ba55a..53f8ca85b713879852e083d5ff9a64ac8acd37e4 100644 (file)
--- a/src/sensorhub/hrm_led_green_batch.cpp
+++ b/src/sensorhub/hrm_led_green_batch.cpp
@@ -17,7 +17,7 @@
 
 #include <stdlib.h>
 #include <sensor_common.h>
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <sensor_log.h>
 #include <util.h>
 

diff --git a/src/sensorhub/sensorhub.h b/src/sensorhub/sensorhub.h
index eceb5ad44bddb9f805ac9a575a7ed4e7f358bef2..2b80dbdba753a69d9364e463cf01197c94321aa7 100644 (file)
--- a/src/sensorhub/sensorhub.h
+++ b/src/sensorhub/sensorhub.h
@@ -19,7 +19,7 @@
 #define _SENSORHUB_DEVICE_H_
 
 #include <vector>
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 
 #include "sensorhub_controller.h"
 #include "sensorhub_manager.h"

diff --git a/src/sensorhub/sensorhub_controller.h b/src/sensorhub/sensorhub_controller.h
index 40bf786c57e5968e8940a5d2f0afc0e0e8fb58c3..bb206d3fb33b5b58838d4b432d475d107246e8bb 100644 (file)
--- a/src/sensorhub/sensorhub_controller.h
+++ b/src/sensorhub/sensorhub_controller.h
@@ -18,7 +18,7 @@
 #ifndef _SENSORHUB_CONTROLLER_H_
 #define _SENSORHUB_CONTROLLER_H_
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 
 class sensorhub_controller {
 public: