linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Alexander Holler <holler@ahsoftware.de>
To: Lars-Peter Clausen <lars@metafoo.de>
Cc: linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org,
	Jonathan Cameron <jic23@cam.ac.uk>,
	rtc-linux@googlegroups.com,
	Alessandro Zummo <a.zummo@towertech.it>,
	srinivas pandruvada <srinivas.pandruvada@intel.com>
Subject: Re: [PATCH 3/3] iio: add rtc-driver for HID sensors of type time
Date: Sun, 09 Dec 2012 19:16:46 +0100	[thread overview]
Message-ID: <50C4D58E.4020702@ahsoftware.de> (raw)
In-Reply-To: <50C4BEFF.2010304@ahsoftware.de>

Am 09.12.2012 17:40, schrieb Alexander Holler:
> Am 09.12.2012 13:55, schrieb Lars-Peter Clausen:
>> On 12/09/2012 01:21 PM, Alexander Holler wrote:
>>> This driver makes the time from HID sensors (hubs) which are offering
>>> such available like any other RTC does.
>>>
>>> Currently the time can only be read. Setting the time must be done
>>> through sending a report, which currently isn't supported by
>>> hid-sensor-hub.
>>>
>>> It is necessary that all values like year, month etc, are send as
>>> 8bit values (1 byte each) and all of them in 1 report. Also the
>>> spec HUTRR39b doesn't define the range of the year field, we
>>> tread it as 0 - 99 because that's what most RTCs I know about are
>>> offering.
>>
>> I don't think we should register a IIO device for this. Just an RTC
>> device
>> should be fully sufficient.
>>
>
> You would have to implement all the used services hid-sensor-hub offers
> in the rtc-driver again.

Besides that, time is part of the spec for HID sensors, which makes 
sense. Making that available as a RTC to read/set the time is imho the 
best to way support that. If you want a driver which doesn't use the HID 
sensor hub framework (or iio), you would either have to define your own 
standard or make sure it is interoperable with the HID sensor hub 
framework, which might not be trivial.

Regards,

Alexander

  reply	other threads:[~2012-12-09 18:17 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-09 12:21 [PATCH 0/3] iio: HID sensor time (as RTC) Alexander Holler
2012-12-09 12:21 ` [PATCH 1/3] iio: hid-sensors: respect CONFIG_IIO_TRIGGER Alexander Holler
2012-12-09 12:21 ` [PATCH 2/3] iio: Add Usage IDs for HID time sensors Alexander Holler
2012-12-09 12:21 ` [PATCH 3/3] iio: add rtc-driver for HID sensors of type time Alexander Holler
2012-12-09 12:55   ` Lars-Peter Clausen
2012-12-09 16:40     ` Alexander Holler
2012-12-09 18:16       ` Alexander Holler [this message]
2012-12-09 19:20         ` Lars-Peter Clausen
2012-12-10 13:12           ` Alexander Holler
2012-12-10 14:51             ` [PATCH 3/3 v2] " Alexander Holler
2012-12-10 17:05               ` Lars-Peter Clausen
2012-12-10 19:45                 ` Alexander Holler
2012-12-10 20:22                   ` Lars-Peter Clausen
2012-12-10 21:26                     ` Alexander Holler
2012-12-10 21:39                       ` Lars-Peter Clausen
2012-12-10 21:42                         ` Jonathan Cameron
2012-12-10 22:50                           ` Alexander Holler
2012-12-11  9:31                             ` Jonathan Cameron
2012-12-11  9:40                               ` Lars-Peter Clausen
2012-12-11 12:39                                 ` Alexander Holler
2012-12-11 13:54                                   ` Jonathan Cameron
2012-12-11 18:21                                     ` [PATCH 1/4 v2] iio: hid-sensors: respect CONFIG_IIO_TRIGGER Alexander Holler
2012-12-11 18:21                                       ` [PATCH 2/4 RESEND] iio: Add Usage IDs for HID time sensors Alexander Holler
2012-12-15 11:06                                         ` Jonathan Cameron
2012-12-15 12:41                                           ` Alexander Holler
2012-12-15 12:45                                             ` [PATCH 1/4 " Alexander Holler
2012-12-15 12:45                                               ` [PATCH 2/4 RESEND] iio: merge hid-sensor-attributes.h into hid-sensor-hub.h Alexander Holler
2013-01-03  9:41                                                 ` Jiri Kosina
2013-01-06 11:50                                                   ` Jonathan Cameron
2012-12-15 12:45                                               ` [PATCH 3/4 v5 RESEND] rtc: add rtc-driver for HID sensors of type time Alexander Holler
2013-01-03 22:39                                                 ` Andrew Morton
2013-01-04  9:18                                                   ` Jiri Kosina
2013-01-04 13:10                                                     ` Alexander Holler
2013-01-06 11:50                                                       ` Jonathan Cameron
2012-12-15 12:45                                               ` [PATCH 4/4 RESEND] hid: iio: rename struct hid_sensor_iio_common to hid_sensor_common Alexander Holler
2013-01-03  9:42                                                 ` Jiri Kosina
2013-01-06 11:50                                                   ` Jonathan Cameron
2013-01-03  9:40                                               ` [PATCH 1/4 RESEND] iio: Add Usage IDs for HID time sensors Jiri Kosina
2013-01-06 11:51                                                 ` Jonathan Cameron
2012-12-11 18:21                                       ` [PATCH 3/4] iio: merge hid-sensor-attributes.h into hid-sensor-hub.h Alexander Holler
2012-12-12 15:45                                         ` Pandruvada, Srinivas
2012-12-12 20:10                                           ` Alexander Holler
2012-12-12 20:28                                             ` [PATCHi 5/4] hid: iio: rename struct hid_sensor_iio_common to hid_sensor_common Alexander Holler
2012-12-12 21:04                                               ` Pandruvada, Srinivas
2012-12-11 18:21                                       ` [PATCH 4/4 v3] rtc: add rtc-driver for HID sensors of type time Alexander Holler
2012-12-12  9:51                                         ` Lars-Peter Clausen
2012-12-12 10:14                                           ` Alexander Holler
2012-12-12 10:18                                             ` Lars-Peter Clausen
2012-12-12 11:11                                             ` [PATCH 4/4 v4] " Alexander Holler
2012-12-14  9:42                                               ` Lars-Peter Clausen
2012-12-14 13:08                                                 ` Alexander Holler
2012-12-14 14:15                                                   ` Alexander Holler
2012-12-14 14:29                                                     ` Alexander Holler
2012-12-14 14:34                                                       ` Lars-Peter Clausen
2012-12-14 15:24                                                         ` Alexander Holler
2012-12-14 16:33                                                           ` Lars-Peter Clausen
2012-12-14 21:24                                                             ` Alexander Holler
2012-12-14 22:02                                                               ` [PATCH 4/4 v5] " Alexander Holler
2012-12-15 10:54                                       ` [PATCH 1/4 v2] iio: hid-sensors: respect CONFIG_IIO_TRIGGER Jonathan Cameron
2012-12-15 12:37                                         ` Alexander Holler
2012-12-16 22:15                                 ` [PATCH 3/3 v2] iio: add rtc-driver for HID sensors of type time Alessandro Zummo
2012-12-17  7:38                                   ` Alexander Holler
2012-12-10 22:20                 ` Alexander Holler
2012-12-10 22:36                   ` Lars-Peter Clausen
2012-12-11  0:01                     ` Alexander Holler
2012-12-11 10:35                       ` Alan Cox

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50C4D58E.4020702@ahsoftware.de \
    --to=holler@ahsoftware.de \
    --cc=a.zummo@towertech.it \
    --cc=jic23@cam.ac.uk \
    --cc=lars@metafoo.de \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rtc-linux@googlegroups.com \
    --cc=srinivas.pandruvada@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).