11a59d1b8SThomas Gleixner // SPDX-License-Identifier: GPL-2.0-or-later
2509ca1a9SAnssi Hannula /*
3509ca1a9SAnssi Hannula * Force feedback support for Linux input subsystem
4509ca1a9SAnssi Hannula *
5509ca1a9SAnssi Hannula * Copyright (c) 2006 Anssi Hannula <anssi.hannula@gmail.com>
6509ca1a9SAnssi Hannula * Copyright (c) 2006 Dmitry Torokhov <dtor@mail.ru>
7509ca1a9SAnssi Hannula */
8509ca1a9SAnssi Hannula
9509ca1a9SAnssi Hannula /* #define DEBUG */
10509ca1a9SAnssi Hannula
11509ca1a9SAnssi Hannula #include <linux/input.h>
12*c727e46fSErick Archer #include <linux/limits.h>
13509ca1a9SAnssi Hannula #include <linux/module.h>
14509ca1a9SAnssi Hannula #include <linux/mutex.h>
15*c727e46fSErick Archer #include <linux/overflow.h>
1683680cdbSGeert Uytterhoeven #include <linux/sched.h>
175a0e3ad6STejun Heo #include <linux/slab.h>
18509ca1a9SAnssi Hannula
19509ca1a9SAnssi Hannula /*
20509ca1a9SAnssi Hannula * Check that the effect_id is a valid effect and whether the user
21509ca1a9SAnssi Hannula * is the owner
22509ca1a9SAnssi Hannula */
check_effect_access(struct ff_device * ff,int effect_id,struct file * file)23509ca1a9SAnssi Hannula static int check_effect_access(struct ff_device *ff, int effect_id,
24509ca1a9SAnssi Hannula struct file *file)
25509ca1a9SAnssi Hannula {
26509ca1a9SAnssi Hannula if (effect_id < 0 || effect_id >= ff->max_effects ||
27509ca1a9SAnssi Hannula !ff->effect_owners[effect_id])
28509ca1a9SAnssi Hannula return -EINVAL;
29509ca1a9SAnssi Hannula
30509ca1a9SAnssi Hannula if (file && ff->effect_owners[effect_id] != file)
31509ca1a9SAnssi Hannula return -EACCES;
32509ca1a9SAnssi Hannula
33509ca1a9SAnssi Hannula return 0;
34509ca1a9SAnssi Hannula }
35509ca1a9SAnssi Hannula
36509ca1a9SAnssi Hannula /*
37509ca1a9SAnssi Hannula * Checks whether 2 effects can be combined together
38509ca1a9SAnssi Hannula */
check_effects_compatible(struct ff_effect * e1,struct ff_effect * e2)39509ca1a9SAnssi Hannula static inline int check_effects_compatible(struct ff_effect *e1,
40509ca1a9SAnssi Hannula struct ff_effect *e2)
41509ca1a9SAnssi Hannula {
42509ca1a9SAnssi Hannula return e1->type == e2->type &&
43509ca1a9SAnssi Hannula (e1->type != FF_PERIODIC ||
44509ca1a9SAnssi Hannula e1->u.periodic.waveform == e2->u.periodic.waveform);
45509ca1a9SAnssi Hannula }
46509ca1a9SAnssi Hannula
47509ca1a9SAnssi Hannula /*
48509ca1a9SAnssi Hannula * Convert an effect into compatible one
49509ca1a9SAnssi Hannula */
compat_effect(struct ff_device * ff,struct ff_effect * effect)50509ca1a9SAnssi Hannula static int compat_effect(struct ff_device *ff, struct ff_effect *effect)
51509ca1a9SAnssi Hannula {
52509ca1a9SAnssi Hannula int magnitude;
53509ca1a9SAnssi Hannula
54509ca1a9SAnssi Hannula switch (effect->type) {
55509ca1a9SAnssi Hannula case FF_RUMBLE:
56509ca1a9SAnssi Hannula if (!test_bit(FF_PERIODIC, ff->ffbit))
57509ca1a9SAnssi Hannula return -EINVAL;
58509ca1a9SAnssi Hannula
59509ca1a9SAnssi Hannula /*
608a4dda79SDan Murphy * calculate magnitude of sine wave as average of rumble's
61509ca1a9SAnssi Hannula * 2/3 of strong magnitude and 1/3 of weak magnitude
62509ca1a9SAnssi Hannula */
63509ca1a9SAnssi Hannula magnitude = effect->u.rumble.strong_magnitude / 3 +
64509ca1a9SAnssi Hannula effect->u.rumble.weak_magnitude / 6;
65509ca1a9SAnssi Hannula
66509ca1a9SAnssi Hannula effect->type = FF_PERIODIC;
67509ca1a9SAnssi Hannula effect->u.periodic.waveform = FF_SINE;
68509ca1a9SAnssi Hannula effect->u.periodic.period = 50;
698c374ef4SCharles Keepax effect->u.periodic.magnitude = magnitude;
70509ca1a9SAnssi Hannula effect->u.periodic.offset = 0;
71509ca1a9SAnssi Hannula effect->u.periodic.phase = 0;
72509ca1a9SAnssi Hannula effect->u.periodic.envelope.attack_length = 0;
73509ca1a9SAnssi Hannula effect->u.periodic.envelope.attack_level = 0;
74509ca1a9SAnssi Hannula effect->u.periodic.envelope.fade_length = 0;
75509ca1a9SAnssi Hannula effect->u.periodic.envelope.fade_level = 0;
76509ca1a9SAnssi Hannula
77509ca1a9SAnssi Hannula return 0;
78509ca1a9SAnssi Hannula
79509ca1a9SAnssi Hannula default:
80509ca1a9SAnssi Hannula /* Let driver handle conversion */
81509ca1a9SAnssi Hannula return 0;
82509ca1a9SAnssi Hannula }
83509ca1a9SAnssi Hannula }
84509ca1a9SAnssi Hannula
85509ca1a9SAnssi Hannula /**
86509ca1a9SAnssi Hannula * input_ff_upload() - upload effect into force-feedback device
87509ca1a9SAnssi Hannula * @dev: input device
88509ca1a9SAnssi Hannula * @effect: effect to be uploaded
89509ca1a9SAnssi Hannula * @file: owner of the effect
90509ca1a9SAnssi Hannula */
input_ff_upload(struct input_dev * dev,struct ff_effect * effect,struct file * file)91509ca1a9SAnssi Hannula int input_ff_upload(struct input_dev *dev, struct ff_effect *effect,
92509ca1a9SAnssi Hannula struct file *file)
93509ca1a9SAnssi Hannula {
94509ca1a9SAnssi Hannula struct ff_device *ff = dev->ff;
95509ca1a9SAnssi Hannula struct ff_effect *old;
96509ca1a9SAnssi Hannula int ret = 0;
97509ca1a9SAnssi Hannula int id;
98509ca1a9SAnssi Hannula
99509ca1a9SAnssi Hannula if (!test_bit(EV_FF, dev->evbit))
100509ca1a9SAnssi Hannula return -ENOSYS;
101509ca1a9SAnssi Hannula
102509ca1a9SAnssi Hannula if (effect->type < FF_EFFECT_MIN || effect->type > FF_EFFECT_MAX ||
103509ca1a9SAnssi Hannula !test_bit(effect->type, dev->ffbit)) {
1042afebafdSOliver Neukum dev_dbg(&dev->dev, "invalid or not supported effect type in upload\n");
105509ca1a9SAnssi Hannula return -EINVAL;
106509ca1a9SAnssi Hannula }
107509ca1a9SAnssi Hannula
108509ca1a9SAnssi Hannula if (effect->type == FF_PERIODIC &&
109509ca1a9SAnssi Hannula (effect->u.periodic.waveform < FF_WAVEFORM_MIN ||
110509ca1a9SAnssi Hannula effect->u.periodic.waveform > FF_WAVEFORM_MAX ||
111509ca1a9SAnssi Hannula !test_bit(effect->u.periodic.waveform, dev->ffbit))) {
1122afebafdSOliver Neukum dev_dbg(&dev->dev, "invalid or not supported wave form in upload\n");
113509ca1a9SAnssi Hannula return -EINVAL;
114509ca1a9SAnssi Hannula }
115509ca1a9SAnssi Hannula
116509ca1a9SAnssi Hannula if (!test_bit(effect->type, ff->ffbit)) {
117509ca1a9SAnssi Hannula ret = compat_effect(ff, effect);
118509ca1a9SAnssi Hannula if (ret)
119509ca1a9SAnssi Hannula return ret;
120509ca1a9SAnssi Hannula }
121509ca1a9SAnssi Hannula
122509ca1a9SAnssi Hannula mutex_lock(&ff->mutex);
123509ca1a9SAnssi Hannula
124509ca1a9SAnssi Hannula if (effect->id == -1) {
125509ca1a9SAnssi Hannula for (id = 0; id < ff->max_effects; id++)
126509ca1a9SAnssi Hannula if (!ff->effect_owners[id])
127509ca1a9SAnssi Hannula break;
128509ca1a9SAnssi Hannula
129509ca1a9SAnssi Hannula if (id >= ff->max_effects) {
130509ca1a9SAnssi Hannula ret = -ENOSPC;
131509ca1a9SAnssi Hannula goto out;
132509ca1a9SAnssi Hannula }
133509ca1a9SAnssi Hannula
134509ca1a9SAnssi Hannula effect->id = id;
135509ca1a9SAnssi Hannula old = NULL;
136509ca1a9SAnssi Hannula
137509ca1a9SAnssi Hannula } else {
138509ca1a9SAnssi Hannula id = effect->id;
139509ca1a9SAnssi Hannula
140509ca1a9SAnssi Hannula ret = check_effect_access(ff, id, file);
141509ca1a9SAnssi Hannula if (ret)
142509ca1a9SAnssi Hannula goto out;
143509ca1a9SAnssi Hannula
144509ca1a9SAnssi Hannula old = &ff->effects[id];
145509ca1a9SAnssi Hannula
146509ca1a9SAnssi Hannula if (!check_effects_compatible(effect, old)) {
147509ca1a9SAnssi Hannula ret = -EINVAL;
148509ca1a9SAnssi Hannula goto out;
149509ca1a9SAnssi Hannula }
150509ca1a9SAnssi Hannula }
151509ca1a9SAnssi Hannula
152509ca1a9SAnssi Hannula ret = ff->upload(dev, effect, old);
153509ca1a9SAnssi Hannula if (ret)
154509ca1a9SAnssi Hannula goto out;
155509ca1a9SAnssi Hannula
156656acd2bSDmitry Torokhov spin_lock_irq(&dev->event_lock);
157509ca1a9SAnssi Hannula ff->effects[id] = *effect;
158509ca1a9SAnssi Hannula ff->effect_owners[id] = file;
159656acd2bSDmitry Torokhov spin_unlock_irq(&dev->event_lock);
160509ca1a9SAnssi Hannula
161509ca1a9SAnssi Hannula out:
162509ca1a9SAnssi Hannula mutex_unlock(&ff->mutex);
163509ca1a9SAnssi Hannula return ret;
164509ca1a9SAnssi Hannula }
165509ca1a9SAnssi Hannula EXPORT_SYMBOL_GPL(input_ff_upload);
166509ca1a9SAnssi Hannula
167509ca1a9SAnssi Hannula /*
168509ca1a9SAnssi Hannula * Erases the effect if the requester is also the effect owner. The mutex
169509ca1a9SAnssi Hannula * should already be locked before calling this function.
170509ca1a9SAnssi Hannula */
erase_effect(struct input_dev * dev,int effect_id,struct file * file)171509ca1a9SAnssi Hannula static int erase_effect(struct input_dev *dev, int effect_id,
172509ca1a9SAnssi Hannula struct file *file)
173509ca1a9SAnssi Hannula {
174509ca1a9SAnssi Hannula struct ff_device *ff = dev->ff;
175509ca1a9SAnssi Hannula int error;
176509ca1a9SAnssi Hannula
177509ca1a9SAnssi Hannula error = check_effect_access(ff, effect_id, file);
178509ca1a9SAnssi Hannula if (error)
179509ca1a9SAnssi Hannula return error;
180509ca1a9SAnssi Hannula
181656acd2bSDmitry Torokhov spin_lock_irq(&dev->event_lock);
182509ca1a9SAnssi Hannula ff->playback(dev, effect_id, 0);
183656acd2bSDmitry Torokhov ff->effect_owners[effect_id] = NULL;
184656acd2bSDmitry Torokhov spin_unlock_irq(&dev->event_lock);
185509ca1a9SAnssi Hannula
186509ca1a9SAnssi Hannula if (ff->erase) {
187509ca1a9SAnssi Hannula error = ff->erase(dev, effect_id);
188656acd2bSDmitry Torokhov if (error) {
189656acd2bSDmitry Torokhov spin_lock_irq(&dev->event_lock);
190656acd2bSDmitry Torokhov ff->effect_owners[effect_id] = file;
191656acd2bSDmitry Torokhov spin_unlock_irq(&dev->event_lock);
192656acd2bSDmitry Torokhov
193509ca1a9SAnssi Hannula return error;
194509ca1a9SAnssi Hannula }
195656acd2bSDmitry Torokhov }
196509ca1a9SAnssi Hannula
197509ca1a9SAnssi Hannula return 0;
198509ca1a9SAnssi Hannula }
199509ca1a9SAnssi Hannula
200509ca1a9SAnssi Hannula /**
201e4477d2dSRandy Dunlap * input_ff_erase - erase a force-feedback effect from device
202509ca1a9SAnssi Hannula * @dev: input device to erase effect from
2034e3e4629SShailendra Verma * @effect_id: id of the effect to be erased
204509ca1a9SAnssi Hannula * @file: purported owner of the request
205509ca1a9SAnssi Hannula *
206509ca1a9SAnssi Hannula * This function erases a force-feedback effect from specified device.
207509ca1a9SAnssi Hannula * The effect will only be erased if it was uploaded through the same
208509ca1a9SAnssi Hannula * file handle that is requesting erase.
209509ca1a9SAnssi Hannula */
input_ff_erase(struct input_dev * dev,int effect_id,struct file * file)210509ca1a9SAnssi Hannula int input_ff_erase(struct input_dev *dev, int effect_id, struct file *file)
211509ca1a9SAnssi Hannula {
212509ca1a9SAnssi Hannula struct ff_device *ff = dev->ff;
213509ca1a9SAnssi Hannula int ret;
214509ca1a9SAnssi Hannula
215509ca1a9SAnssi Hannula if (!test_bit(EV_FF, dev->evbit))
216509ca1a9SAnssi Hannula return -ENOSYS;
217509ca1a9SAnssi Hannula
218509ca1a9SAnssi Hannula mutex_lock(&ff->mutex);
219509ca1a9SAnssi Hannula ret = erase_effect(dev, effect_id, file);
220509ca1a9SAnssi Hannula mutex_unlock(&ff->mutex);
221509ca1a9SAnssi Hannula
222509ca1a9SAnssi Hannula return ret;
223509ca1a9SAnssi Hannula }
224509ca1a9SAnssi Hannula EXPORT_SYMBOL_GPL(input_ff_erase);
225509ca1a9SAnssi Hannula
226509ca1a9SAnssi Hannula /*
227e8b95728SDmitry Torokhov * input_ff_flush - erase all effects owned by a file handle
228e8b95728SDmitry Torokhov * @dev: input device to erase effect from
229e8b95728SDmitry Torokhov * @file: purported owner of the effects
230e8b95728SDmitry Torokhov *
231e8b95728SDmitry Torokhov * This function erases all force-feedback effects associated with
232e8b95728SDmitry Torokhov * the given owner from specified device. Note that @file may be %NULL,
233e8b95728SDmitry Torokhov * in which case all effects will be erased.
234509ca1a9SAnssi Hannula */
input_ff_flush(struct input_dev * dev,struct file * file)235e8b95728SDmitry Torokhov int input_ff_flush(struct input_dev *dev, struct file *file)
236509ca1a9SAnssi Hannula {
237509ca1a9SAnssi Hannula struct ff_device *ff = dev->ff;
238509ca1a9SAnssi Hannula int i;
239509ca1a9SAnssi Hannula
2402afebafdSOliver Neukum dev_dbg(&dev->dev, "flushing now\n");
241509ca1a9SAnssi Hannula
242509ca1a9SAnssi Hannula mutex_lock(&ff->mutex);
243509ca1a9SAnssi Hannula
244509ca1a9SAnssi Hannula for (i = 0; i < ff->max_effects; i++)
245509ca1a9SAnssi Hannula erase_effect(dev, i, file);
246509ca1a9SAnssi Hannula
247509ca1a9SAnssi Hannula mutex_unlock(&ff->mutex);
248509ca1a9SAnssi Hannula
249509ca1a9SAnssi Hannula return 0;
250509ca1a9SAnssi Hannula }
251e8b95728SDmitry Torokhov EXPORT_SYMBOL_GPL(input_ff_flush);
252509ca1a9SAnssi Hannula
253509ca1a9SAnssi Hannula /**
254509ca1a9SAnssi Hannula * input_ff_event() - generic handler for force-feedback events
255509ca1a9SAnssi Hannula * @dev: input device to send the effect to
256509ca1a9SAnssi Hannula * @type: event type (anything but EV_FF is ignored)
257509ca1a9SAnssi Hannula * @code: event code
258509ca1a9SAnssi Hannula * @value: event value
259509ca1a9SAnssi Hannula */
input_ff_event(struct input_dev * dev,unsigned int type,unsigned int code,int value)260509ca1a9SAnssi Hannula int input_ff_event(struct input_dev *dev, unsigned int type,
261509ca1a9SAnssi Hannula unsigned int code, int value)
262509ca1a9SAnssi Hannula {
263509ca1a9SAnssi Hannula struct ff_device *ff = dev->ff;
264509ca1a9SAnssi Hannula
265509ca1a9SAnssi Hannula if (type != EV_FF)
266509ca1a9SAnssi Hannula return 0;
267509ca1a9SAnssi Hannula
268509ca1a9SAnssi Hannula switch (code) {
269509ca1a9SAnssi Hannula case FF_GAIN:
270379d7cfaSDan Carpenter if (!test_bit(FF_GAIN, dev->ffbit) || value > 0xffffU)
271509ca1a9SAnssi Hannula break;
272509ca1a9SAnssi Hannula
273509ca1a9SAnssi Hannula ff->set_gain(dev, value);
274509ca1a9SAnssi Hannula break;
275509ca1a9SAnssi Hannula
276509ca1a9SAnssi Hannula case FF_AUTOCENTER:
277379d7cfaSDan Carpenter if (!test_bit(FF_AUTOCENTER, dev->ffbit) || value > 0xffffU)
278509ca1a9SAnssi Hannula break;
279509ca1a9SAnssi Hannula
280509ca1a9SAnssi Hannula ff->set_autocenter(dev, value);
281509ca1a9SAnssi Hannula break;
282509ca1a9SAnssi Hannula
283509ca1a9SAnssi Hannula default:
284dec3eb01SDmitry Torokhov if (check_effect_access(ff, code, NULL) == 0)
285509ca1a9SAnssi Hannula ff->playback(dev, code, value);
286509ca1a9SAnssi Hannula break;
287509ca1a9SAnssi Hannula }
288509ca1a9SAnssi Hannula
289509ca1a9SAnssi Hannula return 0;
290509ca1a9SAnssi Hannula }
291509ca1a9SAnssi Hannula EXPORT_SYMBOL_GPL(input_ff_event);
292509ca1a9SAnssi Hannula
293509ca1a9SAnssi Hannula /**
294509ca1a9SAnssi Hannula * input_ff_create() - create force-feedback device
295509ca1a9SAnssi Hannula * @dev: input device supporting force-feedback
296509ca1a9SAnssi Hannula * @max_effects: maximum number of effects supported by the device
297509ca1a9SAnssi Hannula *
298509ca1a9SAnssi Hannula * This function allocates all necessary memory for a force feedback
299509ca1a9SAnssi Hannula * portion of an input device and installs all default handlers.
300509ca1a9SAnssi Hannula * @dev->ffbit should be already set up before calling this function.
301509ca1a9SAnssi Hannula * Once ff device is created you need to setup its upload, erase,
302509ca1a9SAnssi Hannula * playback and other handlers before registering input device
303509ca1a9SAnssi Hannula */
input_ff_create(struct input_dev * dev,unsigned int max_effects)30405be8b81SDan Carpenter int input_ff_create(struct input_dev *dev, unsigned int max_effects)
305509ca1a9SAnssi Hannula {
306509ca1a9SAnssi Hannula struct ff_device *ff;
30705be8b81SDan Carpenter size_t ff_dev_size;
308509ca1a9SAnssi Hannula int i;
309509ca1a9SAnssi Hannula
310509ca1a9SAnssi Hannula if (!max_effects) {
3112afebafdSOliver Neukum dev_err(&dev->dev, "cannot allocate device without any effects\n");
312509ca1a9SAnssi Hannula return -EINVAL;
313509ca1a9SAnssi Hannula }
314509ca1a9SAnssi Hannula
31533b96d93SElias Vanderstuyft if (max_effects > FF_MAX_EFFECTS) {
31633b96d93SElias Vanderstuyft dev_err(&dev->dev, "cannot allocate more than FF_MAX_EFFECTS effects\n");
31733b96d93SElias Vanderstuyft return -EINVAL;
31833b96d93SElias Vanderstuyft }
31933b96d93SElias Vanderstuyft
320*c727e46fSErick Archer ff_dev_size = struct_size(ff, effect_owners, max_effects);
321*c727e46fSErick Archer if (ff_dev_size == SIZE_MAX) /* overflow */
32205be8b81SDan Carpenter return -EINVAL;
32305be8b81SDan Carpenter
32405be8b81SDan Carpenter ff = kzalloc(ff_dev_size, GFP_KERNEL);
325509ca1a9SAnssi Hannula if (!ff)
326509ca1a9SAnssi Hannula return -ENOMEM;
327509ca1a9SAnssi Hannula
328509ca1a9SAnssi Hannula ff->effects = kcalloc(max_effects, sizeof(struct ff_effect),
329509ca1a9SAnssi Hannula GFP_KERNEL);
330509ca1a9SAnssi Hannula if (!ff->effects) {
331509ca1a9SAnssi Hannula kfree(ff);
332509ca1a9SAnssi Hannula return -ENOMEM;
333509ca1a9SAnssi Hannula }
334509ca1a9SAnssi Hannula
335509ca1a9SAnssi Hannula ff->max_effects = max_effects;
336509ca1a9SAnssi Hannula mutex_init(&ff->mutex);
337509ca1a9SAnssi Hannula
338509ca1a9SAnssi Hannula dev->ff = ff;
339e8b95728SDmitry Torokhov dev->flush = input_ff_flush;
340509ca1a9SAnssi Hannula dev->event = input_ff_event;
341bf3204cbSDmitry Torokhov __set_bit(EV_FF, dev->evbit);
342509ca1a9SAnssi Hannula
343509ca1a9SAnssi Hannula /* Copy "true" bits into ff device bitmap */
344948cea14SAnshul Garg for_each_set_bit(i, dev->ffbit, FF_CNT)
345bf3204cbSDmitry Torokhov __set_bit(i, ff->ffbit);
346509ca1a9SAnssi Hannula
347509ca1a9SAnssi Hannula /* we can emulate RUMBLE with periodic effects */
348509ca1a9SAnssi Hannula if (test_bit(FF_PERIODIC, ff->ffbit))
349bf3204cbSDmitry Torokhov __set_bit(FF_RUMBLE, dev->ffbit);
350509ca1a9SAnssi Hannula
351509ca1a9SAnssi Hannula return 0;
352509ca1a9SAnssi Hannula }
353509ca1a9SAnssi Hannula EXPORT_SYMBOL_GPL(input_ff_create);
354509ca1a9SAnssi Hannula
355509ca1a9SAnssi Hannula /**
356721a730eSRoger Quadros * input_ff_destroy() - frees force feedback portion of input device
357e4477d2dSRandy Dunlap * @dev: input device supporting force feedback
358509ca1a9SAnssi Hannula *
359509ca1a9SAnssi Hannula * This function is only needed in error path as input core will
360509ca1a9SAnssi Hannula * automatically free force feedback structures when device is
361509ca1a9SAnssi Hannula * destroyed.
362509ca1a9SAnssi Hannula */
input_ff_destroy(struct input_dev * dev)363509ca1a9SAnssi Hannula void input_ff_destroy(struct input_dev *dev)
364509ca1a9SAnssi Hannula {
365bf3204cbSDmitry Torokhov struct ff_device *ff = dev->ff;
366bf3204cbSDmitry Torokhov
367bf3204cbSDmitry Torokhov __clear_bit(EV_FF, dev->evbit);
368bf3204cbSDmitry Torokhov if (ff) {
369bf3204cbSDmitry Torokhov if (ff->destroy)
370bf3204cbSDmitry Torokhov ff->destroy(ff);
371bf3204cbSDmitry Torokhov kfree(ff->private);
3726a47081cSJari Vanhala kfree(ff->effects);
373bf3204cbSDmitry Torokhov kfree(ff);
374509ca1a9SAnssi Hannula dev->ff = NULL;
375509ca1a9SAnssi Hannula }
376509ca1a9SAnssi Hannula }
377509ca1a9SAnssi Hannula EXPORT_SYMBOL_GPL(input_ff_destroy);
378