Thread (8 messages) 8 messages, 4 authors, 2016-06-21

Re: [PATCH] Alps I2C HID Touchpad-Stick support

flat view

From: Jiri Kosina <jikos@kernel.org>
Date: 2016-06-21 08:23:45

On Mon, 20 Jun 2016, Dmitry Torokhov wrote:
quoted
+struct u1_dev *priv;
I do not understand why you need this global. You can allocate arbitrary
memory in probe() and you actually already do that, why do you need
second copy?
Mea culpa, I raised this concern already quite some time ago, and missed 
the fact that it wasn't fixed in further submissions :/ 

Thanks,

-- 
Jiri Kosina
SUSE Labs
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help