Re: [PATCH] staging: vt6655: fix sparse warning: argument type

2014-12-21 Thread Jeremiah Mahler
Eddie, all, On Sat, Dec 20, 2014 at 10:27:55PM -0700, Eddie Kovsky wrote: > Fixes following warning generated by sparse: > > drivers/staging/vt6655/baseband.c:2180:45: warning: incorrect type in > argument 1 (different address spaces) > drivers/staging/vt6655/baseband.c:2180:45:expected stru

Re: [PATCH] Staging: comedi: fix spacing issues in ni_stc.h

2014-12-21 Thread Jeremiah Mahler
Sage, On Sun, Dec 21, 2014 at 02:49:29AM -0500, Sage Vaillancourt wrote: > Jeremiah, > > I will get this patching business eventually. When you say "gratuitous > spacing" do you refer to this line? > > > #define G_Gate_Polarity _bit13 > > Because it looks fine in the code, but there's a tab tha

[PATCH] kernel: fixed a space coding style issue

2014-12-21 Thread Gangadhar Vukkesala
fixed a space coding style issue in pid.c which was found when running checkpatch.pl on pid.c Signed-off-by: Gangadhar Vukkesala --- kernel/pid.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/pid.c b/kernel/pid.c index cd36a5e..ad9d780 100644 --- a/kernel/pid.

Re: [PATCH] kernel: fixed a space coding style issue

2014-12-21 Thread Jeremiah Mahler
Gangadhar, On Sun, Dec 21, 2014 at 02:02:02PM +0530, Gangadhar Vukkesala wrote: > fixed a space coding style issue in pid.c which was found when > running checkpatch.pl on pid.c > > Signed-off-by: Gangadhar Vukkesala > --- > kernel/pid.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletio

Re: [PATCH v2 1/2] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
On Saturday 20 December 2014 19:07:49 Guenter Roeck wrote: > On 12/20/2014 09:27 AM, Steven Honeyman wrote: > > On 20 December 2014 at 17:20, Guenter Roeck wrote: > >> I would think that it should be relatively safe > >> to assume that the first fan always exists > > > > That would be a bad idea,

Re: [PATCH v2 1/2] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
On Saturday 20 December 2014 19:38:32 Guenter Roeck wrote: > > + if (fan_mult <= 0) { > > Wonder what to do in the < 0 case. It might be better to make > the variable and the module parameter an unsigned int to > cover that case, since a negative multiplier doesn't really > make sense. Same for

Re: [PATCH v2 1/2] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
On Saturday 20 December 2014 20:02:31 Guenter Roeck wrote: > > + for (fan = 0; fan < ARRAY_SIZE(i8k_fan_mult); ++fan) { > > + i8k_fan_mult[fan] = I8K_FAN_DEFAULT_MULT; > > + if (i8k_get_fan_speed(fan) > I8K_FAN_MAX_RPM) { > > +

[PATCH v3] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
This patch adds new function i8k_get_fan_nominal_speed() for doing SMM call which will return nominal fan RPM for specified fan speed. It returns nominal RPM value at which fan operate when speed (0, 1, 2, 3) is set. It looks like RPM value is not accurate, but still provides very useful informatio

Re: MAINTAINERS: clk framework git tree moved to kernel.org

2014-12-21 Thread Geert Uytterhoeven
Hi Mike, Stephen, On Sun, Dec 21, 2014 at 2:14 AM, Linux Kernel Mailing List wrote: > Gitweb: > http://git.kernel.org/linus/;a=commit;h=baeb0d9b98c3450e22f8f8e4a6619b8b1d5106ff > Commit: baeb0d9b98c3450e22f8f8e4a6619b8b1d5106ff > Parent: f956165f07839a0643e4e01081787190692a7c29 > Ref

Re: [PATCH] powerpc: Wire up sys_execveat() syscall

2014-12-21 Thread Stephen Rothwell
Hi Pranith, On Sat, 20 Dec 2014 11:47:18 -0500 Pranith Kumar wrote: > > Wire up sys_execveat(). This passes the selftests for the system call. Thanks for this, but ... > diff --git a/arch/powerpc/include/asm/systbl.h > b/arch/powerpc/include/asm/systbl.h > index ce9577d..778844a 100644 > --- a

Re: [PATCH] staging: vt6655: fix sparse warning: argument type

2014-12-21 Thread Malcolm Priestley
On 21/12/14 08:17, Jeremiah Mahler wrote: Eddie, all, On Sat, Dec 20, 2014 at 10:27:55PM -0700, Eddie Kovsky wrote: Fixes following warning generated by sparse: drivers/staging/vt6655/baseband.c:2180:45: warning: incorrect type in argument 1 (different address spaces) drivers/staging/vt6655/b

Re: [GIT PULL] one vdso fix for x86/urgent

2014-12-21 Thread Ingo Molnar
* Andy Lutomirski wrote: > Hi Ingo, etc, > > Please consider pulling for x86/urgent. This fixes a longstanding, > albeit relatively minor, issue in the x86 vdso randomization > algorithm. Note that this isn't super-urgent, as this bug isn't > directly exploitable, and it's as old as the vdso

Re: [PATCH 1/2] TTY: add support for "tty slave" devices.

2014-12-21 Thread Sebastian Reichel
Hi Neil, On Sat, Dec 20, 2014 at 11:09:20AM +1100, NeilBrown wrote: > A "tty slave" is a device connected via UART. > It may need a driver to, for example, power the device on > when the tty is opened, and power it off when the tty > is released. How about (reads a bit easier to me, but I'm not a

Re: [RFC PATCH] CodingStyle: remove what nowadays might be considered polemic

2014-12-21 Thread Alexander Holler
Am 19.12.2014 um 14:36 schrieb Alexander Holler: In times where things like checkpatch do exist and are mandated to be used, it would be easy to warn if too many levels of indentation are used (e.g. by counting leading tabs). The paragraph before already says that more than 3 levels of indentati

linux-next: Tree for Dec 21

2014-12-21 Thread Stephen Rothwell
Hi all, There will only be intermittent releases of linux-next between now and Jan 5. Changes since 20141219: The idle tree gained a conflict against Linus' tree. The akpm tree lost a patch that turned up elsewhere. Non-merge commits (relative to Linus' tree): 375 445 files changed, 10391 ins

[RFC PATCH] drivers/intel_powerclamp : Redesign implementation of idle injection

2014-12-21 Thread Preeti U Murthy
The powerclamp driver injects idle periods to stay within the thermal constraints. The driver does a fake idle by spawning per-cpu threads that call the mwait instruction. This behavior of fake idle can confuse the other kernel subsystems. For instance it calls into the nohz tick handlers, which a

Re: [RFC PATCH 1/2] common: dma-mapping: introduce dma_get_parent_cfg() helper

2014-12-21 Thread Will Deacon
On Thu, Dec 18, 2014 at 05:20:12PM +, Catalin Marinas wrote: > On Thu, Dec 18, 2014 at 12:09:17AM +, Arnd Bergmann wrote: > > Will, I think we may have a problem on ARM64 now, since we only replaced > > set_arch_dma_coherent_ops on ARM32 but not ARM64. Can you send a fix for > > this? Witho

[PATCH] Staging: gdm724x: fix space before comma coding style issue in gdm_mux.c

2014-12-21 Thread Robert Nachlinger
This is a patch to the gdm_mux.c file that fixes a space before comma coding style issue found by the checkpatch.pl tool. Signed-off-by: Robert Nachlinger --- drivers/staging/gdm724x/gdm_mux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gdm724x/gdm_mux.c

Re: [RFC PATCH] CodingStyle: remove what nowadays might be considered polemic

2014-12-21 Thread Alexander Holler
Am 21.12.2014 um 11:23 schrieb Alexander Holler: Uups, there is maximum of 8 levels of intendation (not 9, the first tab And, please, no words about my broken english. Turn on your human fuzzy and do s/intendation/indentation/ ;) Regards, Alexander Holler -- To unsubscribe from this list: s

[PATCH] ceph: Use %zu to format size_t in ceph_fill_inline_data()

2014-12-21 Thread Geert Uytterhoeven
On 32-bit, where size_t is unsigned int: fs/ceph/addr.c: In function ‘ceph_fill_inline_data’: fs/ceph/addr.c:1419: warning: format ‘%lu’ expects type ‘long unsigned int’, but argument 10 has type ‘size_t’ Signed-off-by: Geert Uytterhoeven --- fs/ceph/addr.c | 2 +- 1 file changed, 1 insertion(

Re: [PATCH V5 13/23] MIPS: BMIPS: Flush the readahead cache after DMA

2014-12-21 Thread Florian Fainelli
2014-12-20 4:44 GMT-08:00 Jonas Gorski : > On Sat, Dec 20, 2014 at 2:39 AM, Kevin Cernekee wrote: >> On Mon, Dec 15, 2014 at 1:43 AM, Jonas Gorski wrote: >>> On Fri, Dec 12, 2014 at 11:07 PM, Kevin Cernekee wrote: BMIPS 3300/435x/438x CPUs have a readahead cache that is separate from t

[PATCH] selftests/exec: Use %zu to format size_t

2014-12-21 Thread Geert Uytterhoeven
On 32-bit: execveat.c: In function 'check_execveat_pathmax': execveat.c:183: warning: format '%lu' expects type 'long unsigned int', but argument 3 has type 'size_t' execveat.c:187: warning: format '%lu' expects type 'long unsigned int', but argument 2 has type 'size_t' Signed-off-by: Geert Uyt

[PATCH selftest fails!] m68k: Wire up execveat

2014-12-21 Thread Geert Uytterhoeven
Check success of execveat(3, '../execveat', 0)... [OK] Check success of execveat(5, 'execveat', 0)... [OK] Check success of execveat(6, 'execveat', 0)... [OK] Check success of execveat(-100, '/root/selftest-exec/exec/execveat', 0)... [OK] Check success of execveat(99, '/root/selftest-exec/exec/exec

[PATCH] fs/affs: Fix casting in printed messages

2014-12-21 Thread Geert Uytterhoeven
- "inode.i_ino" is "unsigned long", - "loff_t" is always "unsigned long long", - "sector_t" should be cast to "unsigned long long" for printing, - "u32" should not be cast to "unsigned int" for printing. Signed-off-by: Geert Uytterhoeven --- fs/affs/amigaffs.c | 6 +++--- fs/affs/dir.c

Re: [PATCH] ceph: Use %zu to format size_t in ceph_fill_inline_data()

2014-12-21 Thread Ilya Dryomov
On Sun, Dec 21, 2014 at 1:56 PM, Geert Uytterhoeven wrote: > On 32-bit, where size_t is unsigned int: > > fs/ceph/addr.c: In function ‘ceph_fill_inline_data’: > fs/ceph/addr.c:1419: warning: format ‘%lu’ expects type ‘long unsigned int’, > but argument 10 has type ‘size_t’ > > Signed-off-by: Geer

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Oded Gabbay
On 12/20/2014 11:25 PM, Greg KH wrote: On Sat, Dec 20, 2014 at 10:46:12PM +0200, Oded Gabbay wrote: When amdkfd and radeon are compiled inside the kernel image (not as modules), radeon will load before amdkfd and will set *kfd2kgd to its interface structure. Therefore, we must not set *kfd2kgd

Re: [PATCH] net: ceph: crypto.c: Remove some unused functions

2014-12-21 Thread Ilya Dryomov
On Sat, Dec 20, 2014 at 7:52 PM, Rickard Strandqvist wrote: > Removes some functions that are not used anywhere: > ceph_encrypt() ceph_decrypt() ceph_crypto_key_encode() > > This was partially found by using a static code analysis program called > cppcheck. I'd like to keep this stuff even if it

[PATCH v2] ALSA: atmel: fix building the ac97 driver for at91-multiplatform

2014-12-21 Thread Alexandre Belloni
From: Arnd Bergmann at91 will no longer export the mach/cpu.h and mach/hardware.h header files in the future, which would break building the atmel ac97c driver. Since the cpu_is_* check is only used to find out whether we are running on avr32 or arm/at91, we can hardcode that check in the ARM ca

Re: [PATCH] net: ceph: armor.c: Remove unused function

2014-12-21 Thread Ilya Dryomov
On Sun, Dec 21, 2014 at 1:36 AM, Rickard Strandqvist wrote: > Remove the function ceph_armor() that is not used anywhere. > > This was partially found by using a static code analysis program called > cppcheck. It's ceph_unarmor()'s counterpart - so same as for "[PATCH] net: ceph: crypto.c: Remov

Re: [PATCH] gpu: drm: nouveau: core: subdev: clock: base.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
2014-12-21 3:46 GMT+01:00 Ben Skeggs : > - Original Message - >> From: "Rickard Strandqvist" >> To: "David Airlie" , "Ben Skeggs" >> Cc: "Rickard Strandqvist" , >> "Alexandre Courbot" , "Ilia >> Mirkin" , dri-de...@lists.freedesktop.org, >> linux-kernel@vger.kernel.org >> Sent: Sunday,

Re: [PATCH] net: ceph: ceph_strings.c: Remove unused function

2014-12-21 Thread Ilya Dryomov
On Sat, Dec 20, 2014 at 3:34 PM, Rickard Strandqvist wrote: > Remove the function ceph_pool_op_name() that is not used anywhere. > > This was partially found by using a static code analysis program called > cppcheck. It's part of pool op infrastructure and I'd rather we use it in monc_show() tha

Re: [PATCH hisi] Input: i2c_atmel_read() can be static

2014-12-21 Thread zhangfei
On 12/19/2014 09:56 AM, kbuild test robot wrote: drivers/input/touchscreen/atmel_mXT224E.c:460:5: sparse: symbol 'i2c_atmel_read' was not declared. Should it be static? drivers/input/touchscreen/atmel_mXT224E.c:496:5: sparse: symbol 'i2c_atmel_write' was not declared. Should it be static? dri

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Oded Gabbay
On 12/21/2014 01:27 PM, Christian König wrote: Am 20.12.2014 um 21:46 schrieb Oded Gabbay: When amdkfd and radeon are compiled inside the kernel image (not as modules), radeon will load before amdkfd and will set *kfd2kgd to its interface structure. Therefore, we must not set *kfd2kgd to NULL

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Christian König
Am 20.12.2014 um 21:46 schrieb Oded Gabbay: When amdkfd and radeon are compiled inside the kernel image (not as modules), radeon will load before amdkfd and will set *kfd2kgd to its interface structure. Therefore, we must not set *kfd2kgd to NULL when amdkfd is loaded because it will override rad

Re: [alsa-devel] [PATCH] sound: pci: hda: patch_analog.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
2014-12-21 9:00 GMT+01:00 Raymond Yau : >> >> Remove the function ad198x_ch_mode_get() that is not used anywhere. >> >> This was partially found by using a static code analysis program called >> cppcheck. >> >> Signed-off-by: Rickard Strandqvist >> >> --- >> sound/pci/hda/patch_analog.c |9 --

[3.18 regression]: machine hangs on s2ram

2014-12-21 Thread Rolf Eike Beer
Since I upgraded to 3.18 my machine (W740SU based laptop) hangs when I do s2ram. For me it looks like the preparations work, but the machine is then not switched off. The only way to recover is a hard powercycle. Since the machine has no serial port I can't look what's happening then, but the lo

Re: [PATCH v2 1/2] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Guenter Roeck
On 12/21/2014 01:13 AM, Pali Rohár wrote: On Saturday 20 December 2014 19:38:32 Guenter Roeck wrote: + if (fan_mult <= 0) { Wonder what to do in the < 0 case. It might be better to make the variable and the module parameter an unsigned int to cover that case, since a negative multiplier

Re: [PATCH v3] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Guenter Roeck
On 12/21/2014 01:20 AM, Pali Rohár wrote: This patch adds new function i8k_get_fan_nominal_speed() for doing SMM call which will return nominal fan RPM for specified fan speed. It returns nominal RPM value at which fan operate when speed (0, 1, 2, 3) is set. It looks like RPM value is not accurat

Re: [PATCH v3] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
On Sunday 21 December 2014 12:57:08 Guenter Roeck wrote: > > -#define I8K_FAN_MULT 30 > > +#define I8K_FAN_MAX_RPM3 > > > > #define I8K_MAX_TEMP 127 > > > > #define I8K_FN_NONE 0x00 > > > > @@ -64,7 +66,7 @@ static DEFINE_MUTEX(i8k_m

Re: Linux 3.19-rc1 - merge window closed

2014-12-21 Thread Guenter Roeck
On Sat, Dec 20, 2014 at 11:11:39PM -0800, Nicholas A. Bellinger wrote: > Hi Linus, > > On Sat, 2014-12-20 at 17:43 -0800, Linus Torvalds wrote: > > So it's been a day less than two weeks, and the merge window is closed. > > ... > > Just hit the following build failure with CONFIG_AUDIT=y: > >

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Christian König
Am 21.12.2014 um 12:34 schrieb Oded Gabbay: On 12/21/2014 01:27 PM, Christian König wrote: Am 20.12.2014 um 21:46 schrieb Oded Gabbay: When amdkfd and radeon are compiled inside the kernel image (not as modules), radeon will load before amdkfd and will set *kfd2kgd to its interface structure

Re: [PATCH v3] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Guenter Roeck
On 12/21/2014 04:09 AM, Pali Rohár wrote: On Sunday 21 December 2014 12:57:08 Guenter Roeck wrote: -#define I8K_FAN_MULT 30 +#define I8K_FAN_MAX_RPM3 #define I8K_MAX_TEMP 127 #define I8K_FN_NONE 0x00 @@ -64,7 +66,7 @@ static DEFINE_MUTEX(i8

[PATCH] sound: pci: hda: patch_analog.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: ad198x_ch_mode_get() ad198x_ch_mode_info() ad198x_ch_mode_info() This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- sound/pci/hda/patch_analog.c | 33 -

[PATCH] staging: iio: adc: ad7192: fix space before , coding style issue

2014-12-21 Thread Mohammad Jamal
This patch solves the space before , error of the checkpatch.pl Signed-off-by: Mohammad Jamal --- drivers/staging/iio/adc/ad7192.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/adc/ad7192.c b/drivers/staging/iio/adc/ad7192.c index f6526aa..6f8ce6c 100

[PATCH] staging : iio: meter: ade7759: fix space before , coding style issue

2014-12-21 Thread Mohammad Jamal
This patch solves the space before , coding style issue found by checkpatch in ade7759.c Signed-off-by: Mohammad Jamal --- drivers/staging/iio/meter/ade7759.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/meter/ade7759.c b/drivers/staging/iio/meter/a

Re: [PATCH v2 1/1] ARM: dts: kirkwood: enable phy driver for SATA controller on 88f6192

2014-12-21 Thread Andrew Lunn
On Tue, Dec 16, 2014 at 07:22:18PM +0100, Evgeni Dobrev wrote: > This patch enables the phy drivers for the SATA controller on Marvell's > 88f6192. > Without them it is not possible to use SATA drives attached to this processor. > > Signed-off-by: Evgeni Dobrev Hi Evgeni Added to mvebu/dt

Re: [PATCH] misc: cxl: sysfs.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
2014-12-21 5:05 GMT+01:00 Michael Neuling : >> Remove the function mmio_size_show() that is not used anywhere. > > Did you compile check this patch? > > drivers/misc/cxl/sysfs.c:291:74: error: ‘mmio_size_show’ undeclared here > (not in a function) > > It's used here: > static struct devi

regression bisected; KVM: entry failed, hardware error 0x80000021

2014-12-21 Thread Jamie Heilman
With v3.19-rc1 when I run qemu-system-x86_64 -machine pc,accel=kvm I get: KVM: entry failed, hardware error 0x8021 If you're running a guest on an Intel machine without unrestricted mode support, the failure can be most likely due to the guest entering an invalid state for Intel VT. For examp

[PATCH] media: radio: wl128x: fmdrv_rx.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
Remove the function fm_rx_get_rds_system() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/media/radio/wl128x/fmdrv_rx.c | 16 drivers/media/radio/wl128x/fmdrv_rx.h |

[PATCH] arch: x86: kernel: apb_timer.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
Remove the function is_apbt_capable() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- arch/x86/kernel/apb_timer.c |8 1 file changed, 8 deletions(-) diff --git a/arch/x86/kernel/ap

[PATCH] staging: rtl8723au: hal: rtl8723au_led.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: rtl8723au_DeInitSwLeds() rtl8723au_InitSwLeds() This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/staging/rtl8723au/hal/rtl8723au_led.c| 23 --

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Oded Gabbay
On 12/21/2014 02:19 PM, Christian König wrote: Am 21.12.2014 um 12:34 schrieb Oded Gabbay: On 12/21/2014 01:27 PM, Christian König wrote: Am 20.12.2014 um 21:46 schrieb Oded Gabbay: When amdkfd and radeon are compiled inside the kernel image (not as modules), radeon will load before amdkfd

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Oded Gabbay
On 12/21/2014 03:06 PM, Oded Gabbay wrote: On 12/21/2014 02:19 PM, Christian König wrote: Am 21.12.2014 um 12:34 schrieb Oded Gabbay: On 12/21/2014 01:27 PM, Christian König wrote: Am 20.12.2014 um 21:46 schrieb Oded Gabbay: When amdkfd and radeon are compiled inside the kernel image (n

[PATCH 1/2] doc: usbmon: fix wording "be reading until"

2014-12-21 Thread Jeremiah Mahler
The usbmon documentation uses "be reading until" which is an unusual wording. Change it to "read until it" which is more clear. Signed-off-by: Jeremiah Mahler --- Documentation/usb/usbmon.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/usb/usbmon.txt b/Docu

[PATCH 2/2] doc: usbmon: fix spelling s/unpriviledged/unprivileged/

2014-12-21 Thread Jeremiah Mahler
Signed-off-by: Jeremiah Mahler --- Documentation/usb/usbmon.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/usb/usbmon.txt b/Documentation/usb/usbmon.txt index be2ccb9..28425f7 100644 --- a/Documentation/usb/usbmon.txt +++ b/Documentation/usb/usbmon.txt @@ -2

[PATCH] clk: mxs: Fix invalid 32-bit access to frac registers

2014-12-21 Thread Stefan Wahren
According to i.MX23 and i.MX28 reference manual the fractional clock control registers must be addressed by byte instructions. This patch fixes the erroneous 32-bit access to these registers and extends the comment in the init functions. Btw the imx23 init now uses a R-M-W sequence just like imx2

[PATCH] staging: vt6655: fix sparse warnings: incorrect argument type

2014-12-21 Thread Mike Krinkin
this patch fixes following sparse warnings: drivers/staging/vt6655/device_main.c:1503:25: warning: incorrect type in argument 1 (different address spaces) drivers/staging/vt6655/device_main.c:1503:25:expected void [noderef] * drivers/staging/vt6655/device_main.c:1503:25:got struct vnt_pr

Re: [PATCH] powerpc: Wire up sys_execveat() syscall

2014-12-21 Thread Pranith Kumar
On Sun, Dec 21, 2014 at 4:56 AM, Stephen Rothwell wrote: > Hi Pranith, > > On Sat, 20 Dec 2014 11:47:18 -0500 Pranith Kumar > wrote: >> >> Wire up sys_execveat(). This passes the selftests for the system call. > > Thanks for this, but ... > >> diff --git a/arch/powerpc/include/asm/systbl.h >> b

[PATCH 1/1] powerpc: Wire up sys_execveat() syscall

2014-12-21 Thread Pranith Kumar
Wire up sys_execveat(). This passes the selftests for the system call. Check success of execveat(3, '../execveat', 0)... [OK] Check success of execveat(5, 'execveat', 0)... [OK] Check success of execveat(6, 'execveat', 0)... [OK] Check success of execveat(-100, '/home/pranith/linux/...ftests/exec/

[PATCH] x86, fpu: Use a symbolic name for asm operand

2014-12-21 Thread Borislav Petkov
From: Borislav Petkov Fix up the else-case in fpu_fxsave() which seems like it has been overlooked. Correct comment style in restore_fpu_checking() while at it. Signed-off-by: Borislav Petkov --- arch/x86/include/asm/fpu-internal.h | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(

Re: [PATCH] ARM: dts: Bump SD card pin drive strength up on rk3288-evb

2014-12-21 Thread Heiko Stübner
Am Mittwoch, 10. Dezember 2014, 12:55:29 schrieb Doug Anderson: > It seems that ever since (536f6b9 mmc: dw_mmc: Reset DMA before > enabling IDMAC) landed upstream that SD cards have been very unhappy > on rk3288-evb. They were a little unhappy before that change, but > after that change they're R

Re: [PATCH v2] clk: rockchip: add CLK_IGNORE_UNUSED flag to fix USB Host

2014-12-21 Thread Heiko Stübner
Am Freitag, 12. Dezember 2014, 22:05:52 schrieb Julien CHAUVEAU: > This patch adds CLK_IGNORE_UNUSED flag to hclk_usb_peri, hclk_usbotg0 > and hclk_usbotg1 because these clocks must remain enabled to use the > USB controllers in host mode. > > This fixes a regression introduced by commit 78eaf6095

Re: [PATCH 1/2] CHROMIUM: clk: rockchip: add clock IDs for the PVTM clocks

2014-12-21 Thread Heiko Stübner
Hi Dmitry, This looks ok and also like it's meant to go upstream, because it still has the CHROMIUM-prefix? No need to resend, I remove them myself, just want to make sure here :-) Heiko Am Donnerstag, 18. Dezember 2014, 16:13:45 schrieb Dmitry Torokhov: > From: Huang Lin > > Process-Voltag

[PATCH] Thermal: fix platform_no_drv_owner.cocci warnings

2014-12-21 Thread kbuild test robot
drivers/thermal/int340x_thermal/int3402_thermal.c:234:6-11: 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: Aaron Lu Signed-off-by: Fengguang Wu --- int340

Re: [PATCH] clk: rockchip: Fix clock gate for rk3188 hclk_emem_peri

2014-12-21 Thread Heiko Stübner
Am Freitag, 12. Dezember 2014, 17:50:39 schrieb Romain Perier: > Do not disable clock gate "hclk_emem_peri", otherwise EMAC clocks no longer > work and it breaks ethernet on RK3066 and RK3188. It fixes a regression > introduced by commit 78eaf6095cc763c1a228ebac5682852f04e85205 ("clk: > rockchip: d

[PATCH] zram: clean up zram_meta_alloc()

2014-12-21 Thread Sergey Senozhatsky
a trivial clean up of zram_meta_alloc() error handling. Signed-off-by: Sergey Senozhatsky --- drivers/block/zram/zram_drv.c | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c index bd8bda3..369fe56 1006

Re: [PATCH selftest fails!] m68k: Wire up execveat

2014-12-21 Thread Andreas Schwab
Geert Uytterhoeven writes: > Check success of execveat(5, '...', > 0)... [FAIL] (child 792 exited with 126 not 127) POSIX says (http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_08_02): If a command is not found, the ex

[PATCH] ASoC: Intel: fix platform_no_drv_owner.cocci warnings

2014-12-21 Thread kbuild test robot
sound/soc/intel/cht_bsw_rt5672.c:273: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: Mengdong Lin Signed-off-by: Fengguang Wu --- cht_bsw_rt5672.c |

[GIT PULL] thermal-soc management updates for v3.19-rc2

2014-12-21 Thread Eduardo Valentin
Hello Rui, Please pull from git://git.kernel.org/pub/scm/linux/kernel/git/evalenti/linux-soc-thermal.git fixes to receive Thermal-SoC Management fixes for v3.19-rc2 with top-most 503ccc3fec4a56cdcfedc507cd1ea0d85e1fbfa2: thermal: cpu_cooling: return ERR_PTR() for !CPU_THERMAL or !THERMAL_

Re: [PATCH] staging: vt6655: fix sparse warnings: incorrect argument type

2014-12-21 Thread Malcolm Priestley
On 21/12/14 13:56, Mike Krinkin wrote: this patch fixes following sparse warnings: drivers/staging/vt6655/device_main.c:1503:25: warning: incorrect type in argument 1 (different address spaces) drivers/staging/vt6655/device_main.c:1503:25:expected void [noderef] * drivers/staging/vt6655/

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Christian König
There should be, but when the modules are compiled in, they are loaded based on link order only, if they are in the same group, and the groups are loaded by a pre-defined order. Is that really still up to date? I've seen effort to change that something like 10+ years ago when Rusty reworked the

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Oded Gabbay
On 12/21/2014 05:57 PM, Christian König wrote: There should be, but when the modules are compiled in, they are loaded based on link order only, if they are in the same group, and the groups are loaded by a pre-defined order. Is that really still up to date? I've seen effort to change that some

Re: [PATCH 1/3] amdkfd: Don't clear *kfd2kgd on kfd_module_init

2014-12-21 Thread Christian König
Am 21.12.2014 um 17:03 schrieb Oded Gabbay: On 12/21/2014 05:57 PM, Christian König wrote: There should be, but when the modules are compiled in, they are loaded based on link order only, if they are in the same group, and the groups are loaded by a pre-defined order. Is that really still up

[PATCH 6/6] f2fs: avoid double lock for cp_rwsem

2014-12-21 Thread Jaegeuk Kim
The __f2fs_add_link is covered by cp_rwsem all the time. This calls init_inode_metadata, which conducts some acl operations including memory allocation with GFP_KERNEL previously. But, under memory pressure, f2fs_write_data_page can be called, which also grabs cp_mutex too. Basically, it's safe sin

[PATCH 3/6] f2fs: add key functions for f2fs_io_tracer

2014-12-21 Thread Jaegeuk Kim
This patch adds two key functions to trace process ids and IOs. The basic idea is to 1. remain process ids, pids, in page->private. 2. show pids in IO traces. So, later we can retrieve process information according to IO traces. Signed-off-by: Jaegeuk Kim --- fs/f2fs/trace.c | 86 ++

[PATCH 5/6] f2fs: activate f2fs_trace_ios

2014-12-21 Thread Jaegeuk Kim
This patch activates f2fs_trace_ios. Signed-off-by: Jaegeuk Kim --- fs/f2fs/data.c | 3 +++ fs/f2fs/file.c | 2 ++ fs/f2fs/super.c | 2 ++ 3 files changed, 7 insertions(+) diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index d86f8b1..20aa3c3 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -

[PATCH 2/6] f2fs: add f2fs_io_tracer support

2014-12-21 Thread Jaegeuk Kim
This patch adds: o initial trace.c and trace.h with skeleton functions o Kconfig and Makefile to activate this feature Signed-off-by: Jaegeuk Kim --- fs/f2fs/Kconfig | 10 ++ fs/f2fs/Makefile | 1 + fs/f2fs/trace.c | 24 fs/f2fs/trace.h | 24 +++

[PATCH 1/6] f2fs: use f2fs_io_info to clean up messy parameters during IO path

2014-12-21 Thread Jaegeuk Kim
This patch cleans up parameters on IO paths. The key idea is to use f2fs_io_info adding a parameter, block address, and then use this structure as parameters. Signed-off-by: Jaegeuk Kim --- fs/f2fs/checkpoint.c | 24 ++- fs/f2fs/data.c | 66 +++--

[PATCH 4/6] f2fs: activate f2fs_trace_pid

2014-12-21 Thread Jaegeuk Kim
This patch activates f2fs_trace_pid. Signed-off-by: Jaegeuk Kim --- fs/f2fs/checkpoint.c | 3 +++ fs/f2fs/node.c | 2 ++ fs/f2fs/segment.c| 2 ++ 3 files changed, 7 insertions(+) diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c index 0ac7c39..333 100644 --- a/fs/f2fs/check

Re: [v7, 5/5] hwmon: ina2xx: allow to change the averaging rate at run-time

2014-12-21 Thread Guenter Roeck
On Sat, Dec 13, 2014 at 05:27:21PM +0100, Bartosz Golaszewski wrote: > The averaging rate of ina226 is hardcoded to 16 in the driver. Make it > modifiable at run-time via a new sysfs attribute. > > While we're at it - add an additional variable to ina2xx_data, which holds > the current configurati

Re: [PATCH v3] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
On Sunday 21 December 2014 13:23:32 Guenter Roeck wrote: > On 12/21/2014 04:09 AM, Pali Rohár wrote: > > On Sunday 21 December 2014 12:57:08 Guenter Roeck wrote: > >>> -#define I8K_FAN_MULT 30 > >>> +#define I8K_FAN_MAX_RPM 3 > >>> > >>>#define I8K_MAX_TEMP 1

[PATCH] sound: pci: asihpi: hpifunc.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: hpi_stream_response_to_legacy() hpi_outstream_host_buffer_free() hpi_outstream_host_buffer_get_info() hpi_outstream_host_buffer_allocate() hpi_outstream_set_time_scale() hpi_outstream_ancillary_read() hpi_outstream_ancillary_get_info() hpi_outstrea

Re: [PATCH v3] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Steven Honeyman
On 21 December 2014 at 16:37, Pali Rohár wrote: > On Sunday 21 December 2014 13:23:32 Guenter Roeck wrote: >> On 12/21/2014 04:09 AM, Pali Rohár wrote: >> > On Sunday 21 December 2014 12:57:08 Guenter Roeck wrote: >> >>> -#define I8K_FAN_MULT 30 >> >>> +#define I8K_FAN_MAX_RPM

[PATCH] video: fbdev: omap2: dss: rfbi.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: rfbi_init_display() rfbi_display_disable() rfbi_display_enable() rfbi_set_interface_timings() rfbi_set_data_lines() rfbi_set_pixel_size() rfbi_set_size() rfbi_update() rfbi_configure() rfbi_enable_te() rfbi_setup_te() rfbi_write_pixels() rfbi_write

[PATCH v3 0/2] lprocfs_write_frac_u64_helper cleanup

2014-12-21 Thread Chris Rorvick
Added Andreas as reviewer (thanks!) v2: Added a second patch to address Dan Carpenter's concern with the complexity of passing the sign through `mult'. Compile tested only. Chris Rorvick (2): drivers: staging: lustre: Use mult if units not specified drivers: staging: lustre: Track sign sepa

[PATCH v3 2/2] drivers: staging: lustre: Track sign separately

2014-12-21 Thread Chris Rorvick
The `mult' parameter is negated if the user data begins with a '-' so that the final value has the appropriate sign. But `mult' is only used if the user data does not include a "units" suffix. In this case, `mult' is overridden with the numeric scale conveyed by the units suffix, but retains the

[PATCH v3 1/2] drivers: staging: lustre: Use mult if units not specified

2014-12-21 Thread Chris Rorvick
Units can be passed to lprocfs_write_frac_u64_helper() via a suffix (e.g., "...K", "...M", etc.) tacked onto the value. A comment states that "specified units override the multiplier," though the multiplier is overridden regardless. Update the conditional logic so that it only applies when units

Re: [RFC PATCH] CodingStyle: remove what nowadays might be considered polemic

2014-12-21 Thread Joe Perches
On Sun, 2014-12-21 at 11:23 +0100, Alexander Holler wrote: > Am 19.12.2014 um 14:36 schrieb Alexander Holler: > > In times where things like checkpatch do exist and are mandated to be used, > > it would be easy to warn if too many levels of indentation are used (e.g. > > by counting leading tabs).

[PATCH] arch: mips: fw: arc: file.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: ArcSetFileInformation() ArcGetFileInformation() ArcSeek() ArcGetReadStatus() ArcClose() ArcOpen() ArcGetDirectoryEntry() This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- a

[PATCH] dma: imx-dma.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
Remove the function is_imx21_dma() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/dma/imx-dma.c |5 - 1 file changed, 5 deletions(-) diff --git a/drivers/dma/imx-dma.c b/driver

[PATCH] i8k: Make fan module parameters an unsigned

2014-12-21 Thread Pali Rohár
Setting negative fan multiplier or maximal fan speed does make any sense and can cause problems. So ensure that negative values will not be accepted. Signed-off-by: Pali Rohár --- drivers/char/i8k.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/

[PATCH v4] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
This patch adds new function i8k_get_fan_nominal_speed() for doing SMM call which will return nominal fan RPM for specified fan speed. It returns nominal RPM value at which fan operate when speed (0, 1, 2, 3) is set. It looks like RPM value is not accurate, but still provides very useful informatio

[BUG] rtl8192se: panic accessing unmapped memory in skb

2014-12-21 Thread Eric Biggers
Hi, I have a RTL8192SE wireless card, attached via PCI. Usually it works with no issues, but I recently had a kernel panic occur in the rtl8192se driver. The kernel version is 3.18. Based on my analysis of the panic dump, the panic was caused by a memory access violation in this block of code i

Re: [PATCH v3] i8k: Autodetect maximal fan speed and fan RPM multiplier

2014-12-21 Thread Pali Rohár
On Sunday 21 December 2014 17:55:51 Steven Honeyman wrote: > On 21 December 2014 at 16:37, Pali Rohár wrote: > > On Sunday 21 December 2014 13:23:32 Guenter Roeck wrote: > >> On 12/21/2014 04:09 AM, Pali Rohár wrote: > >> > On Sunday 21 December 2014 12:57:08 Guenter Roeck wrote: > >> >>> -#defin

[PATCH] net: wireless: rtlwifi: btcoexist: halbtc8723b1ant.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: ex_halbtc8723b1ant_periodical() ex_halbtc8723b1ant_coex_dm_reset() ex_halbtc8723b1ant_pnp_notify() ex_halbtc8723b1ant_halt_notify() ex_halbtc8723b1ant_bt_info_notify() ex_halbtc8723b1ant_special_packet_notify() ex_halbtc8723b1ant_connect_notify() e

[PATCH] scripts: dtc: libfdt: fdt_rw.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: fdt_delprop() fdt_add_mem_rsv() This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- scripts/dtc/libfdt/fdt_rw.c | 32 - scripts/dtc/libfdt/libfd

[PATCH] net: ethernet: micrel: ksz884x.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
Remove the function port_cfg_dis_learn() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/net/ethernet/micrel/ksz884x.c |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers

[PATCH] gpu: drm: radeon: radeon_device.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
Remove the function radeon_doorbell_free() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/gpu/drm/radeon/radeon.h|1 - drivers/gpu/drm/radeon/radeon_device.c | 14

[PATCH] thermal: ti-soc-thermal: ti-bandgap.c: Remove unused function

2014-12-21 Thread Rickard Strandqvist
Remove the function ti_bandgap_write_thot() that is not used anywhere. This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/thermal/ti-soc-thermal/ti-bandgap.c | 13 - drivers/thermal/ti-soc-thermal/ti-ba

[PATCH] staging: rtl8192e: rtllib_rx.c: Remove some unused functions

2014-12-21 Thread Rickard Strandqvist
Removes some functions that are not used anywhere: update_ibss_network() rtllib_SignalStrengthTranslate() This was partially found by using a static code analysis program called cppcheck. Signed-off-by: Rickard Strandqvist --- drivers/staging/rtl8192e/rtllib_rx.c | 35 ---

  1   2   3   >