From: Roy Im <roy.im.opensource@diasemi.com>
To: Pavel Machek <pavel@ucw.cz>, Roy Im <roy.im.opensource@diasemi.com>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Support Opensource <Support.Opensource@diasemi.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH V4 2/3] dt-bindings: input: Add document bindings for DA7280
Date: Mon, 27 Aug 2018 05:02:24 +0000 [thread overview]
Message-ID: <DB7PR10MB2169FA41DB1196ACCDFEE333850B0@DB7PR10MB2169.EURPRD10.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <20180826065157.GA25309@amd>
Hi Pavel,
On Sunday, August 26, 2018 3:52 PM, Pavel Machek wrote:
>
> > +++ b/Documentation/devicetree/bindings/input/dlg,da7280.txt
> > @@ -0,0 +1,101 @@
> > +Dialog Semiconductor DA7280 Haptics bindings
> > +
> > +Required properties:
> > +- compatible: Should be "dlg,da7280".
> > +- reg: Specifies the I2C slave address.
> > +
> > +- interrupt-parent : Specifies the phandle of the interrupt
> > +controller to
> > + which the IRQs from DA7280 are delivered to.
> > +
> > +- dlg,vib-mode:
> > + "LRA-MODE" - Linear Resonance Actuator mode.
> > + "ERM-BAR" - Bar type Eccentric Rotating Mass mode.
> > + "ERM-COIN" - Coin type Eccentric Rotating Mass mode.
>
> So this is some kind of "vibration motor" on steroids)?
I will add some descriptions more here in v5.
> > +- dlg,op-mode: Haptic operation mode.
> > + Possible values:
> > + 1 - Direct register override mode triggered by i2c(default),
> > + 2 - PWM data source mode setting duty to 0 - 0xffff(0% - 100%),
>
> Space after 0xffff?
I will remove this in v5.
>
> > + 3 - Register triggered waveform memory mode, the pattern assigned
> > + to the PS_SEQ_ID played as much times as PS_SEQ_LOOP,
> > + 4 - Edge triggered waveform memory mode, external GPI(N) condtrol
>
> "control"
I will fix this in v5
>
> > + are required to enable/disable and it needs to keep
> > + device enabled by sending magnitude (X > 0),
> > + the pattern is assigned to the GPI(N)_SEQUENCE_ID below
>
> "below."
I will fix this in v5
>
> > +- dlg,imax-microamp: Actuator max current rating.
> > + Valid values: 0 - 252000.
> > + Default: 130000.
> > +- dlg,impd-micro-ohms: the impedance of the actuator in micro ohm,
>
> "ohms"
I will fix this in v5.
>
> > + as read from its datasheet.
>
> "as read" does not sound english.
I will correct this in v5.
>
> > +- dlg,mem-array: use in case that memory registers should be updated,
> > + Please fill the whole buffers(100 bytes) to avoid any error in driver.
> > + For example,
> > + dlg,mem-array = <
> > + 0x01 0x02 0x03 0x04 0x05 0x06 0x07 0x08 0x09 0x0A
> > + ...
> > + 0x71 0x72 0x73 0x74 0x75 0x76 0x77 0x78 0x79 0x7A
> > + 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00
> > + 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00 0x00
> > + >;
>
> I'd not understand what this is from this description. (And I'd put
> example below... in the exampls section).
I will improve and update the description including the example section in v5.
Regards,
Roy
>
> > +For further information, see device datasheet.
> > +
> > +======
> > +
> > +Example:
> > +
> > + haptics: da7280-haptics@4a {
> > + compatible = "dlg,da7280";
> > + reg = <0x4a>;
> > + interrupt-parent = <&gpio6>;
> > + interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
> > + dlg,vib-mode = "LRA-MODE";
> > + dlg,op-mode = <1>;
> > + dlg,nom-microvolt = <2000000>;
> > + dlg,abs-max-microvolt = <2000000>;
> > + dlg,imax-microamp = <170000>;
> > + dlg,resonant-freq-hz = <180>;
> > + dlg,impd-micro-ohms = <10500000>;
> > + dlg,freq-track-enable;
> > + dlg,rapid-stop-enable;
> > + };
>
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures)
> http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next prev parent reply other threads:[~2018-08-27 5:02 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-21 5:44 [PATCH V4 0/3] da7280: haptic driver submission Roy Im
2018-08-21 5:44 ` [PATCH V4 1/3] MAINTAINERS: da7280 updates to the Dialog Semiconductor search terms Roy Im
2018-08-21 5:44 ` [PATCH V4 2/3] dt-bindings: input: Add document bindings for DA7280 Roy Im
2018-08-26 6:51 ` Pavel Machek
2018-08-27 5:02 ` Roy Im [this message]
2018-08-21 5:44 ` [PATCH V4 3/3] Input: new da7280 haptic driver Roy Im
2018-08-23 15:28 ` Randy Dunlap
2018-08-24 5:35 ` Roy Im
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=DB7PR10MB2169FA41DB1196ACCDFEE333850B0@DB7PR10MB2169.EURPRD10.PROD.OUTLOOK.COM \
--to=roy.im.opensource@diasemi.com \
--cc=Support.Opensource@diasemi.com \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pavel@ucw.cz \
--cc=robh+dt@kernel.org \
/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).