All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: ivanverbovoy@gmail.com
Cc: Support Opensource <support.opensource@diasemi.com>,
	 Roy Im <roy.im.opensource@diasemi.com>,
	linux-input@vger.kernel.org, linux-kernel@vger.kernel.org,
	 stable@vger.kernel.org
Subject: Re: [PATCH] Input: da7280 - do not advertise FF_GAIN
Date: Mon, 9 Nov 2026 18:26:05 -0800	[thread overview]
Message-ID: <avKAtJ8VJ7mL2gP4@google.com> (raw)
In-Reply-To: <20261109-da7280-fixes-v1-1-09cc3650261c@gmail.com>

On Mon, Nov 09, 2026 at 07:15:55PM +0200, Ivan Verbovoy via B4 Relay wrote:
> From: Ivan Verbovoy <ivanverbovoy@gmail.com>
> 
> The driver sets FF_GAIN in the device's force feedback capabilities but
> provides no set_gain() callback. input_ff_event() checks the capability
> bit, not the callback, before calling ff->set_gain(), so an EV_FF/FF_GAIN
> event written to the event node by any process with write access to it
> calls through a NULL pointer.
> 
> The driver has no global gain to set: a constant effect carries its own
> level, and a periodic effect plays a pattern from the device's waveform
> memory. Drop FF_GAIN from the capabilities, so that input_ff_event()
> ignores such an event.
> 
> Fixes: cd3f609823a5 ("Input: new da7280 haptic driver")
> Cc: stable@vger.kernel.org
> Signed-off-by: Ivan Verbovoy <ivanverbovoy@gmail.com>

Applied, thank you.

-- 
Dmitry

      reply	other threads:[~2026-11-10  2:26 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-09 17:15 [PATCH] Input: da7280 - do not advertise FF_GAIN Ivan Verbovoy via B4 Relay
2026-11-09 17:15 ` Ivan Verbovoy
2026-11-10  2:26 ` Dmitry Torokhov [this message]

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=avKAtJ8VJ7mL2gP4@google.com \
    --to=dmitry.torokhov@gmail.com \
    --cc=ivanverbovoy@gmail.com \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=roy.im.opensource@diasemi.com \
    --cc=stable@vger.kernel.org \
    --cc=support.opensource@diasemi.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.