usb: gadget: f_fs: remove redundant ffs_data_get()
authorRobert Baldyga <r.baldyga@samsung.com>
Thu, 9 Oct 2014 07:41:16 +0000 (09:41 +0200)
committerFelipe Balbi <balbi@ti.com>
Thu, 23 Oct 2014 14:55:42 +0000 (09:55 -0500)
commita3058a5d82e296daaca07411c3738a9ddd79f302
tree5556fb3b080d2fae4fa365c5f6c1115142351f3e
parentf1113be1bff674b8410fcb301d6551ab5a43193e
usb: gadget: f_fs: remove redundant ffs_data_get()

During FunctionFS bind, ffs_data_get() function was called twice
(in functionfs_bind() and in ffs_do_functionfs_bind()), while on unbind
ffs_data_put() was called once (in functionfs_unbind() function).
In result refcount never reached value 0, and ffs memory resources has
been never released.

Since ffs_data_get() call in ffs_do_functionfs_bind() is redundant
and not neccessary, we remove it to have equal number of gets ans puts,
and free allocated memory after refcount reach 0.

Fixes: 5920cda (usb: gadget: FunctionFS: convert to new function
interface with backward compatibility)
Cc: <stable@vger.kernel.org> # v3.14+
Signed-off-by: Robert Baldyga <r.baldyga@samsung.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/function/f_fs.c