Re: [PATCH] Staging: TIDSPBRIDGE: Use vm_iomap_memory for mmap-ing instead of remap_pfn_range

2013-12-11 Thread Dan Carpenter
On Wed, Dec 11, 2013 at 09:45:52AM +0200, Ivajlo Dimitrov wrote: I can pick your changes and re-send the original patch with them incorporated if there are no objections. Are you fine with that? Do it on top of staging-next, don't redo the original. regards, dan carpenter

Re: staging: comedi: USB devs not working / some comedi core reorganization

2013-12-11 Thread Bernd Porr
Hi all, I've drilled down further. After an async command has run the channel list is kfree'd in do_become_nonbusy. The pointer is not NULL but I guess that's been kfree'd already somewhere else and there is no need to do it here? The comedi_command_test is actually OK. Ignore the report

Re: [PATCH] Staging: TIDSPBRIDGE: Use vm_iomap_memory for mmap-ing instead of remap_pfn_range

2013-12-11 Thread Ivaylo Dimitrov
On 11.12.2013 10:33, Dan Carpenter wrote: On Wed, Dec 11, 2013 at 09:45:52AM +0200, Ivajlo Dimitrov wrote: I can pick your changes and re-send the original patch with them incorporated if there are no objections. Are you fine with that? Do it on top of staging-next, don't redo the original.

Re: [PATCH] staging: comedi: drivers: fix return value of comedi_load_firmware()

2013-12-11 Thread Ian Abbott
On 2013-12-10 23:31, H Hartley Sweeten wrote: Some of the callback functions that upload the firmware in the comedi drivers return a positive value indicating the number of bytes sent to the device. Detect this condition and just return '0' to indicate a successful upload. Acked-by: Ian

Re: [PATCH] Staging: TIDSPBRIDGE: Use vm_iomap_memory for mmap-ing instead of remap_pfn_range

2013-12-11 Thread Dan Carpenter
On Wed, Dec 11, 2013 at 11:57:04AM +0200, Ivaylo Dimitrov wrote: On 11.12.2013 10:33, Dan Carpenter wrote: On Wed, Dec 11, 2013 at 09:45:52AM +0200, Ivajlo Dimitrov wrote: I can pick your changes and re-send the original patch with them incorporated if there are no objections. Are you fine

Re: [PATCH 1/3] vme_user: Ensure driver compiles after VME bridges

2013-12-11 Thread Martyn Welch
On 05/12/13 00:29, Aaron Sierra wrote: Martyn, Your suggestion resolves the kernel panic associated with the vme_user driver being initialized before the VME bus core driver, but it doesn't address the issue of the vme_user driver being registered before any buses have been probed:

[PATCH] staging: comedi: drivers: fix kernel oops when channel list has invalid pointer

2013-12-11 Thread Bernd Porr
Hi all, I think with this patch everything seems to be running fine now. The kernel oops was actually caused by a bug in one of my userspace programs (forgot to set the channel list in cmd) but of course that shouldn't have been possible. That's now properly dealt with and we get a bad

Re: [PATCH v2] vme_user: Update API to work in mixed environments

2013-12-11 Thread Martyn Welch
On 09/12/13 16:05, Aaron Sierra wrote: This patch updates the vme_master and vme_slave structures to use types with well defined size and to prevent the compiler from inserting padding (between enable and vme_addr for one). The original vme_master and vme_slave structs would be different

[PATCH 0/2] staging/comedi: a couple of remove device bug fixes

2013-12-11 Thread Ian Abbott
Fix some bugs related to removing dynamically allocated comedi devices that have open file objects. 1) staging/comedi: keep reference to class device after destroyed 2) staging/comedi: bug fix for module usage count on device removal drivers/staging/comedi/comedi_fops.c | 39

[PATCH 1/2] staging/comedi: keep reference to class device after destroyed

2013-12-11 Thread Ian Abbott
When a dynamically allocated `struct comedi_device` gets automatically unconfigured by a call to `comedi_auto_unconfig()` from a lower-level driver's bus removal function (e.g. when a USB device is disconnected), the class device in `dev-class_dev` (where `dev` points to the `struct

Re: [PATCH] staging: comedi: drivers: fix kernel oops when channel list has invalid pointer

2013-12-11 Thread Ian Abbott
On 2013-12-11 11:58, Bernd Porr wrote: From d83a3e0cda7559e9b91759ab4ef8a6c3eb19fbc0 Mon Sep 17 00:00:00 2001 From: Bernd Porrm...@berndporr.me.uk Date: Wed, 11 Dec 2013 11:45:09 + Subject: [PATCH 1/1] If the channel list is not set in userspace we get an error at

[PATCH] staging: comedi: fix result of memdup_user for user chanlist

2013-12-11 Thread Ian Abbott
From: Bernd Porr m...@berndporr.me.uk If the channel list is not set in userspace we get an error at PTR_ERR(async-cmd.chanlist). However, do_become_nonbusy(dev, s) cleans up this pointer which causes a kernel ooops. Setting the channel list in async to NULL and checking this in do_become_nonbusy

Re: [PATCH v1 9/9] staging: android: binder: Add binder compat layer

2013-12-11 Thread Octavian Purdila
On Wed, Dec 11, 2013 at 5:21 AM, Arve Hjønnevåg a...@android.com wrote: Assuming you are talking about a kernel compat layer that translates the flat_binder_object structs as they pass between 32 bit and 64 bit processes, that will not always work. The data portion of the message sometimes

Re: [PATCH v1 9/9] staging: android: binder: Add binder compat layer

2013-12-11 Thread Arve Hjønnevåg
On Wed, Dec 11, 2013 at 10:10 AM, Octavian Purdila octavian.purd...@intel.com wrote: On Wed, Dec 11, 2013 at 5:21 AM, Arve Hjønnevåg a...@android.com wrote: Assuming you are talking about a kernel compat layer that translates the flat_binder_object structs as they pass between 32 bit and 64

Re: [PATCH 00/31] ARM: tegra: use common reset and DMA bindings

2013-12-11 Thread Stephen Warren
On 11/15/2013 01:53 PM, Stephen Warren wrote: From: Stephen Warren swar...@nvidia.com This series implements a common reset framework driver for Tegra, and updates all relevant Tegra drivers to use it. It also removes the custom DMA bindings and replaced them with the standard DMA DT

Re: [PATCH] Staging: TIDSPBRIDGE: Use vm_iomap_memory for mmap-ing instead of remap_pfn_range

2013-12-11 Thread Greg KH
On Wed, Dec 11, 2013 at 01:27:17PM +0300, Dan Carpenter wrote: On Wed, Dec 11, 2013 at 11:57:04AM +0200, Ivaylo Dimitrov wrote: On 11.12.2013 10:33, Dan Carpenter wrote: On Wed, Dec 11, 2013 at 09:45:52AM +0200, Ivajlo Dimitrov wrote: I can pick your changes and re-send the original patch

[PATCH -next] [media] radio-bcm2048: fix missing unlock on error in bcm2048_rds_fifo_receive()

2013-12-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function bcm2048_rds_fifo_receive() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/media/bcm2048/radio-bcm2048.c | 1 + 1 file changed, 1