Re: [PATCH v5 08/11] input: raspberrypi-ts: Release firmware handle when not needed

2020-12-01 Thread Dmitry Torokhov
Hi Nicolas, On Mon, Nov 23, 2020 at 07:38:29PM +0100, Nicolas Saenz Julienne wrote: > Use devm_rpi_firmware_get() so as to make sure we release RPi's firmware > interface when unbinding the device. I do not believe this comment is correct any longer. Otherwise: Acked-by: Dmit

Re: [PATCH v4 01/11] firmware: raspberrypi: Keep count of all consumers

2020-11-12 Thread Dmitry Torokhov
On Thu, Nov 12, 2020 at 07:52:14PM +0200, Andy Shevchenko wrote: > On Thu, Nov 12, 2020 at 6:40 PM Nicolas Saenz Julienne > wrote: > > > > When unbinding the firmware device we need to make sure it has no > > consumers left. Otherwise we'd leave them with a firmware handle > > pointing at freed me

Re: [PATCH v3 07/11] input: raspberrypi-ts: Release firmware handle when not needed

2020-11-11 Thread Dmitry Torokhov
Hi Nicolas, On Wed, Nov 04, 2020 at 11:39:33AM +0100, Nicolas Saenz Julienne wrote: > Use devm_rpi_firmware_get() so as to make sure we release RPi's firmware > interface when unbinding the device. Unless I am mistaken this driver does not really need the firmware structure past rpi_ts_probe(), a

Re: [PATCH 0/3] Modernize tasklet callback API

2020-07-16 Thread Dmitry Torokhov
On Thu, Jul 16, 2020 at 12:14 PM Kees Cook wrote: > > On Thu, Jul 16, 2020 at 09:57:18AM +0200, Peter Zijlstra wrote: > > On Wed, Jul 15, 2020 at 08:08:44PM -0700, Kees Cook wrote: > > > Hi, > > > > > > This is the infrastructure changes to prepare the tasklet API for > > > conversion to passing t

Re: [PATCH 01/13] staging: gasket: core: hold reference to pci_dev while used

2018-07-30 Thread Dmitry Torokhov
On Sun, Jul 29, 2018 at 12:37 PM Todd Poynor wrote: > > From: Todd Poynor > > Hold a reference on the struct pci_dev while a pointer to it is held in > the gasket data structures. > > Signed-off-by: Todd Poynor > --- > drivers/staging/gasket/gasket_core.c | 4 ++-- > 1 file changed, 2 insertion

Re: [PATCH 04/13] staging: gasket: core: allow root access based on user namespace

2018-07-30 Thread Dmitry Torokhov
Hi Todd, On Sun, Jul 29, 2018 at 12:37 PM Todd Poynor wrote: > @@ -1064,7 +1067,8 @@ static int gasket_open(struct inode *inode, struct file > *filp) > char task_name[TASK_COMM_LEN]; > struct gasket_cdev_info *dev_info = > container_of(inode->i_cdev, struct gasket_cde

Re: [PATCH 11/18] staging: gasket: always allow root open for write

2018-07-15 Thread Dmitry Torokhov
On Sun, Jul 15, 2018 at 12:32 PM Greg Kroah-Hartman wrote: > > On Sun, Jul 15, 2018 at 12:11:47PM +0300, Dmitry Torokhov wrote: > > On Sun, Jul 15, 2018 at 12:05 PM Greg Kroah-Hartman > > wrote: > > > > > > On Fri, Jul 13, 2018 at 10:58:09PM -0700, Todd Poy

Re: [PATCH 11/18] staging: gasket: always allow root open for write

2018-07-15 Thread Dmitry Torokhov
On Sun, Jul 15, 2018 at 12:05 PM Greg Kroah-Hartman wrote: > > On Fri, Jul 13, 2018 at 10:58:09PM -0700, Todd Poynor wrote: > > From: Todd Poynor > > > > Always allow root to open device for writing. > > > > Drop special-casing of ioctl permissions for root vs

Re: [PATCH 11/18] staging: gasket: always allow root open for write

2018-07-14 Thread Dmitry Torokhov
On Sat, Jul 14, 2018 at 8:58 AM Todd Poynor wrote: > > From: Todd Poynor > > Always allow root to open device for writing. > > Drop special-casing of ioctl permissions for root vs. owner. > > Reported-by: Dmitry Torokhov > Signed-off-by: Zhongze Hu > Signed-off-

Re: [PATCH 14/18] staging: gasket: fix class create bug handling

2018-07-14 Thread Dmitry Torokhov
On Sat, Jul 14, 2018 at 8:59 AM Todd Poynor wrote: > > From: Todd Poynor > > class_create() never returns NULL, and this driver should never return > PTR_ERR(NULL) anyway. > > Reported-by: Dmitry Torokhov > Signed-off-by: Zhongze Hu > Signed-off-by: Todd Poynor R

Re: [PATCH 06/18] staging: gasket: fix deadlock in pci driver unregister path

2018-07-14 Thread Dmitry Torokhov
On Sat, Jul 14, 2018 at 8:58 AM Todd Poynor wrote: > > From: Todd Poynor > > g_mutex held across pci_unregister_driver() call, also held in > gasket_pci_remove(), which deadlocks. > > Reported-by: Dmitry Torokhov > Signed-off-by: Zhongze Hu > Signed-off-by: To

Re: [PATCH 10/18] staging: gasket: gasket_open use container_of()

2018-07-14 Thread Dmitry Torokhov
On Sat, Jul 14, 2018 at 8:58 AM Todd Poynor wrote: > > From: Todd Poynor > > Use container_of(), drop unnecessary NULL check. > > Reported-by: Dmitry Torokhov > Signed-off-by: Zhongze Hu > Signed-off-by: Todd Poynor > --- > drivers/staging/gasket/gasket_core.c

Re: [PATCH v2 2/5] x86: add enum for hypervisors to replace x86_hyper

2017-11-10 Thread Dmitry Torokhov
hyperv.c| 4 ++-- > arch/x86/kernel/cpu/vmware.c | 4 ++-- > arch/x86/kernel/kvm.c | 4 ++-- > arch/x86/xen/enlighten_hvm.c | 4 ++-- > arch/x86/xen/enlighten_pv.c | 4 ++-- > drivers/hv/vmbus_drv.c| 2 +- > drivers/input/mouse/vmmouse.c |

[PATCH] android: binder: check result of binder_get_thread() in binder_poll()

2017-05-08 Thread Dmitry Torokhov
If binder_get_thread() fails to give us a thread data, we should avoid dereferencing a NULL pointer and return POLLERR instead. Signed-off-by: Dmitry Torokhov --- drivers/android/binder.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/drivers/android/binder.c b

[PATCH] [media] Staging: media: radio-bcm2048: remove incorrect __exit markups

2017-03-01 Thread Dmitry Torokhov
: Dmitry Torokhov --- drivers/staging/media/bcm2048/radio-bcm2048.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c index 37bd439ee08b..1fba377f816b 100644 --- a/drivers/staging/media

Re: [PATCH] driver-core: fix odd_ptr_err.cocci warnings

2016-11-10 Thread Dmitry Torokhov
[ resending as plain text ] On Thu, Nov 10, 2016 at 12:18 PM, Dmitry Torokhov wrote: > > > On Thu, Nov 10, 2016 at 12:16 PM, Julia Lawall wrote: >> >> PTR_ERR should access the value just tested by IS_ERR >> >> Generated by: scripts/coccinelle/tests/odd_ptr_er

Re: [PATCH] android: fix warning when releasing active sync point

2015-12-15 Thread Dmitry Torokhov
On Tue, Dec 15, 2015 at 11:00 AM, Gustavo Padovan wrote: > 2015-12-15 Daniel Vetter : > >> On Mon, Dec 14, 2015 at 05:29:55PM -0800, Dmitry Torokhov wrote: >> > Userspace can close the sync device while there are still active fence >> > points, in which case kernel

Re: [PATCH] android: fix warning when releasing active sync point

2015-12-15 Thread Dmitry Torokhov
On Tue, Dec 15, 2015 at 1:26 AM, Daniel Vetter wrote: > On Mon, Dec 14, 2015 at 05:29:55PM -0800, Dmitry Torokhov wrote: >> Userspace can close the sync device while there are still active fence >> points, in which case kernel produces the following warning: >> >> [ 4

Re: [PATCH] android: fix warning when releasing active sync point

2015-12-15 Thread Dmitry Torokhov
On Tue, Dec 15, 2015 at 5:30 AM, Gustavo Padovan wrote: > 2015-12-14 Dmitry Torokhov : > >> Userspace can close the sync device while there are still active fence >> points, in which case kernel produces the following warning: >> >> [ 43.

Re: [PATCH] android: fix warning when releasing active sync point

2015-12-15 Thread Dmitry Torokhov
On Tue, Dec 15, 2015 at 2:01 AM, Maarten Lankhorst wrote: > Op 15-12-15 om 02:29 schreef Dmitry Torokhov: >> Userspace can close the sync device while there are still active fence >> points, in which case kernel produces the following warning: >> >> [ 43.853

Re: [PATCH] android: fix warning when releasing active sync point

2015-12-15 Thread Dmitry Torokhov
On Tue, Dec 15, 2015 at 5:50 AM, Frank Binns wrote: > Is this not the issue fixed by 8e43c9c75? No because if we start teardown without waiting for the fence to be signaled it will still be on the active_list. Thanks. -- Dmitry ___ devel mailing list

[PATCH] android: unconditionally remove callbacks in sync_fence_free()

2015-12-14 Thread Dmitry Torokhov
_fence, we guarantee that each callback has either completed (since fence_remove_callback() grabs the fence lock) or been removed. Signed-off-by: Andrew Bresticker Signed-off-by: Dmitry Torokhov --- drivers/staging/android/sync.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff

[PATCH] android: fix warning when releasing active sync point

2015-12-14 Thread Dmitry Torokhov
lback (disable_signaling) to fence operations so that drivers can do proper clean ups when we remove last callback for given fence. Reviewed-by: Andrew Bresticker Signed-off-by: Dmitry Torokhov --- drivers/dma-buf/fence.c| 6 +- drivers/staging/android/sync.c | 8 include/

Re: [PATCH] Input: hyperv-keyboard - implement Type Clipboard Text

2014-08-18 Thread Dmitry Torokhov
On Mon, Aug 18, 2014 at 03:54:51AM +, Dexuan Cui wrote: > > -Original Message- > > From: Dmitry Torokhov > > Sent: Saturday, August 16, 2014 0:58 AM > > To: Dexuan Cui > > > For each char in the string, the host sends 2 events (key down/up with the &

Re: [PATCH] Input: hyperv-keyboard - implement Type Clipboard Text

2014-08-15 Thread Dmitry Torokhov
Hi Dexuan, On Fri, Aug 15, 2014 at 02:41:59AM -0700, Dexuan Cui wrote: > In the menu of the Hyper-V's Virtual Machine Connection, there is a feature > called "Clipboard | Type clipboard text", which can be used to copy a string > in the host's clipboard into the guest's current input focus(text co

Re: [PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-13 Thread Dmitry Torokhov
On Wed, Aug 13, 2014 at 05:24:35AM +, Dexuan Cui wrote: > > -Original Message- > > From: Dmitry Torokhov > > Sent: Wednesday, August 13, 2014 1:55 AM > > > > > To decouple the dependency between the hyperv-keyboard and i8042 > > > > > mod

Re: [PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-12 Thread Dmitry Torokhov
On Tue, Aug 12, 2014 at 07:15:25AM +, Dexuan Cui wrote: > > -Original Message- > > From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- > > ow...@vger.kernel.org] On Behalf Of Greg KH > > > > What exactly needs to be done to fix this "correctly" that is going to > > > > take t

Re: [PATCH] Input: hyperv-keyboard: register as a wakeup source

2014-08-06 Thread Dmitry Torokhov
On Wed, Aug 6, 2014 at 5:19 PM, Dexuan Cui wrote: > > > -Original Message- > > From: Dmitry Torokhov [mailto:dmitry.torok...@gmail.com] > > Sent: Thursday, August 7, 2014 4:34 AM > > To: Dexuan Cui > > On Fri, Aug 01, 2014 at 07:28:24AM -0700, Dexuan Cui

Re: [PATCH] Input: hyperv-keyboard: register as a wakeup source

2014-08-06 Thread Dmitry Torokhov
On Fri, Aug 01, 2014 at 07:28:24AM -0700, Dexuan Cui wrote: > With this patch, we can press a key to wake up the VM after the VM executes > "echo freeze > /sys/power/state". > > This addresses part of https://bugzilla.redhat.com/show_bug.cgi?id=1086100 I am sorry but this bug is not open to the p

Re: Status of RMI4 drivers?

2014-07-06 Thread Dmitry Torokhov
On Sat, Jul 5, 2014 at 8:39 PM, Greg KH wrote: > On Sat, Jul 05, 2014 at 04:37:37PM +0100, Alan Cox wrote: >> On Fri, 2014-07-04 at 21:48 +0300, Kristina Martšenko wrote: >> > Hi, >> > >> > I'm going over some "older" drivers in the staging tree and wanted to >> > ask about cptm1217 and ste_rmi4.

Re: [PATCH 1/1] Drivers: input: serio:hyperv-keyoard: Handle 0xE1 prefix

2014-01-12 Thread Dmitry Torokhov
On Sat, Jan 11, 2014 at 07:41:50PM -0800, K. Y. Srinivasan wrote: > Handle the 0xE1 prefix. > > Signed-off-by: K. Y. Srinivasan Applied, thank you. > --- > drivers/input/serio/hyperv-keyboard.c |4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/drivers/input/seri

Re: [PATCH V2 1/1] Drivers: input: serio: New driver to support Hyper-V synthetic keyboard

2013-09-23 Thread Dmitry Torokhov
On Monday, September 23, 2013 11:08:12 PM Olaf Hering wrote: > On Wed, Sep 18, Dmitry Torokhov wrote: > > This looks much better. Could you tell me if the patch below (on top of > > yours) still works? > > The help text is slightly incorrect, its a dash not underscore:

Re: [PATCH V2 1/1] Drivers: input: serio: New driver to support Hyper-V synthetic keyboard

2013-09-19 Thread Dmitry Torokhov
On Wed, Sep 18, 2013 at 11:27:51PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Dmitry Torokhov [mailto:dmitry.torok...@gmail.com] > > Sent: Wednesday, September 18, 2013 2:01 PM > > To: KY Srinivasan > > Cc: gre...@linuxfoundati

Re: [PATCH V2 1/1] Drivers: input: serio: New driver to support Hyper-V synthetic keyboard

2013-09-18 Thread Dmitry Torokhov
ech Pavlik for helping me with the > details of the AT keyboard driver. I would also like to thank > Dan Carpenter and > Dmitry Torokhov for their detailed review of this > driver. > > I have addressed all the comments of Dan and Dmitry in this version of > the patch This

Re: [PATCH 1/1] Drivers: input: serio: New driver to support Hyper-V synthetic keyboard

2013-09-16 Thread Dmitry Torokhov
On Mon, Sep 16, 2013 at 06:29:45PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Dmitry Torokhov [mailto:dmitry.torok...@gmail.com] > > Sent: Monday, September 16, 2013 10:10 AM > > To: KY Srinivasan > > Cc: Dan Carpenter; o...@a

Re: [PATCH 1/1] Drivers: input: serio: New driver to support Hyper-V synthetic keyboard

2013-09-16 Thread Dmitry Torokhov
On Mon, Sep 16, 2013 at 03:52:18PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Dmitry Torokhov [mailto:dmitry.torok...@gmail.com] > > Sent: Monday, September 16, 2013 8:20 AM > > To: KY Srinivasan > > Cc: gre...@linuxfoundati

Re: [PATCH 1/1] Drivers: input: serio: New driver to support Hyper-V synthetic keyboard

2013-09-16 Thread Dmitry Torokhov
On Mon, Sep 16, 2013 at 04:56:03PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > > Sent: Monday, September 16, 2013 8:06 AM > > To: KY Srinivasan > > Cc: o...@aepfle.de; gre...@linuxfoundation.org; jasow...@redhat.com; >

Re: [PATCH 5/6] Staging/iio/adc/touchscreen/MXS: add interrupt driven touch detection

2013-09-16 Thread Dmitry Torokhov
On Sun, Sep 15, 2013 at 11:56:25AM +0100, Jonathan Cameron wrote: > On 09/11/13 09:18, Juergen Beisert wrote: > > For battery driven systems it is a very bad idea to collect the touchscreen > > data within a kernel busy loop. > > > > This change uses the features of the hardware to delay and accum

Re: [PATCH 1/1] Drivers: input: serio: New driver to support Hyper-V synthetic keyboard

2013-09-16 Thread Dmitry Torokhov
Hi K. Y. On Sun, Sep 15, 2013 at 10:28:54PM -0700, K. Y. Srinivasan wrote: > Add a new driver to support synthetic keyboard. On the next generation > Hyper-V guest firmware, many legacy devices will not be emulated and this > driver will be required. > > I would like to thank Vojtech Pavlik for