[PATCH] usb: gadget: uvc: Fix return value in case of error

2016-07-16 Thread Christophe JAILLET
If this memory allocation fail, we will return 0, which means success. Return -ENOMEM instead. Signed-off-by: Christophe JAILLET --- drivers/usb/gadget/function/uvc_configfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/function/uvc_configfs.c b/driver

[for-next][PATCH 4/7] tracing: Use outer () on __get_str() definition

2016-07-16 Thread Steven Rostedt
From: Daniel Bristot de Oliveira __get_str(str)'s definition includes a (char *) operator overloading that is not protected with outer (). This patch adds () around __get_str()'s definition, enabling some code cleanup. Link: http://lkml.kernel.org/r/20ac1a10c2ec4ccd23e4a8ef34101fb6e4157d37.146

[for-next][PATCH 5/7] tracing, RAS: Cleanup on __get_str() usage

2016-07-16 Thread Steven Rostedt
From: Daniel Bristot de Oliveira __get_str(msg) does not need (char *) operator overloading to access mgs's elements anymore. This patch substitutes ((char *)__get_str(msg))[0] usage to __get_str(msg)[0]. It is just a code cleanup, no changes on tracepoint ABI. Link: http://lkml.kernel.org/r/6

[for-next][PATCH 1/7] tracing: Using for_each_set_bit() to simplify trace_pid_write()

2016-07-16 Thread Steven Rostedt
From: Wei Yongjun Using for_each_set_bit() to simplify the code. Link: http://lkml.kernel.org/r/1467645004-11169-1-git-send-email-weiyj...@163.com Signed-off-by: Wei Yongjun Signed-off-by: Steven Rostedt --- kernel/trace/trace.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-)

[for-next][PATCH 0/7] tracing: Some last updates for 4.8

2016-07-16 Thread Steven Rostedt
git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next Head SHA1: 78aebca2c955c1c5aeb48e12645e13fe3c3461f2 Daniel Bristot de Oliveira (4): tracing: Use outer () on __get_str() definition tracing, RAS: Cleanup on __get_str() usage tracing: Use __get_str

[for-next][PATCH 2/7] tracing: Have HIST_TRIGGERS select TRACING

2016-07-16 Thread Steven Rostedt
From: Tom Zanussi The kbuild test robot reported a compile error if HIST_TRIGGERS was enabled but nothing else that selected TRACING was configured in. HIST_TRIGGERS should directly select it and not rely on anything else to do it. Link: http://lkml.kernel.org/r/57791866.8080...@linux.intel.com

[for-next][PATCH 7/7] printk, tracing: Avoiding unneeded blank lines

2016-07-16 Thread Steven Rostedt
From: Daniel Bristot de Oliveira Printk messages often finish with '\n' to cause a new line. But as each tracepoint is already printed in a new line, printk messages that finish with '\n' ends up adding a blank line to the trace output. For example: kworker/0:1-86[000] d...46.006949

[for-next][PATCH 6/7] tracing: Use __get_str() when manipulating strings

2016-07-16 Thread Steven Rostedt
From: Daniel Bristot de Oliveira Use __get_str(str) rather than __get_dynamic_array(str) when deadling with strings. It is just a code cleanup, no changes on tracepoint ABI. Link: http://lkml.kernel.org/r/ea260df91817411cca2a1f3db2abd88860094788.1467407618.git.bris...@redhat.com Cc: Trond Myk

[for-next][PATCH 3/7] ftrace: Reduce size of function graph entries

2016-07-16 Thread Steven Rostedt
From: Namhyung Kim Currently ftrace_graph_ent{,_entry} and ftrace_graph_ret{,_entry} struct can have padding bytes at the end due to alignment in 64-bit data type. As these data are recorded so frequently, those paddings waste non-negligible space. As the ring buffer maintains alignment properly

[PATCH] drm/hdlcd: Delete an unnecessary check before drm_fbdev_cma_hotplug_event()

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 09:10:40 +0200 The drm_fbdev_cma_hotplug_event() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elf

Re: [PATCH 2/2] nbd: Disallow ioctls on disconnected block device

2016-07-16 Thread Pranay Srivastava
Hi, On Fri, Jun 24, 2016 at 2:59 PM, Markus Pargmann wrote: > After NBD_DO_IT exited the block device may still be used. Make sure > that we handle intended disconnects differently and do not allow any > changed of the nbd device. > > This patch should avoid that the nbd-client connects to a diff

[PATCH] net/mlx5_core/health: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue health->wq was used as per device private health thread. This was done so that system error handling could be processed concurrently. The workqueue has a single workitem(&health->work) and hence doesn't require ordering. It is involved in handling the health of the deviceand is not be

[PATCH] spi: spi-sh: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue has a single workitem(&ss->ws) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System workqueues have been able to handle high level of concurrency for a long tim

[PATCH] drm/ast: Delete an unnecessary check before drm_gem_object_unreference_unlocked()

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 09:54:22 +0200 The drm_gem_object_unreference_unlocked() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Ma

Re: [PATCH] spi: spi-sh: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
Please ignore this mail. I have already sent a patch for this driver Thanks, Bhaktipriya On Sat, Jul 16, 2016 at 1:35 PM, Bhaktipriya Shridhar wrote: > The workqueue has a single workitem(&ss->ws) and hence doesn't require > ordering. Also, it is not being used on a memory reclaim path. Hence,

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Hans Verkuil
On 07/15/2016 08:11 PM, Laurent Pinchart wrote: > Hi Ricardo, > > Thank you for the patch. > > On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: >> Describe the HSV formats >> >> Signed-off-by: Ricardo Ribalda Delgado >> --- >> Documentation/media/uapi/v4l/hsv-formats.rst | 1

[PATCH] drm/ttm: Remove create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
swap_queue was created to handle shrinking in low memory situations. A separate workqueue was used in order to avoid other workqueue tasks from being blocked since work items on swap_queue spend a lot of time waiting for the GPU. Since these long-running work items aren't involved in memory reclai

Re: [PATCH 1/1] irqdomain: Export __irq_domain_alloc_irqs() and irq_domain_free_irqs()

2016-07-16 Thread Marc Zyngier
On Sat, 16 Jul 2016 04:33:59 +0300 Alexander Popov wrote: > On 08.07.2016 11:34, Alexander Popov wrote: > > On 06.07.2016 14:17, Thomas Gleixner wrote: > >> On Fri, 1 Jul 2016, Alexander Popov wrote: > >> > >>> Export __irq_domain_alloc_irqs() and irq_domain_free_irqs() for being > >>> able t

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
Hi Laurent It is actually a very good comment. :) In our case we have implemented the format ourselves in the FPGA and we support both 0-255 and 0-179 Hue ranges. After some weeks of use, only the 0-179 range is used in userpace. The reasons for this is mainly that it is the format used by OpenCV

[PATCH] dwc_eth_qos: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
alloc_workqueue replaces deprecated create_singlethread_workqueue(). A dedicated workqueue has been used since the workitem viz lp->txtimeout_reinit is involved in reinitialization if a TX timeout occurs, which is necessary to guarantee forward progress in packet processing. As a network device ca

Re: [PATCH 1/5] namespaces: move user_ns into ns_common

2016-07-16 Thread kbuild test robot
-user_ns-into-ns_common/20160716-093057 config: openrisc-allyesconfig (attached as .config) compiler: or32-linux-gcc (GCC) 4.5.1-or32-1.0rc1 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin

[PATCH] ARM: pxa: fix GPIO double shifts

2016-07-16 Thread Robert Jarzmik
The commit 9bf448c66d4b ("ARM: pxa: use generic gpio operation instead of gpio register") from Oct 17, 2011, leads to the following static checker warning: arch/arm/mach-pxa/spitz_pm.c:172 spitz_charger_wakeup() warn: double left shift '!gpio_get_value(SPITZ_GPIO_KEY_INT) << (1 << ((SPI

[PATCH] [media] s5p-mfc: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
alloc_workqueue replaces deprecated create_singlethread_workqueue(). The MFC device driver is a v4l2 driver which can encode/decode video raw/elementary streams and has support for all popular video codecs. The driver's watchdog_workqueue has been replaced with system_wq since it queues a single

[PATCH] [media] pvrusb2: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "workqueue" is involved in polling the pvrusb2 hardware (pvr2_hdw). It has a single work item(&hdw->workpoll) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. Sys

RE: [PATCH v3 5/8] thunderbolt: Communication with the ICM (firmware)

2016-07-16 Thread Winkler, Tomas
> -Original Message- > From: Levy, Amir (Jer) > Sent: Thursday, July 14, 2016 17:50 > To: Winkler, Tomas ; > andreas.noe...@gmail.com; gre...@linuxfoundation.org; > bhelg...@google.com > Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; > net...@vger.kernel.org; thunderbolt-lin

[PATCH] s390/cio/chp : Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "chp_wq" is involved in performing pending configure tasks for channel paths. It has a single work item(&cfg_work) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq.

[PATCH] [media] gspca: sonixj: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in updating the JPEG quality of the gspca_dev. It has a single work item(&sd->work) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. Syst

[PATCH] [media] gspca: vicam: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in streaming the camera data. It has a single work item(&sd->work_struct) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System workque

[PATCH] [media] gspca: jl2005bcd: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in streaming the camera data. It has a single work item(&sd->work_struct) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System workque

Re: [PATCH] i2c: efm32: fix a failure path in efm32_i2c_probe()

2016-07-16 Thread Uwe Kleine-König
On Sat, Jul 16, 2016 at 02:36:38AM +0300, Alexey Khoroshilov wrote: > There is the only failure path in efm32_i2c_probe(), > where clk_disable_unprepare() is missed. > > Found by Linux Driver Verification project (linuxtesting.org). > > Signed-off-by: Alexey Khoroshilov oops, this is wrong sinc

[PATCH] [media] gspca: finepix: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in streaming the camera data. It has a single work item(&dev->work_struct) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System workqu

Re: [patch] mtd: maps: sa1100-flash: potential NULL dereference

2016-07-16 Thread Dan Carpenter
I like the Fixes tag because it was my invention. :) It's a separate thing from -stable. It's nice for reviewing so you can see the original intent of the patch you're fixing. Also it forces you to find the original authors and CC them so hopefully they Ack the patch. The other thing is it let

Re: [RESEND PATCH] arm: colibri_pxa270_defconfig: disable IDE subsystem

2016-07-16 Thread Robert Jarzmik
Bartlomiej Zolnierkiewicz writes: > This patch disables deprecated IDE subsystem in colibri_pxa270_defconfig > (no IDE host drivers are selected in this config so there is no valid > reason to enable IDE subsystem itself). Hi Bartlomiej, It seems I have missed that serie a year ago ... I can't r

[PATCH 0/2] Remove improper workqueue usage

2016-07-16 Thread Bhaktipriya Shridhar
This patch set fixes the improper usage of the workqueue API. This includes dropping the freeing of workqueue and removing the deprecated create_singlethread_workqueue instance. Bhaktipriya Shridhar (2): [media] cx25821: Drop Freeing of Workqueue [media] cx25821: Remove deprecated create_singl

[PATCH 1/2] [media] cx25821: Drop Freeing of Workqueue

2016-07-16 Thread Bhaktipriya Shridhar
Workqueues shouldn't be freed. destroy_workqueue should be used instead. destroy_workqueue safely destroys a workqueue and ensures that all pending work items are done before destroying the workqueue. Signed-off-by: Bhaktipriya Shridhar --- drivers/media/pci/cx25821/cx25821-audio-upstream.c | 2

[PATCH 2/2] [media] cx25821: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "_irq_audio_queues" runs the audio upstream handler. It has a single work item(&dev->_audio_work_entry) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System wor

[no subject]

2016-07-16 Thread Pranay Kr Srivastava
Hi Markus, Can you take a look at this. Let me know if this looks ok, I'll resend the whole series again. Regards,

[PATCH v5 2/4] nbd: fix might_sleep warning on socket shutdown.

2016-07-16 Thread Pranay Kr Srivastava
From: "Pranay Kr. Srivastava" spinlocked ranges should be small and not contain calls into huge subfunctions. Fix my mistake and just get the pointer to the socket instead of doing everything with spinlock held. Reported-by: Mikulas Patocka Signed-off-by: Markus Pargmann Changelog: Pranay Kr.

Re: [Nbd] [PATCH 2/2] nbd: Disallow ioctls on disconnected block device

2016-07-16 Thread Alex Bligh
On 16 Jul 2016, at 08:42, Pranay Srivastava wrote: > So instead can't we put a mechanism in place for network address + mac > to be same > for allowing clients to reconnect? Do let me know if this is not of concern. MAC address?! nbd clients connect over IP, and if a router reboots between them

[PATCH] [media] ad9389b: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue work_queue is involved in EDID (Extended Display Identification Data) handling. It has a single work item(&state->edid_handler) and hence doesn't require ordering. It is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of sys

Re: [Nbd] [PATCH 2/2] nbd: Disallow ioctls on disconnected block device

2016-07-16 Thread Pranay Srivastava
On Sat, Jul 16, 2016 at 3:02 PM, Alex Bligh wrote: > > On 16 Jul 2016, at 08:42, Pranay Srivastava wrote: > >> So instead can't we put a mechanism in place for network address + mac >> to be same >> for allowing clients to reconnect? Do let me know if this is not of concern. > > MAC address?! nbd

Re: [PATCH 4/5] mm: show node_pages_scanned per node, not zone

2016-07-16 Thread Minchan Kim
On Fri, Jul 15, 2016 at 02:09:24PM +0100, Mel Gorman wrote: > From: Minchan Kim > > The node_pages_scanned represents the number of scanned pages > of node for reclaim so it's pointless to show it as kilobytes. > > As well, node_pages_scanned is per-node value, not per-zone. > > This patch chan

Re: [PATCH v5 2/4] nbd: fix might_sleep warning on socket shutdown.

2016-07-16 Thread Pranay Srivastava
Hi Markus, On Sat, Jul 16, 2016 at 2:52 PM, Pranay Kr Srivastava wrote: > From: "Pranay Kr. Srivastava" > > spinlocked ranges should be small and not contain calls into huge > subfunctions. Fix my mistake and just get the pointer to the socket > instead of doing everything with spinlock held. >

Re: [PATCH] s390/cio/chp : Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Sebastian Ott
On Sat, 16 Jul 2016, Bhaktipriya Shridhar wrote: > The workqueue "chp_wq" is involved in performing pending > configure tasks for channel paths. > > It has a single work item(&cfg_work) and hence doesn't require > ordering. Also, it is not being used on a memory reclaim path. > Hence, the singleth

[PATCH v5 3/4] make nbd device wait for its users

2016-07-16 Thread Pranay Kr Srivastava
When a timeout occurs or a recv fails, then instead of abruptly killing nbd block device wait for its users to finish. This is more required when filesystem(s) like ext2 or ext3 don't expect their buffer heads to disappear while the filesystem is mou

[PATCH v3 1/9] [media] videodev2.h Add HSV formats

2016-07-16 Thread Ricardo Ribalda Delgado
These formats store the color information of the image in a geometrical representation. The colors are mapped into a cylinder, where the angle is the HUE, the height is the VALUE and the distance to the center is the SATURATION. This is a very useful format for image segmentation algorithms. Signe

[PATCH v3 0/9] Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
HSV formats are extremely useful for image segmentation. This set of patches makes v4l2 aware of this kind of formats. Vivid changes have been divided to ease the reviewing process. We are working on patches for Gstreamer and OpenCV that will make use of these formats. We still need to decide if

[PATCH v3 6/9] [media] vivid: Rename variable

2016-07-16 Thread Ricardo Ribalda Delgado
r_y and g_u now also contain the H and V components on the HSV formats. Rename the variables to reflect this. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 209 +- 1 file changed, 105 insertions(+), 104 deletions(-) diff --git

[PATCH v3 3/9] [media] Documentation: Add Ricardo Ribalda

2016-07-16 Thread Ricardo Ribalda Delgado
My initials were on the Changelog, but there was no link to my name. Signed-off-by: Ricardo Ribalda Delgado --- Documentation/media/uapi/v4l/v4l2.rst | 4 1 file changed, 4 insertions(+) diff --git a/Documentation/media/uapi/v4l/v4l2.rst b/Documentation/media/uapi/v4l/v4l2.rst index 6d23b

Re: [PATCH v5 3/4] make nbd device wait for its users

2016-07-16 Thread Pranay Srivastava
Hi Markus On Sat, Jul 16, 2016 at 4:06 PM, Pranay Kr Srivastava wrote: > When a timeout occurs or a recv fails, then > instead of abruptly killing nbd block device > wait for its users to finish. > > This is more required when filesystem(s) like > ext2 or

[PATCH v3 7/9] [media] vivid: Introduce TPG_COLOR_ENC_LUMA

2016-07-16 Thread Ricardo Ribalda Delgado
Simplifies handling of Gray formats. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 26 +++-- drivers/media/platform/vivid/vivid-vid-common.c | 6 +++--- include/media/v4l2-tpg.h| 1 + 3 files changed, 24

[PATCH v3 2/9] [media] Documentation: Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
Describe the HSV formats Signed-off-by: Ricardo Ribalda Delgado --- Documentation/media/uapi/v4l/hsv-formats.rst | 19 +++ Documentation/media/uapi/v4l/pixfmt-packed-hsv.rst | 158 + Documentation/media/uapi/v4l/pixfmt.rst| 1 + Documentation/media/uapi/v

[PATCH v3 9/9] [media] vivid: Local optimization

2016-07-16 Thread Ricardo Ribalda Delgado
Avoid duplicated data shifts when possible. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c b/drivers/media/common/v4l2-tpg/v4

[PATCH v3 8/9] [media] vivid: Fix YUV555 and YUV565 handling

2016-07-16 Thread Ricardo Ribalda Delgado
precalculate_color() had a optimization that avoided duplicated conversion for YUV formats. This optimization did not take into consideration YUV444, YUV555, YUV565 or limited range quantization. This patch keeps the optimization, but fixes the wrong handling. Signed-off-by: Ricardo Ribalda Delga

Re: [PATCH -v4 2/2] printk: Add kernel parameter to control writes to /dev/kmsg

2016-07-16 Thread Dave Young
On 07/15/16 at 02:45pm, Borislav Petkov wrote: > On Fri, Jul 15, 2016 at 02:21:09PM +0800, Dave Young wrote: > > Sorry for jumping in late. But I just see this today and I really like the > > idea to add a switch to turn off the kmsg writing from userspace because > > I suffer from it also. > > Th

[PATCH v3 4/9] [media] vivid: code refactor for color encoding

2016-07-16 Thread Ricardo Ribalda Delgado
Replace is_yuv with color_enc Which can be used by other color encodings such us HSV. This change should ease the review of the following patches. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 49 +++ drivers/media/platform/vivi

[PATCH v3 5/9] [media] vivid: Add support for HSV formats

2016-07-16 Thread Ricardo Ribalda Delgado
This patch adds support for V4L2_PIX_FMT_HSV24 and V4L2_PIX_FMT_HSV32. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 93 +++-- drivers/media/platform/vivid/vivid-vid-common.c | 14 include/media/v4l2-tpg.h

Re: [PATCH] [media] ad9389b: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread kbuild test robot
-deprecated-create_singlethread_workqueue/20160716-174501 base: git://linuxtv.org/media_tree.git master config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp

Re: [PATCH v3 9/9] [media] vivid: Local optimization

2016-07-16 Thread Ricardo Ribalda Delgado
Hi On Sat, Jul 16, 2016 at 12:41 PM, Ricardo Ribalda Delgado wrote: > - cr = clamp(cr, 16 << 4, 240 << 4); > + y = clamp(y >> 4, 16, 235); > + cb = clamp(cb >> 4, 16, 240); > + cr = clamp(cr > 4, 16, 240); Th

Re: [PATCH net] net: nb8800: Fix SKB leak in nb8800_receive()

2016-07-16 Thread Måns Rullgård
Florian Fainelli writes: > In case nb8800_receive() fails to allocate a fragment, we would leak the > SKB freshly allocated and just return, instead, free it. > > Reported-by: coverity (CID 1341750) > Signed-off-by: Florian Fainelli Acked-by: Mans Rullgard > --- > drivers/net/ethernet/aurora

[PATCH v2] [media] ad9389b: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue work_queue is involved in EDID (Extended Display Identification Data) handling. It has a single work item(&state->edid_handler) and hence doesn't require ordering. It is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of sys

Re: [RFC][PATCH 0/7] Add HDMI audio support for HiKey

2016-07-16 Thread Mark Brown
On Fri, Jul 15, 2016 at 08:38:47PM -0700, John Stultz wrote: > Yea. I'm not sure what the communities policy on Author/SoB lines in > the face of email address changes. Given that the signoff is all about DCO and hence licensing it should probably stay with Linaro. signature.asc Description: PG

Re: [RFC][PATCH 5/7] Kconfig: Allow k3dma driver to be selected for more then HISI3xx platforms

2016-07-16 Thread Mark Brown
On Fri, Jul 15, 2016 at 07:13:25PM -0700, John Stultz wrote: > This allows the k3dma driver to be selected on HiKey > config K3_DMA > tristate "Hisilicon K3 DMA support" > - depends on ARCH_HI3xxx > select DMA_ENGINE > select DMA_VIRTUAL_CHANNELS > help And *every* o

Re: [Nbd] [PATCH 2/2] nbd: Disallow ioctls on disconnected block device

2016-07-16 Thread Wouter Verhelst
On Sat, Jul 16, 2016 at 03:38:40PM +0530, Pranay Srivastava wrote: > Okay. So how about we include some negotiated key which goes in with every > request which the server could maintain for clients that can be checked while > resetting the connection with the same server? Wut? > So am I correct t

[PATCH v2] drm/radeon: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
alloc_workqueue replaces deprecated create_singlethread_workqueue(). Each hardware CRTC has a single flip work queue. When a radeon_flip_work_func item is queued, it needs to be executed ASAP because even a slight delay may cause the flip to be delayed by one refresh cycle. Hence, a dedicated wo

Re: [RFC][PATCH 6/7] ASoC: hisilicon: Add hi6210 i2s audio driver for hdmi audio

2016-07-16 Thread Mark Brown
On Fri, Jul 15, 2016 at 07:13:26PM -0700, John Stultz wrote: > sound/soc/Kconfig | 1 + > sound/soc/Makefile | 1 + > sound/soc/hisilicon/Kconfig| 5 + > sound/soc/hisilicon/Makefile | 2 + > sound/soc/hisilicon/hi6210-hdmi-ca

Re: [PATCH 7/7] dts: hi6220: Add k3-dma and i2s/hdmi audio support

2016-07-16 Thread Mark Brown
On Fri, Jul 15, 2016 at 07:13:27PM -0700, John Stultz wrote: > Add entry for k3-dma driver and i2s/hdmi audio devices. > This enables HDMI audio output. These bindings appear to be undocumented. All new bindings require documentation. > + i2s0: hi6210_i2s { > +

Re: [PATCH] lib/bitmap.c: enhance map pattern

2016-07-16 Thread Ben Hutchings
On Sat, 2016-07-16 at 10:57 +0300, Noam Camus wrote: > From: Noam Camus > > Today there are platforms with many CPUs (up to 4K). > Trying to boot only part of the CPUs may result in too long string. > > For example lets take NPS platform that is part of arch/arc. > This platform have SMP system

Re: [PATCH] [media] vb2: map dmabuf for planes on driver queue instead of vidioc_qbuf

2016-07-16 Thread Luis de Bethencourt
On 15/07/16 17:26, Javier Martinez Canillas wrote: > The buffer planes' dma-buf are currently mapped when buffers are queued > from userspace but it's more appropriate to do the mapping when buffers > are queued in the driver since that's when the actual DMA operation are > going to happen. > > Su

[patch v2] tools/vm/slabinfo: fix an unintentional printf

2016-07-16 Thread Dan Carpenter
The curly braces are missing here so we print stuff unintentionally. Fixes: 9da4714a2d44 ('slub: slabinfo update for cmpxchg handling') Signed-off-by: Dan Carpenter --- v2: Fix typo in git hash diff --git a/tools/vm/slabinfo.c b/tools/vm/slabinfo.c index 7cf6e17..b9d34b3 100644 --- a/tools/vm/sl

Re: [PATCH v7 4/9] acpi/arm64: Add GTDT table parse driver

2016-07-16 Thread Rafael J. Wysocki
On Saturday, July 16, 2016 10:24:35 AM Fu Wei wrote: > Hi Rafeal, > > On 16 July 2016 at 05:22, Rafael J. Wysocki wrote: > > On Saturday, July 16, 2016 12:32:14 AM Fu Wei wrote: > >> Hi Rafael, > >> > >> On 15 July 2016 at 21:07, Rafael J. Wysocki wrote: > >> > On Friday, July 15, 2016 02:15:27

Re: [PATCH] usb: gadget: uvc: Fix return value in case of error

2016-07-16 Thread Laurent Pinchart
Hi Christophe, Thank you for the patch. On Saturday 16 Jul 2016 09:04:40 Christophe JAILLET wrote: > If this memory allocation fail, we will return 0, which means success. > Return -ENOMEM instead. > > Signed-off-by: Christophe JAILLET Reviewed-by: Laurent Pinchart > --- > drivers/usb/gadge

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Laurent Pinchart
Hi Hans, On Saturday 16 Jul 2016 10:19:29 Hans Verkuil wrote: > On 07/15/2016 08:11 PM, Laurent Pinchart wrote: > > On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: > >> Describe the HSV formats > >> > >> Signed-off-by: Ricardo Ribalda Delgado > >> --- > >> > >> Documentation/medi

Re: [Nbd] [PATCH 2/2] nbd: Disallow ioctls on disconnected block device

2016-07-16 Thread Pranay Srivastava
On Sat, Jul 16, 2016 at 4:56 PM, Wouter Verhelst wrote: > On Sat, Jul 16, 2016 at 03:38:40PM +0530, Pranay Srivastava wrote: >> Okay. So how about we include some negotiated key which goes in with every >> request which the server could maintain for clients that can be checked while >> resetting t

Re: [PATCH v5] wlcore: spi: add wl18xx support

2016-07-16 Thread Rob Herring
On Sun, Jul 10, 2016 at 08:32:40AM +, Reizer, Eyal wrote: > Add support for using with both wl12xx and wl18xx. > > - all wilink family needs special init command for entering wspi mode. > extra clock cycles should be sent after the spi init command while the > cs pin is high. > - Use inver

Re: [PATCH V2 1/3] dt-bindings: pwm: Add MediaTek display PWM bindings

2016-07-16 Thread Rob Herring
On Mon, Jul 11, 2016 at 04:18:07PM +0800, Weiqing Kong wrote: > Add MT2701 compatible string. > > Signed-off-by: Weiqing Kong > --- > Documentation/devicetree/bindings/pwm/pwm-mtk-disp.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Acked-by: Rob Herring

Re: [PATCH] radix-tree: fix radix_tree_iter_retry() for tagged iterators.

2016-07-16 Thread Konstantin Khlebnikov
On Fri, Jul 15, 2016 at 10:00 PM, Ross Zwisler wrote: > On Fri, Jul 15, 2016 at 11:52:58AM +0300, Andrey Ryabinin wrote: >> On 07/15/2016 01:25 AM, Ross Zwisler wrote: >> > On Thu, Jul 14, 2016 at 02:19:56PM +0300, Andrey Ryabinin wrote: >> >> radix_tree_iter_retry() resets slot to NULL, but it do

Re: [PATCH v1 4/6] drm/rockchip: dw_hdmi: add RK3399 HDMI support

2016-07-16 Thread Rob Herring
On Mon, Jul 11, 2016 at 07:05:46PM +0800, Yakir Yang wrote: > RK3399 and RK3288 shared the same HDMI IP controller, only some light > difference with GRF configure. > > Signed-off-by: Yakir Yang > --- > .../devicetree/bindings/display/bridge/dw_hdmi.txt | 1 + > .../bindings/display/rockchip/dw

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Hans Verkuil
On 07/16/2016 02:38 PM, Laurent Pinchart wrote: > Hi Hans, > > On Saturday 16 Jul 2016 10:19:29 Hans Verkuil wrote: >> On 07/15/2016 08:11 PM, Laurent Pinchart wrote: >>> On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: Describe the HSV formats Signed-off-by: Ricardo Ri

[GIT PULL] KVM fixes for 4.7-rc8

2016-07-16 Thread Radim Krčmář
Linus, The following changes since commit a99cde438de0c4c0cecc1d1af1a55a75b10bfdef: Linux 4.7-rc6 (2016-07-03 23:01:00 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/kvm.git tags/for-linus for you to fetch changes up to b244c9fc251e14a083a1cbf04bef10bd

[PATCH] net/sched/sch_htb: clamp xstats tokens to fit into 32-bit int

2016-07-16 Thread Konstantin Khlebnikov
In kernel HTB keeps tokens in signed 64-bit in nanoseconds. In netlink protocol these values are converted into pshed ticks (64ns for now) and truncated to 32-bit. In struct tc_htb_xstats fields "tokens" and "ctokens" are declared as unsigned 32-bit but they could be negative thus tool 'tc' prints

Re: 4.1.28: memory leak introduced by "mm/swap.c: flush lru pvecs on compound page arrival"

2016-07-16 Thread Jens Rottmann
Hi again, took lack of response to express reluctance examining vendor kernels. Therefore reproduced and can confirm memory leak on 4.1.28 vanilla x86. Identical symptoms. Regards, Jens From: Jens Rottmann Sent: Friday, July 15, 2016 21:27 To: Lukasz Od

[PATCH v2 comment update] kernel/watchdog: use nmi registers snapshot in hardlockup handler

2016-07-16 Thread Konstantin Khlebnikov
NMI handler doesn't call set_irq_regs(), it's set only by normal IRQ. Thus get_irq_regs() returns NULL or stale registers snapshot with IP/SP pointing to the code interrupted by IRQ which was interrupted by NMI. NULL isn't a problem: in this case watchdog calls dump_stack() and prints full stack tr

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Laurent Pinchart
Hi Hans, On Saturday 16 Jul 2016 15:59:08 Hans Verkuil wrote: > On 07/16/2016 02:38 PM, Laurent Pinchart wrote: >> On Saturday 16 Jul 2016 10:19:29 Hans Verkuil wrote: >>> On 07/15/2016 08:11 PM, Laurent Pinchart wrote: On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: > Descr

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
Hi On Sat, Jul 16, 2016 at 4:12 PM, Laurent Pinchart wrote: > I'd still like to know about it for my personal information :-) Maybe it is just a very cheap gamma. > >> Anyway, I am inclined to use ycbcr_enc as well. > > I'm glad we agree. > Are you thinking about something like this: diff -

[PATCH 0/8] drm/amdgpu: Fine-tuning for three function implementations

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 16:23:21 +0200 Further update suggestions were taken into account after patches were applied from static source code analysis. Markus Elfring (8): Delete an unnecessary check before drm_gem_object_unreference_unlocked() Delete unnecessary checks be

[PATCH 1/8] drm/amdgpu: Delete an unnecessary check before drm_gem_object_unreference_unlocked()

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 11:28:36 +0200 The drm_gem_object_unreference_unlocked() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Ma

4.1.28 is broken due to "mm/swap.c: flush lru pvecs on compound page arrival"

2016-07-16 Thread Mikulas Patocka
Hi The patch c5ad33184354260be6d05de57e46a5498692f6d6 on the kernel v4.1.28 breaks the kernel. The kernel crashes when executing the boot scripts with "kernel panic: Out of memory and no killable processes...". The machine has 512MB ram and 1 core. Note that the upstream kernel 4.7-rc4 with th

Re: 4.1.28: memory leak introduced by "mm/swap.c: flush lru pvecs on compound page arrival"

2016-07-16 Thread Minchan Kim
On Fri, Jul 15, 2016 at 09:27:55PM +0200, Jens Rottmann wrote: > Hi, > > 4.1.y stable commit c5ad33184354260be6d05de57e46a5498692f6d6 (Upstream > commit 8f182270dfec432e93fae14f9208a6b9af01009f) "mm/swap.c: flush lru > pvecs on compound page arrival" in 4.1.28 introduces a memory leak. > > Simply

[PATCH 2/8] drm/amdgpu/powerplay: Delete unnecessary checks before the function call "kfree"

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 12:38:12 +0200 The kfree() function tests whether its argument is NULL and then returns immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gp

[PATCH 3/8] drm/amdgpu: One function call less in amdgpu_cgs_acpi_eval_object() after error detection

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 13:43:44 +0200 The kfree() function was called in one case by the amdgpu_cgs_acpi_eval_object() function during error handling even if the passed variable "obj" contained a null pointer. * Adjust jump targets according to the Linux coding style convent

Re: 4.1.28 is broken due to "mm/swap.c: flush lru pvecs on compound page arrival"

2016-07-16 Thread Minchan Kim
On Sat, Jul 16, 2016 at 10:46:17AM -0400, Mikulas Patocka wrote: > Hi > > The patch c5ad33184354260be6d05de57e46a5498692f6d6 on the kernel v4.1.28 > breaks the kernel. The kernel crashes when executing the boot scripts with > "kernel panic: Out of memory and no killable processes...". The machin

[PATCH 4/8] drm/amdgpu: Delete a variable in amdgpu_cgs_acpi_eval_object()

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 14:00:28 +0200 The local variable "func_no" was assigned a value at two places. But it was not read within this function. Thus delete it. Signed-off-by: Markus Elfring --- drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 2 -- 1 file changed, 2 deletions(-)

[PATCH 5/8] drm/amdgpu: Delete an unnecessary variable initialisation in amdgpu_cgs_acpi_eval_object()

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 14:54:12 +0200 The variable "argument" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 2 +- 1 file changed, 1 insert

[PATCH 5/8] drm/amdgpu: Delete an unnecessary variable initialisation in amdgpu_cgs_acpi_eval_object()

2016-07-16 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 16 Jul 2016 14:54:12 +0200 The variable "argument" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 2 +- 1 file changed, 1 insert

[PATCH v0 00/10] Convert Netgear WNR854T to devicetree

2016-07-16 Thread Jamie Lentin
This is an attempt to resurrect the mainline WNR854T support (I had no luck getting a non-DT kernel to boot). First 2 patches are trivial, the next 3 add DT bindings for the SoC, then the old board file is swapped out for DT. The final 3 patches configure the PHY leds as Netgear intended. There's

[PATCH v0 07/10] arm: orion5x: Remove old non-DT-based WNR854T support

2016-07-16 Thread Jamie Lentin
Signed-off-by: Jamie Lentin --- arch/arm/mach-orion5x/Makefile| 1 - arch/arm/mach-orion5x/wnr854t-setup.c | 185 -- 2 files changed, 186 deletions(-) delete mode 100644 arch/arm/mach-orion5x/wnr854t-setup.c diff --git a/arch/arm/mach-orion5x/Makefile b

[PATCH v0 03/10] arm: orion5x: Add clk support for mv88f5181

2016-07-16 Thread Jamie Lentin
Referring to values in the u-boot port, add support for the mv88f5181 Signed-off-by: Jamie Lentin --- .../devicetree/bindings/clock/mvebu-core-clock.txt | 1 + drivers/clk/mvebu/orion.c | 70 ++ 2 files changed, 71 insertions(+) diff --git a/Documen

[PATCH v0 09/10] net: phy: Re-attempt custom DT configuration after configuration

2016-07-16 Thread Jamie Lentin
marvell,reg-init is generally used to apply a custom LED configuration on boot. However this is then blatted in m88e1121_config_aneg when the interface is brought up. Re-apply any custom configuration afterwards, to keep custom LED configuration. Signed-off-by: Jamie Lentin --- drivers/net/phy/m

[PATCH v0 02/10] arm: orion5x: Add documentation for SoC and board bindings

2016-07-16 Thread Jamie Lentin
Copy the format for kirkwood/dove to orion5x Signed-off-by: Jamie Lentin --- .../bindings/arm/marvell/marvell,orion5x.txt | 23 ++ 1 file changed, 23 insertions(+) create mode 100644 Documentation/devicetree/bindings/arm/marvell/marvell,orion5x.txt diff --git a/Docum

  1   2   3   >