search

sensorplatforms / open-sensor-platform

Open Sensor Platform project
Apache License 2.0
61 stars 37 forks source link

Added Sensor HAL for Acc, Mag and Gyr. #7

Open pankajgoenka opened 8 years ago

pankajgoenka commented 8 years ago

Added Sensor HAL for Acc, Mag and Gyr. Modified the SH-Xpresso-LPC54102 project to include HAL files

Signed-off-by: Pankaj Goenka pgoenka@audience.com

pankajgoenka commented 8 years ago

I think the Driver_Sensor.c needs to be split in to individual sensor types - e.g. Driver_BMC150.c and then use the OSP_BUILD_DRIVER_SENSOR macro to create the descriptor for application use. This scheme needs to be understood properly before implementing.

Done

The definition of SENSOR_FLAG_CONTINUOUS_DATA seems incorrect - if you intended the flag to be at bit position 0 then its value should be 1 and not 0 because value of 0 is meaningless as far as checking for 'flag' is concerned.

I agree with you. Have written to Rian about it since the file was sent to us by him and we retained it as is.