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-backend implementation must define hal_backend_sensor_data

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

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 26996b4..0024728 100755 (executable)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -2,6 +2,11 @@ CMAKE_MINIMUM_REQUIRED(VERSION 2.6)
 PROJECT(sensor-hal-tm1 CXX)
 INCLUDE(GNUInstallDirs)
 
+SET(LIBRARY_NAME "hal-backend-sensor")
+SET(HAL_LIBDIR ${CMAKE_HAL_LIBDIR_PREFIX})
+SET(HAL_LICENSEDIR ${CMAKE_HAL_LICENSEDIR_PREFIX})
+SET(DEPENDENTS "dlog hal-api-common hal-api-sensor")
+
 SET(ACCEL "ON")
 SET(PROXIMITY "ON")
 SET(SENSORHUB "OFF")
@@ -44,9 +49,10 @@ ADD_DEFINITIONS(-DENABLE_SENSORHUB)
 ENDIF()
 
 MESSAGE("Sources: ${SRCS}")
-ADD_LIBRARY(${PROJECT_NAME} SHARED ${SRCS})
-TARGET_LINK_LIBRARIES(${PROJECT_NAME} ${HAL_PKGS_LDFLAGS})
+ADD_LIBRARY(${LIBRARY_NAME} SHARED ${SRCS})
+TARGET_LINK_LIBRARIES(${LIBRARY_NAME} ${HAL_PKGS_LDFLAGS})
 
-INSTALL(TARGETS ${PROJECT_NAME} DESTINATION ${CMAKE_INSTALL_LIBDIR}/sensor)
+INSTALL(FILES ${CMAKE_CURRENT_SOURCE_DIR}/LICENSE.APLv2 DESTINATION ${HAL_LICENSEDIR}/${PROJECT_NAME})
+INSTALL(TARGETS ${LIBRARY_NAME} DESTINATION ${HAL_LIBDIR} COMPONENT RuntimeLibraries)
 
 ADD_SUBDIRECTORY(testcase)

diff --git a/packaging/sensor-hal-tm1.spec b/packaging/sensor-hal-tm1.spec
index 4cbc579..bc34b30 100755 (executable)
--- a/packaging/sensor-hal-tm1.spec
+++ b/packaging/sensor-hal-tm1.spec
@@ -14,7 +14,8 @@ BuildRequires:  pkgconfig(dlog)
 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
 TM1 Sensor HAL
@@ -30,7 +31,7 @@ Sensor Device HAL(Hardware Abstraction Layer) Test Cases
 %setup -q
 
 %build
-%cmake .
+%cmake . -DCMAKE_HAL_LIBDIR_PREFIX=%{_hal_libdir} -DCMAKE_HAL_LICENSEDIR_PREFIX=%{_hal_licensedir}
 make %{?_smp_mflags}
 
 %install
@@ -49,8 +50,8 @@ install -m 0644 %SOURCE1 %{buildroot}%{_libdir}/udev/rules.d
 %files
 %manifest packaging/%{name}.manifest
 %{_libdir}/udev/rules.d/99-sensor.rules
-%{_libdir}/sensor/*.so
-%license LICENSE.APLv2
+%{_hal_libdir}/*.so*
+%{_hal_licensedir}/%{name}/LICENSE.APLv2
 
 %files haltests
 %manifest packaging/%{name}.manifest

diff --git a/src/accel/accel_device.h b/src/accel/accel_device.h
index 6c4b02b..0566fd9 100644 (file)
--- 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>
 #include <functional>

diff --git a/src/create.cpp b/src/create.cpp
deleted file mode 100644 (file)
index 2e9a2d5..0000000
--- a/src/create.cpp
+++ /dev/null
@@ -1,59 +0,0 @@
-/*
- * Copyright (c) 2016 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 "proxi/proxi_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)
-{
-#ifdef ENABLE_ACCEL
-       create_sensor<accel_device>("Accelerometer");
-#endif
-#ifdef ENABLE_PROXIMITY
-       create_sensor<proxi_device>("Proximity");
-#endif
-#ifdef ENABLE_SENSORHUB
-       create_sensor<sensorhub_device>("Sensorhub");
-#endif
-
-       *devices = &devs[0];
-       return devs.size();
-}

diff --git a/src/hal-backend-sensor.cpp b/src/hal-backend-sensor.cpp
new file mode 100644 (file)
index 0000000..3544482
--- /dev/null
+++ b/src/hal-backend-sensor.cpp
@@ -0,0 +1,89 @@
+/*
+ * 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 "proxi/proxi_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_tm1_create(sensor_device_t **devices) {
+#ifdef ENABLE_ACCEL
+  create_sensor<accel_device>("Accelerometer");
+#endif
+#ifdef ENABLE_PROXIMITY
+  create_sensor<proxi_device>("Proximity");
+#endif
+#ifdef ENABLE_SENSORHUB
+  create_sensor<sensorhub_device>("Sensorhub");
+#endif
+
+  *devices = &devs[0];
+  return devs.size();
+}
+
+static int sensor_tm1_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_tm1_create;
+
+  *data = (void *)funcs;
+
+  return 0;
+}
+
+static int sensor_tm1_exit(void *data) {
+  if (!data) return -EINVAL;
+  free(data);
+
+  return 0;
+}
+
+extern "C" hal_backend hal_backend_sensor_data = {
+    .name = "sensor-tm1",
+    .vendor = "Tizen",
+    .abi_version = HAL_ABI_VERSION_TIZEN_6_5,
+    .init = sensor_tm1_init,
+    .exit = sensor_tm1_exit,
+};

diff --git a/src/proxi/proxi_device.h b/src/proxi/proxi_device.h
index f9cf22b..29114d1 100644 (file)
--- a/src/proxi/proxi_device.h
+++ b/src/proxi/proxi_device.h
@@ -18,7 +18,7 @@
 #ifndef _PROXI_DEVICE_H_
 #define _PROXI_DEVICE_H_
 
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <string>
 #include <vector>
 #include <functional>

diff --git a/src/sensorhub/sensorhub.h b/src/sensorhub/sensorhub.h
index 3a5bbcc..630f304 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 0aa5d7c..d4a707e 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:

diff --git a/src/sensorhub/wristup.cpp b/src/sensorhub/wristup.cpp
index fcbecac..81108b3 100644 (file)
--- a/src/sensorhub/wristup.cpp
+++ b/src/sensorhub/wristup.cpp
@@ -16,7 +16,7 @@
  */
 
 #include <algorithm>
-#include <sensor/sensor_hal.h>
+#include <hal/hal-sensor-interface.h>
 #include <sensor_log.h>
 
 #include "sensorhub_manager.h"