[PATCH -next] drivers: CCI: fix the error handle in cci_pmu_probe()

2013-09-21 Thread Wei Yongjun
From: Wei Yongjun This patch fix the error handle of function cci_pmu_probe(): - using IS_ERR() instead of NULL test for the return value of devm_ioremap_resource() since it nerver return NULL. - remove kfree() for devm_kzalloc allocated memory - remove dev_warn() since devm_ioremap_resource()

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-21 Thread Benjamin Herrenschmidt
On Sun, 2013-09-22 at 14:39 +1000, Benjamin Herrenschmidt wrote: > How do you do your per-cpu on x86 ? On powerpc we struggle because we > try to dedicate a register (r13) to this PACA (the per-cpu offset hangs > off it), but we constantly run into issues where gcc copies r13 to > another register

Re: [PATCH 3/3] iio: ti_am335x_adc: Add continuous sampling support

2013-09-21 Thread Zubair Lutfullah :
On Sat, Sep 21, 2013 at 07:28:32PM +0100, Jonathan Cameron wrote: > On 09/19/13 07:24, Zubair Lutfullah wrote: > > Previously the driver had only one-shot reading functionality. > > This patch adds continuous sampling support to the driver. ... > I've added a SELECT IIO_KFIFO_BUF after the

Re: [PATCH V11 0/3] iio: input: ti_am335x_adc: Add continuous sampling support

2013-09-21 Thread Zubair Lutfullah :
On Sat, Sep 21, 2013 at 11:52:21AM +0100, Jonathan Cameron wrote: > Hi Zubair, > > Thanks for persevering with this patch set. Now this > gets to be the example for dealing with hardware buffer > equipped devices ;) > > Applied to the togreg branch of >

[PATCH] arc: include: asm: add generic cmpxchg64(), cmpxchg64_local(), and cmpxchg_local() in cmpxchg.h

2013-09-21 Thread Chen Gang
Some of sub-systems may need cmpxchg64() or another cmpxchg*local() functions, need implement them, or can not pass compiling. The related error (allmodconfig for arc): CC drivers/block/blockconsole.o drivers/block/blockconsole.c: In function ‘bcon_advance_console_bytes’:

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-21 Thread Benjamin Herrenschmidt
On Sat, 2013-09-21 at 19:01 -0700, H. Peter Anvin wrote: > On 09/21/2013 02:45 PM, Benjamin Herrenschmidt wrote: > > > > I really don't like the "larger" irq stack ... probably because I can't > > make it work easily :-) See my previous comment about how we get to > > thread_info on ppc. > > >

[PATCH] arc: include: asm: uaccess.h: include "asm-generic/uaccess-unaligned.h"

2013-09-21 Thread Chen Gang
Need include the default implementation, just like another platforms (e.g. parisc, tile ...), or can not pass compiling. The related error (with allmodconfig for arc): CC [M] fs/btrfs/ioctl.o fs/btrfs/ioctl.c: In function ‘btrfs_ioctl_file_extent_same’: fs/btrfs/ioctl.c:2802: error:

Re: [PATCH,RFC] random: make fast_mix() honor its name

2013-09-21 Thread Theodore Ts'o
The following fast_mix function, with the loop unrolling, is about 70% slower than your proposed version, but it's still four times faster than the original byte-based fast_mix function. This is what I'm considering using as a compromise. Any comments or objections?

Re: [PATCH v2 3/3] ACPI / video: Do not register backlight if win8 and native interface exists

2013-09-21 Thread Aaron Lu
On 09/20/2013 04:36 PM, Jani Nikula wrote: > On Tue, 17 Sep 2013, Aaron Lu wrote: >> According to Matthew Garrett, "Windows 8 leaves backlight control up >> to individual graphics drivers rather than making ACPI calls itself. >> There's plenty of evidence to suggest that the Intel driver for >>

Re: [PATCH v5] clk: si570: Add a driver for SI570 oscillators

2013-09-21 Thread Guenter Roeck
On 09/21/2013 04:40 PM, Soren Brinkmann wrote: Add a driver for SILabs 570, 571, 598, 599 programmable oscillators. The devices generate low-jitter clock signals and are reprogrammable via an I2C interface. Cc: Guenter Roeck Signed-off-by: Soren Brinkmann Looks good to me. Thanks a lot for

Re: [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes

2013-09-21 Thread Guenter Roeck
On 09/21/2013 05:23 PM, Eduardo Valentin wrote: On 21-09-2013 19:56, Guenter Roeck wrote: On 09/21/2013 04:30 PM, Eduardo Valentin wrote: On 21-09-2013 14:06, Guenter Roeck wrote: On 09/18/2013 09:21 AM, Eduardo Valentin wrote: This patch adds to lm75 temperature sensor the possibility to

[PATCH] ipc/sem.c: fix update sem_otime when calling sem_op in semaphore initialization

2013-09-21 Thread Jia He
In commit 0a2b9d4c,the update of semaphore's sem_otime(last semop time) was removed because he wanted to move setting sem->sem_otime to one place. But after that, the initial semop() will not set the otime because its sem_op value is 0(in semtimedop,will not change otime if alter == 1). the error

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-21 Thread H. Peter Anvin
On 09/21/2013 02:45 PM, Benjamin Herrenschmidt wrote: > > I really don't like the "larger" irq stack ... probably because I can't > make it work easily :-) See my previous comment about how we get to > thread_info on ppc. > For the record, I intend to remove thread_info from the stack on x86

[PATCH 00/21] CPUIdle: Minor cleanups for 3.13

2013-09-21 Thread Viresh Kumar
Hi Rafael/Daniel, This is a small cleanup patchset for CPUIdle which can go in 3.13 if it looks okay to you guys.. Mostly trivial patches but few are doing good/significant changes. Tested on my thinkpad with suspend/resume and didn't found any broken stuff with it. I a not very sure about this

[PATCH 21/21] cpuidle: change governor from within cpuidle_replace_governor()

2013-09-21 Thread Viresh Kumar
When I first read cpuidle_replace_governor()'s name I thought it will replace the governor (as per its name) but then found that it just returns the next best governor. And cpuidle_unregister_governor() actually replaces it. We always replace current governor with the next best and this

[PATCH 19/21] cpuidle: create list of registered drivers

2013-09-21 Thread Viresh Kumar
Currently we have multiple definitions of few routines based on following config option: CONFIG_CPU_IDLE_MULTIPLE_DRIVERS. These are present to save space by not creating per-cpu variable for platforms which need only one cpuidle driver to be registered for all CPUs. But this setup has a

[PATCH 14/21] cpuidle: coupled: don't compare cpu masks unnecessarily

2013-09-21 Thread Viresh Kumar
In cpuidle_coupled_register_device() we do following: if (WARN_ON(!cpumask_equal(>coupled_cpus, >coupled_cpus))) coupled->prevent++; This is only required to be done when we are using 'coupled' from an existing cpuidle_device and not when we have just done this:

[PATCH 15/21] cpuidle: free all state kobjects from cpuidle_free_state_kobj()

2013-09-21 Thread Viresh Kumar
Loop for states is currently present on callers side and so is replicated at several places. It would be better to move that inside cpuidle_free_state_kobj() instead. This patch does it. Signed-off-by: Viresh Kumar --- drivers/cpuidle/sysfs.c | 23 --- 1 file changed, 12

[PATCH 16/21] cpuidle: avoid unnecessary kzalloc/free of struct cpuidle_device_kobj

2013-09-21 Thread Viresh Kumar
We always need to allocate struct cpuidle_device_kobj for all CPUs and so there is no real need to have a pointer to it inside struct cpuidle_device. This patch makes a object instance of struct cpuidle_device_kobj inside struct cpuidle_device instead of a pointer. Signed-off-by: Viresh Kumar

[PATCH 18/21] cpuidle: don't call poll_idle_init() for every cpu

2013-09-21 Thread Viresh Kumar
Signed-off-by: Viresh Kumar --- drivers/cpuidle/cpuidle.c | 41 - drivers/cpuidle/driver.c | 42 ++ 2 files changed, 42 insertions(+), 41 deletions(-) diff --git a/drivers/cpuidle/cpuidle.c

[PATCH 13/21] cpuidle: use drv instead of cpuidle_driver in show_current_driver()

2013-09-21 Thread Viresh Kumar
Instances of "struct cpuidle_driver *" are consistently named as "drv" in cpuidle core. Its broken only at one place: show_current_driver(). Fix it for consistency. Signed-off-by: Viresh Kumar --- drivers/cpuidle/sysfs.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 20/21] cpuidle: don't calculate time-diff if entered_state == 0

2013-09-21 Thread Viresh Kumar
If entered_state == 0, we don't need to set dev->last_residency to 'diff' as we will be setting it to zero without using its new value. And so move calculation of diff also inside the "if" statement. Signed-off-by: Viresh Kumar --- drivers/cpuidle/cpuidle.c | 17 + 1 file

[PATCH 17/21] cpuidle: avoid unnecessary kzalloc/free of struct cpuidle_driver_kobj

2013-09-21 Thread Viresh Kumar
For CONFIG_CPU_IDLE_MULTIPLE_DRIVERS, struct cpuidle_device always needs to allocate struct cpuidle_driver_kobj for all CPUs and so there is no real need to have a pointer to it inside struct cpuidle_device. This patch makes a object instance of struct cpuidle_driver_kobj inside struct

[PATCH 08/21] cpuidle: use cpuidle_disabled() instead of "off"

2013-09-21 Thread Viresh Kumar
We have a routine for getting value of "off", better call that instead of using "off" directly. Signed-off-by: Viresh Kumar --- drivers/cpuidle/cpuidle.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index

[PATCH 04/21] cpuidle: make __cpuidle_device_init() return void

2013-09-21 Thread Viresh Kumar
__cpuidle_device_init() doesn't return anything except zero and so doesn't really need a return type other than void. Signed-off-by: Viresh Kumar --- drivers/cpuidle/cpuidle.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/cpuidle/cpuidle.c

[PATCH 07/21] cpuidle: rearrange __cpuidle_register_device() to keep minimal exit points

2013-09-21 Thread Viresh Kumar
This patch rearranges __cpuidle_register_device() a bit in order to reduce the number of exit points of this function. Signed-off-by: Viresh Kumar --- drivers/cpuidle/cpuidle.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/cpuidle/cpuidle.c

[PATCH 09/21] cpuidle: merge two if() statements for checking error cases

2013-09-21 Thread Viresh Kumar
Both return same error message and so better write them in a single line. Signed-off-by: Viresh Kumar --- drivers/cpuidle/cpuidle.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index aec9029..b8c63cb 100644 ---

[PATCH 03/21] cpuidle: make __cpuidle_get_cpu_driver() inline

2013-09-21 Thread Viresh Kumar
__cpuidle_get_cpu_driver() is a single line function and so deserves to be marked inline. Signed-off-by: Viresh Kumar --- drivers/cpuidle/driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpuidle/driver.c b/drivers/cpuidle/driver.c index ced1df6..25455e8

[PATCH 10/21] cpuidle: reduce code duplication inside cpuidle_idle_call()

2013-09-21 Thread Viresh Kumar
We are doing this twice in cpuidle_idle_call() routine: drv->states[next_state].flags & CPUIDLE_FLAG_TIMER_STOP Would be better if we actually store this in a local variable and use that. That would remove code duplication as well as make this piece of code run fast (in case compiler

[PATCH 06/21] cpuidle: rearrange code in __cpuidle_driver_init()

2013-09-21 Thread Viresh Kumar
This is trivial patch that just reorders few statements in __cpuidle_driver_init() routine, so that we don't need both 'continue' and 'break' in the for loop. Functionally it shouldn't change anything. Signed-off-by: Viresh Kumar --- drivers/cpuidle/driver.c | 10 -- 1 file changed, 4

[PATCH 12/21] cpuidle: call cpuidle_get_driver() from after taking cpuidle_driver_lock

2013-09-21 Thread Viresh Kumar
cpuidle_driver_lock is taken correctly at most of the places but at few places calls to cpuidle_get_driver() are done from outside of this lock. Fix them by calling cpuidle_get_driver() after taking cpuidle_driver_lock. Signed-off-by: Viresh Kumar --- drivers/cpuidle/driver.c | 3 ++-

[PATCH 05/21] cpuidle: make __cpuidle_driver_init() return void

2013-09-21 Thread Viresh Kumar
__cpuidle_driver_init() doesn't return anything except zero and so doesn't really need a return type other than void. Signed-off-by: Viresh Kumar --- drivers/cpuidle/driver.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/cpuidle/driver.c

[PATCH 11/21] cpuidle: replace multiline statements with single line in cpuidle_idle_call()

2013-09-21 Thread Viresh Kumar
Few statements in cpuidle_idle_call() are broken into multiple lines, whereas they can actually come in a single line. Convert those to single line. Signed-off-by: Viresh Kumar --- drivers/cpuidle/cpuidle.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

nice to be your friend

2013-09-21 Thread Jane Ruba
Hello dear, I am Miss Jane Ruba. I am presently single and searching for a trusted and God fearing person. I hope you are fine in perfect condition of health. I saw your email today when I was searching for a good friend and took interest on it, if you don't mind I will like you to contact me

[PATCH 02/21] cpuidle: Fix comments in cpuidle core

2013-09-21 Thread Viresh Kumar
Few comments in cpuidle core files have trivial mistakes. This patch fixes them. Signed-off-by: Viresh Kumar --- drivers/cpuidle/coupled.c | 2 +- drivers/cpuidle/cpuidle.c | 2 +- drivers/cpuidle/driver.c | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git

[PATCH 01/21] cpuidle: fix indentation of cpumask

2013-09-21 Thread Viresh Kumar
cpumask is indented using spaces instead of tabs. Fix it. Signed-off-by: Viresh Kumar --- include/linux/cpuidle.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/cpuidle.h b/include/linux/cpuidle.h index 781addc..c082425 100644 --- a/include/linux/cpuidle.h +++

Re: [PATCH v4 2/3] Cpufreq: s5pv210 cpufreq fixes for CCF

2013-09-21 Thread Viresh Kumar
On 21 September 2013 23:12, Mateusz Krawczuk wrote: > Use common clock framework api to get clock. > > Signed-off-by: Mateusz Krawczuk > Signed-off-by: Kyungmin Park > --- > drivers/cpufreq/s5pv210-cpufreq.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Acked-by: Viresh Kumar

Re: [PATCH] mm/ksm: return NULL when doesn't get mergeable page

2013-09-21 Thread Jianguo Wu
On 2013/9/19 16:33, Petr Holasek wrote: > On Mon, 16 Sep 2013, Jianguo Wu wrote: >> In get_mergeable_page() local variable page is not initialized, >> it may hold a garbage value, when find_mergeable_vma() return NULL, >> get_mergeable_page() may return a garbage value to the caller. >> >> So

Re: [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes

2013-09-21 Thread Eduardo Valentin
On 21-09-2013 19:56, Guenter Roeck wrote: > On 09/21/2013 04:30 PM, Eduardo Valentin wrote: >> On 21-09-2013 14:06, Guenter Roeck wrote: >>> On 09/18/2013 09:21 AM, Eduardo Valentin wrote: This patch adds to lm75 temperature sensor the possibility to expose itself as thermal zone device,

Reworking dm-writeboost [was: Re: staging: Add dm-writeboost]

2013-09-21 Thread Akira Hayakawa
Mike, > We don't need to go through staging. If the dm-writeboost target is > designed well and provides a tangible benefit it doesn't need > wide-spread users as justification for going in. The users will come if > it is implemented well. OK. The benefit of introducing writeboost will be

Re: [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes

2013-09-21 Thread Guenter Roeck
On 09/21/2013 04:30 PM, Eduardo Valentin wrote: On 21-09-2013 14:06, Guenter Roeck wrote: On 09/18/2013 09:21 AM, Eduardo Valentin wrote: This patch adds to lm75 temperature sensor the possibility to expose itself as thermal zone device, registered on the thermal framework. The thermal zone

Re: [PATCH RESEND] printk: report boot console names during cut-over

2013-09-21 Thread Frederic Weisbecker
On Fri, Sep 20, 2013 at 08:34:01AM -0700, Kees Cook wrote: > This reports the names of boot consoles as they're being disabled to > help identify which is which during cut-over. Helps answer the question > "which boot console actually got activated?" once the regular console > is running, mostly

[PATCH v5] clk: si570: Add a driver for SI570 oscillators

2013-09-21 Thread Soren Brinkmann
Add a driver for SILabs 570, 571, 598, 599 programmable oscillators. The devices generate low-jitter clock signals and are reprogrammable via an I2C interface. Cc: Guenter Roeck Signed-off-by: Soren Brinkmann --- v5: - fix typo in bindings documentation - mention the unit for the

Re: [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes

2013-09-21 Thread Eduardo Valentin
On 21-09-2013 14:06, Guenter Roeck wrote: > On 09/18/2013 09:21 AM, Eduardo Valentin wrote: >> This patch adds to lm75 temperature sensor the possibility >> to expose itself as thermal zone device, registered on the >> thermal framework. >> >> The thermal zone is built only if a device tree node

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-21 Thread Frederic Weisbecker
On Sun, Sep 22, 2013 at 07:45:01AM +1000, Benjamin Herrenschmidt wrote: > On Sat, 2013-09-21 at 13:58 -0500, Frederic Weisbecker wrote: > > > Now certainly what needs to be fixed then is archs that don't have > > __ARCH_IRQ_EXIT_IRQS_DISABLED > > or archs that have any other significant

Re: [PATCH][RFC] Fix breakage in ffs_fs_mount()

2013-09-21 Thread Greg KH
On Sat, Sep 21, 2013 at 03:51:26PM +0100, Al Viro wrote: > On Sat, Sep 21, 2013 at 01:10:48AM +0200, Michal Nazarewicz wrote: > > On Fri, Sep 20 2013, Al Viro wrote: > > > There's a bunch of failure exits in ffs_fs_mount() with > > > seriously broken recovery logics. Most of that appears to

Re: [PATCH 15/19] staging: Change variable type to bool

2013-09-21 Thread Greg KH
On Sun, Sep 22, 2013 at 12:27:47AM +0200, Peter Senna Tschudin wrote: > The variable ownbit_flag is only assigned the values true and false. > Change its type to bool. > > The simplified semantic patch that find this problem is as > follows (http://coccinelle.lip6.fr/): > > @exists@ > type T; >

Re: [PATCH] Documentation/kbuild/kconfig.txt: Doesn't make listnewconfig replacing yes "" | make oldconfig ?

2013-09-21 Thread Yann E. MORIN
Regid, All, On 2013-09-12 03:35 +0300, Regid Ichira spake thusly: > Signed-off-by: Regid Ichira > --- > Documentation/kbuild/kconfig.txt | 11 ++- > 1 file changed, 2 insertions(+), 9 deletions(-) There was a trailing whitespace. I fixed it here, and pulled it in my tree for -next.

[PATCH] LSM: MntRestrict blocks mounts on symlink targets

2013-09-21 Thread Kees Cook
On systems where certain filesystem contents cannot be entirely trusted, it is beneficial to block mounts on symlinks. This makes sure that malicious filesystem contents cannot trigger the over-mounting of trusted filesystems. (For example, a bind-mounted subdirectory of /var cannot be redirected

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

2013-09-21 Thread Maxime Ripard
On Sun, Sep 22, 2013 at 12:49:45AM +0800, cinifr wrote: > Thanks. I have modify my patch as you said. > It is in attachments. Maybe I remail the new patch? Yes, remail the whole serie and change the prefix to PATCHv3 (you can use the --subject-prefix option of git format-patch to do so). Maxime

[PATCH 10/19] net: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
There is the rc variable on both myri10ge_ss_lock_napi and myri10ge_ss_lock_poll functions. In both cases rc is only assigned the values true and false. Both functions already return bool. Change rc type to bool. The simplified semantic patch that find this problem is as follows

[PATCH 02/19] hv: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable execute_shutdown is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 09/19] net: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable rc is only assigned the values true and false. The function bnx2x_prev_is_path_marked already returns bool. Change rc type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b =

[PATCH 03/19] infiniband: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variables found and is_uctx_pd are only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 11/19] wireless: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variables cancel_scan_cmd, enable_data, hs_activate and valid are only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ...

[PATCH 04/19] isdn: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable noCh is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:

[PATCH 05/19] media: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable vco_select is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 12/19] wireless: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable continual is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 07/19] mmc: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable is_rpmb is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:

[PATCH 13/19] rtc: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable want_irq is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:

[PATCH 06/19] media: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable frame_ready is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 18/19] nilfs2: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable update_sr is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 17/19] staging: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable Trigger is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:

[PATCH 15/19] staging: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable ownbit_flag is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 08/19] net: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable ret is only assigned the values true and false. The function atl1c_read_eeprom already returns bool. Change ret type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...;

[PATCH 16/19] staging: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variables bMultiBand and bKeyTableFull are only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b =

[PATCH 19/19] net: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable fully_acked is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

[PATCH 14/19] scsi: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable success is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:

[PATCH 01/19] arm: Change variable type to bool

2013-09-21 Thread Peter Senna Tschudin
The variable pll_overclock is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\)

Re: [PATCH V3] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-09-21 Thread Arnd Bergmann
On Saturday 21 September 2013, Kishon Vijay Abraham I wrote: > { > u32 val; > void __iomem *val1; > void __iomem *dbi_base = pp->dbi_base; > > /* Program viewport 0 : INBOUND : MEMORY*/ > val = PCIE_ATU_REGION_INBOUND | (0 & 0xF); >

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-21 Thread Benjamin Herrenschmidt
On Sat, 2013-09-21 at 13:58 -0500, Frederic Weisbecker wrote: > Now certainly what needs to be fixed then is archs that don't have > __ARCH_IRQ_EXIT_IRQS_DISABLED > or archs that have any other significant opportunity to nest interrupt. Interesting. I notice we don't define it on powerpc but we

Re: [PATCH,RFC] random: make fast_mix() honor its name

2013-09-21 Thread Theodore Ts'o
BTW, just to give another example of the difference between the mixing funtions, try compiling the following with and without ORIG_MIX defined... - Ted #include #include #include #include /* #define ORIG_MIX */ #define ADD_ROTATE #define

Re: [PATCH,RFC] random: make fast_mix() honor its name

2013-09-21 Thread Theodore Ts'o
On Mon, Sep 16, 2013 at 11:40:27AM -0400, Jörn Engel wrote: > > Here is a patch to make add_interrupt_randomness() significantly > cheaper without significantly impacting the quality. The second part > is my personal opinion and others might disagree. > > So far this has only seen userspace

Re: [PATCH 09/17] Move unicode to ASCII conversion to shared function.

2013-09-21 Thread Roy Franz
On Fri, Sep 20, 2013 at 8:00 AM, H. Peter Anvin wrote: > On 09/20/2013 04:27 AM, Matt Fleming wrote: >> On Wed, 18 Sep, at 09:48:44PM, Roy Franz wrote: >>> Would it be acceptable to fix the naming/comments, and convert values >>> above 126 to '?' >>> in the current patchset, and address a more

Re: [PATCH 15/26] ARM: kirkwood: remove custom .init_time hook

2013-09-21 Thread Andrew Lunn
On Wed, Sep 18, 2013 at 07:53:48PM +0200, Sebastian Hesselbarth wrote: > With arch/arm calling of_clk_init(NULL) from time_init(), we can now > remove custom .init_time hooks. > > Signed-off-by: Sebastian Hesselbarth > --- > Cc: Olof Johansson > Cc: Arnd Bergmann > Cc: Jason Cooper > Cc:

Re: [PATCH 1/1] tty: disassociate_ctty() sends the extra SIGCONT

2013-09-21 Thread Peter Hurley
On 09/21/2013 02:34 PM, Oleg Nesterov wrote: Peter, sorry for delay, I was sick. On 09/17, Peter Hurley wrote: On 09/15/2013 11:50 AM, Oleg Nesterov wrote: Put the "!on_exit" check back to restore the old behaviour. Cc: sta...@vger.kernel.org # v3.10+ Signed-off-by: Oleg Nesterov

Re: [PATCH 0/3] KVM: Make kvm_lock non-raw

2013-09-21 Thread Michael S. Tsirkin
On Fri, Sep 20, 2013 at 08:04:19PM +0200, Jan Kiszka wrote: > On 2013-09-20 19:51, Paul Gortmaker wrote: > > [Re: [PATCH 0/3] KVM: Make kvm_lock non-raw] On 16/09/2013 (Mon 18:12) Paul > > Gortmaker wrote: > > > >> On 13-09-16 10:06 AM, Paolo Bonzini wrote: > >>> Paul Gortmaker reported a BUG on

Re: [PATCH] block: Device driver for sTec's PCIe Kronos Card.

2013-09-21 Thread Jens Axboe
On Fri, Sep 20 2013, Andrew Morton wrote: > On Tue, 17 Sep 2013 14:20:55 -0600 Jens Axboe wrote: > > > > So, it looks like this driver needs a bunch of work before it's ready > > > to go in. Or, maybe it's better to submit it with a TODO list for the > > > staging tree instead? > > > > Not

Re: [PATCH] block: Add nr_bios to block_rq_remap tracepoint

2013-09-21 Thread Jens Axboe
On Tue, Sep 17 2013, Jun'ichi Nomura wrote: > Adding the number of bios in a remapped request to 'block_rq_remap' > tracepoint. > > Request remapper clones bios in a request to track the completion > status of each bio. So the number of bios can be useful information > for investigation. > >

Re: [PATCH] hotplug: Optimize {get,put}_online_cpus()

2013-09-21 Thread Oleg Nesterov
On 09/21, Oleg Nesterov wrote: > > As for the patch itself, I am not sure. Forgot to mention... and with this patch cpu_hotplug_done() loses the "release" semantics, not sure this is fine. Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

[GIT PULL] Please pull an NFS client bugfix

2013-09-21 Thread Myklebust, Trond
Hi Linus, The following changes since commit 272b98c6455f00884f0350f775c5342358ebb73f: Linux 3.12-rc1 (2013-09-16 16:17:51 -0400) are available in the git repository at: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git tags/nfs-for-3.12-3 for you to fetch changes up to

Re: [RFC GIT PULL] softirq: Consolidation and stack overrun fix

2013-09-21 Thread Frederic Weisbecker
2013/9/20 Linus Torvalds : > On Fri, Sep 20, 2013 at 9:26 AM, Frederic Weisbecker > wrote: >> >> Now just for clarity, what do we then do with inline sofirq executions: on >> local_bh_enable() >> for example, or explicit calls to do_softirq() other than irq exit? > > If we do a softirq because

Re: [PATCH 0/4] ipc: shm and msg fixes

2013-09-21 Thread Linus Torvalds
On Sat, Sep 21, 2013 at 11:30 AM, Davidlohr Bueso wrote: > > IPC uses security_xxx_free() at two levels: for freeing the structure > (ie: shm_destroy()) and cleaning up upon error when creating the > structure (ie: newseg()). For both I believe we can actually use RCU. > What do you think of the

You Have Won

2013-09-21 Thread Andres Jacob
Immediately Email Mr. Adrian and Gillian Bayford for details -- 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

Re: [PATCH 1/1] tty: disassociate_ctty() sends the extra SIGCONT

2013-09-21 Thread Oleg Nesterov
Peter, sorry for delay, I was sick. On 09/17, Peter Hurley wrote: > > On 09/15/2013 11:50 AM, Oleg Nesterov wrote: > >> Put the "!on_exit" check back to restore the old behaviour. >> >> Cc: sta...@vger.kernel.org # v3.10+ >> Signed-off-by: Oleg Nesterov >> Reported-by: Karel Srot > >

[PATCH v6] gpio: pcf857x: Add OF support

2013-09-21 Thread Laurent Pinchart
Add DT bindings for the pcf857x-compatible chips and parse the device tree node in the driver. Signed-off-by: Laurent Pinchart --- .../devicetree/bindings/gpio/gpio-pcf857x.txt | 71 ++ drivers/gpio/gpio-pcf857x.c| 44 +++--- 2 files

Re: [PATCH 0/4] ipc: shm and msg fixes

2013-09-21 Thread Davidlohr Bueso
Hi Eric, On Fri, 2013-09-20 at 14:08 -0400, Eric Paris wrote: > > > Note that Linus suggested a good alternative to patches 1 and 3: use > > > kfree_rcu() and delay the freeing of the security structure. I would > > > much prefer that approach to doing security checks with the lock held, > > >

[PATCH] drivers: bluetooth: btusb: Added support for Belkin F8065bf usb bluetooth device

2013-09-21 Thread Ken O'Brien
Adding generic rule on encountering Belkin bluetooth usb device F8065bf. Relevant section from /sys/kernel/debug/usb/devices: T: Bus=03 Lev=01 Prnt=01 Port=00 Cnt=01 Dev#= 2 Spd=12 MxCh= 0 D: Ver= 2.00 Cls=ff(vend.) Sub=01 Prot=01 MxPS=64 #Cfgs= 1 P: Vendor=050d ProdID=065a Rev= 1.12 S:

Re: [PATCH 06/16] hwmon: tmp102: expose to thermal fw via DT nodes

2013-09-21 Thread Guenter Roeck
On 09/18/2013 09:23 AM, Eduardo Valentin wrote: This patch adds to tmp102 temperature sensor the possibility to expose itself as thermal zone device, registered on the thermal framework. The thermal zone is built only if a device tree node describing a thermal zone for this sensor is present

Re: [PATCHv3 05/16] hwmon: lm75: expose to thermal fw via DT nodes

2013-09-21 Thread Guenter Roeck
On 09/18/2013 09:21 AM, Eduardo Valentin wrote: This patch adds to lm75 temperature sensor the possibility to expose itself as thermal zone device, registered on the thermal framework. The thermal zone is built only if a device tree node describing a thermal zone for this sensor is present

Re: [PATCH 01/12] inet*.h: Remove extern from function prototypes

2013-09-21 Thread David Miller
Series applied, thanks Joe. -- 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/

[PATCH] modpost: Optionally ignore secondary errors seen if a single module build fails

2013-09-21 Thread Guenter Roeck
Commit ea4054a23 (modpost: handle huge numbers of modules) added support for building a large number of modules. Unfortunately, the commit changed the semantics of the makefile: Instead of passing only existing object files to modpost, make now passes all expected object files. If make was

[PATCH v4 1/3] clk: samsung: Add clock driver for s5pc110/s5pv210

2013-09-21 Thread Mateusz Krawczuk
This patch adds new, Common Clock Framework-based clock driver for Samsung S5PV210 SoCs. The driver is just added, without enabling it yet. Signed-off-by: Mateusz Krawczuk Signed-off-by: Kyungmin Park --- .../bindings/clock/samsung,s5pv210-clock.txt | 75 +++

[PATCH v4 3/3] ARM: s5pv210: Migrate clock handling to Common Clock Framework

2013-09-21 Thread Mateusz Krawczuk
This patch migrates the s5pv210 platform to use new clock driver using Common Clock Framework. Signed-off-by: Mateusz Krawczuk Signed-off-by: Kyungmin Park --- arch/arm/mach-s5pv210/Kconfig | 9 + arch/arm/mach-s5pv210/Makefile| 4 ++-- arch/arm/mach-s5pv210/common.c

[PATCH v4 2/3] Cpufreq: s5pv210 cpufreq fixes for CCF

2013-09-21 Thread Mateusz Krawczuk
Use common clock framework api to get clock. Signed-off-by: Mateusz Krawczuk Signed-off-by: Kyungmin Park --- drivers/cpufreq/s5pv210-cpufreq.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/cpufreq/s5pv210-cpufreq.c b/drivers/cpufreq/s5pv210-cpufreq.c index

[PATCH v4 0/3] ARM: S5PV210: move to common clk framework

2013-09-21 Thread Mateusz Krawczuk
This patch series is the new s5pv210 clock implementation (using common clk framework). This implementation is compatible with device tree definition and board files. This patch series is based on linux-next and has been tested on goni and aquila boards using board file. Since v3: Replace

Re: [PATCH 3/3] iio: ti_am335x_adc: Add continuous sampling support

2013-09-21 Thread Jonathan Cameron
On 09/19/13 07:24, Zubair Lutfullah wrote: > Previously the driver had only one-shot reading functionality. > This patch adds continuous sampling support to the driver. > > Continuous sampling starts when buffer is enabled. > HW IRQ wakes worker thread that pushes samples to userspace. > Sampling

[PATCH 11/12] netrom.h: Remove extern from function prototypes

2013-09-21 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare

[PATCH 12/12] p8022.h: Remove extern from function prototypes

2013-09-21 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare

[PATCH 02/12] ip*.h: Remove extern from function prototypes

2013-09-21 Thread Joe Perches
There are a mix of function prototypes with and without extern in the kernel sources. Standardize on not using extern for function prototypes. Function prototypes don't need to be written with extern. extern is assumed by the compiler. Its use is as unnecessary as using auto to declare

  1   2   3   4   >