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

2015-03-28 Thread Richard Weinberger
On Sat, Mar 28, 2015 at 9:21 PM, Shirish Gajera gshirishf...@gmail.com wrote: This patch fixes the checkpatch.pl warning: WARNING: line over 80 characters All line over 80 characters in driver/staging/speakup/* are fixed. Signed-off-by: Shirish Gajera gshirishf...@gmail.com ---

[PATCH 2/3] rtc: __rtc_read_time: reduce log level

2015-03-28 Thread Aaro Koskinen
__rtc_read_time logs should be debug logs instead of error logs. For example, when the RTC clock is not set, it's not really useful to print a kernel error log every time someone tries to read the clock: ~ # hwclock -r [ 604.508263] rtc rtc0: read_time: fail to read

[PATCH 1/3] rtc: initialize rtc name early

2015-03-28 Thread Aaro Koskinen
In some error cases RTC name is used before it is initialized: rtc-rs5c372 0-0032: clock needs to be set rtc-rs5c372 0-0032: rs5c372b found, 24hr, driver version 0.6 rtc (null): read_time: fail to read rtc-rs5c372 0-0032: rtc core: registered rtc-rs5c372 as rtc0

[PATCH 3/3] rtc: hctosys: use function name in the error log

2015-03-28 Thread Aaro Koskinen
Use function name in the error log instead of __FILE__. Signed-off-by: Aaro Koskinen aaro.koski...@iki.fi --- drivers/rtc/hctosys.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/rtc/hctosys.c b/drivers/rtc/hctosys.c index 6c719f2..7748a61 100644 ---

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Pali Rohár
On Saturday 28 March 2015 13:54:51 Steven Honeyman wrote: On 28 March 2015 at 11:04, Paul Bolle pebo...@tiscali.nl wrote: On Sat, 2015-03-28 at 11:24 +0100, Pali Rohár wrote: --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -1703,6 +1703,17 @@ config SENSORS_ULTRA45

Re: [RFC] bpf: Suggestion on bpf syscall interface

2015-03-28 Thread Daniel Borkmann
On 03/28/2015 06:21 PM, Alexei Starovoitov wrote: On 3/28/15 4:36 AM, He Kuang wrote: Hi, Alexei In our end-end IO module project, we use bpf maps to record configurations. According to current bpf syscall interface, we should specify map_fd to lookup/update bpf maps, so we are restricted to

Re: [PATCH v2 01/10] iio: core: Introduce ENERGY channel type

2015-03-28 Thread Hartmut Knaack
Irina Tirdea schrieb am 11.01.2015 um 20:10: Human activity sensors report the energy burnt by the user. One of this devices is Freescale's MMA9553L (http://www.freescale.com/files/sensors/doc/ref_manual/MMA9553LSWRM.pdf) that computes the number of calories based on weight and step rate.

Re: [PATCH v4 00/15] checkpatch clean-up of rtl8723au's rtw_security.c

2015-03-28 Thread Joe Perches
On Sat, 2015-03-28 at 08:52 -0400, M. Vefa Bicakci wrote: These commits address a number of checkpatch.pl warnings and errors in rtl8723au's rtw_security.c. Hello. This is one of those patchsets that should be done against Greg's staging-testing branch. There are some commits against

Re: [RFC PATCH 00/11] an introduction of library operating system for Linux (LibOS)

2015-03-28 Thread Richard Weinberger
Am 27.03.2015 um 16:17 schrieb Antti Kantee: Let me try to offer some insight. I've been working on something similar in mainline NetBSD for almost 8 years now, so in addition to ideas popping into my head I've also tested them out in the real world. I do think that all operating systems

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Pali Rohár
On Saturday 28 March 2015 15:23:15 Guenter Roeck wrote: + ---help--- + This hwmon driver adds support for reporting temperature of different +sensors and controls the fans on Dell laptops via System Management + Mode provided by Dell BIOS. + + When option I8K

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Guenter Roeck
On 03/28/2015 03:00 PM, Pali Rohár wrote: On Saturday 28 March 2015 13:54:51 Steven Honeyman wrote: On 28 March 2015 at 11:04, Paul Bolle pebo...@tiscali.nl wrote: On Sat, 2015-03-28 at 11:24 +0100, Pali Rohár wrote: --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -1703,6 +1703,17

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Pali Rohár
On Saturday 28 March 2015 23:19:04 Steven Honeyman wrote: On 28 March 2015 at 22:00, Pali Rohár pali.ro...@gmail.com wrote: On Saturday 28 March 2015 13:54:51 Steven Honeyman wrote: i8k-virtual-0 Adapter: Virtual device Processor Fan: 3171 RPM CPU:+52.0°C Ambient:

[PATCH v5 02/15] staging: rtl8723au: Fix before/around/after whitespace issues

2015-03-28 Thread M. Vefa Bicakci
Correct a number of space(s) required before/around/after checkpatch.pl issues in a number of functions in rtl8723au's rtw_security.c. Signed-off-by: M. Vefa Bicakci m@runbox.com --- v5: Correct even more whitespace issues. v2: Correct one more whitespace issue (length-4) in

[PATCH v5 09/15] staging: rtl8723au: that open brace should be on the previous line

2015-03-28 Thread M. Vefa Bicakci
Correct two instances of the checkpatch.pl error indicating that the opening curly braces should not be on new lines: ERROR: that open brace { should be on the previous line Signed-off-by: M. Vefa Bicakci m@runbox.com --- v3: Make sure that all edited lines are at most 80 characters

[PATCH v5 15/15] staging: rtl8723au: Remove unneeded comments

2015-03-28 Thread M. Vefa Bicakci
This commit removes a number of unneeded comments. Two of the aforementioned comments were most likely meant to aid with version control, whereas the remaining two comments relate to (now unused) local variable names. Signed-off-by: M. Vefa Bicakci m@runbox.com ---

[PATCH v5 10/15] staging: rtl8723au: No spaces at the start of a line

2015-03-28 Thread M. Vefa Bicakci
Prior to this commit, a large block of constants used to represent an AES S-box table were indented with spaces in rtl8723au's rtw_security.c. Correct the checkpatch.pl warnings indicating that spaces should not be used to indent lines: WARNING: please, no spaces at the start of a line

[PATCH v5 13/15] staging: rtl8723au: Rework two byte array comparisons

2015-03-28 Thread M. Vefa Bicakci
Prior to this commit, rtl8723au's rtw_security.c had two instances of byte array comparisons (for CRC checks) where the individual elements of the byte arrays were compared one by one and an error trace would be output if the byte arrays were determined to be different. This commit improves the

[PATCH v5 11/15] staging: rtl8723au: Adjust whitespace in and around comments

2015-03-28 Thread M. Vefa Bicakci
As the subject indicates, adjust whitespace in and around comments in rtl8723au's rtw_security.c. Signed-off-by: M. Vefa Bicakci m@runbox.com --- drivers/staging/rtl8723au/core/rtw_security.c | 113 +- 1 file changed, 57 insertions(+), 56 deletions(-) diff --git

[PATCH v5 01/15] staging: rtl8723au: Reformat whitespace to increase readability

2015-03-28 Thread M. Vefa Bicakci
Adjust the whitespace in the signature, local variable declaration and initialization parts of a number of functions to increase readability in rtl8723au's rtw_security.c. Signed-off-by: M. Vefa Bicakci m@runbox.com --- v2: Make sure that the arcfour_encrypt function's argument list is

[PATCH v5 07/15] staging: rtl8723au: Remove unneeded curly braces

2015-03-28 Thread M. Vefa Bicakci
Correct a number of checkpatch.pl warnings in rtl8723au's rtw_security.c related to the existence of unnecessary curly braces around single statement blocks: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: M. Vefa Bicakci m@runbox.com ---

Re: [virtio-dev] Re: [PATCH v5] Add virtio-input driver.

2015-03-28 Thread Michael S. Tsirkin
On Thu, Mar 26, 2015 at 01:10:40PM +0100, Michael S. Tsirkin wrote: On Thu, Mar 26, 2015 at 11:49:25AM +0100, Gerd Hoffmann wrote: virtio-input is basically evdev-events-over-virtio, so this driver isn't much more than reading configuration from config space and forwarding incoming events

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

2015-03-28 Thread Richard Weinberger
Am 28.03.2015 um 22:18 schrieb Joe Perches: On Sat, 2015-03-28 at 21:40 +0100, Richard Weinberger wrote: On Sat, Mar 28, 2015 at 9:21 PM, Shirish Gajera gshirishf...@gmail.com wrote: This patch fixes the checkpatch.pl warning: [] diff --git a/drivers/staging/speakup/main.c

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Paul Bolle
On Sat, 2015-03-28 at 22:55 +0100, Pali Rohár wrote: I have tested compilation only with out of tree make command with manual CONFIG_SENSORS_DELL_SMM and CONFIG_I8K options. Not sure what you mean here. I can't get things past this error: drivers/hwmon/Kconfig:1721: syntax error

Re: [rtc-linux] Re: [PATCH] MAINTAINERS: Add Alexandre Belloni as an RTC maintainer

2015-03-28 Thread Alessandro Zummo
On Sat, 28 Mar 2015 22:29:20 +0100 Alexandre Belloni alexandre.bell...@free-electrons.com wrote: I'll keep that in mind. Could you give me permissions on the rtc-linux patchwork? Sure. Let me find the way to do it :) -- Best regards, Alessandro Zummo - CEO, Tower Technologies -

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

2015-03-28 Thread Shirish Gajera
On Sun, Mar 29, 2015 at 12:54:45AM +0100, Richard Weinberger wrote: Am 29.03.2015 um 00:44 schrieb Shirish Gajera: On Sat, Mar 28, 2015 at 02:35:19PM -0700, Joe Perches wrote: On Sat, 2015-03-28 at 22:22 +0100, Richard Weinberger wrote: Am 28.03.2015 um 22:18 schrieb Joe Perches: On Sat,

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Guenter Roeck
On 03/28/2015 03:44 PM, Paul Bolle wrote: Guenter Roeck schreef op za 28-03-2015 om 15:20 [-0700]: Maybe that is the plan, but it is not what is happening. I8K selects SENSORS_DELL_SMM, so a boolean I8K forces SENSORS_DELL_SMM to be built into the kernel. What you suggest would work if I8K

Re: d63e2e1f3df breaks sparc/T5-8

2015-03-28 Thread Yinghai Lu
On Sat, Mar 28, 2015 at 7:48 AM, David Ahern david.ah...@oracle.com wrote: On 3/27/15 11:26 PM, Yinghai Lu wrote: looks like the offset for mem64 is not right. Please try attached v2. still have problem. [139295.760918] pci_sun4v f02dbcfc: PCI host bridge to bus :00 [139295.831448]

alx: Wake-on-LAN with Qualcomm Atheros QCA8171

2015-03-28 Thread MegaBrutal
Hi Johannes, I have an ASRock B85M motherboard with Qualcomm Atheros QCA8171 NIC. In dmesg, I see the NIC is claimed by your alx driver: [0.530351] alx :04:00.0 eth0: Qualcomm Atheros AR816x/AR817x Ethernet [xx:xx:xx:xx:xx:xx] In lspci, the NIC shows up like this: 04:00.0 Ethernet

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Steven Honeyman
On 28 March 2015 at 22:00, Pali Rohár pali.ro...@gmail.com wrote: On Saturday 28 March 2015 13:54:51 Steven Honeyman wrote: i8k-virtual-0 Adapter: Virtual device Processor Fan: 3171 RPM CPU:+52.0°C Ambient:+46.0°C SODIMM: +40.0°C - Ambient and SODIMM look

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

2015-03-28 Thread Shirish Gajera
On Sat, Mar 28, 2015 at 02:35:19PM -0700, Joe Perches wrote: On Sat, 2015-03-28 at 22:22 +0100, Richard Weinberger wrote: Am 28.03.2015 um 22:18 schrieb Joe Perches: On Sat, 2015-03-28 at 21:40 +0100, Richard Weinberger wrote: On Sat, Mar 28, 2015 at 9:21 PM, Shirish Gajera

[PATCH] Documentation/ABI: iio: fix typo

2015-03-28 Thread Martin Kepplinger
Signed-off-by: Martin Kepplinger mart...@posteo.de --- Documentation/ABI/testing/sysfs-bus-iio | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio index 9a70c31..74a3493 100644 ---

Re: [rtc-linux] Re: [PATCH] MAINTAINERS: Add Alexandre Belloni as an RTC maintainer

2015-03-28 Thread Alexandre Belloni
Hi Alessandro, On 25/03/2015 at 23:58:09 +0100, Alessandro Zummo wrote : A few things I consider important: - a driver for a new hardware usually causes no harm to the subsystem - a patch for an existing driver should be approved by the original author, if possible. there are a few

Re: [PATCH 3/5] dmaengine: bcm2835-dma: Fix memory leak when stopping a running transfer

2015-03-28 Thread Peter Ujfalusi
Thanks, On 03/28/2015 05:57 AM, Stephen Warren wrote: On 03/27/2015 05:35 AM, Peter Ujfalusi wrote: The vd-node is removed from the lists when the transfer started so the vchan_get_all_descriptors() will not find it. This results memory leak. Acked-by: Stephen Warren swar...@wwwdotorg.org

Question on ARC's thread_info

2015-03-28 Thread Richard Weinberger
Vineet, I'm working on a patch series which touches sturct thread_info. ARC's thread_info has the following comment: * - if the contents of this structure are changed, the assembly constants * must also be changed Is this still true? Unless I'm reading the code wrong that case should be

[PATCH v5 03/15] staging: rtl8723au: else should follow close brace

2015-03-28 Thread M. Vefa Bicakci
Correct checkpatch.pl errors in rtl8723au's rtw_security.c indicating that an else statement should follow the closing brace of the previous if/else if code block: ERROR: else should follow close brace '}' Signed-off-by: M. Vefa Bicakci m@runbox.com ---

[PATCH v5 12/15] staging: rtl8723au: suspect code indent for conditional statements

2015-03-28 Thread M. Vefa Bicakci
Correct a number of indentation-with-spaces-and-tabs issues in rtl8723au's rtw_security.c, according to checkpatch.pl: WARNING: suspect code indent for conditional statements Signed-off-by: M. Vefa Bicakci m@runbox.com --- v3: Make sure that all edited lines are at most 80

[PATCH v5 04/15] staging: rtl8723au: Fix the indentation of two lines

2015-03-28 Thread M. Vefa Bicakci
Correct the indentation of two lines in rtw_tkip_encrypt23a function in rtl8723au's rtw_security.c. Signed-off-by: M. Vefa Bicakci m@runbox.com --- drivers/staging/rtl8723au/core/rtw_security.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v5 08/15] staging: rtl8723au: trailing statements should be on next line

2015-03-28 Thread M. Vefa Bicakci
Correct a number of checkpatch.pl errors in rtl8723au's rtw_security.c related to trailing statements: ERROR: trailing statements should be on next line Signed-off-by: M. Vefa Bicakci m@runbox.com --- drivers/staging/rtl8723au/core/rtw_security.c | 25 + 1

[PATCH v5 14/15] staging: rtl8723au: Use __func__ in trace logs

2015-03-28 Thread M. Vefa Bicakci
Rework the trace log-related lines in rtl8723au's rtw_security.c to use the __func__ GCC magic variable instead of hardcoding the function names into the trace log strings. This also corrects a copy-paste-related typo in the function named rtw_tkip_decrypt23a. Thanks to Jes Sorensen for the

[PATCH] clk: at91: pll: fix input range validity check

2015-03-28 Thread Boris Brezillon
The PLL impose a certain input range to work correctly, but it appears that this input range does not apply on the input clock (or parent clock) but on the input clock after it has passed the PLL divisor. Fix the implementation accordingly. Cc: sta...@vger.kernel.org # v3.14+ Signed-off-by: Boris

[PATCH v2] lib: bitmap_[empty,full]: remove code duplication

2015-03-28 Thread Yury Norov
Function 'bitmap_empty' has it's own implementation. But it's clearly as simple as: find_first_bit(src, nbits) == nbits The same is true for 'bitmap_full'. --- include/linux/bitmap.h | 8 lib/bitmap.c | 30 -- 2 files changed, 4

Re: d63e2e1f3df breaks sparc/T5-8

2015-03-28 Thread Yinghai Lu
On Sat, Mar 28, 2015 at 11:16 AM, David Miller da...@davemloft.net wrote: PCI addresses being 64-bit or not is an attribute of the PCI controller and the geography of the bridges behind it, not the cpu architecture. Good point. We should add one choice in pci subsystem Kconfig. Thanks

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

2015-03-28 Thread Richard Weinberger
Am 29.03.2015 um 00:44 schrieb Shirish Gajera: On Sat, Mar 28, 2015 at 02:35:19PM -0700, Joe Perches wrote: On Sat, 2015-03-28 at 22:22 +0100, Richard Weinberger wrote: Am 28.03.2015 um 22:18 schrieb Joe Perches: On Sat, 2015-03-28 at 21:40 +0100, Richard Weinberger wrote: On Sat, Mar 28,

[PATCH v5 00/15] checkpatch clean-up of rtl8723au's rtw_security.c

2015-03-28 Thread M. Vefa Bicakci
These commits address a number of checkpatch.pl warnings and errors in rtl8723au's rtw_security.c. Prior to this set of commits, checkpatch.pl reported the following: total: 75 errors, 124 warnings, 1631 lines checked After applying this set of commits, checkpatch.pl reports the

[PATCH v5 05/15] staging: rtl8723au: Reorganize a few functions to remove indentation

2015-03-28 Thread M. Vefa Bicakci
Prior to this commit, functions rtw_tkip_encrypt23a and rtw_tkip_decrypt23a had large if blocks which contained the majority of the logic in the functions. Rework these functions so that if the negated version of the aforementioned if blocks' conditions are true, we return from the function with

[PATCH v5 06/15] staging: rtl8723au: else is not generally useful after a return

2015-03-28 Thread M. Vefa Bicakci
Correct a checkpatch.pl warning regarding rtl8723au's rtw_security.c::crc32_init pointing out that having an else statement after a break or a return is not useful. drivers/staging/rtl8723au/core/rtw_security.c:105: WARNING: else is not generally useful after a break or return

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

2015-03-28 Thread Shirish Gajera
This patch fixes the checkpatch.pl warning: WARNING: line over 80 characters All line over 80 characters in driver/staging/speakup/* are fixed. Signed-off-by: Shirish Gajera gshirishf...@gmail.com --- drivers/staging/speakup/main.c | 9 ++--- drivers/staging/speakup/serialio.h

[PATCH 1/3] mfd: menelaus: delete omap_has_menelaus

2015-03-28 Thread Aaro Koskinen
Delete unused macro. Signed-off-by: Aaro Koskinen aaro.koski...@iki.fi --- include/linux/mfd/menelaus.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/include/linux/mfd/menelaus.h b/include/linux/mfd/menelaus.h index f097e89..a1e12bf3 100644 --- a/include/linux/mfd/menelaus.h +++

[PATCH 3/3] mfd: menelaus: use macro for magic number

2015-03-28 Thread Aaro Koskinen
Use macro to check a register bit. Signed-off-by: Aaro Koskinen aaro.koski...@iki.fi --- drivers/mfd/menelaus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/menelaus.c b/drivers/mfd/menelaus.c index 917fa86..c2ca665 100644 --- a/drivers/mfd/menelaus.c +++

[PATCH 2/3] mfd: menelaus: drop support for SW controller VCORE

2015-03-28 Thread Aaro Koskinen
Drop support for SW controlled VCORE, nobody uses it. Signed-off-by: Aaro Koskinen aaro.koski...@iki.fi --- drivers/mfd/menelaus.c | 23 --- include/linux/mfd/menelaus.h | 1 - 2 files changed, 24 deletions(-) diff --git a/drivers/mfd/menelaus.c

[PATCH 0/3] mfd: menelaus: couple simple cleanups

2015-03-28 Thread Aaro Koskinen
Hi, I came across these while trying to start DT conversion for menelaus (http://marc.info/?t=14197028735r=1w=2). While the DT work failed and is still pending, I think it's still worth to apply these as they are independent and they remove some cruft from the tree. A. Aaro Koskinen (3):

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

2015-03-28 Thread Joe Perches
On Sat, 2015-03-28 at 22:22 +0100, Richard Weinberger wrote: Am 28.03.2015 um 22:18 schrieb Joe Perches: On Sat, 2015-03-28 at 21:40 +0100, Richard Weinberger wrote: On Sat, Mar 28, 2015 at 9:21 PM, Shirish Gajera gshirishf...@gmail.com wrote: This patch fixes the checkpatch.pl warning:

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Pali Rohár
On Saturday 28 March 2015 12:04:20 Paul Bolle wrote: On Sat, 2015-03-28 at 11:24 +0100, Pali Rohár wrote: --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -1703,6 +1703,17 @@ config SENSORS_ULTRA45 This driver provides support for the Ultra45 workstation

Re: PANIC: double fault, error_code: 0x0 in 4.0.0-rc3-2, kvm related?

2015-03-28 Thread Maciej W. Rozycki
On Wed, 18 Mar 2015, Andy Lutomirski wrote: I posted the same problem to the opensuse kernel list shortly before turning to LKML. There, Michal Kubecek noted: I encountered a similar problem recently. The thing is, x86 specification says that on a double fault, RIP and RSP registers are

[PATCH] clk: check -determine/round_rate() return value in clk_calc_new_rates

2015-03-28 Thread Boris Brezillon
-determine_rate() and -round_rate() can return the closest rate to the requested one or an error code. clk_calc_new_rates is assuming these functions can't return a negative value, which leads to a undefined behavior when the clk implementation returns such an error code. Fix this by returning

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

2015-03-28 Thread Joe Perches
On Sat, 2015-03-28 at 21:40 +0100, Richard Weinberger wrote: On Sat, Mar 28, 2015 at 9:21 PM, Shirish Gajera gshirishf...@gmail.com wrote: This patch fixes the checkpatch.pl warning: [] diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup/main.c [] @@ -423,7 +423,8 @@

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Paul Bolle
Guenter Roeck schreef op za 28-03-2015 om 15:20 [-0700]: Maybe that is the plan, but it is not what is happening. I8K selects SENSORS_DELL_SMM, so a boolean I8K forces SENSORS_DELL_SMM to be built into the kernel. What you suggest would work if I8K would depend on SENSORS_DELL_SMM, but then

Re: [PATCH v4 00/15] checkpatch clean-up of rtl8723au's rtw_security.c

2015-03-28 Thread M. Vefa Bicakci
On Sat, March 28, 2015 12:52:54 Joe Perches wrote: On Sat, 2015-03-28 at 08:52 -0400, M. Vefa Bicakci wrote: These commits address a number of checkpatch.pl warnings and errors in rtl8723au's rtw_security.c. Hello. This is one of those patchsets that should be done against Greg's

[PATCH] clk: at91: usb: propagate rate modification to the parent clk

2015-03-28 Thread Boris Brezillon
The at91sam9n12 and at91sam9x5 usb clocks do not propagate rate modification requests to their parents. This causes a bug when the PLLB is left uninitialized by the bootloader (PLL multiplier set to 0, or in other words, PLL rate = 0 Hz). Implement the determinate_rate method and propagate the

[PATCH] Do not use arbitrary large movablecore to calculate kernelcore

2015-03-28 Thread Zhihui Zhang
If kernelcore is not set, then we are working with a very large kernelcore for nothing - no movable zone will be created. If kernelcore is set, then it is not respected at all. Signed-off-by: Zhihui Zhang zzhs...@gmail.com --- mm/page_alloc.c | 6 +- 1 file changed, 5 insertions(+), 1

Re: [PATCH 1/2] Input: Add binding document for a generic ADC keypad

2015-03-28 Thread Dmitry Torokhov
On Fri, Mar 27, 2015 at 05:43:17PM -0700, Andrew Bresticker wrote: Add a binding document for a generic ADC keypad. Buttons on an ADC keypad are connected in a resistor ladder to an ADC. The binding describes the mapping of ADC channel and voltage ranges to buttons. Signed-off-by: Andrew

[PATCH] README: Update references to version 4

2015-03-28 Thread Pranith Kumar
Since we bumped the version to 4.0, let us update the references to match that in the README file. Signed-off-by: Pranith Kumar bobby.pr...@gmail.com --- README | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/README b/README index

Re: [PATCH 2/2] lib/lcm.c: lcm(n,0)=lcm(0,n) is 0, not n

2015-03-28 Thread Mike Snitzer
On Tue, Dec 9, 2014 at 4:03 PM, Rasmus Villemoes li...@rasmusvillemoes.dk wrote: Return the mathematically correct answer when an argument is 0. Signed-off-by: Rasmus Villemoes li...@rasmusvillemoes.dk --- lib/lcm.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

Re: [PATCH] Input: ALPS - Detect trackstick presence for v7 protocol

2015-03-28 Thread Dmitry Torokhov
On Tue, Mar 24, 2015 at 08:27:29AM +0100, Hans de Goede wrote: Hi, On 23-03-15 18:02, Pali Rohár wrote: On Monday 23 March 2015 12:39:48 Hans de Goede wrote: Hi, On 22-03-15 14:46, Pali Rohár wrote: This patch adds detection of trackstick for v7 protocol devices. Code in this patch is

Re: [PATCH v2] vfs: avoid recopying file names in getname_flags

2015-03-28 Thread Boqun Feng
Ping. This patch has been tested by 0day test bot. Thanks, Boqun Feng On Thu, Mar 26, 2015 at 02:45:52AM +0800, Boqun Feng wrote: In the current implementation of getname_flags, a file name in the user-space will be recopied if it takes more space that EMBEDDED_NAME_MAX, however, at this

Re: [PATCH] tools: iio: generic_buffer: Fix generic scale extraction

2015-03-28 Thread Jonathan Cameron
On 27/03/15 12:57, Daniel Baluta wrote: On Fri, Mar 27, 2015 at 1:53 PM, Irina Tirdea irina.tir...@intel.com wrote: When using generic_buffer to read data, the scale is not properly detected for scale shared by type. This is caused by a problem with the generation of generic name out of the

Re: ia32_sysenter_target does not preserve EFLAGS

2015-03-28 Thread Denys Vlasenko
On Sat, Mar 28, 2015 at 10:46 AM, Ingo Molnar mi...@kernel.org wrote: * Denys Vlasenko vda.li...@googlemail.com wrote: This is a C function. [...] Arguably that's a self-inflicted wound of uclibc: nothing keeps it from taking advantage of the syscall ABI and avoiding the double

Re: [PATCH 1/3] iio: configfs: Add configfs support to IIO

2015-03-28 Thread Jonathan Cameron
On 27/03/15 17:17, Lars-Peter Clausen wrote: On 03/25/2015 06:00 PM, Daniel Baluta wrote: This creates an IIO configfs subsystem named iio, which has one default group named triggers. This allows us to easily create/destroy software triggers. One must create a driver which implements

Re: [PATCH 3/3] iio: Documentation: Add initial documentaton for IIO

2015-03-28 Thread Jonathan Cameron
On 25/03/15 17:00, Daniel Baluta wrote: This file wants to be a starting point document for anyone wanting to use IIO configfs support or adding new IIO configfs functionality. Signed-off-by: Daniel Baluta daniel.bal...@intel.com Looks like a good flexible structure to me. Thanks! ---

[PATCH] of/unittest: Fix of_platform_depopulate test case

2015-03-28 Thread Grant Likely
The previous commit, of/unittest: early return from test skips tests exposed broken tests for the of_platform_unpopulate() function. The problem was the populate and depopulate calls were not symmetrical like they were intended to be, and unpopulate depends on the parent device to have it's

Re: [PATCH v3 3/3] Documentation: update the of_unittest.txt

2015-03-28 Thread Grant Likely
On Tue, 17 Mar 2015 03:20:00 + , Wang Long long.wangl...@huawei.com wrote: Since the directory drivers/of/testcase-data is renamed to drivers/of/unittest-data. so we should update the path in the of_selftest.txt. When the kernel is built with OF_UNITTEST enabled, the output dtb is

Re: [RFC] vmstat: Avoid waking up idle-cpu to service shepherd work

2015-03-28 Thread Viresh Kumar
On 28 March 2015 at 17:27, viresh kumar viresh.ku...@linaro.org wrote: On 28 March 2015 at 15:23, Peter Zijlstra pet...@infradead.org wrote: Well, for one your patch is indeed disgusting. Yeah, I agree :) Sigh.. Sorry for the series of *nonsense* mails before the last one. Its some

Re: [PATCH v3 2/3] Documentation: rename of_selftest.txt to of_unittest.txt

2015-03-28 Thread Grant Likely
On Tue, 17 Mar 2015 03:19:59 + , Wang Long long.wangl...@huawei.com wrote: Since the test of the devicetree's OF api use unittest as its name. so we should rename of_selftest.txt to of_unittest.txt. Signed-off-by: Wang Long long.wangl...@huawei.com Applied, thanks. g. ---

Re: [Patch 0/5] dt: unittest: fix breakage and warnings

2015-03-28 Thread Grant Likely
On Fri, 13 Mar 2015 22:25:51 -0700 , Frank Rowand frowand.l...@gmail.com wrote: Running checkpatch on early versions of my patchset to fix the devicetree make dependency issues exposed a large number of warnings, including some that are actual bugs.

[PATCH] x86, doc: Remove cmdline_size from list of fields to be filled in for EFI handover

2015-03-28 Thread Alex Smith
The cmdline_size field in setup_header is listed as read-only, and is never read by the kernel. Therefore, there is no need for it to be set by a boot loader implementing the EFI handover protocol, so remove it from the list of fields required to be set by a boot loader. Signed-off-by: Alex Smith

[PATCH v4 11/15] staging: rtl8723au: Adjust whitespace in and around comments

2015-03-28 Thread M. Vefa Bicakci
As the subject indicates, adjust whitespace in and around comments in rtl8723au's rtw_security.c. Signed-off-by: M. Vefa Bicakci m@runbox.com --- drivers/staging/rtl8723au/core/rtw_security.c | 114 +- 1 file changed, 57 insertions(+), 57 deletions(-) diff --git

[PATCH v4 04/15] staging: rtl8723au: Fix the indentation of two lines

2015-03-28 Thread M. Vefa Bicakci
Correct the indentation of two lines in rtw_tkip_encrypt23a function in rtl8723au's rtw_security.c. Signed-off-by: M. Vefa Bicakci m@runbox.com --- drivers/staging/rtl8723au/core/rtw_security.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Steven Honeyman
On 28 March 2015 at 11:04, Paul Bolle pebo...@tiscali.nl wrote: On Sat, 2015-03-28 at 11:24 +0100, Pali Rohár wrote: --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -1703,6 +1703,17 @@ config SENSORS_ULTRA45 This driver provides support for the Ultra45 workstation

[PATCH v4 14/15] staging: rtl8723au: Use __func__ in trace logs

2015-03-28 Thread M. Vefa Bicakci
Rework the trace log-related lines in rtl8723au's rtw_security.c to use the __func__ GCC magic variable instead of hardcoding the function names into the trace log strings. This also corrects a copy-paste-related typo in the function named rtw_tkip_decrypt23a. Thanks to Jes Sorensen for the

[PATCH v4 15/15] staging: rtl8723au: Remove unneeded comments

2015-03-28 Thread M. Vefa Bicakci
This commit removes a number of unneeded comments. Two of the aforementioned comments were most likely meant to aid with version control, whereas the remaining two comments relate to (now unused) local variable names. Signed-off-by: M. Vefa Bicakci m@runbox.com ---

Re: security problem with seccomp-filter

2015-03-28 Thread Kees Cook
On Thu, Mar 26, 2015 at 11:39 PM, Richard Weinberger richard.weinber...@gmail.com wrote: Cc'ing seccomp folks. On Fri, Mar 27, 2015 at 6:56 AM, Felix von Leitner felix-linuxker...@fefe.de wrote: Hi, I have had some great success with seccomp-filter a while ago, so I decided to use it to

Re: [PATCH v4 2/4] mfd: lubbock_cplds: add lubbock IO board

2015-03-28 Thread Arnd Bergmann
On Saturday 28 March 2015, Robert Jarzmik wrote: It's not as much a problem as a generic question : does a driver belong to arch/* ? Personaly it would have been far simpler for me to have it through the pxa tree, but I want to be sure it's the right place. Others will follow, pxa

Re: [RFC] vmstat: Avoid waking up idle-cpu to service shepherd work

2015-03-28 Thread Peter Zijlstra
On Sat, Mar 28, 2015 at 05:27:23PM +0530, viresh kumar wrote: So probably we need to make 'base' aligned to 8 bytes ? Yeah, something like the below (at the very end) should ensure the thing is cacheline aligned, that should give us a fair few bits. So, what you are suggesting is something

Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver without /proc/i8k

2015-03-28 Thread Paul Bolle
On Sat, 2015-03-28 at 11:24 +0100, Pali Rohár wrote: --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -1703,6 +1703,17 @@ config SENSORS_ULTRA45 This driver provides support for the Ultra45 workstation environmental sensors. +config SENSORS_DELL_SMM +

Re: ia32_sysenter_target does not preserve EFLAGS

2015-03-28 Thread Denys Vlasenko
On Sat, Mar 28, 2015 at 1:39 AM, Linus Torvalds torva...@linux-foundation.org wrote: What part of don't leak kernel data did you have trouble understanding? IOW, this is a *security* issue. Stop arguing for crazy shit. We can zero the registers instead of saving/restoring them. push/pop pair

[PATCH] btrfs: unlock i_mutex after attempting to delete subvolume during send

2015-03-28 Thread Omar Sandoval
Whenever the check for a send in progress introduced in commit 521e0546c970 (btrfs: protect snapshots from deleting during send) is hit, we return without unlocking inode-i_mutex. This is easy to see with lockdep enabled: [ +0.59] [

Re: [PATCH] btrfs: unlock i_mutex after attempting to delete subvolume during send

2015-03-28 Thread Filipe David Manana
On Sat, Mar 28, 2015 at 11:02 AM, Omar Sandoval osan...@osandov.com wrote: Whenever the check for a send in progress introduced in commit 521e0546c970 (btrfs: protect snapshots from deleting during send) is hit, we return without unlocking inode-i_mutex. This is easy to see with lockdep

Re: Revised futex(2) man page for review

2015-03-28 Thread Peter Zijlstra
On Sat, Mar 28, 2015 at 09:53:21AM +0100, Michael Kerrisk (man-pages) wrote: So, please take a look at the page below. At this point, I would most especially appreciate help with the FIXMEs. For people who cannot read that troff gibberish (me).. --- FUTEX(2) Linux

[PATCH v4 10/15] staging: rtl8723au: No spaces at the start of a line

2015-03-28 Thread M. Vefa Bicakci
Prior to this commit, a large block of constants used to represent an AES S-box table were indented with spaces in rtl8723au's rtw_security.c. Correct the checkpatch.pl warnings indicating that spaces should not be used to indent lines: WARNING: please, no spaces at the start of a line

[PATCH v4 01/15] staging: rtl8723au: Reformat whitespace to increase readability

2015-03-28 Thread M. Vefa Bicakci
Adjust the whitespace in the signature, local variable declaration and initialization parts of a number of functions to increase readability in rtl8723au's rtw_security.c. Signed-off-by: M. Vefa Bicakci m@runbox.com --- v2: Make sure that the arcfour_encrypt function's argument list is

[PATCH v4 12/15] staging: rtl8723au: suspect code indent for conditional statements

2015-03-28 Thread M. Vefa Bicakci
Correct a number of indentation-with-spaces-and-tabs issues in rtl8723au's rtw_security.c, according to checkpatch.pl: WARNING: suspect code indent for conditional statements Signed-off-by: M. Vefa Bicakci m@runbox.com --- v3: Make sure that all edited lines are at most 80

[PATCH v4 00/15] checkpatch clean-up of rtl8723au's rtw_security.c

2015-03-28 Thread M. Vefa Bicakci
These commits address a number of checkpatch.pl warnings and errors in rtl8723au's rtw_security.c. Prior to this set of commits, checkpatch.pl reported the following: total: 76 errors, 138 warnings, 1621 lines checked After applying this set of commits, checkpatch.pl reports the

Re: [RFC][PATCH 09/10] v4l: Export enums used by tracepoints to user space

2015-03-28 Thread Xie XiuQi
On 2015/3/28 5:37, Steven Rostedt wrote: #define show_field(field)\ - __print_symbolic(field, \ - { V4L2_FIELD_ANY, ANY },\ - { V4L2_FIELD_NONE,

Re: [PATCH] mm/x86: AMD Bulldozer ASLR fix

2015-03-28 Thread Kees Cook
On Fri, Mar 27, 2015 at 7:44 AM, Borislav Petkov b...@alien8.de wrote: On Fri, Mar 27, 2015 at 12:38:21PM +0100, Hector Marco-Gisbert wrote: A bug in Linux ASLR implementation which affects some AMD processors has been found. The issue affects to all Linux process even if they are not using

Re: [PATCH 2/2] show nohz_full cpus in sysfs

2015-03-28 Thread Rik van Riel
On 03/28/2015 12:18 AM, Mike Galbraith wrote: On Fri, 2015-03-27 at 17:50 -0400, r...@redhat.com wrote: From: Rik van Riel r...@redhat.com Currently there is no way to query which CPUs are in nohz_full mode from userspace. Hm, they're both (as of your last set) invariant. So are most of

[GIT PULL] x86 fix

2015-03-28 Thread Ingo Molnar
Linus, Please pull the latest x86-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86-urgent-for-linus # HEAD: b3494a4ab20f6bdf74cdf2badf7918bb65ee8a00 x86/asm/entry: Check for syscall exit work with IRQs disabled Fix x86 syscall exit code bug

Re: [PATCH 3/4] [SMB3] Fix dereference before null check warning

2015-03-28 Thread Shirish Pargaonkar
Looks correct. Acked-by: Shirish Pargaonkar shirishpargaon...@gmail.com On Fri, Mar 27, 2015 at 12:28 AM, Steve French smfre...@gmail.com wrote: null tcon is not likely in these paths in current code, but obviously it does clarify the code to check for null (if at all) before derefrencing

Re: [PATCH 1/2] hwmon: Rename i8k driver to dell-smm-hwmon and move it to hwmon tree

2015-03-28 Thread Guenter Roeck
On 03/28/2015 03:24 AM, Pali Rohár wrote: This commit moves i8k driver to hwmon tree under name dell-smm-hwmon which is better name then abbreviation i8k. For backward compatibility is added macro MODULE_ALIAS(i8k) so modprobe will load driver also old name i8k. CONFIG_I8K compile option was not

Re: [PATCH] IIO: Add support for L3GD20H gyroscope

2015-03-28 Thread Jonathan Cameron
On 24/03/15 10:29, Linus Walleij wrote: On Mon, Mar 23, 2015 at 2:40 PM, Robert Dolca robert.do...@intel.com wrote: It can be used exactly like L3GD20 but it has a different WhoAmI register value. Signed-off-by: Robert Dolca robert.do...@intel.com Reviewed-by: Linus Walleij

Re: [PATCH] tools: iio: Add help option for generic_buffer

2015-03-28 Thread Jonathan Cameron
On 27/03/15 12:42, Alin Arhip wrote: From: Alin Gabriel Arhip alin.g.ar...@intel.com This patch adds usage instructions and -h parameter to view available command line parameters. Signed-off-by: Alin Gabriel Arhip alin.g.ar...@intel.com Good in principal, but I wonder if we can't make it

  1   2   3   4   >