[PATCH 1/1] Staging: iio: Coding style correction

2015-02-07 Thread Tolga Ceylan
Line over 80 characters corrected Signed-off-by: Tolga Ceylan --- drivers/staging/iio/meter/ade7854-i2c.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/meter/ade7854-i2c.c b/drivers/staging/iio/meter/ade7854-i2c.c index 5b33c7f..5d0671a 100644 --- a/d

[PATCH] pci: spear: Drop __initdata from struct platform_driver spear13xx_pcie_driver

2015-02-07 Thread Matwey V. Kornilov
spear13xx_pcie_driver.driver is allocated in text.init section and then the pointer to it is passed futher. This patch is to avoid crashes like the following, when freed memory is used: #0 __device_attach (drv=0xc0ed5608 , data=0xdb622610) at ../drivers/base/dd.c:409 #1 0xc07a4798 in bus_for_

[PATCH] staging: wlan-ng: Use setup_timer

2015-02-07 Thread Vaishali Thakkar
This patch introduces the use of function setup_timer. This is done using Coccinelle and semantic patch used is as follows: @@ expression x,y,z; @@ - init_timer (&x); + setup_timer (&x, y, z); - x.function = y; - x.data = z; Signed-off-by: Vaishali Thakkar --- drivers/staging/wlan-ng/hfa384x_

[PATCH] staging: dgnc: Use setup_timer

2015-02-07 Thread Vaishali Thakkar
This patch introduces the use of function setup_timer. This is done using Coccinelle and semantic patch used is as follows: @@ expression x,y,z; @@ - init_timer (&x); + setup_timer (&x, y, z); - x.function = y; - x.data = z; Signed-off-by: Vaishali Thakkar --- drivers/staging/dgnc/dgnc_driver

Re: [PATCH] mtd: avoid registering reboot notifier twice

2015-02-07 Thread Brian Norris
On Sun, Feb 01, 2015 at 02:08:50AM +0100, Niklas Cassel wrote: > Calling mtd_device_parse_register with the same mtd_info > (e.g. registering several partitions on a single device) > would add the same reboot notifier twice, causing an > infinte loop in notifier_chain_register during boot up. > >

[PATCH] staging: comedi: Use setup_timer

2015-02-07 Thread Vaishali Thakkar
This patch introduces the use of function setup_timer. This is done using Coccinelle and semantic patch used is as follows: @@ expression x,y,z; @@ - init_timer (&x); + setup_timer (&x, y, z); - x.function = y; - x.data = z; Signed-off-by: Vaishali Thakkar --- drivers/staging/comedi/drivers/c

[PATCH] x86/xen: Cleanup file path in a comment

2015-02-07 Thread Alexander Kuleshov
There is no head.S now, arch/x86/kernel/head_32.S and arch/x86/kernel/head_64.S instead Signed-off-by: Alexander Kuleshov --- arch/x86/xen/xen-head.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/xen/xen-head.S b/arch/x86/xen/xen-head.S index 674b2225..c73e603

Re: [PATCH net-next v2 0/7] r8152: adjust the code

2015-02-07 Thread David Miller
From: Hayes Wang Date: Fri, 6 Feb 2015 11:30:44 +0800 > V2: > Correct the subject of patch #5. Replace "link feed" with "line feed". > > v1: > Code adjustment. Series applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord..

Re: [PATCHv2] rds: Make rds_message_copy_from_user() return 0 on success.

2015-02-07 Thread David Miller
From: Sowmini Varadhan Date: Thu, 5 Feb 2015 17:41:43 -0500 > Commit 083735f4b01b ("rds: switch rds_message_copy_from_user() to iov_iter") > breaks rds_message_copy_from_user() semantics on success, and causes it > to return nbytes copied, when it should return 0. This commit fixes that bug. >

Re: [PATCH] net: rds: Remove repeated function names from debug output

2015-02-07 Thread David Miller
From: Rasmus Villemoes Date: Thu, 5 Feb 2015 23:17:20 +0100 > The macro rdsdebug is defined as > > pr_debug("%s(): " fmt, __func__ , ##args) > > Hence it doesn't make sense to include the name of the calling > function explicitly in the format string passed to rdsdebug. > > Signed-off-by: R

[PATCH v9 1/3] i2c: iProc: define Broadcom iProc I2C binding

2015-02-07 Thread Ray Jui
Document the I2C device tree binding for Broadcom iProc family of SoCs Signed-off-by: Ray Jui Reviewed-by: Scott Branden Reviewed-by: Kevin Cernekee --- .../devicetree/bindings/i2c/brcm,iproc-i2c.txt | 37 1 file changed, 37 insertions(+) create mode 100644 Document

[PATCH v9 3/3] ARM: dts: add I2C device nodes for Broadcom Cygnus

2015-02-07 Thread Ray Jui
Add I2C device nodes and its properties in bcm-cygnus.dtsi but keep them disabled there. Individual I2C devices can be enabled in board specific dts file when I2C slave devices are enabled in the future Signed-off-by: Ray Jui Reviewed-by: Scott Branden Reviewed-by: Kevin Cernekee --- arch/arm/

[PATCH v9 2/3] i2c: iproc: Add Broadcom iProc I2C Driver

2015-02-07 Thread Ray Jui
Add initial support to the Broadcom iProc I2C controller found in the iProc family of SoCs. The iProc I2C controller has separate internal TX and RX FIFOs, each has a size of 64 bytes. The iProc I2C controller supports two bus speeds including standard mode (100kHz) and fast mode (400kHz) Signed-

[PATCH v9 0/3] Add I2C support to Broadcom iProc

2015-02-07 Thread Ray Jui
This patchset contains the initial I2C support for Broadcom iProc family of SoCs. The iProc I2C controller has separate internal TX and RX FIFOs, each has a size of 64 bytes. The iProc I2C controller supports two bus speeds including standard mode (100 kHz) and fast mode (400 kHz) Synced code bas

Re: [PATCH v8 2/3] i2c: iproc: Add Broadcom iProc I2C Driver

2015-02-07 Thread Ray Jui
On 2/7/2015 9:50 AM, Wolfram Sang wrote: > Hi Ray, > > On Fri, Feb 06, 2015 at 05:28:26PM -0800, Ray Jui wrote: >> Add initial support to the Broadcom iProc I2C controller found in the >> iProc family of SoCs. >> >> The iProc I2C controller has separate internal TX and RX FIFOs, each has >> a si

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Mathieu Desnoyers
- Original Message - > From: "Michael Cree" > To: "Mathieu Desnoyers" > Cc: "Greg KH" , linux-al...@vger.kernel.org, > "Richard Henderson" , "Ivan > Kokshaysky" , "Matt Turner" , > "Huang Ying" , > linux-kernel@vger.kernel.org, "Paul McKenney" , > "David Howells" , > "Pranith Kumar" ,

[PATCH Resend] staging: speakup: Fix warning of line over 80 characters.

2015-02-07 Thread Shirish Gajera
This patch fixes the checkpatch.pl warning: WARNING: line over 80 characters All line over 80 characters in driver/staging/speakup/* are fixed. Signed-off-by: Shirish Gajera --- drivers/staging/speakup/main.c | 12 drivers/staging/speakup/serialio.h | 3 ++- driver

Re: [PATCH] brd: detect zero writes for saving ram

2015-02-07 Thread Akinobu Mita
2015-02-08 0:18 GMT+09:00 Akinobu Mita : > 2015-02-08 0:03 GMT+09:00 Akinobu Mita : >> This introduces a module parameter to detect zero writes and not to >> allocate memory. Read requests for unallocated (unwritten) region >> end up by reading zero. So this can save zeroed memory consumption >>

Re: [RFC PATCH 3/8] kmod - teach call_usermodehelper() to use a namespace

2015-02-07 Thread Ian Kent
On Fri, 2015-02-06 at 07:08 -0500, Jeff Layton wrote: > On Thu, 05 Feb 2015 10:34:11 +0800 > Ian Kent wrote: > > > The call_usermodehelper() function executes all binaries in the > > global "init" root context. This doesn't allow a binary to be run > > within a namespace (eg. the namespace of a c

Re: [PATCH v2] staging: speakup: Fix warning of line over 80 characters.

2015-02-07 Thread Greg KH
On Sat, Feb 07, 2015 at 06:34:39PM -0800, shirish gajera wrote: > On Sat, Feb 7, 2015 at 4:33 PM, Greg KH wrote: > > On Sat, Feb 07, 2015 at 09:46:37AM -0800, shirish gajera wrote: > > On Sat, Feb 7, 2015 at 1:09 AM, Greg KH > wrote: > > > >     On Sat, Jan 31, 2015 at 02:19:

[PATCH v2] Staging: fbtft: Fix Sparse warnings

2015-02-07 Thread Frederic Jacob
This fixes the folowing sparse warnings: fb_hx8340bn.c:111:6: warning: symbol 'set_addr_win' was not declared. Should it be static? fbtft_device.c:32:19: warning: symbol 'spi_device' was not declared. Should it be static? fbtft_device.c:33:24: warning: symbol 'p_device' was not declared. Should

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Greg KH
On Sun, Feb 08, 2015 at 02:12:04PM +1300, Michael Cree wrote: > On Sun, Feb 08, 2015 at 08:59:41AM +0800, Greg KH wrote: > > On Sun, Feb 08, 2015 at 01:47:29PM +1300, Michael Cree wrote: > > > On Sat, Feb 07, 2015 at 10:30:44PM +, Mathieu Desnoyers wrote: > > > > > On Fri, Feb 06, 2015 at 09:08

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Michael Cree
On Sun, Feb 08, 2015 at 08:59:41AM +0800, Greg KH wrote: > On Sun, Feb 08, 2015 at 01:47:29PM +1300, Michael Cree wrote: > > On Sat, Feb 07, 2015 at 10:30:44PM +, Mathieu Desnoyers wrote: > > > > On Fri, Feb 06, 2015 at 09:08:21PM -0500, Mathieu Desnoyers wrote: > > > > > A lockless_dereference

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Michael Cree
On Sat, Feb 07, 2015 at 10:30:44PM +, Mathieu Desnoyers wrote: > > On Fri, Feb 06, 2015 at 09:08:21PM -0500, Mathieu Desnoyers wrote: > > > A lockless_dereference() appears to be missing in llist_del_first(). > > > It should only matter for Alpha in practice. What could one anticipate to be th

[PATCH 1/2] tracing: Add condition check to RCU lockdep checks

2015-02-07 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The trace_tlb_flush() tracepoint can be called when a CPU is going offline. When a CPU is offline, RCU is no longer watching that CPU and since the tracepoint is protected by RCU, it must not be called. To prevent the tlb_flush tracepoint from being called when th

[PATCH 2/2] x86/tlb/trace: Do not trace on CPU that is offline

2015-02-07 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" When taking a CPU down for suspend and resume, a tracepoint may be called when the CPU has been designated offline. As tracepoints require RCU for protection, they must not be called if the current CPU is offline. Unfortunately, trace_tlb_flush() is called in thi

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Greg KH
On Sun, Feb 08, 2015 at 01:47:29PM +1300, Michael Cree wrote: > On Sat, Feb 07, 2015 at 10:30:44PM +, Mathieu Desnoyers wrote: > > > On Fri, Feb 06, 2015 at 09:08:21PM -0500, Mathieu Desnoyers wrote: > > > > A lockless_dereference() appears to be missing in llist_del_first(). > > > > It should

[PATCH 0/2] [GIT PULL] tracing: Fix tlb_flush TP called in RCU ignored location

2015-02-07 Thread Steven Rostedt
Linus, During testing Sedat Dilek hit a "suspicious RCU usage" splat that pointed out a real bug. During suspend and resume the tlb_flush tracepoint is called when the CPU is going offline. As the CPU has been noted as offline, RCU is ignoring that CPU, which means that it can not use RCU protect

Re: [PATCH v2] staging: speakup: Fix warning of line over 80 characters.

2015-02-07 Thread Greg KH
On Sat, Feb 07, 2015 at 09:46:37AM -0800, shirish gajera wrote: > On Sat, Feb 7, 2015 at 1:09 AM, Greg KH wrote: > > On Sat, Jan 31, 2015 at 02:19:43PM -0800, Shirish Gajera wrote: > > This patch fixes the checkpatch.pl warning: > > > > WARNING: line over 80 characters > > >

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Greg KH
On Sat, Feb 07, 2015 at 04:18:14PM -0800, Matt Turner wrote: > On Sat, Feb 7, 2015 at 2:30 PM, Mathieu Desnoyers > wrote: > > - Original Message - > >> From: "Greg KH" > >> To: "Mathieu Desnoyers" > >> Cc: "Huang Ying" , linux-kernel@vger.kernel.org, > >> "Paul McKenney" , > >> "David H

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Matt Turner
On Sat, Feb 7, 2015 at 2:30 PM, Mathieu Desnoyers wrote: > - Original Message - >> From: "Greg KH" >> To: "Mathieu Desnoyers" >> Cc: "Huang Ying" , linux-kernel@vger.kernel.org, "Paul >> McKenney" , >> "David Howells" , "Pranith Kumar" >> , sta...@vger.kernel.org >> Sent: Saturday, Feb

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Paul E. McKenney
On Sun, Feb 08, 2015 at 06:16:25AM +0800, Greg KH wrote: > On Fri, Feb 06, 2015 at 09:08:21PM -0500, Mathieu Desnoyers wrote: > > A lockless_dereference() appears to be missing in llist_del_first(). > > It should only matter for Alpha in practice. > > Meta-comment, do we really care about Alpha an

Re: [PATCH v3] net: bluetooth: hci_sock: Use 'const u32 *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-07 Thread Chen Gang S
On 2/8/15 03:52, Joe Perches wrote: > On Sat, 2015-02-07 at 21:24 +0800, Chen Gang S wrote: >> hci_test_bit() does not modify 2nd parameter, so it is better to let it >> be constant, or may cause build warning. The related warning (with >> allmodconfig under xtensa): >> >> net/bluetooth/hci_sock.

Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

2015-02-07 Thread Dave Hansen
Feel free to add to the pair: Acked-by: Dave Hansen -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH] vxlan: Wrong type passed to %pIS

2015-02-07 Thread Cong Wang
On Sat, Feb 7, 2015 at 4:34 AM, Rasmus Villemoes wrote: > On Sat, Feb 07 2015, Cong Wang wrote: > >> On Fri, Feb 6, 2015 at 6:17 PM, Rasmus Villemoes >> wrote: >>> src_ip is a pointer to a union vxlan_addr, one member of which is a >>> struct sockaddr. Passing a pointer to src_ip is wrong; one s

Please respond!!!

2015-02-07 Thread Antonio Vinal
Hello friend! I would like to contact you personally for an important proposal that could of interest to you. I send this email only to know if this email address is functional. I have something very important to discuss with you. Contact me for details by: Email: fernrodyu...@aol.jp with your

Re: mmc does not work in qemu n900

2015-02-07 Thread Pali Rohár
On Friday 30 January 2015 15:23:34 Pali Rohár wrote: > On Tuesday 27 January 2015 10:17:39 Pali Rohár wrote: > > On Tuesday 27 January 2015 00:21:04 Aaro Koskinen wrote: > > > Hi, > > > > > > On Mon, Jan 26, 2015 at 10:04:59PM +0100, Pali Rohár wrote: > > > > problem? Why any of these two patches

Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

2015-02-07 Thread Sedat Dilek
On Sat, Feb 7, 2015 at 11:14 PM, Paul E. McKenney wrote: > On Sat, Feb 07, 2015 at 04:52:05PM -0500, Steven Rostedt wrote: >> On Sat, 7 Feb 2015 12:09:48 -0800 >> "Paul E. McKenney" wrote: >> >> >The tag sequence has the meaning of: >> > git cherry-pick a1f84a3 >> > git cherry-pick

[PATCH] ASoC: sirf: atlas7: fix platform_no_drv_owner.cocci warnings

2015-02-07 Thread kbuild test robot
sound/soc/sirf/atlas7-iacc.c:625:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci CC: Rongjun Ying Signed-off-by: Fengguang Wu --- atlas7-iacc.c |1 -

[PATCH 2/3] drivers: usb: storage: cypress_atacb.c: trivial checkpatch fixes

2015-02-07 Thread Bas Peters
Fixes errors thrown by checkpatch over a space issue and the incorrect indentation of a switch statement. Signed-off-by: Bas Peters --- drivers/usb/storage/cypress_atacb.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/drivers/usb/storage/cypress_atacb.c b

[PATCH 1/3] drivers: usb: storage: alauda.c: properly place braces after function declarations

2015-02-07 Thread Bas Peters
This patch places braces on a new line following function declarations. Signed-off-by: Bas Peters --- drivers/usb/storage/alauda.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/usb/storage/alauda.c b/drivers/usb/storage/alauda.c index 62c2d9d..4b55a

[PATCH 0/3] drivers: usb: storage: fix some checkpatch errors

2015-02-07 Thread Bas Peters
This patchset adresses checkpatch errors in a few of the files in usb storage. More to follow. Bas Peters (3): drivers: usb: storage: alauda.c: properly place braces after function declarations drivers: usb: storage: cypress_atacb.c: trivial checkpatch fixes drivers: usb: storage: da

[PATCH 3/3] drivers: usb: storage: datafab.c: clean up a variety of checkpatch errors.

2015-02-07 Thread Bas Peters
This patch cleans up a variety of checkpatch errors: Bunch of space issues. C99 comments converted to /* */ format. Some switch statement indentations. "foo * bar" -> "foo *bar" Signed-off-by: Bas Peters --- drivers/usb/storage/datafab.c | 183 +++

Re: [PATCH/RFC 0/4] Probe deferral for IOMMU DT integration

2015-02-07 Thread a...@arndb.de
Laura Abbott hat am 6. Februar 2015 um 01:31 geschrieben: > > The requirement for this is based on a previous patch to add clock > support to the ARM SMMU driver[2]. Once we have clock support, it's > possible that the driver itself may need to be defered which breaks > a bunch of assumptions abou

Re: [PATCH] ARM: pxa: pxa27x skip default device initialization with DT

2015-02-07 Thread Daniel Mack
On 02/07/2015 10:18 PM, Robert Jarzmik wrote: > When booting via DT, the default PXA devices must not have been probed > before, otherwise the augmented information from the device tree is > ignored. > > This is the twin commit of commit 82ce44d104dc ("ARM: pxa3xx: skip > default device initializa

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Mathieu Desnoyers
- Original Message - > From: "Greg KH" > To: "Mathieu Desnoyers" > Cc: "Huang Ying" , linux-kernel@vger.kernel.org, "Paul > McKenney" , > "David Howells" , "Pranith Kumar" > , sta...@vger.kernel.org > Sent: Saturday, February 7, 2015 5:16:25 PM > Subject: Re: [PATCH] llist: Fix missing

Re: [PATCH] llist: Fix missing lockless_dereference()

2015-02-07 Thread Greg KH
On Fri, Feb 06, 2015 at 09:08:21PM -0500, Mathieu Desnoyers wrote: > A lockless_dereference() appears to be missing in llist_del_first(). > It should only matter for Alpha in practice. Meta-comment, do we really care about Alpha anymore? Is it still consered an "active" arch we support? I haven'

Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

2015-02-07 Thread Paul E. McKenney
On Sat, Feb 07, 2015 at 04:52:05PM -0500, Steven Rostedt wrote: > On Sat, 7 Feb 2015 12:09:48 -0800 > "Paul E. McKenney" wrote: > > >The tag sequence has the meaning of: > > git cherry-pick a1f84a3 > > git cherry-pick 1b9508f > > git cherry-pick fd21073 > > git cherry-pick

Re: [PATCH 2/3] clk: hi6220: Clock driver support for Hisilicon hi6220 SoC

2015-02-07 Thread Tyler Baker
Hi Bintian, On 7 February 2015 at 10:05, Brent Wang wrote: > Hello Tyler, > > Thank you very much for helping test this patchset! Not a problem. > > 2015-02-07 2:10 GMT+08:00 Tyler Baker : >> Hi Bintian, >> >> This patch applied to next-20150204 is producing build failures on >> various ARM def

[PATCH 2/7] drivers: isdn: act2000: capi.c: fix checkpatch errors

2015-02-07 Thread Bas Peters
This patch fixes the following checkpatch errors: 1. trailing statement 1. assignment of variable in if condition 1. incorrectly placed brace after function definition Signed-off-by: Bas Peters --- drivers/isdn/act2000/capi.c | 9 ++--- 1 file changed, 6 insertions(+)

[PATCH 3/7] drivers: isdn: act2000: remove assignments of variables in if conditions

2015-02-07 Thread Bas Peters
This patch removes all assignments of if conditions, which is not in accordance with the CodingStyle. --- drivers/isdn/act2000/module.c | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c

[PATCH 2/6] drivers: usb: core: devio.c: fix whitespace errors thrown by checkpatch.pl

2015-02-07 Thread Bas Peters
This patch fixes errors generated by checkpatch.pl relating to whitespace issues. Signed-off-by: Bas Peters --- drivers/usb/core/devio.c | 61 1 file changed, 31 insertions(+), 30 deletions(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/c

[PATCH 4/6] drivers: usb: core: hub.c: remove NULL initialization of static variables.

2015-02-07 Thread Bas Peters
NULL initialization of static variables is unnecessary as GCC kindly does this for us. Signed-off-by: Bas Peters --- drivers/usb/core/hub.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index aeb50bb..82983d9 100644 --- a/

[PATCH 6/6] drivers: usb: core: endpoint.c: fix trivial whitespace issue

2015-02-07 Thread Bas Peters
Changes space-based indentation to tab-based indentation. Signed-off-by: Bas Peters --- drivers/usb/core/endpoint.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/core/endpoint.c b/drivers/usb/core/endpoint.c index 39a2402..101983b 100644 --- a/drivers/usb/core/e

[PATCH 0/6] drivers: usb: core: fix various checkpatch errors.

2015-02-07 Thread Bas Peters
This patchset adresses various checkpatch errors found when running the checkpatch script on the directory. Bas Peters (6): drivers: usb: core: devio.c: remove assignment of variables in if conditions. drivers: usb: core: devio.c: fix whitespace errors thrown by checkpatch.pl

[PATCH 5/6] drivers: usb: core: hub.c: remove assignment of variables in if conditions.

2015-02-07 Thread Bas Peters
As specified in the CodingStyle. Signed-off-by: Bas Peters --- drivers/usb/core/hub.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index 82983d9..9afe8b0 100644 --- a/drivers/usb/core/hub.c +++ b/drivers/usb/core/hub.c

[PATCH 1/6] drivers: usb: core: devio.c: remove assignment of variables in if conditions.

2015-02-07 Thread Bas Peters
This patch removes assignment of variables in if conditions in accordance witht the CodingStyle. Signed-off-by: Bas Peters --- drivers/usb/core/devio.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index 0

[PATCH 3/6] drivers: usb: core: hcd.c: remove assignment of variables in if conditions.

2015-02-07 Thread Bas Peters
This patch removes assignment of variables in if conditions, as specified in CodingStyle. Signed-off-by: Bas Peters --- drivers/usb/core/hcd.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c index 11cee55..37c40

[PATCH 6/7] drivers: isdn: act2000: fix wrongly positioned brace.

2015-02-07 Thread Bas Peters
Trivial, but why not? :) Signed-off-by: Bas Peters --- drivers/isdn/act2000/module.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 889ffcb..9ba98ce 100644 --- a/drivers/isdn/act2000/module.c +++ b/drivers

[PATCH 7/7] drivers: isdn: act2000: capi.c: add macro \ and fix brace

2015-02-07 Thread Bas Peters
This patch adds the \ that was accidentally deleted in patch 2. It also adds a brace after the else statement, which is required due to the fact that the if statement has braces. --- drivers/isdn/act2000/capi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/isdn/a

Re: [PATCH 1/3] drivers: isdn: isdnloop: isdnloop.c: remove assignment of variables in if conditions, in accordance with the CodingStyle.

2015-02-07 Thread Bas Peters
Please discard all these e-mails, something went wrong and I sent the wrong directory of patches. 2015-02-07 22:54 GMT+01:00 Bas Peters : > Signed-off-by: Bas Peters > --- > drivers/isdn/isdnloop/isdnloop.c | 18 ++ > 1 file changed, 10 insertions(+), 8 deletions(-) > > diff --gi

[PATCH v2] ext4: fix indirect punch hole corruption

2015-02-07 Thread Omar Sandoval
Commit 4f579ae7de56 (ext4: fix punch hole on files with indirect mapping) rewrote FALLOC_FL_PUNCH_HOLE for ext4 files with indirect mapping. However, the case where the punch happens within one level of indirection is incorrect. It assumes that the partial branches returned from ext4_find_shared wi

[PATCH 0/2] drivers: isdn: icn: fix checkpatch errors

2015-02-07 Thread Bas Peters
This patch cleans up all checkpatch errors in the icn directory. Bas Peters (2): drivers: isdn: icn: icn.c: clean up all checkpatch errors drivers: isdn: icn: icn.h Clean up trivial checkpatch errors. drivers/isdn/icn/icn.c | 52 ++ drivers/isd

Re: [PATCH V2 0/7] drivers: isdn: act2000: fix checkpatch errors.

2015-02-07 Thread Bas Peters
Please discard this 2015-02-07 22:53 GMT+01:00 Bas Peters : > This patchset adresses many checkpatch errors found in the act2000 driver. > > Bas Peters (7): > drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors > drivers: isdn: act2000: capi.c: fix checkpatch errors > drivers: isdn:

Re: [PATCH 1/7] drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors

2015-02-07 Thread Bas Peters
Please discard this 2015-02-07 22:53 GMT+01:00 Bas Peters : > This patch adresses various checkpatch errors: > 3 assignments in if conditions > 1 return value enclosed in parenthesis > > Signed-off-by: Bas Peters > --- > drivers/isdn/act2000/act2000_isa.c | 11 +++ > 1 fi

[PATCH 2/2] drivers: isdn: icn: icn.h Clean up trivial checkpatch errors.

2015-02-07 Thread Bas Peters
Signed-off-by: Bas Peters --- drivers/isdn/icn/icn.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/isdn/icn/icn.h b/drivers/isdn/icn/icn.h index b713466..05daed2 100644 --- a/drivers/isdn/icn/icn.h +++ b/drivers/isdn/icn/icn.h @@ -54,7 +54,7 @@ typedef struct ic

[PATCH 5/7] drivers: isdn: act2000: module.c: remove parenthesres around return values.

2015-02-07 Thread Bas Peters
return is not a function, therefore parentheses are not needed. --- drivers/isdn/act2000/module.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 9359b36..889ffcb 100644 --- a/drivers/isdn/act2000/

[PATCH 0/3] Fix checkpatch errors in drivers/isdn/isdnloop

2015-02-07 Thread Bas Peters
This patchset adresses various checkpatch errors in the abovementioned driver. Bas Peters (3): drivers: isdn: isdnloop: isdnloop.c: remove assignment of variables in if conditions, in accordance with the CodingStyle. drivers: isdn: isdnloop: isdnloop.c: Fix brace positions according to

[PATCH 1/3] drivers: isdn: isdnloop: isdnloop.c: remove assignment of variables in if conditions, in accordance with the CodingStyle.

2015-02-07 Thread Bas Peters
Signed-off-by: Bas Peters --- drivers/isdn/isdnloop/isdnloop.c | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/isdn/isdnloop/isdnloop.c b/drivers/isdn/isdnloop/isdnloop.c index 5a4da94..af96317 100644 --- a/drivers/isdn/isdnloop/isdnloop.c +++ b/dri

[PATCH 1/2] drivers: isdn: icn: icn.c: clean up all checkpatch errors

2015-02-07 Thread Bas Peters
This patch cleans up various trivial checkpatch errors such as variable declarations in if statements, return values in parenthesis and a wrongly placed brace. Signed-off-by: Bas Peters --- drivers/isdn/icn/icn.c | 52 ++ 1 file changed, 31 inserti

[PATCH 1/7] drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors

2015-02-07 Thread Bas Peters
This patch adresses various checkpatch errors: 3 assignments in if conditions 1 return value enclosed in parenthesis Signed-off-by: Bas Peters --- drivers/isdn/act2000/act2000_isa.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/isdn/act20

[PATCH V2 0/7] drivers: isdn: act2000: fix checkpatch errors.

2015-02-07 Thread Bas Peters
This patchset adresses many checkpatch errors found in the act2000 driver. Bas Peters (7): drivers: isdn: act2000: act2000_isa.c: Fix checkpatch errors drivers: isdn: act2000: capi.c: fix checkpatch errors drivers: isdn: act2000: remove assignments of variables in if conditions driver

[PATCH 4/7] drivers: isdn: act2000: module.c: remove NULL-initialization of static variable.

2015-02-07 Thread Bas Peters
GCC takes care of this for us, thus it is not needed and theoretically only hoggs memory, allbeit only a bit. --- drivers/isdn/act2000/module.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 352916a..9359b36 1

Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

2015-02-07 Thread Steven Rostedt
On Sat, 7 Feb 2015 12:09:48 -0800 "Paul E. McKenney" wrote: >The tag sequence has the meaning of: > git cherry-pick a1f84a3 > git cherry-pick 1b9508f > git cherry-pick fd21073 > git cherry-pick > > Does that do what you need? Note, for this case it really doesn't apply,

Re: [PATCH] HID: sony: Enable Gasia third-party PS3 controllers

2015-02-07 Thread Lauri Kasanen
Hi, On Sat, 7 Feb 2015 17:31:33 +0100 Antonio Ospite wrote: > > > +#include > > > > Please don't. > > HID should be transport agnostic, so please refrain from directly call usb. > > > > I agree with Benjamin here. > > > > + > > > + ret = usb_interrupt_msg(dev, usb_sndintpipe(dev, 0x02),

[PATCH] ARM: pxa: pxa27x skip default device initialization with DT

2015-02-07 Thread Robert Jarzmik
When booting via DT, the default PXA devices must not have been probed before, otherwise the augmented information from the device tree is ignored. This is the twin commit of commit 82ce44d104dc ("ARM: pxa3xx: skip default device initialization when booting via DT"). Signed-off-by: Robert Jarzmik

Re: [PATCH 2/6] drivers: isdn: act2000: capi.c: fix checkpatch errors

2015-02-07 Thread Joe Perches
On Sat, 2015-02-07 at 21:55 +0100, Bas Peters wrote: > I thought it might have been > useful and a good way to learn Maybe pick a device you have (or maybe buy something in the staging directory like a realtek wireless device) and play with adding support for something in it. -- To unsubscribe

[PATCH] usb/core: Remove unneeded #ifdef and associated dead code

2015-02-07 Thread Andreas Ruprecht
In commit ceb6c9c862c8 ("USB / PM: Drop CONFIG_PM_RUNTIME from the USB core"), all occurrences of CONFIG_PM_RUNTIME in the USB core code were replaced by CONFIG_PM. This created the following structure of #ifdef blocks in drivers/usb/core/hub.c: [...] #ifdef CONFIG_PM #ifdef CONFIG_PM /* alway

Re: [PATCH 2/6] drivers: isdn: act2000: capi.c: fix checkpatch errors

2015-02-07 Thread Bas Peters
2015-02-07 21:43 GMT+01:00 Paul Bolle : > [Adding Tilman.] > > On Sat, 2015-02-07 at 11:19 -0800, Joe Perches wrote: >> Does anyone still use these cards? > > 0) Good question. > > 1) None of the (two dozen) commits in drivers/isdn/act2000/ added since > v2.6.12 appear to be triggered complaints, s

[PATCH 4/4] tridentfb: Add DDC support

2015-02-07 Thread Ondrej Zary
Add DDC support for Trident cards. Tested on TGUI9440, TGUI9680, 3DImage 9750, Blade3D 9880 and Blade XP. Signed-off-by: Ondrej Zary --- drivers/video/fbdev/Kconfig |9 ++ drivers/video/fbdev/tridentfb.c | 192 ++- 2 files changed, 196 insertions(+),

[PATCH 3/4] fb_ddc: Allow I2C adapters without SCL read capability

2015-02-07 Thread Ondrej Zary
i2c-algo-bit allows I2C adapters without SCL read capability to work but fb_ddc_read fails to work on them. Fix fb_ddc_read to work with I2C adapters not capable of reading SCL. Signed-off-by: Ondrej Zary --- drivers/video/fbdev/core/fb_ddc.c |8 +--- 1 file changed, 5 insertions(+), 3

[PATCH 2/4] tridentfb: Fix set_lwidth on TGUI9440 and CYBER9320

2015-02-07 Thread Ondrej Zary
According to X.Org driver, chips older than TGUI9660 have only 1 width bit in AddColReg. Touching the 2nd one causes I2C/DDC to fail on TGUI9440. Set only 1 bit of width in AddColReg on TGUI9440 and CYBER9320. Signed-off-by: Ondrej Zary --- drivers/video/fbdev/tridentfb.c | 10 -- 1 f

[PATCH 1/4] tridentfb: fix hang on Blade3D with CONFIG_CC_OPTIMIZE_FOR_SIZE

2015-02-07 Thread Ondrej Zary
When the kernel is compiled with -Os (CONFIG_CC_OPTIMIZE_FOR_SIZE), tridentfb hangs the machine upon load with Blade3D cards unless acceleration is disabled. This is caused by memcpy() which copies data byte-by-byte (rep movsb) when compiled with -Os. The card does not like that - it requires 32-b

Re: [PATCH 2/6] drivers: isdn: act2000: capi.c: fix checkpatch errors

2015-02-07 Thread Paul Bolle
[Adding Tilman.] On Sat, 2015-02-07 at 11:19 -0800, Joe Perches wrote: > Does anyone still use these cards? 0) Good question. 1) None of the (two dozen) commits in drivers/isdn/act2000/ added since v2.6.12 appear to be triggered complaints, suggestions, etc. of actual users. 2) Broader picture:

Re: [PATCH 2/2] KVM: x86: optimize delivery of TSC deadline timer interrupt

2015-02-07 Thread Paolo Bonzini
On 06/02/2015 21:51, Marcelo Tosatti wrote: > On Fri, Feb 06, 2015 at 01:16:59PM +0100, Paolo Bonzini wrote: >> The newly-added tracepoint shows the following results on >> the tscdeadline_latency test: >> >> qemu-kvm-8387 [002] 6425.558974: kvm_vcpu_wakeup: poll time >> 10407 ns

Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

2015-02-07 Thread Sedat Dilek
On Sat, Feb 7, 2015 at 9:09 PM, Paul E. McKenney wrote: > On Sat, Feb 07, 2015 at 10:20:02AM -0500, Steven Rostedt wrote: >> On Sat, 7 Feb 2015 09:01:34 +0100 >> Sedat Dilek wrote: >> >> >> > - Tested-by's >> > - Reference of 2/2 to 1/2 >> >> The two are together in the series and fix two differe

Re: [PATCH] classmate-laptop: fix sparse warning

2015-02-07 Thread Darren Hart
On Thu, Feb 05, 2015 at 02:57:44PM +, Lad Prabhakar wrote: > From: "Lad, Prabhakar" > > this patch fixes following sparse warning: > > classmate-laptop.c:523:61: warning: Using plain integer as NULL pointer > > Signed-off-by: Lad, Prabhakar Queued, thanks Lad. -- Darren Hart Intel Open

Re: [PATCH] Sony-laptop: fix sparse warning

2015-02-07 Thread Darren Hart
On Thu, Feb 05, 2015 at 02:49:41PM +, Lad Prabhakar wrote: > From: "Lad, Prabhakar" > > this patch fixes following sparse warning: > > sony-laptop.c:1035:29: warning: symbol 'sony_bl_props' was not declared. > Should it be static? > > Signed-off-by: Lad, Prabhakar Queued, thanks Lad. --

Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

2015-02-07 Thread Paul E. McKenney
On Sat, Feb 07, 2015 at 10:20:02AM -0500, Steven Rostedt wrote: > On Sat, 7 Feb 2015 09:01:34 +0100 > Sedat Dilek wrote: > > > > - Tested-by's > > - Reference of 2/2 to 1/2 > > The two are together in the series and fix two different bugs. They do > not need to reference each other. > > > - CC

Re: [PATCH] Sony-laptop: fix sparse warning

2015-02-07 Thread Darren Hart
On Sat, Feb 07, 2015 at 12:33:16PM +0100, Bjørn Mork wrote: > Darren Hart writes: > > On Thu, Feb 05, 2015 at 04:00:32PM +, Lad, Prabhakar wrote: > > > >> Lad, Prabhakar > >> Lad, Prabhakar > > > > I think there may be a problem with this in an unquoted email address. The > > unquoted local

Re: [PATCH 1/5] rcu,nohz: add state parameter to context_tracking_user_enter/exit

2015-02-07 Thread Paul E. McKenney
On Sat, Feb 07, 2015 at 09:30:41AM +0100, Frederic Weisbecker wrote: > On Fri, Feb 06, 2015 at 11:14:53PM -0800, Paul E. McKenney wrote: > > On Fri, Feb 06, 2015 at 10:34:21PM -0800, Paul E. McKenney wrote: > > > On Fri, Feb 06, 2015 at 10:53:34PM -0500, Rik van Riel wrote: > > > > -BEGIN PGP S

Re: [PATCH v3] net: bluetooth: hci_sock: Use 'const u32 *' instead of 'void *' for 2nd parameter of hci_test_bit()

2015-02-07 Thread Joe Perches
On Sat, 2015-02-07 at 21:24 +0800, Chen Gang S wrote: > hci_test_bit() does not modify 2nd parameter, so it is better to let it > be constant, or may cause build warning. The related warning (with > allmodconfig under xtensa): > > net/bluetooth/hci_sock.c: In function 'hci_sock_sendmsg': > net

Re: [PATCH 2/2] x86/tbl/trace: Do not trace on CPU that is offline

2015-02-07 Thread Sedat Dilek
On Sat, Feb 7, 2015 at 4:20 PM, Steven Rostedt wrote: > On Sat, 7 Feb 2015 09:01:34 +0100 > Sedat Dilek wrote: > > >> - Tested-by's >> - Reference of 2/2 to 1/2 > > The two are together in the series and fix two different bugs. They do > not need to reference each other. > >> - CC: stable v3.17+/

Re: [PATCH 2/6] drivers: isdn: act2000: capi.c: fix checkpatch errors

2015-02-07 Thread Joe Perches
On Sat, 2015-02-07 at 20:51 +0300, Sergei Shtylyov wrote: > On 02/07/2015 08:06 PM, Bas Peters wrote: > > > This patch fixes the following checkpatch errors: > > 1. trailing statement > > 1. assignment of variable in if condition > > 1. incorrectly placed brace after function definitio

Re: [PATCH 7/7] drivers: isdn: act2000: capi.c: add macro \ and fix brace

2015-02-07 Thread Julia Lawall
On Sat, 7 Feb 2015, Bas Peters wrote: > This patch adds the \ that was accidentally deleted in patch 2. It also adds > a brace after the else statement, which is required due to the fact that the > if statement has braces. You should fix the patch that was incorrect, rather than submitting a p

Re: [PATCH 7/7] drivers: isdn: act2000: capi.c: add macro \ and fix brace

2015-02-07 Thread Sergei Shtylyov
On 02/07/2015 10:05 PM, Bas Peters wrote: This patch adds the \ that was accidentally deleted in patch 2. It also adds a brace after the else statement, which is required due to the fact that the if statement has braces. This won't do, fix up the patch #2 please. And please wrap your chan

Re: [PATCH 2/7] drivers: isdn: act2000: capi.c: fix checkpatch errors

2015-02-07 Thread Sergei Shtylyov
On 02/07/2015 10:05 PM, Bas Peters wrote: This patch fixes the following checkpatch errors: 1. trailing statement 1. assignment of variable in if condition 1. incorrectly placed brace after function definition Signed-off-by: Bas Peters --- drivers/isdn/act2000/capi.

[PATCH 4/7] drivers: isdn: act2000: module.c: remove NULL-initialization of static variable.

2015-02-07 Thread Bas Peters
GCC takes care of this for us, thus it is not needed and theoretically only hoggs memory, allbeit only a bit. --- drivers/isdn/act2000/module.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 352916a..9359b36 1

[PATCH 5/7] drivers: isdn: act2000: module.c: remove parenthesres around return values.

2015-02-07 Thread Bas Peters
return is not a function, therefore parentheses are not needed. --- drivers/isdn/act2000/module.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/isdn/act2000/module.c b/drivers/isdn/act2000/module.c index 9359b36..889ffcb 100644 --- a/drivers/isdn/act2000/

[PATCH 7/7] drivers: isdn: act2000: capi.c: add macro \ and fix brace

2015-02-07 Thread Bas Peters
This patch adds the \ that was accidentally deleted in patch 2. It also adds a brace after the else statement, which is required due to the fact that the if statement has braces. --- drivers/isdn/act2000/capi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/isdn/a

  1   2   3   >