Re: [PATCH] Haptic class support (v2)

[Kyungmin Park <kmpark@infradead.org>]

On Sun, Oct 11, 2009 at 6:05 PM, Pavel Machek <pavel@ucw.cz> wrote:
> On Tue 2009-10-06 16:45:33, Kyungmin Park wrote:
>> This patch includes two haptic devices, isa1000 and isa1200
>> ISA1000 is gpio based haptic, but isa1200 is based on I2C
>> Both are working on Samsung SoCs and tested.
>>
>> To enable the haptic, echo 1 > /sys/class/haptic/${name}/enable
>> You can adjust the level by echo ${level} > /sys/class/haptic/${name}/enable
>> or
>> With oneshot feature, echo ${msec time} > /sys/class/haptic/${name}/oneshot
>
> New device classes should certainly be documented in Documentation/.
>
> For example... is level 0-100?

Basically. range from 0 to 255. also each device can set max level in
case of isa1000 it's 6.

>  Is it valid to change level while 'oneshot' is running?

we usually use it under 1 sec. but I think it's possible to change it
but not tried.

>  Does oneshot use last level set?

Right. it used last set value.

If you don't mind to update these at Documentation please wait until
commit first patches merged.

Thank you,
Kyungmin Park