Re: [PATCH] Check for Null return of function of affs_bread in function affs_truncate

2014-07-12 Thread Nick Krause
On Fri, Jul 11, 2014 at 11:05 AM, Dan Carpenter wrote: > On Sat, Jun 21, 2014 at 01:59:15AM +0200, Thomas Gleixner wrote: >> On Fri, 20 Jun 2014, Nick Krause wrote: >> >> > Ok that's fine I would return as if it's a NULL the other parts of the >> > function can't continue. >> > Nick >> > >> > On T

Re: [PATCH] powerpc: Disable RELOCATABLE for COMPILE_TEST with PPC64

2014-07-12 Thread Nick Krause
On Mon, Jul 7, 2014 at 11:10 PM, Benjamin Herrenschmidt wrote: > On Mon, 2014-07-07 at 20:03 -0700, Guenter Roeck wrote: >> ping ... >> >> Ben, I know this is not perfect, but on the other side it is simple >> and would be easy to backport. With this in place, a complete/clean >> fix would not be

[PATCH] checkpatch: Add for_each tests to indentation and brace tests

2014-07-12 Thread Joe Perches
All the various for_each loop macros were not tested for trailing brace on the following lines and for bad indentation. Add them. Reported-by: Greg KH Signed-off-by: Joe Perches --- On Sat, 2014-07-12 at 16:08 -0700, Greg KH wrote: > The following function: > > $ cat foo.c > > static int foo

[PULL REQUEST] slave dmaenegine fixes

2014-07-12 Thread Vinod Koul
Hi Linus, Here is the fixes pull request for 3.16. We have two small fixes. First one from Daniel to handle 0-length packets for usb cppi dma. Second by Russell for imx-sdam cyclic residue reporting The following changes since commit 4c834452aad01531db949414f94f817a86348d59: Linus Torvalds (1):

Re: [GIT PULL] at91: dt for 3.17 #1

2014-07-12 Thread Olof Johansson
On Wed, Jul 09, 2014 at 07:29:39PM +0200, Nicolas Ferre wrote: > Arnd, Olof, Kevin, > > This is a big pull-request concerning DT on AT91. The bulk of it is the move > of > the remaining SoCs to CCF. Some fixes and little additions are also included. > Alexandre's work were taken as a pull-request

Re: [GIT PULL] at91: drivers for 3.17 #1

2014-07-12 Thread Olof Johansson
On Wed, Jul 09, 2014 at 06:53:57PM +0200, Nicolas Ferre wrote: > Arnd, Olof, Kevin, > > This update delayed to 3.17, is about replacing the existing calls to the > older, non-standard drivers by the use of the newer "pwm-atmel" which takes > advantage of the PWM framework. > All concerned maintain

Re: linux-next: Tree for Jul 11

2014-07-12 Thread Nick Krause
On Sat, Jul 12, 2014 at 6:26 PM, Nick Krause wrote: > On Fri, Jul 11, 2014 at 3:18 AM, Stephen Rothwell > wrote: >> Hi all, >> >> Changes since 20140710: >> >> My fixes tree contains: >> powerpc: Disable RELOCATABLE for COMPILE_TEST with PPC64 >> >> The tty.current tree gained a build fa

Re: [PATCH 00/83] AMD HSA kernel driver

2014-07-12 Thread Jerome Glisse
On Sat, Jul 12, 2014 at 09:55:49PM +, Gabbay, Oded wrote: > On Fri, 2014-07-11 at 17:18 -0400, Jerome Glisse wrote: > > On Thu, Jul 10, 2014 at 10:51:29PM +, Gabbay, Oded wrote: > > > On Thu, 2014-07-10 at 18:24 -0400, Jerome Glisse wrote: > > > > On Fri, Jul 11, 2014 at 12:45:27AM +0300,

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-12 Thread Marek Vasut
On Sunday, July 13, 2014 at 05:07:10 AM, Chen Gang wrote: > Several drivers need 'devm_ioremap_resource' which need HAS_IOMEM enabled. > So let them depend on HAS_IOMEM. > > The related error (with allmodconfig under score): > > MODPOST 1365 modules > ERROR: "devm_ioremap_resource" [drivers

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-12 Thread Chen Gang
After this last patch, score architecture can pass allmodconfig. :-) And also find a compiler issue, I will try to fix it, but shall not notify kernel mailing list, again. The related issue is below (it seems a kernel issue, but in fact, it is a compiler's issue): CC [M] drivers/staging/lus

Re: checkpatch.pl misses list_for_each_entry() coding style issues

2014-07-12 Thread Joe Perches
On Sat, 2014-07-12 at 16:08 -0700, Greg KH wrote: > static int foo_init(void) > { > > list_for_each_entry(foo, &foo, list) > { > do_something_foo(foo); > } > > return 0; > } > > Will not be caught by checkpatch: > > $ ./scripts/checkpatch.pl --file foo.c >

[PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

2014-07-12 Thread Chen Gang
Several drivers need 'devm_ioremap_resource' which need HAS_IOMEM enabled. So let them depend on HAS_IOMEM. The related error (with allmodconfig under score): MODPOST 1365 modules ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined! ERROR: "devm_ioremap_resource" [dri

[PATCH] drivers: firewire: Let several sub-modules depend on HAS_DMA

2014-07-12 Thread Chen Gang
Several sub-modules of 'firewire' need HAS_DMA, so let them depend on it. FIREWIRE_NET and FIREWIRE_OHCI use 'core-iso.c' which also needs HAS_DMA, so need 'ifdef' the related function by CONFIG_HAS_DMA in 'core-iso.c'. The related error (with allmodconfig under score): MODPOST 1365 modules

Re: [PATCH] reformat_with_checkpatch: Add automation to checkpatch

2014-07-12 Thread Joe Perches
On Sat, 2014-07-12 at 12:28 -0700, Greg KH wrote: > On Sat, Jul 12, 2014 at 11:29:37AM -0700, Joe Perches wrote: > > On Sat, 2014-07-12 at 10:55 -0700, Greg KH wrote: > > > Yes, the warnings are all due to line-length, but Joe, you shouldn't add > > > a patch that causes more checkpatch warnings th

[PATCH] drivers/rtc/Kconfig: Let several drivers depend on HAS_IOMEM to avoid compiling issue

2014-07-12 Thread Chen Gang
Some drivers need 'devm_ioremap_resource' or 'devm_ioremap' which need HAS_IOMEM, so let them depend on it. The related error (with allmodconfig under score): MODPOST 1365 modules ERROR: "devm_ioremap_resource" [drivers/rtc/rtc-xgene.ko] undefined! ERROR: "devm_ioremap_resource" [drivers/

[PATCH] drivers/crypto/Kconfig: Let 'DEV_QCE' depend on both HAS_DMA and HAS_IOMEM

2014-07-12 Thread Chen Gang
'DEV_QCE' needs both HAS_DMA and HAS_IOMEM, so let it depend on them. The related error (with allmodconfig under score): MODPOST 1365 modules ERROR: "devm_ioremap_resource" [drivers/crypto/qce/qcrypto.ko] undefined! ERROR: "dma_map_sg" [drivers/crypto/qce/qcrypto.ko] undefined! ERROR: "

Re: [PATCH v3 4/4] ARM: tegra: roth: add display DT node

2014-07-12 Thread Alexandre Courbot
On Fri, Jul 11, 2014 at 10:14 PM, Thierry Reding wrote: > On Tue, Jul 08, 2014 at 09:32:14PM +0900, Alexandre Courbot wrote: >> Tegra DSI support has been fixed to support continuous clock behavior that >> the panel used on SHIELD requires, so finally add its device tree node >> since it is functi

Re: [RESEND PATCH] staging: android: Clean up binder_buffer_size()

2014-07-12 Thread Greg Kroah-Hartman
On Sat, Jul 12, 2014 at 09:50:09PM -0300, Lucas Tanure wrote: > Kernel coding style. Remove useless else statement after return. > > Signed-off-by: Lucas Tanure > --- > drivers/staging/android/binder.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) What tree are you making this patch

[PATCH] dgnc: DGNC_VERIFY_BOARD macro in do while block

2014-07-12 Thread Jeremiah Mahler
Enclose the body of the multi-statement DGNC_VERIFY_BOARD macro inside a do - while block as per Documentation/CodingStyle. Fixes 1 error found by checkpatch.pl. Signed-off-by: Jeremiah Mahler --- drivers/staging/dgnc/dgnc_sysfs.c | 20 +++- 1 file changed, 11 insertions(+), 9 d

Re: [PATCH v2] not adding modules range to kcore if it's equal to vmcore range

2014-07-12 Thread Baoquan He
On some ARCHs modules range is eauql to vmalloc range. E.g on i686 "#define MODULES_VADDR VMALLOC_START" "#define MODULES_END VMALLOC_END" This will cause 2 duplicate program segments in /proc/kcore, makes user confused. In this patch a judgment added to check if modules range is equal to vma

[RESEND PATCH] staging: android: Clean up binder_buffer_size()

2014-07-12 Thread Lucas Tanure
Kernel coding style. Remove useless else statement after return. Signed-off-by: Lucas Tanure --- drivers/staging/android/binder.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/android/binder.c b/drivers/staging/android/binder.c index cfe4bc8..1a0966b 10064

[PATCH] not adding modules range to kcore if it's equal to vmcore range

2014-07-12 Thread Baoquan He
On some ARCHs modules range is eauql to vmalloc range. E.g on i686 This will cause 2 duplicate program segments in /proc/kcore, makes user confused. In this patch a judgment added to check if modules range is equal to vmalloc range. If yes, just skip adding the modules range. Signed-off-by: Baoqu

Re: [PATCH 0/4] New Qualcomm PMIC pin controller drivers

2014-07-12 Thread Bjorn Andersson
On Thu, Jul 10, 2014 at 6:39 AM, Ivan T. Ivanov wrote: > On Wed, 2014-07-09 at 07:02 -0700, Bjorn Andersson wrote: [..] >> Correct Ivan; we do however share the same issues related to how to do >> interrupt handling, > > Yep, but do we actually need to do interrupt handling in driver? > Interrupts

Re: [PATCH 10/11] perf, tools, test: Add test case for alias and JSON parsing

2014-07-12 Thread Andi Kleen
> missing the tests/aliases.o file > > AR libperf.a > ar: tests/aliases.o: No such file or directory Fixed here (and in git) --- From: Andi Kleen Add a simple test case to perf test that runs perf download and parses all the available events, including json events. This needs adding

[PATCH] drivers: tty: Fix use-after-free in pty_common_install

2014-07-12 Thread Rasmus Villemoes
In 2c964a2f "drivers: tty: Merge alloc_tty_struct and initialize_tty_struct", I messed up the refactorization of pty_common_install, causing use-after-free and NULL pointer derefs on various error paths. This should fix it. Reported-by: Julia Lawall Reported-by: Dan Carpenter Signed-off-by: Rasm

Re: bit fields && data tearing

2014-07-12 Thread Benjamin Herrenschmidt
On Sat, 2014-07-12 at 22:51 +0200, Oleg Nesterov wrote: > OK, looks like this is compiler bug, > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52080 > > Thanks to Dan who informed me privately. So yes, there's is this compiler bug which means a bitfield access can cause a r-m-w access to a neig

Re: [PATCHv6] perf tools: enable close-on-exec flag on perf file descriptor

2014-07-12 Thread Jiri Olsa
On Mon, Jun 30, 2014 at 10:28:47PM +0200, Yann Droneaud wrote: > In commit a21b0b354d4a ('perf: Introduce a flag to enable > close-on-exec in perf_event_open()'), flag PERF_FLAG_FD_CLOEXEC > was added to perf_event_open(2) syscall to allows userspace > to atomically enable close-on-exec behavor whe

RE: scsi-mq V2

2014-07-12 Thread Elliott, Robert (Server Storage)
> I will see if that solves the problem with the scsi-mq-3 tree, or > at least some of the bisect trees leading up to it. scsi-mq-3 is still going after 45 minutes. I'll leave it running overnight. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a messa

Re: [PATCH 10/11] perf, tools, test: Add test case for alias and JSON parsing

2014-07-12 Thread Jiri Olsa
On Fri, Jul 11, 2014 at 04:59:24PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Add a simple test case to perf test that runs perf download and parses > all the available events, including json events. > > This needs adding an all event iterator to pmu.c > > v2: Rename identifiers > v3: Only

checkpatch.pl misses list_for_each_entry() coding style issues

2014-07-12 Thread Greg KH
Hi Joe, The following function: $ cat foo.c static int foo_init(void) { list_for_each_entry(foo, &foo, list) { do_something_foo(foo); } return 0; } Will not be caught by checkpatch: $ ./scripts/checkpatch.pl --file foo.c total: 0 errors, 0 wa

Re: [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up

2014-07-12 Thread Kukjin Kim
On 07/05/14 05:15, Arnd Bergmann wrote: On Friday 04 July 2014 19:48:00 Tomasz Figa wrote: This huge series is a (hopefully final) attempt to convert Samsung S5PV210 into a DT-only and multiplatform-aware platform. It consists of several steps to gradually replace legacy code with one meeting cu

[PATCH 2/2] staging: comedi: addi_apci_1564: use addi_watchdog module to init watchdog subdevice

2014-07-12 Thread Chase Southwood
Use the addi_watchdog module to provide support for the watchdog subdevice. Also, rearrange the subdevice init blocks so that the order makes sense. Digital input/output subdevices and subdevices for DI/DO interrupt support, followed by timer/counter/watchdog subdevices is the new order. Signed-o

[PATCH 1/2] staging: comedi: addi_apci_1564: driver no longer needs to include addi_common.h

2014-07-12 Thread Chase Southwood
This driver no longer depends on anything in addi_common.h, save for a few headers that it was including indirectly. Remove the include of addi_common.h and add the includes of and directly. Signed-off-by: Chase Southwood Cc: Ian Abbott Cc: H Hartley Sweeten --- drivers/staging/comedi/drive

Re: linux-next: Tree for Jul 11

2014-07-12 Thread Nick Krause
On Fri, Jul 11, 2014 at 3:18 AM, Stephen Rothwell wrote: > Hi all, > > Changes since 20140710: > > My fixes tree contains: > powerpc: Disable RELOCATABLE for COMPILE_TEST with PPC64 > > The tty.current tree gained a build failure so I used the version from > next-20140710. > > The irqchip

[PATCH] timekeeping: Fix missing memory barriers in NMI safe CLOCK_MONOTONIC[_RAW]

2014-07-12 Thread Mathieu Desnoyers
Commit c7b080e148d9 "timekeeping: Provide fast and NMI safe access to CLOCK_MONOTONIC[_RAW]" lacks memory barriers. The following scenario demonstrates a race condition where the reader can see a corrupted clock value. Initial conditions: tkf->seq = 0 tkf->base[0] and tkf->base[1] are initialize

Re: Blackfin Build fixs

2014-07-12 Thread Nick Krause
On Fri, Jul 11, 2014 at 12:18 PM, Nick Krause wrote: > Hey Steven, > I having been testing what builds are still failing from Steven > Rothwell's tests and there seem to be two failing > still for blackfin. I am tracing the issues to make it easier for you > as a maintainer. BF561-ACVILON_defconf

Re: [PATCHv2 2/5] mailbox/omap: add support for parsing dt devices

2014-07-12 Thread Pavel Machek
On Fri 2014-07-11 17:04:09, Suman Anna wrote: > Logic has been added to the OMAP2+ mailbox code to parse the > mailbox dt nodes and construct the different sub-mailboxes > associated with the instance. The DT representation of the > sub-mailbox devices is different from legacy platform data > repre

[PATCH 3/4] perf tools: Factor eprintf to allow different debug variables

2014-07-12 Thread Jiri Olsa
This way we can easily reuse current debug functions for another debug variables other than verbose. Cc: Arnaldo Carvalho de Melo Cc: Corey Ashford Cc: David Ahern Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter Zijlstra Signed-off-by: Jiri Olsa ---

[PATCH 2/4] perf tools: Move pr_* debug macros into debug object

2014-07-12 Thread Jiri Olsa
Moving pr_* debug macros to have it with in same object as debug variables, becase we will change them to use verbose variable in next patch. Cc: Arnaldo Carvalho de Melo Cc: Corey Ashford Cc: David Ahern Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: Namhyung Kim Cc: Paul Mackerras Cc: Peter

[PATCH 4/4] perf tools: Add --debug optionto set debug variable

2014-07-12 Thread Jiri Olsa
Adding --debug option as a way to setup debug variables. Starting with support for verbose, more will come. It's possible to use it now with report command: $ perf report --debug verbose $ perf report --debug verbose=2 I'll need this support to add separated debug variable for ordered events

[PATCH 1/4] perf tools: Remove verbose from functions prototypes

2014-07-12 Thread Jiri Olsa
And use verbose as an global object in following fuunctions: __map_groups__fprintf_maps __map_groups__fprintf_removed_maps map_groups__fprintf_maps map_groups__fprintf Also making map_groups__fprintf_maps static. Cc: Arnaldo Carvalho de Melo Cc: Corey Ashford Cc: David Ahern Cc: Freder

[PATCH 0/4] perf tools: Add support for more debug variables

2014-07-12 Thread Jiri Olsa
hi, adding support for more debug variables and --debug option as a way to setup debug variables. Starting with support for verbose, more will come. It's possible to use it now with report command: $ perf report --debug verbose $ perf report --debug verbose=2 I need this for ordered events ob

Re: [PATCH 00/83] AMD HSA kernel driver

2014-07-12 Thread Gabbay, Oded
On Fri, 2014-07-11 at 17:18 -0400, Jerome Glisse wrote: > On Thu, Jul 10, 2014 at 10:51:29PM +, Gabbay, Oded wrote: > > On Thu, 2014-07-10 at 18:24 -0400, Jerome Glisse wrote: > > > On Fri, Jul 11, 2014 at 12:45:27AM +0300, Oded Gabbay wrote: > > > > This patch set implements a Heterogeneou

RE: scsi-mq V2

2014-07-12 Thread Elliott, Robert (Server Storage)
> -Original Message- > From: Benjamin LaHaise [mailto:b...@kvack.org] > Sent: Friday, 11 July, 2014 9:55 AM > To: Elliott, Robert (Server Storage) > Cc: Christoph Hellwig; Jeff Moyer; Jens Axboe; dgilb...@interlog.com; James > Bottomley; Bart Van Assche; linux-s...@vger.kernel.org; linux-

[PATCH] staging: android: Coding style fix

2014-07-12 Thread Modestas Stankus
This patch fixes warnings of checkpatch.pl script: WARNING: void function return statements are not generally useful + return; +} Task of Eudyptula challenge. --- drivers/staging/android/ion/ion_carveout_heap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/android/ion/i

Re: [PATCH V8 2/2] serial/uart/8250: Add tunable RX interrupt trigger I/F of FIFO buffers

2014-07-12 Thread Greg KH
On Thu, Jul 10, 2014 at 04:16:37PM -0700, Greg Kroah-Hartman wrote: > On Thu, Jul 10, 2014 at 03:31:57PM +0100, One Thousand Gnomes wrote: > > > I really don't like the way that the tty core has been changed to handle > > > multiple attribute groups, as I feel tty drivers shouldn't be creating > >

Re: [PATCH 2/2] x86_64,signal: Remove 'fs' and 'gs' from sigcontext

2014-07-12 Thread Andy Lutomirski
On Sat, Jul 12, 2014 at 11:52 AM, Andi Kleen wrote: > On Sat, Jul 12, 2014 at 11:40:03AM -0700, H. Peter Anvin wrote: >> Because you are doing something weird (like Pin, for example) and take an >> asynchronous fault? > > But even for pin that would need executing 16 bit code, or really weird > 3

[PATCH v2 2/2] arch/x86/xen: Silence compiler warnings

2014-07-12 Thread Daniel Kiper
Compiler complains in the following way when x86 32-bit kernel with Xen support is build: CC arch/x86/xen/enlighten.o arch/x86/xen/enlighten.c: In function ‘xen_start_kernel’: arch/x86/xen/enlighten.c:1726:3: warning: right shift count >= width of type [enabled by default] Such line conta

[PATCH v2 1/2] xen: Silence compiler warnings

2014-07-12 Thread Daniel Kiper
Add inline keyword to silence the following compiler warnings if xen_efi_probe() is not used: CC arch/x86/xen/setup.o In file included from arch/x86/xen/xen-ops.h:7:0, from arch/x86/xen/setup.c:31: include/xen/xen-ops.h:43:35: warning: ‘xen_efi_probe’ defined but not used

[PATCH v2 0/2] xen: Silence compiler warnings

2014-07-12 Thread Daniel Kiper
Hi, Here are two patches that follow earlier Xen dom0 EFI series and fix some compiler warnings found during detailed build tests. Both patches should be applied to: git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git next v2 contains fix suggested by Konrad Rzeszutek Wilk. Daniel

Re: bit fields && data tearing

2014-07-12 Thread Oleg Nesterov
OK, looks like this is compiler bug, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52080 Thanks to Dan who informed me privately. On 07/12, Oleg Nesterov wrote: > > Hello, > > I am not sure I should ask here, but since Documentation/memory-barriers.txt > mentions load/store tearing perhaps my que

[GIT PULL] TTY/Serial fixes for 3.16-rc5

2014-07-12 Thread Greg KH
The following changes since commit cd3de83f147601356395b57a8673e9c5ff1e59d1: Linux 3.16-rc4 (2014-07-06 12:37:51 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git/ tags/tty-3.16-rc5 for you to fetch changes up to 34c4eda809e1efb16c5

[GIT PULL] USB driver fixes for 3.16-rc5

2014-07-12 Thread Greg KH
The following changes since commit cd3de83f147601356395b57a8673e9c5ff1e59d1: Linux 3.16-rc4 (2014-07-06 12:37:51 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git/ tags/usb-3.16-rc5 for you to fetch changes up to eb82a3d846fab00c4b9

[GIT PULL] char/misc driver fixes for 3.16-rc5

2014-07-12 Thread Greg KH
The following changes since commit a497c3ba1d97fc69c1e78e7b96435ba8c2cb42ee: Linux 3.16-rc2 (2014-06-21 19:02:54 -1000) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git/ tags/char-misc-3.16-rc5 for you to fetch changes up to 6d827fb

[GIT PULL] Staging driver fixes for 3.16-rc5

2014-07-12 Thread Greg KH
The following changes since commit cd3de83f147601356395b57a8673e9c5ff1e59d1: Linux 3.16-rc4 (2014-07-06 12:37:51 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/ tags/staging-3.16-rc5 for you to fetch changes up to bdac8ca90e0

Re: [patch 54/55] timekeeping: Provide fast and NMI safe access to CLOCK_MONOTONIC[_RAW]

2014-07-12 Thread Mathieu Desnoyers
- Original Message - > From: "Thomas Gleixner" > To: "Mathieu Desnoyers" > Cc: "LKML" , "John Stultz" > , "Peter Zijlstra" > , "Steven Rostedt" > Sent: Saturday, July 12, 2014 4:04:59 PM > Subject: Re: [patch 54/55] timekeeping: Provide fast and NMI safe access to > CLOCK_MONOTONIC[_RA

[PATCH 2/2] pinctrl: remove all usage of gpio_remove ret val in driver/pinctl

2014-07-12 Thread abdoulaye berthe
Signed-off-by: abdoulaye berthe --- drivers/pinctrl/pinctrl-adi2.c | 9 - drivers/pinctrl/pinctrl-as3722.c | 11 ++- drivers/pinctrl/pinctrl-baytrail.c | 5 + drivers/pinctrl/pinctrl-coh901.c | 10 ++ drivers/pinctrl/pinctrl-rockchip.c | 1

[PATCH 0/3] remove all usage of gpio_remove retval

2014-07-12 Thread abdoulaye berthe
This removes all usage of gpio_remove retval driver/gpio: 0001-gpio-remove-all-usage-of-gpio_remove-retval-in-drive driver/pinctl: 0002-pinctrl-remove-all-usage-of-gpio_remove-ret-val-in-d all the rest: 0003-driver-gpio-remove-all-usage-of-gpio_remove-retval-i abdoulaye berthe (1): driver:gpio r

[PATCH 1/1] gpio: remove all usage of gpio_remove retval in driver/gpio

2014-07-12 Thread abdoulaye berthe
Signed-off-by: abdoulaye berthe --- drivers/gpio/gpio-74x164.c | 8 +++- drivers/gpio/gpio-adnp.c | 9 + drivers/gpio/gpio-adp5520.c| 8 +--- drivers/gpio/gpio-adp5588.c| 6 +- drivers/gpio/gpio-amd8111.c| 3 +-- drivers/gpio/gpio-arizona.c| 3 +

[PATCH 3/3] driver:gpio remove all usage of gpio_remove retval in driver

2014-07-12 Thread abdoulaye berthe
this remove all reference to gpio_remove retval in all driver except pinctrl and gpio. the same thing is done for gpio and pinctrl in two different patches. Signed-off-by: abdoulaye berthe --- arch/arm/common/scoop.c| 10 ++ arch/mips/txx9/generic/setup.c

[PATCH v2] lib/Kconfig.debug: introduce CONFIG_CC_STACK_USAGE

2014-07-12 Thread Konstantin Khlebnikov
This patch adds option for collecting stack usage statistics via gcc option '-fstack-usage' (needs gcc 4.6 or newer). For each .o file gcc dumps stack-frame sizes into .su file. File format: ::: select.c:870:5:do_sys_poll 1040static rtnetlink.c:1880:12:rtnl_newlink552

Re: [PATCH] media: em28xx-dvb - fix em28xx_dvb_resume() to not unregister i2c and dvb

2014-07-12 Thread Antti Palosaari
Reviewed-by: Antti Palosaari Antti On 07/09/2014 04:21 PM, Shuah Khan wrote: em28xx_dvb_resume() unregisters i2c tuner, i2c demod, and dvb. This erroneous cleanup results in i2c tuner, i2c demod, and dvb devices unregistered and removed during resume. This error is a result of merge conflict b

Re: [PATCH] staging: android: Cleanup style issues

2014-07-12 Thread Greg KH
On Sat, Jul 12, 2014 at 01:23:10PM -0700, Greg KH wrote: > On Sat, Jul 12, 2014 at 09:55:56PM +0200, Peter Senna Tschudin wrote: > > This patch fixes the following checkpatch warnings: > > - Remove else after return > > - Add space after declaration > > > > Tested by compilation only. > > > > S

Re: [PATCH] media: em28xx - add error handling for KWORLD dvb_attach failures

2014-07-12 Thread Antti Palosaari
Looks correct! All the other devices, excluding those few PCTV ones I have added, has that same bug... Deadlocks if tuner attach fails. Reviewed-by: Antti Palosaari regards Antti On 07/09/2014 11:36 PM, Shuah Khan wrote: Add error hanlding when EM2870_BOARD_KWORLD_A340 dvb_attach() for fe

Re: [PATCH] staging: android: Cleanup style issues

2014-07-12 Thread Greg KH
On Sat, Jul 12, 2014 at 09:55:56PM +0200, Peter Senna Tschudin wrote: > This patch fixes the following checkpatch warnings: > - Remove else after return > - Add space after declaration > > Tested by compilation only. > > Signed-off-by: Peter Senna Tschudin > --- > drivers/staging/android/sync

Re: [PATCH] media: em28xx-dvb unregister i2c tuner and demod after fe detach

2014-07-12 Thread Antti Palosaari
Moikka Shuah! I suspect that patch makes no sense. On DVB there is runtime PM controlled by DVB frontend. It wakes up all FE sub-devices when frontend device is opened and sleeps when closed. FE release() is not relevant at all for those sub-devices which are implemented as a proper I2C clien

Re: [PATCH] lib/Kconfig.debug: introduce CONFIG_CC_STACK_USAGE

2014-07-12 Thread Konstantin Khlebnikov
Damn, I forget to add them to the .gitignore On Sun, Jul 13, 2014 at 12:01 AM, Konstantin Khlebnikov wrote: > This patch adds option for collecting stack usage statistics > via gcc option '-fstack-usage' (needs gcc 4.6 or newer). > For each .o file gcc dumps stack-frame sizes into .su file. > > F

Re: [patch 54/55] timekeeping: Provide fast and NMI safe access to CLOCK_MONOTONIC[_RAW]

2014-07-12 Thread Thomas Gleixner
On Sat, 12 Jul 2014, Thomas Gleixner wrote: > On Sat, 12 Jul 2014, Mathieu Desnoyers wrote: > > I'm perhaps missing something here, but what happens with the > > following scenario ? > > > > Initial conditions: > > > > tkf->seq = 0 > > tkf->base[0] and tkf->base[1] are initialized. > > > > CPU 0

[PATCH] lib/Kconfig.debug: introduce CONFIG_CC_STACK_USAGE

2014-07-12 Thread Konstantin Khlebnikov
This patch adds option for collecting stack usage statistics via gcc option '-fstack-usage' (needs gcc 4.6 or newer). For each .o file gcc dumps stack-frame sizes into .su file. File format: ::: select.c:870:5:do_sys_poll 1040static rtnetlink.c:1880:12:rtnl_newlink552

[PATCH 1/1] doc: fix docbook generation

2014-07-12 Thread Pranith Kumar
kernel/timer.c and kernel/hrtimer.c have moved to kernel/time directory. This causes docbook generation to fail. The following commit fixes it by updating the references. Signed-off-by: Pranith Kumar --- Documentation/DocBook/device-drivers.tmpl | 4 ++-- 1 file changed, 2 insertions(+), 2 delet

[PATCH] staging: android: Cleanup style issues

2014-07-12 Thread Peter Senna Tschudin
This patch fixes the following checkpatch warnings: - Remove else after return - Add space after declaration Tested by compilation only. Signed-off-by: Peter Senna Tschudin --- drivers/staging/android/sync_debug.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a

Re: [patch 35/55] mfd: cros_ec_spi: Use ktime_get_ns()

2014-07-12 Thread Thomas Gleixner
On Fri, 11 Jul 2014, Lee Jones wrote: > > Replace the ever recurring: > > ts = ktime_get_ts(); > > ns = timespec_to_ns(&ts); > > with > > ns = ktime_get_ns(); > > > > Signed-off-by: Thomas Gleixner > > Cc: Samuel Ortiz > > Only Sam? How old is your version of MAINTAINERS? Hmm. I

Re: slub/debugobjects: lockup when freeing memory

2014-07-12 Thread Paul E. McKenney
On Sat, Jul 12, 2014 at 02:03:57PM -0400, Sasha Levin wrote: > On 06/20/2014 11:40 AM, Paul E. McKenney wrote: > > rcu: Export debug_init_rcu_head() and and debug_init_rcu_head() > > > > Currently, call_rcu() relies on implicit allocation and initialization > > for the debug-objects handling of RC

Re: [patch 54/55] timekeeping: Provide fast and NMI safe access to CLOCK_MONOTONIC[_RAW]

2014-07-12 Thread Thomas Gleixner
On Sat, 12 Jul 2014, Mathieu Desnoyers wrote: > - Original Message - > > From: "Thomas Gleixner" > > On Fri, 11 Jul 2014, Mathieu Desnoyers wrote: > > > Any reason why the updater wouldn't do: > > > > > > tkf->seq++; > > > smp_wmb(); > > > update(tkf->base[1 - (tkf->seq & 0x01)], tk); > >

Re: [patch 54/55] timekeeping: Provide fast and NMI safe access to CLOCK_MONOTONIC[_RAW]

2014-07-12 Thread Thomas Gleixner
On Sat, 12 Jul 2014, Mathieu Desnoyers wrote: > I'm perhaps missing something here, but what happens with the > following scenario ? > > Initial conditions: > > tkf->seq = 0 > tkf->base[0] and tkf->base[1] are initialized. > > CPU 0 CPU 1 >

[PATCH 1/1] scripts/get_maintainer: increase threshold for --follow to reduce time

2014-07-12 Thread Pranith Kumar
get_maintainer tries to follow files with a matching threshold of default 50%. This is not really necessary as we do not change a file and move it in the same commit usually. Increasing the threshold to 90% should be sufficient. I encountered this on a recent commit where I was touching 12 files.

Re: [PATCH] reformat_with_checkpatch: Add automation to checkpatch

2014-07-12 Thread Greg KH
On Sat, Jul 12, 2014 at 11:29:37AM -0700, Joe Perches wrote: > On Sat, 2014-07-12 at 10:55 -0700, Greg KH wrote: > > Yes, the warnings are all due to line-length, but Joe, you shouldn't add > > a patch that causes more checkpatch warnings than before :) > > Yeah, that was intentional though. > >

[PATCH] staging: iio: adc: ad7192: fix coding style

2014-07-12 Thread Marcus Folkesson
Line over 80 characters. This is for Eudyptula Challenge. Signed-off-by: Marcus Folkesson --- drivers/staging/iio/adc/ad7192.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/adc/ad7192.c b/drivers/staging/iio/adc/ad7192.c index 83bb44b..c25c9ab 100644 -

Re: Things I wish I'd known about Inotify

2014-07-12 Thread Michael Kerrisk (man-pages)
Late follow up on this thread..., since another question occurred in discussions with Jake. On Fri, Apr 4, 2014 at 2:43 PM, Jan Kara wrote: > On Fri 04-04-14 09:35:50, Michael Kerrisk (man-pages) wrote: >> On 04/03/2014 10:52 PM, Jan Kara wrote: >> > On Thu 03-04-14 08:34:44, Michael Kerrisk (man

Re: [PATCH 2/2] x86_64,signal: Remove 'fs' and 'gs' from sigcontext

2014-07-12 Thread Andi Kleen
On Sat, Jul 12, 2014 at 11:40:03AM -0700, H. Peter Anvin wrote: > Because you are doing something weird (like Pin, for example) and take an > asynchronous fault? But even for pin that would need executing 16 bit code, or really weird 32bit code. AFAIK for 32bit the only good use case was NX emula

Re: [PATCH 2/2] x86_64,signal: Remove 'fs' and 'gs' from sigcontext

2014-07-12 Thread H. Peter Anvin
Because you are doing something weird (like Pin, for example) and take an asynchronous fault? On July 12, 2014 11:37:48 AM PDT, Andi Kleen wrote: >> This seems like it's asking for trouble. I think wxe'd have to >> separately save the selectors and the base registers to avoid >breaking >> somet

Re: [PATCH 2/2] x86_64,signal: Remove 'fs' and 'gs' from sigcontext

2014-07-12 Thread Andi Kleen
> This seems like it's asking for trouble. I think wxe'd have to > separately save the selectors and the base registers to avoid breaking > something, especially once wrgsbase, etc are enabled. I agree, it would likely break existing code. > Linus, for context, the other patch in this series sav

Re: [PATCH] reformat_with_checkpatch: Add automation to checkpatch

2014-07-12 Thread Joe Perches
On Sat, 2014-07-12 at 10:55 -0700, Greg KH wrote: > Yes, the warnings are all due to line-length, but Joe, you shouldn't add > a patch that causes more checkpatch warnings than before :) Yeah, that was intentional though. This script does a series of relatively discrete changes. Lindent would mo

[no subject]

2014-07-12 Thread Gregory K Quintana
unsubscribe linux-kernel -- 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/

hi

2014-07-12 Thread laura wintour
Bitte sei nicht böse über diese Mail, es ist wegen der Dringlichkeit der meine Situation hier. Ich bin Laura Wintour Sirleaf, 23 Jahre alt Nursing Student, Cote Dvoire, aber derzeit Asylsuchende in Lome Republik Togo, als Folge des Krieges in meinem Land, mein Vater eine Selige Erinnerung Ende D

bit fields && data tearing

2014-07-12 Thread Oleg Nesterov
Hello, I am not sure I should ask here, but since Documentation/memory-barriers.txt mentions load/store tearing perhaps my question is not completely off-topic... I am fighting with mysterious RHEL bug, it can be reproduced on ppc and s390 but not on x86. Finally I seem to understand the problem,

[PATCH V2 3/9] i40e: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c b/drivers/net/ethernet/intel/i40e/i40e_lan_hmc.c index 870ab1e..7c6bf89 100644 --- a/drivers/net/ethernet/intel/i

[PATCH V2 5/9] ps3_gelic: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/ethernet/toshiba/ps3_gelic_wireless.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/ethernet/toshiba/ps3_gelic_wireless.c b/drivers/net/ethernet/toshiba/ps3_gelic_wireless.c index d568af1..2553ed5 100644 --- a/drivers/net/ethern

[PATCH V2 8/9] rtlwifi: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/wireless/rtlwifi/rtl8192se/fw.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/fw.c b/drivers/net/wireless/rtlwifi/rtl8192se/fw.c index 380e7d4..73fa4a1 100644 --- a/drivers/net/wireless/rtlwifi/rtl8192se/

[PATCH V2 6/9] orinoco_usb: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/wireless/orinoco/orinoco_usb.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/orinoco/orinoco_usb.c b/drivers/net/wireless/orinoco/orinoco_usb.c index c90939c..d3cf7c3 100644 --- a/drivers/net/wireless/orinoco/orinoco_usb.c

[PATCH V2 7/9] wcn36xx: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/wireless/ath/wcn36xx/main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c index 4ab5370..b71d2b3 100644 --- a/drivers/net/wireless/ath/wcn36xx/main.c +++ b/drive

[PATCH V2 1/9] igb: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/ethernet/intel/igb/e1000_82575.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/intel/igb/e1000_82575.c b/drivers/net/ethernet/intel/igb/e1000_82575.c index a2db388..168a5ee 100644 --- a/drivers/net/ethernet/intel/igb/e1000

[PATCH V2 4/9] i40evf: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/ethernet/intel/i40evf/i40evf_main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/ethernet/intel/i40evf/i40evf_main.c b/drivers/net/ethernet/intel/i40evf/i40evf_main.c index 7fc5f3b..7195566 100644 --- a/drivers/net/ethernet/inte

[PATCH V2 2/9] ixgbe: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 1 - drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c | 4 drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 2 -- 3 files changed, 7 deletions(-) diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c

[PATCH V2 9/9] slip: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- drivers/net/slip/slhc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/slip/slhc.c b/drivers/net/slip/slhc.c index 1252d9c..079f7ad 100644 --- a/drivers/net/slip/slhc.c +++ b/drivers/net/slip/slhc.c @@ -396,7 +396,6 @@ found:

[PATCH V2 0/9] drivers/net: remove unnecessary break after goto

2014-07-12 Thread Fabian Frederick
Small patchset addressing break redundancy on drivers/net branch (suggested by Joe Perches). V2: cc to maintainers of each section. Fabian Frederick (9): igb: remove unnecessary break after goto ixgbe: remove unnecessary break after goto i40e: remove unnecessary break after goto i40evf:

Re: slub/debugobjects: lockup when freeing memory

2014-07-12 Thread Sasha Levin
On 06/20/2014 11:40 AM, Paul E. McKenney wrote: > rcu: Export debug_init_rcu_head() and and debug_init_rcu_head() > > Currently, call_rcu() relies on implicit allocation and initialization > for the debug-objects handling of RCU callbacks. If you hammer the > kernel hard enough with Sasha's modif

Re: [PATCH] reformat_with_checkpatch: Add automation to checkpatch

2014-07-12 Thread Greg KH
On Sat, Jul 12, 2014 at 12:30:43PM +0300, Dan Carpenter wrote: > On Fri, Jul 11, 2014 at 06:40:16PM -0700, Joe Perches wrote: > > On Fri, 2014-07-11 at 18:34 -0700, Greg KH wrote: > > > On Fri, Jul 11, 2014 at 06:21:27PM -0700, Joe Perches wrote: > > > > A simple script to run checkpatch --fix for

Re: [PATCH v3 1/2] iio: adc: add driver for Rockchip saradc

2014-07-12 Thread Jonathan Cameron
On July 12, 2014 4:14:58 PM GMT+01:00, "Heiko Stübner" wrote: >Am Samstag, 12. Juli 2014, 15:35:04 schrieb Jonathan Cameron: >> On 09/07/14 11:22, Heiko Stübner wrote: >> > The ADC is a 3-channel signal-ended 10-bit Successive Approximation >> > Register (SAR) A/D Converter. It uses the supply a

Re: [PATCH] pci: Add include guard to include/linux/pci_ids.h

2014-07-12 Thread Bjorn Helgaas
On Sat, Jul 12, 2014 at 6:09 AM, Rasmus Villemoes wrote: > Bjorn Helgaas writes: > >> On Thu, Jul 10, 2014 at 02:05:11PM +0200, Rasmus Villemoes wrote: >>> Adding an include guard frees the preprocessor from reparsing over >>> 2600 #defines in the cases where pci_ids.h is somehow included more >>

  1   2   3   >