From: Markus Elfring
Date: Fri, 17 Feb 2017 21:34:27 +0100
The script "checkpatch.pl" pointed information out like the following.
WARNING: void function return statements are not generally useful
Thus remove such a statement in the affected functions.
Signed-off-by: Markus Elfring
---
driver
On Sat, Feb 18, 2017 at 02:10:43PM -0600, Alan Tull wrote:
> On Sat, Feb 18, 2017 at 6:45 AM, Nadathur, Sundar
> wrote:
>
> > Hi all,
> >Interesting discussion. The discussion so far has brought out many
> > concerns such as OS independence. There is an existing format, well-known
> > to de
From: Markus Elfring
Date: Fri, 17 Feb 2017 20:48:59 +0100
Add a missing character in this description and adjust
the comment formatting.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/mad.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/infinib
From: Markus Elfring
Date: Fri, 17 Feb 2017 20:20:43 +0100
Replace the specification of a data structure by a pointer dereference
as the parameter for the operator "sizeof" to make the corresponding size
determination a bit safer according to the Linux coding style convention.
Signed-off-by: Mar
On Sun, 2017-02-19 at 01:50 +0530, simran singhal wrote:
> Simplify function returns by merging assignment and return into
> one command line.
> Found with Coccinelle
> @@
> local idexpression ret;
> expression e;
> @@
>
> -ret =
> +return
> e;
> -return ret;
[]
> diff --git a/drivers/staging
From: Markus Elfring
Date: Fri, 17 Feb 2017 20:15:02 +0100
Replace the specification of a data type by a pointer dereference
as the parameter for the operator "sizeof" to make the corresponding size
determination a bit safer according to the Linux coding style convention.
Signed-off-by: Markus E
From: Markus Elfring
Date: Fri, 17 Feb 2017 20:05:38 +0100
Replace the specification of a data structure by a pointer dereference
as the parameter for the operator "sizeof" to make the corresponding size
determination a bit safer according to the Linux coding style convention.
Signed-off-by: Mar
From: Markus Elfring
Date: Sat, 18 Feb 2017 21:34:32 +0100
Several update suggestions were taken into account
from static source code analysis.
Markus Elfring (29):
Use kcalloc() in mlx4_ib_alloc_pv_bufs()
Improve another size determination in mlx4_ib_alloc_pv_bufs()
Improve another size d
On Sun, 19 Feb 2017, simran singhal wrote:
> Simplify function returns by merging assignment and return into
> one command line.
> Found with Coccinelle
> @@
> local idexpression ret;
> expression e;
> @@
>
> -ret =
> +return
> e;
> -return ret;
>
> Signed-off-by: simran singhal
> ---
> d
Hi,
Please don't apply this patch and instead apply the switchtec driver as
we submitted in v2. As per the discussion in [1], not using the cdev's
kobj parent results in incorrect reference counting and a possible use
of the cdev after its containing structure is freed.
I've also done a quick rev
Simplify function returns by merging assignment and return into
one command line.
Found with Coccinelle
@@
local idexpression ret;
expression e;
@@
-ret =
+return
e;
-return ret;
Signed-off-by: simran singhal
---
drivers/staging/vt6655/card.c | 2 +-
1 file changed, 1 insertion(+), 1 delet
commit b101829a029a ("mfd: axp20x: Fix AXP806 access errors on cold boot")
was intended to fix the case where a board uses an AXP806 in slave mode,
but the boot loader leaves it in master mode for lack of AXP806 support.
But now the driver breaks on boards where the PMIC is operating in master
mode
On Sat, Feb 18, 2017 at 6:45 AM, Nadathur, Sundar
wrote:
> Hi all,
>Interesting discussion. The discussion so far has brought out many
> concerns such as OS independence. There is an existing format, well-known to
> developers, with widespread support, and which is quite extensible:
> Type
commit b101829a029a ("mfd: axp20x: Fix AXP806 access errors on cold boot")
was intended to fix the case where a board uses an AXP806 in slave mode,
but the boot loader leaves it in master mode for lack of AXP806 support.
But now the driver breaks on boards where the PMIC is operating in master
mode
On Sat, 18 Feb 2017, Joe Perches wrote:
> On Sat, 2017-02-18 at 20:12 +0100, Julia Lawall wrote:
> > On Sun, 19 Feb 2017, simran singhal wrote:
> > > Extra parentheses were causing checkpatch issues
> > > and were removed.
> []
> > > diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c
> > >
The AXP806 (and very similar AXP808) PMICs can be wired to either master
mode or slave mode on a board-by-board basis. A register on the PMIC needs
to be set accordingly for the PMIC to respond to other register accesses.
The "xpowers,master-mode" property introduced in this patch series lets a
boa
On 17/02/17 16:03, Fabrice Gasnier wrote:
> Document Industrial I/O GPIO trigger support.
>
> Signed-off-by: Fabrice Gasnier
Whilst I think this needs redoing for the interrupt trigger driver,
please be aware that bindings must be OS agnostic i.e. you can't
name things after linux specific subsys
On Sat, 2017-02-18 at 20:12 +0100, Julia Lawall wrote:
> On Sun, 19 Feb 2017, simran singhal wrote:
> > Extra parentheses were causing checkpatch issues
> > and were removed.
[]
> > diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c
> > b/drivers/staging/rtl8192e/rtl819x_HTProc.c
[]
> > @@ -66
> Am 18.02.2017 um 19:08 schrieb Pavel Machek :
>
>>> And I agree that kernel should _not_ attempt rescaling itself, as it
>>> would lose precision.
>>
>> With an almost 1:1 mapping you won't loose precision.
>
> How do you propose to do that?
something like
xinput --set-prop --type=float "TS
On 17/02/17 16:16, Lars-Peter Clausen wrote:
> On 02/17/2017 05:03 PM, Fabrice Gasnier wrote:
>> GPIOs can be used to generate triggers for any IIO device.
>> Introduce generic GPIO trigger driver. It offers some options
>> to tune polarity (e.g. rising/falling edge), and label so trigger
>> name m
On Sat, 2017-02-18 at 21:32 +0530, Arushi Singhal wrote:
> Unnecessary parentheses as reported by checkpatch.pl
[]
> diff --git a/drivers/staging/ks7010/ks_hostif.c
> b/drivers/staging/ks7010/ks_hostif.c
[]
> @@ -121,19 +121,19 @@ int get_current_ap(struct ks_wlan_private *priv, struct
> link_ap_
On 17/02/17 16:23, Daniel Baluta wrote:
> On Fri, Feb 17, 2017 at 6:03 PM, Fabrice Gasnier
> wrote:
>> Hi,
>>
>> I send this as an RFC, as I'm digging to try to get as generic
>> as possible GPIO based trigger support, as discussed earlier in
>> other patchset "Add EXTI GPIO trigger support to ST
On Sun, 19 Feb 2017, simran singhal wrote:
> Extra parentheses were causing checkpatch issues
> and were removed.
>
> Signed-off-by: simran singhal
> ---
> drivers/staging/rtl8192e/rtl819x_HTProc.c | 12 ++--
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/st
On Sat, Feb 18, 2017 at 09:35:47AM +0100, Romain Perier wrote:
> The PCI pool API is deprecated. This commits replaces the PCI pool old
> API by the appropriated function with the DMA pool API.
Did not apply on linux-next-20170217
>
> Signed-off-by: Romain Perier
> ---
> drivers/scsi/megaraid
Use 'pci_dma_mapping_error()' instead of 'dma_mapping_error()' because it
is more consistent with the rest of the code. It is also a little less
verbove.
While at it, remove some '(dma_addr_t)' casts in order to be consistent
within the file.
Signed-off-by: Christophe JAILLET
---
drivers/scsi/h
This patch has locking problem. I've got lockdep splat under LTP.
[ 6633.115456] ==
[ 6633.115502] [ INFO: possible circular locking dependency detected ]
[ 6633.115553] 4.9.10-debug+ #9 Tainted: G L
[ 6633.115584] --
On Sat, Feb 18, 2017 at 09:35:36AM +0100, Romain Perier wrote:
Tested all patches by compilation and checkpatch. All of them compile
fine, but patches 11 and 12 need some fixes. You can resend as
PATCH instead of RFC.
> The current PCI pool API are simple macro functions direct expanded to
> the
Align * on last line, to conform to the kernal coding
style for block comments.
Signed-off-by: simran singhal
---
v2:
-Modified commit message
drivers/staging/rtl8192e/rtl819x_HT.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HT.h
b/
On Sat, Feb 18, 2017 at 09:35:48AM +0100, Romain Perier wrote:
> The PCI pool API is deprecated. This commits replaces the PCI pool old
> API by the appropriated function with the DMA pool API.
Please run checkpatch, fix the style issue and resend.
>
> Signed-off-by: Romain Perier
> ---
> driv
On Sat, Feb 18, 2017 at 09:35:47AM +0100, Romain Perier wrote:
Hi Romain,
Checkpatch gives some warnings you can fix related to NULL tests before
dma_pool_destroy(), and you changed indentation style in some of your
changes. Some times it is important to keep consistency within a file
even if the
Extra parentheses were causing checkpatch issues
and were removed.
Signed-off-by: simran singhal
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c
b/drivers/staging/rtl8192e/rt
I tried to send this patch yesterday and it appeared to work. However,
I did not see an email from the list. Assuming it failed and trying
again. I apologize if this is a repeat.
In Linux kernel 4.9-rc1, the function ib_get_dma_mr()
was removed and a second parameter was added to ib_alloc_pd().
As this broke the building of the ko2iblnd module in
staging, the Kconfig for LNet has marked ko2iblnd as broken
and stopped building it.
This patch fixes this breakage by:
- Remov
On Thu, Feb 16, 2017 at 09:41:59AM -0600, Tom Lendacky wrote:
> create mode 100644 Documentation/x86/amd-memory-encryption.txt
> create mode 100644 arch/x86/include/asm/mem_encrypt.h
> create mode 100644 arch/x86/kernel/mem_encrypt_boot.S
> create mode 100644 arch/x86/kernel/mem_encrypt_init.c
On Sat, Feb 18, 2017 at 09:29:17AM -0800, Steve Longerbeam wrote:
> On 02/18/2017 01:23 AM, Russell King - ARM Linux wrote:
> >On Fri, Feb 17, 2017 at 05:12:44PM -0800, Steve Longerbeam wrote:
> >>Hi Russell,
> >>
> >>I signed-off on this but after more review I'm not sure this is right.
> >>
> >>T
> > And I agree that kernel should _not_ attempt rescaling itself, as it
> > would lose precision.
>
> With an almost 1:1 mapping you won't loose precision.
How do you propose to do that?
Pavel
--
(english) http://www.live
Hi!
> > I guess I can. But I'll only have one 80x25 screen to look at...
> >
> > .config is attached.
>
> Ah this is x86-32, interesting! I'm going to try to boot that, we never know.
>
> Thanks a lot!
Happens on x86-64, too; I'm running that normally, but for testing,
32-bit kernel is easier.
Le 18/02/2017 à 14:06, Greg Kroah-Hartman a écrit :
> On Sat, Feb 18, 2017 at 09:35:36AM +0100, Romain Perier wrote:
>> The current PCI pool API are simple macro functions direct expanded to
>> the appropriated dma pool functions. The prototypes are almost the same
>> and semantically, they are v
On Fri, Feb 17, 2017 at 06:48:13PM +0100, Boris Petkov wrote:
> LGTM.
>
> Acked-by: me
Well, it looks good but actually trying it is a different story. For
example:
$ ./perf stat -e amd_nb/event=0xe0,umask=0x1f/ sleep 1
still says because argc is not 0.
So how about the below diff instead?
$
On Sat, Feb 18, 2017 at 6:28 PM, Dmitry Vyukov wrote:
> On Fri, Feb 10, 2017 at 6:17 AM, Jason Wang wrote:
>>
>>
>> On 2017年02月10日 02:10, Michael S. Tsirkin wrote:
>>>
>>> On Thu, Feb 09, 2017 at 05:02:31AM -0500, Jason Wang wrote:
- Original Message -
>
> Hello,
>
>
On 02/16/2017 02:57 PM, Russell King - ARM Linux wrote:
On Thu, Feb 16, 2017 at 02:27:41PM -0800, Steve Longerbeam wrote:
On 02/16/2017 02:20 PM, Russell King - ARM Linux wrote:
On Wed, Feb 15, 2017 at 06:19:02PM -0800, Steve Longerbeam wrote:
In version 4:
With this version, I get:
[28
On 02/18/2017 01:23 AM, Russell King - ARM Linux wrote:
On Fri, Feb 17, 2017 at 05:12:44PM -0800, Steve Longerbeam wrote:
Hi Russell,
I signed-off on this but after more review I'm not sure this is right.
The CSI-2 receiver really has no control over frame rate. It's output
frame rate is the
On Fri, Feb 10, 2017 at 6:17 AM, Jason Wang wrote:
>
>
> On 2017年02月10日 02:10, Michael S. Tsirkin wrote:
>>
>> On Thu, Feb 09, 2017 at 05:02:31AM -0500, Jason Wang wrote:
>>>
>>> - Original Message -
Hello,
I've got the following report while running syzkaller fuzzer on
On Sat, 18 Feb 2017, SIMRAN SINGHAL wrote:
>
>
> On Saturday, February 18, 2017 at 9:46:48 PM UTC+5:30, Julia Lawall wrote:
>
>
> On Sat, 18 Feb 2017, SIMRAN SINGHAL wrote:
>
> >
> >
> > On Saturday, February 18, 2017 at 9:00:51 PM UTC+5:30, Julia
> Lawall wrote:
>
The mmc host was added in meson_mmc_probe, but never removed in
meson_mmc_remove.
Fix that by removing the host before deallocating other resources.
Signed-off-by: Michał Zegan
Tested-by: Michał Zegan
---
changes since v1:
rebased on top of patchset at
https://patchwork.kernel.org/patch/9581057/
This patch fixes the case where meson_mmc_probe function fails before
core_clk is enabled.
Originally, that would result in a try to disable the core clock, and if
it was not already enabled, it would result in a kernel warning.
This issue is similar to the one with div_clk.
Fix it by introducing a
On Sat, 18 Feb 2017, Arushi Singhal wrote:
> The following patch the checkpatch.pl warning:
> drivers/staging/ks7010/ks_hostif.c warning: please, no spaces at the
> start of a line
>
> Signed-off-by: Arushi Singhal
> ---
> drivers/staging/ks7010/ks_hostif.c | 4 ++--
> 1 file changed, 2 insert
Unnecessary parentheses should be avoided as reported by checkpatch.pl
Signed-off-by: Arushi Singhal
---
drivers/staging/ks7010/ks_hostif.c | 24
drivers/staging/ks7010/ks_wlan_net.c | 20 ++--
2 files changed, 22 insertions(+), 22 deletions(-)
diff --
On Sat, Feb 18, 2017 at 10:10:02PM +0530, Arushi Singhal wrote:
> The following patch the checkpatch.pl warning:
> drivers/staging/ks7010/ks_hostif.c warning: please, no spaces at the
> start of a line
>
> Signed-off-by: Arushi Singhal
> ---
> drivers/staging/ks7010/ks_hostif.c | 4 ++--
> 1 fil
The following patch the checkpatch.pl warning:
drivers/staging/ks7010/ks_hostif.c warning: please, no spaces at the
start of a line
Signed-off-by: Arushi Singhal
---
drivers/staging/ks7010/ks_hostif.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/ks7010/
On Monday 30 January 2017 11:04 PM, Sudip Mukherjee wrote:
On Wednesday 18 January 2017 09:23 PM, Sudip Mukherjee wrote:
It is possible to update the backlight power and the brightness using
the sysfs and on writing it either returns the count or if the callback
function does not exist then retu
On Sat, 18 Feb 2017, SIMRAN SINGHAL wrote:
>
>
> On Saturday, February 18, 2017 at 9:00:51 PM UTC+5:30, Julia Lawall wrote:
>
>
> On Sat, 18 Feb 2017, simran singhal wrote:
>
> > Align * on a line, to conform to the kernal coding
> > style for block comments.
>
> It is no
On Fri, Feb 10, 2017 at 7:48 PM, Pau Espin Pedrol wrote:
> Otherwise those are printed several times when (un)plugging the AC
> connector.
If we have a user space tool that supports them better not to ignore.
--
With Best Regards,
Andy Shevchenko
On Sat, 18 Feb 2017, simran singhal wrote:
> Remove blank line after opening brace to fix the
> checkpatch issue blank lines aren't necessary after
> a open brace '{'.
>
> Signed-off-by: simran singhal
Acked-by: Julia Lawall
> ---
> drivers/staging/rtl8192e/rtl819x_HT.h | 2 --
> 1 file cha
This patch fixes the checkpatch.pl issue:
Please don't use multiple blank lines
Signed-off-by: simran singhal
---
drivers/staging/rtl8192e/rtl819x_HT.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HT.h
b/drivers/staging/rtl8192e/rtl819x_HT.h
index f53
Unnecessary parentheses as reported by checkpatch.pl
Signed-off-by: Arushi Singhal
---
drivers/staging/ks7010/ks_hostif.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/ks7010/ks_hostif.c
b/drivers/staging/ks7010/ks_hostif.c
index 1fbd
I'm announcing the release of the 4.4.50 kernel.
All users of the 4.4 kernel series must upgrade.
The updated 4.4.y git tree can be found at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
linux-4.4.y
and can be browsed at the normal kernel.org git web browser:
diff --git a/Makefile b/Makefile
index 5fab6d4068b5..10993715abb8 100644
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,6 @@
VERSION = 4
PATCHLEVEL = 4
-SUBLEVEL = 49
+SUBLEVEL = 50
EXTRAVERSION =
NAME = Blurry Fish Butt
diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c
b/drivers/net/etherne
diff --git a/Makefile b/Makefile
index d2fe757a979d..18b0c5adad3b 100644
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,6 @@
VERSION = 4
PATCHLEVEL = 9
-SUBLEVEL = 10
+SUBLEVEL = 11
EXTRAVERSION =
NAME = Roaring Lionus
diff --git a/arch/x86/kernel/fpu/core.c b/arch/x86/kernel/fpu/core.c
index ebb4e
I'm announcing the release of the 4.9.11 kernel.
All users of the 4.9 kernel series must upgrade.
The updated 4.9.y git tree can be found at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
linux-4.9.y
and can be browsed at the normal kernel.org git web browser:
On Sat, 18 Feb 2017, at 02:07 PM, Arun Raghavan wrote:
> This dumps some information in logs when a process exceeds its CPU or RT
> limits (soft and hard). Makes debugging easier when userspace triggers
> these limits.
>
> Signed-off-by: Arun Raghavan
> ---
> kernel/time/posix-cpu-timers.c | 1
Remove blank line after opening brace to fix the
checkpatch issue blank lines aren't necessary after
a open brace '{'.
Signed-off-by: simran singhal
---
drivers/staging/rtl8192e/rtl819x_HT.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HT.h
b/drivers/sta
Some of the block comments styles are not encouraged:
for example:
/*
block comment without introductory *
*/
and
/*
* block comment with line terminating */
Signed-off-by: Arushi Singhal
---
drivers/staging/ks7010/ks7010_sdio.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff -
Align * on a line, to conform to the kernal coding
style for block comments.
Signed-off-by: simran singhal
---
drivers/staging/rtl8192e/rtl819x_HT.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HT.h
b/drivers/staging/rtl8192e/rtl819x_HT.h
On Sat, 18 Feb 2017, simran singhal wrote:
> Align * on a line, to conform to the kernal coding
> style for block comments.
It is not clear what is means to align * on a line. What line?
What has been done is to add a space at the beginning of the line. But
then the line goes over 80 charact
Guys,
please fix the 0day bot reporting. See below for more info.
On Sat, Feb 18, 2017 at 01:01:53PM +0800, Fengguang Wu wrote:
> Greetings,
>
> 0day kernel testing robot got the below dmesg and the first bad commit is
>
> https://github.com/0day-ci/linux
> Borislav-Petkov/x86-Optimize-clear_p
On 02/17/2017 11:08 PM, Alexey Khoroshilov wrote:
> hisi_spi_nor_probe() ignores clk_prepare_enable() error code.
> The patch fixes that.
>
> Found by Linux Driver Verification project (linuxtesting.org).
>
> Signed-off-by: Alexey Khoroshilov
Acked-by: Marek Vasut
> ---
> drivers/mtd/spi-nor
On 18/02/2017 04:29, Andy Lutomirski wrote:
> There's no code here because the patch is trivial, but I want to run
> the idea by you all first to see if there are any issues.
>
> VMX is silly and forces the TSS limit to the minimum on VM exits. KVM
> wastes lots of cycles bumping it back up to
Hi Dmitry,
> Am 17.02.2017 um 23:56 schrieb Dmitry Torokhov :
>
> Do not manipulate evbit/keybit directly, use helper for that.
>
> Signed-off-by: Dmitry Torokhov
> ---
> drivers/input/touchscreen/tsc2007_core.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/
On Sat, Feb 18, 2017 at 10:39:17AM +0100, Pavel Machek wrote:
> Hi!
>
> [I droped some CCs here, you may want to check the CC list].
Good!
>
> > > These are different bugs.
> > >
> > > On x60, I see failures doing hotplug/unplug in a loop, or lot of
> > > suspends. Someone seen it in v4.8-stab
Hi Ansuel,
On Fri, Feb 17, 2017 at 09:42:17PM +0100, Ansuel Smith wrote:
> In others board we have the sata led set to funcion with the sata led trigger
> by default.
> This patch makes the same for these board that have sata led but disabled by
> not associating it to any trigger
Please line w
Hi Bartosz,
[auto build test ERROR on arm-soc/for-next]
[also build test ERROR on next-20170217]
[cannot apply to v4.10-rc8]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Bartosz-Golaszewski/AR
On Sat, 18 Feb 2017, Arushi Singhal wrote:
> function definition argument 'struct vb_device_info *' and 'unsigned
> long' should also have an identifier name.
>
> Signed-off-by: Arushi Singhal
Acked-by: Julia Lawall
> ---
> drivers/staging/xgifb/vb_init.h | 3 +--
> 1 file changed, 1 insert
On Sat, 18 Feb 2017, simran singhal wrote:
> This patch removes the explicit NULL comparison. This issue was found by
> checkpatch.
>
> Signed-off-by: simran singhal
Acked-by: Julia Lawall
> ---
>
> v2:
>-Modified commit message
>
> drivers/staging/rtl8192e/rtl819x_BAProc.c | 6 +++---
function definition argument 'struct vb_device_info *' and 'unsigned
long' should also have an identifier name.
Signed-off-by: Arushi Singhal
---
drivers/staging/xgifb/vb_init.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/xgifb/vb_init.h b/drivers/stagin
This patch removes the explicit NULL comparison. This issue was found by
checkpatch.
Signed-off-by: simran singhal
---
v2:
-Modified commit message
drivers/staging/rtl8192e/rtl819x_BAProc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8192e/r
function definition argument 'struct vb_device_info *' and 'unsigned
long' should also have an identifier name.
Signed-off-by: Arushi Singhal
---
drivers/staging/xgifb/vb_init.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/xgifb/vb_init.h b/drivers/stagin
m/0day-ci/linux/commits/Eddie-Kovsky/provide-check-for-ro_after_init-memory-sections/20170218-141040
:: branch date: 2 hours ago
:: commit date: 2 hours ago
>> drivers/hv/vmbus_drv.c:1058:2-45: code aligned with following code on line
>> 1059
drivers/hv/vmbus_drv.c:1129:2-45:
On Sat, 18 Feb 2017, Arushi Singhal wrote:
> function definition argument 'struct vb_device_info *' and 'unsigned
> long' should also have an identifier name.
This could be a bit more clear as "Add variable names to function
prototype parameter list."
>
> Signed-off-by: Arushi Singhal
> ---
>
function definition argument 'struct vb_device_info *' and 'unsigned
long' should also have an identifier name.
Signed-off-by: Arushi Singhal
---
drivers/staging/xgifb/vb_init.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/xgifb/vb_init.h b/drivers/staging/
On Sat, 18 Feb 2017, simran singhal wrote:
> This patch corrects check generated by checkpatch.pl by
There is not much information in the first line. Use the imperative:
Replace comparisons to NULL by ...
> replacing comparison to null statements with equivalent
> statements in the form of "
On Sat, Feb 18, 2017 at 09:35:36AM +0100, Romain Perier wrote:
> The current PCI pool API are simple macro functions direct expanded to
> the appropriated dma pool functions. The prototypes are almost the same
> and semantically, they are very similar. I propose to use the DMA pool
> API directly a
On Sat, 18 Feb 2017, simran singhal wrote:
> This patch renames pBA to pba to avoid camelcase.
I think that this was tried in the last round and was rejected, because BA
is meaningful. You also see BA in the name of a header file. The kind of
camel case that one should remove is whenTextIsWritt
On Sat, Feb 18, 2017 at 11:25:19AM +0100, Maarten Brock wrote:
> Hello Greg,
>
> Does that also mean that this isn't possible in the sc16is7xx.c driver in
> sc16is7xx_spi_probe() line 1358 and sc16is7xx_i2c_probe() line 1419 ?
>
> If so, should these checks be removed?
Probably.
This patch renames pBA to pba to avoid camelcase.
Signed-off-by: simran singhal
---
drivers/staging/rtl8192e/rtl819x_BAProc.c | 100 +++---
1 file changed, 50 insertions(+), 50 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_BAProc.c
b/drivers/staging/rtl8192
On Sat, Feb 18, 2017 at 09:35:37AM +0100, Romain Perier wrote:
> The PCI pool API is deprecated. This commits replaces the PCI pool old
> API by the appropriated function with the DMA pool API.
>
no new errors added, tested by compilation only.
> Signed-off-by: Romain Perier
> Acked-by: Peter S
On February 17, 2017 6:30 PM, Moritz Fischer wrote:
On Fri, Feb 17, 2017 at 04:28:37PM -0600, Yves Vandervennet wrote:
> Moritz,
>
> whatever solution we decide to go with has to work with other OS'es.
> The last thing we want to do is to have wrappers that are Linux specific.
I do agree tha
Hi Philipp and Russell,
On Fri, Feb 17, 2017 at 04:04:30PM +0100, Philipp Zabel wrote:
> On Fri, 2017-02-17 at 14:22 +0200, Sakari Ailus wrote:
> > Hi Philipp, Steve and Russell,
> >
> > On Fri, Feb 17, 2017 at 12:43:38PM +0100, Philipp Zabel wrote:
> > > On Thu, 2017-02-16 at 14:27 -0800, Steve
'qedr_alloc_pbl_tbl()' can not return NULL.
In qedr_init_user_queue():
- simplify the test for the return value, no need to test for NULL
- propagate the error pointer if needed, otherwise 0 (success) is returned.
This is spurious.
In init_mr_info():
- test the return value with IS_ERR
- p
> Am 17.02.2017 um 22:27 schrieb Dmitry Torokhov :
>
> On Fri, Dec 30, 2016 at 08:36:30PM +, Jonathan Cameron wrote:
>> On 28/12/16 14:53, H. Nikolaus Schaller wrote:
>>> The tsc2007 chip not only has a resistive touch screen controller but
>>> also an external AUX adc imput which can be used
Hi Sebastian,
> Am 18.02.2017 um 04:22 schrieb Sebastian Reichel :
>
> Hi,
>
> On Fri, Feb 17, 2017 at 12:40:41PM -0800, Dmitry Torokhov wrote:
>>> AFAIK there is no mainline board using the DT except ours (and the upcoming
>>> OMAP5-Pyra), so we shouldn't care too much. If you prefer, you can r
Hi Pavel,
> Am 18.02.2017 um 10:15 schrieb Pavel Machek :
>
> Hi!
>
>>> Well, there is some historic confusion wether this driver reports resistance
>>> or pressure.
>>>
>>> The unpatched tsc2007 driver does it wrong (please test!) and we fix it on
>>> the fly (because a separate patch is much
Hi Dmitry,
> Am 17.02.2017 um 21:40 schrieb Dmitry Torokhov :
>
> Hi Nikolaus,
>
> On Sat, Jan 28, 2017 at 10:44:35PM +0100, H. Nikolaus Schaller wrote:
>> Hi Dmitry,
>>
>>> Am 28.01.2017 um 20:33 schrieb Dmitry Torokhov :
>>>
>>> Hi Nikolaus,
>>>
>>> On Wed, Dec 28, 2016 at 03:53:16PM +0100,
From: Borislav Petkov
Update to the new file paths, remove them from introductory comments.
Signed-off-by: Borislav Petkov
---
arch/x86/events/amd/core.c | 2 +-
arch/x86/events/intel/cstate.c | 2 +-
arch/x86/events/intel/rapl.c
Hi Dmitry,
> Am 17.02.2017 um 21:43 schrieb Dmitry Torokhov :
>
> On Wed, Feb 15, 2017 at 03:03:07PM +0100, H. Nikolaus Schaller wrote:
>> Hi Dmitry,
>>
>>> Am 12.02.2017 um 16:18 schrieb H. Nikolaus Schaller :
>>>
>>> Hi Dmitry,
>>>
Am 28.01.2017 um 19:16 schrieb H. Nikolaus Schaller :
>
On Friday 17 February 2017 23:38:12 Finn Thain wrote:
> On Thu, 16 Feb 2017, Ondrej Zary wrote:
> > On Tuesday 31 January 2017 02:31:45 Finn Thain wrote:
> > [...]
> >
> > > Are you trying to figure out which commands are going to disconnect
> > > during a transfer? This is really a function of the
Hello Greg,
Does that also mean that this isn't possible in the sc16is7xx.c driver
in
sc16is7xx_spi_probe() line 1358 and sc16is7xx_i2c_probe() line 1419 ?
If so, should these checks be removed?
Kind regards,
Maarten
On 2017-01-30 20:37, Greg Kroah-Hartman wrote:
On Fri, Jan 27, 2017 at 04:
'pktdma_get_regs()' never returns NULL. It returns an error pointer or a
valid pointer.
So test its return value with IS_ERR.
Signed-off-by: Christophe JAILLET
---
After checking with IS_ERR, it could be better to propagate the error code
instead of returning -ENODEV uncondionnaly
v2: fix typo l
-handling/20170218-164533
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
wget
https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross
-O ~/bin/make.cross
chmod +x ~/bin/make.cross
Commit-ID: 336a9cde10d641e70bac67d90ae91b3190c3edca
Gitweb: http://git.kernel.org/tip/336a9cde10d641e70bac67d90ae91b3190c3edca
Author: Marc Zyngier
AuthorDate: Fri, 15 Jan 2016 17:41:09 +
Committer: Thomas Gleixner
CommitDate: Sat, 18 Feb 2017 10:58:39 +0100
hrtimer: Catch invalid
101 - 200 of 225 matches
Mail list logo