Hi ,
On Thu, Jul 19, 2012 at 11:47 PM, srinivas pandruvada
<srinivas.pandruvada@xxxxxxxxx> wrote:
> Added usage id processing for ALS. This uses IIO
> interfaces for triggerred buffer to present data to user
> mode.This uses HID sensor framework for registering callback
> events from the sensor hub.
>
> Signed-off-by: srinivas pandruvada <srinivas.pandruvada@xxxxxxxxx>
> ---
> drivers/iio/Kconfig | 1 +
> drivers/iio/Makefile | 1 +
> drivers/iio/light/Kconfig | 16 ++
> drivers/iio/light/Makefile | 6 +
> drivers/iio/light/hid-sensor-als.c | 397 ++++++++++++++++++++++++++++++++++++
> 5 files changed, 421 insertions(+), 0 deletions(-)
[...]
> +
> +/* Function to deinitialize the processing for usage id */
> +static int als_exit(struct platform_device *pdev)
Can this be merged in the remove function ?
Also the return value is not checked.
> +{
> + int ret = 0;
> + struct iio_dev *indio_dev = platform_get_drvdata(pdev);
> + struct hid_sensor_attributes *st = iio_priv(indio_dev);
> + struct als_state *als_state = st->private;
> +
> + iio_device_unregister(indio_dev);
> + hid_sensor_remove_trigger(indio_dev);
> + iio_triggered_buffer_cleanup(indio_dev);
> + kfree(indio_dev->channels);
> + iio_device_free(indio_dev);
> + kfree(als_state);
> +
> + return ret;
> +}
> +
> +static struct hid_sensor_hub_callbacks als_callbacks = {
> + .send_event = als_proc_event,
> + .capture_sample = als_capture_sample,
> +};
> +
> +static int __devinit hid_als_probe(struct platform_device *pdev)
> +{
> + int ret;
> + struct hid_sensor_hub_device *hsdev = pdev->dev.platform_data;
> +
> + ret = als_init(pdev, hsdev, HID_USAGE_SENSOR_ALS);
> + if (ret < 0) {
> + dev_err(&pdev->dev, "als_init failed\n");
> + return ret;
> + }
> + als_callbacks.pdev = pdev;
> + ret = sensor_hub_register_callback(hsdev, HID_USAGE_SENSOR_ALS,
> + &als_callbacks);
> +
> + return 0;
> +}
> +
> +static int __devinit hid_als_remove(struct platform_device *pdev)
> +{
> + struct hid_sensor_hub_device *hsdev = pdev->dev.platform_data;
> +
> + als_exit(pdev);
> +
> + return sensor_hub_remove_callback(hsdev, HID_USAGE_SENSOR_ALS);
> +}
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
