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 struct

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 that's only

[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 gangs.freelan...@gmail.com --- kernel/pid.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/pid.c b/kernel/pid.c index

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 gangs.freelan...@gmail.com --- kernel/pid.c |4 ++-- 1 file changed, 2

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 linux@roeck- us.net wrote: I would think that it should be relatively safe to assume that the first fan always exists That would be a

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 fan_max.

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

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 linux-kernel@vger.kernel.org wrote: Gitweb: http://git.kernel.org/linus/;a=commit;h=baeb0d9b98c3450e22f8f8e4a6619b8b1d5106ff Commit: baeb0d9b98c3450e22f8f8e4a6619b8b1d5106ff Parent:

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 bobby.pr...@gmail.com 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

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)

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

2014-12-21 Thread Ingo Molnar
* Andy Lutomirski l...@amacapital.net 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

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

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

[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

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? Without

[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 robert.nachlin...@googlemail.com --- drivers/staging/gdm724x/gdm_mux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

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:

[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 ge...@linux-m68k.org --- fs/ceph/addr.c | 2 +- 1 file

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 j...@openwrt.org: On Sat, Dec 20, 2014 at 2:39 AM, Kevin Cernekee cerne...@gmail.com wrote: On Mon, Dec 15, 2014 at 1:43 AM, Jonas Gorski j...@openwrt.org wrote: On Fri, Dec 12, 2014 at 11:07 PM, Kevin Cernekee cerne...@gmail.com wrote: BMIPS

[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

[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,

[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 ge...@linux-m68k.org --- fs/affs/amigaffs.c | 6 +++---

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 ge...@linux-m68k.org 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’

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

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 rickard_strandqv...@spectrumdigital.se 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

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

2014-12-21 Thread Alexandre Belloni
From: Arnd Bergmann a...@arndb.de 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

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 rickard_strandqv...@spectrumdigital.se 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

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 bske...@redhat.com: - Original Message - From: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se To: David Airlie airl...@linux.ie, Ben Skeggs bske...@redhat.com Cc: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se, Alexandre

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 rickard_strandqv...@spectrumdigital.se 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

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?

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

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 superquad.vort...@gmail.com: 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 rickard_strandqv...@spectrumdigital.se

[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

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

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_mutex);

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

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

[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 rickard_strandqv...@spectrumdigital.se ---

[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 md.jamalmohiud...@gmail.com --- drivers/staging/iio/adc/ad7192.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/adc/ad7192.c

[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 md.jamalmohiud...@gmail.com --- drivers/staging/iio/meter/ade7759.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/meter/ade7759.c

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 evg...@studio-punkt.com Hi Evgeni

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 mi...@neuling.org: 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

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

[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 rickard_strandqv...@spectrumdigital.se --- drivers/media/radio/wl128x/fmdrv_rx.c | 16

[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 rickard_strandqv...@spectrumdigital.se --- arch/x86/kernel/apb_timer.c |8 1 file changed, 8

[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 rickard_strandqv...@spectrumdigital.se ---

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

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

[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 jmmah...@gmail.com --- Documentation/usb/usbmon.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

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

2014-12-21 Thread Jeremiah Mahler
Signed-off-by: Jeremiah Mahler jmmah...@gmail.com --- 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 +++

[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

[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] asn:2*noident drivers/staging/vt6655/device_main.c:1503:25:got

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 s...@canb.auug.org.au wrote: Hi Pranith, On Sat, 20 Dec 2014 11:47:18 -0500 Pranith Kumar bobby.pr...@gmail.com wrote: Wire up sys_execveat(). This passes the selftests for the system call. Thanks for this, but ... diff --git

[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,

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

2014-12-21 Thread Borislav Petkov
From: Borislav Petkov b...@suse.de 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 b...@suse.de --- arch/x86/include/asm/fpu-internal.h | 10 ++ 1 file changed, 6

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

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 78eaf6095cc7

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 h...@rock-chips.com

[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 aaron...@intel.com Signed-off-by:

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: disable

[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 sergey.senozhat...@gmail.com --- drivers/block/zram/zram_drv.c | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/drivers/block/zram/zram_drv.c

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

2014-12-21 Thread Andreas Schwab
Geert Uytterhoeven ge...@linux-m68k.org 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

[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 mengdong@intel.com Signed-off-by: Fengguang Wu

[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

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] asn:2*noident

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

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

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

[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

[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 jaeg...@kernel.org --- fs/f2fs/trace.c |

[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 jaeg...@kernel.org --- 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 +++

[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 jaeg...@kernel.org --- fs/f2fs/Kconfig | 10 ++ fs/f2fs/Makefile | 1 + fs/f2fs/trace.c | 24 fs/f2fs/trace.h |

[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 jaeg...@kernel.org --- 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 jaeg...@kernel.org --- 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

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 configuration

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 127

[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()

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 pali.ro...@gmail.com 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()

[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

[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 are

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

[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 rickard_strandqv...@spectrumdigital.se --- drivers/dma/imx-dma.c |5 - 1 file changed, 5 deletions(-) diff

[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 pali.ro...@gmail.com --- drivers/char/i8k.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-)

[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

[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

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 pali.ro...@gmail.com 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:

[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()

[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 rickard_strandqv...@spectrumdigital.se --- scripts/dtc/libfdt/fdt_rw.c | 32

[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 rickard_strandqv...@spectrumdigital.se --- drivers/net/ethernet/micrel/ksz884x.c |6 -- 1 file

[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 rickard_strandqv...@spectrumdigital.se --- drivers/gpu/drm/radeon/radeon.h|1 -

[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 rickard_strandqv...@spectrumdigital.se --- drivers/thermal/ti-soc-thermal/ti-bandgap.c | 13

[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 rickard_strandqv...@spectrumdigital.se ---

  1   2   3   4   5   6   >