From: Surender Polsani
Fixed the following checkpatch.pl warnings:
octal permissions are more preferable than symbolic permissions
Signed-off-by: Surender Polsani
---
drivers/staging/iio/light/tsl2x7x_core.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --g
Several atomisp files use:
ccflags-y += -Werror
As, on media, our usual procedure is to use W=1, and atomisp
has *a lot* of warnings with such flag enabled,like:
./drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_common/host/system_local.h:62:26:
warning: 'DDR_BASE' defin
On Thu, May 18, 2017 at 10:45 AM, Mauro Carvalho Chehab
wrote:
> Several atomisp files use:
> ccflags-y += -Werror
>
> As, on media, our usual procedure is to use W=1, and atomisp
> has *a lot* of warnings with such flag enabled,like:
>
> ./drivers/staging/media/atomisp/pci/atomisp2/css24
On Thu, May 18, 2017 at 02:06:29PM +0530, surenderpols...@gmail.com wrote:
> From: Surender Polsani
>
> Fixed the following checkpatch.pl warnings:
> octal permissions are more preferable than symbolic permissions
>
> Signed-off-by: Surender Polsani
> ---
> drivers/staging/iio/light/tsl2x7x_co
Em Thu, 18 May 2017 11:05:24 +0200
Arnd Bergmann escreveu:
> On Thu, May 18, 2017 at 10:45 AM, Mauro Carvalho Chehab
> wrote:
> > Several atomisp files use:
> > ccflags-y += -Werror
> >
> > As, on media, our usual procedure is to use W=1, and atomisp
> > has *a lot* of warnings with suc
Fixed a trailing */ issue.
Signed-off-by: srishti
---
drivers/staging/bcm2835-audio/bcm2835-ctl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/bcm2835-audio/bcm2835-ctl.c
b/drivers/staging/bcm2835-audio/bcm2835-ctl.c
index a4ffa1b..38fabab 100644
--- a
On Thu, May 18, 2017 at 03:37:48PM +0530, srishti wrote:
> Fixed a trailing */ issue.
>
> Signed-off-by: srishti
Fix your From header and your signed-off-by line to use your legal name
as if you were signing a legal document.
regards,
dan carpenter
_
From: srishti sharma
Fixed a trailing */ issue.
Signed-off-by: srishti sharma
---
drivers/staging/bcm2835-audio/bcm2835-ctl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/bcm2835-audio/bcm2835-ctl.c
b/drivers/staging/bcm2835-audio/bcm2835-ctl.c
index
From: srishti sharma
Fixed a trailing */ issue.
Signed-off-by: srishti sharma
---
drivers/staging/bcm2835-audio/bcm2835-ctl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/bcm2835-audio/bcm2835-ctl.c
b/drivers/staging/bcm2835-audio/bcm2835-ctl.c
index
Enabling the arche platform for compile testing showed a harmless
warning with CONFIG_PM=n:
drivers/staging/greybus/arche-platform.c:632:12: error: 'arche_platform_resume'
defined but not used [-Werror=unused-function]
drivers/staging/greybus/arche-platform.c:618:12: error:
'arche_platform_suspe
On Thu, May 18, 2017 at 03:28:00PM +0200, Arnd Bergmann wrote:
> Enabling the arche platform for compile testing showed a harmless
> warning with CONFIG_PM=n:
>
> drivers/staging/greybus/arche-platform.c:632:12: error:
> 'arche_platform_resume' defined but not used [-Werror=unused-function]
> dri
On Mon, May 08, 2017 at 11:25:55PM +0100, Mark Railton wrote:
> Cleared up some errors and warnings in
> drivers/staging/media/atomisp/i2c/ap1302.c
>
> Signed-off-by: Mark Railton
> ---
> drivers/staging/media/atomisp/i2c/ap1302.c | 83
> ++
> 1 file changed, 50 inse
Hi Mauro,
Here's the set of accumulated atomisp staging patches that I had in my
to-review mailbox. After this, my queue is empty, the driver is all
yours! Good Luck! :)
thanks,
greg k-h
Avraham Shukron (3):
staging: media: atomisp: fixed sparse warnings
staging: media: atomisp: fixed cod
From: Joe Perches
__printf validation adds format and argument validation.
Fix the various broken format/argument mismatches.
Signed-off-by: Joe Perches
Signed-off-by: Greg Kroah-Hartman
---
.../isp/kernels/sdis/sdis_1.0/ia_css_sdis.host.c | 6 +++---
.../isp/kernels/sdis/sdis_2/ia_css
From: Fabrizio Perria
Fix checkpatch warning: removed unnecessary initialization of
static variable "skip_fwload" to 0 in source atomisp_v4l2.c
Signed-off-by: Fabrizio Perria
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/media/atomisp/pci/atomisp2/atomisp_v4l2.c | 2 +-
1 file changed
From: Valentin Vidic
Fixes a sparse warning:
drivers/staging/media/atomisp/platform/intel-mid/intel_mid_pcihelpers.c:35:5:
warning: symbol 'qos' was not declared. Should it be static?
Signed-off-by: Valentin Vidic
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/media/atomisp/platform/
From: Guru Das Srinagesh
Fix sparse warnings: "symbol not declared; should it be static?"
Signed-off-by: Guru Das Srinagesh
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/media/atomisp/pci/atomisp2/atomisp_fops.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
dif
From: Dan Carpenter
We're putting the NUL terminators one space beyond where they belong.
This doesn't show up in testing because all but the callers put a NUL in
the correct place themselves. LOL. It causes a static checker warning
about buffer overflows.
Fixes: a49d25364dfb ("staging/atomisp
From: Avraham Shukron
Fix for error (not warnings) reported by checkpatch.pl
Specifically:
- missing whitespace around "=" and after ","
- indentation with spaces instead of tabs
- lines starting with a whitespace
This patch does not affect the compiled code in any way.
Signed-off-by: Avraha
From: Avraham Shukron
Added "static" storage class to 4 not-declared functions
Signed-off-by: Avraham Shukron
Signed-off-by: Greg Kroah-Hartman
---
.../media/atomisp/platform/intel-mid/atomisp_gmin_platform.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git
a/dr
From: Manny Vindiola
This is a patch to the atomisp_tpg.c file that fixes up a missing
blank line warning found by the checkpatch.pl tool
Signed-off-by: Manny Vindiola
Signed-off-by: Greg Kroah-Hartman
---
drivers/staging/media/atomisp/pci/atomisp2/atomisp_tpg.c | 1 +
1 file changed, 1 inser
From: Dan Carpenter
We should verify that "ix < max_len" before we test whether we have
reached the NUL terminator.
Fixes: a49d25364dfb ("staging/atomisp: Add support for the Intel IPU v2")
Reported-by: David Binderman
Signed-off-by: Dan Carpenter
Signed-off-by: Greg Kroah-Hartman
---
.../pc
From: Avraham Shukron
Fix for warnings reported by checkpatch.pl:
- Multiline comment style
- Bare "unsigned"
- Missing blank line after declarations
- Un-needed braces around single-statement branch
Signed-off-by: Avraham Shukron
Signed-off-by: Greg Kroah-Hartman
---
.../platform/intel-m
From: Guru Das Srinagesh
Fixes sparse warning "dubious: x & !y" in logical expression.
Signed-off-by: Guru Das Srinagesh
Signed-off-by: Greg Kroah-Hartman
---
.../media/atomisp/pci/atomisp2/css2400/runtime/binary/src/binary.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --
From: Mauro Carvalho Chehab
Several atomisp files use:
ccflags-y += -Werror
As, on media, our usual procedure is to use W=1, and atomisp
has *a lot* of warnings with such flag enabled,like:
./drivers/staging/media/atomisp/pci/atomisp2/css2400/hive_isp_css_common/host/system_local.h:62:
From: Hans de Goede
With gcc-7.1.1 I was getting the following compile error:
error: ‘*’ in boolean context, suggest ‘&&’ instead
The problem is the definition of CEIL_DIV:
#define CEIL_DIV(a, b) ((b) ? ((a) + (b) - 1) / (b) : 0)
Which when called as: CEIL_DIV(x, y * z) triggers this er
On Thu, May 04, 2017 at 12:13:38PM -0400, Mathias Rav wrote:
> Prefer kstrtouint_from_user to copy_from_user+simple_strtoul.
>
> The helper function lprocfs_wr_uint() is only used to implement
> "dump_granted_max" in debugfs.
>
> Note the slight change in semantics: The previous implementation us
On Thu, May 11, 2017 at 09:17:07PM +1000, Sergiy Redko wrote:
> Fixed incorrect type in assignment warning found by sparse
> by changing the type of a local variable
>
> Signed-off-by: Sergiy Redko
> ---
> drivers/staging/lustre/lnet/lnet/lib-socket.c | 2 +-
> 1 file changed, 1 insertion(+), 1
On Tue, May 16, 2017 at 11:20:18AM +0200, Matej Dujava wrote:
> This patch adds tabs into macro definitions so all rhs are on same column.
> Move MHz macro from ddk_chip.c to ddk_chip.h.
>
> Signed-off-by: Matej Dujava
> ---
> drivers/staging/sm750fb/ddk750_chip.c| 18
> drivers/sta
On Thu, May 18, 2017 at 04:20:15PM +0530, srishti wrote:
> From: srishti sharma
>
> Fixed a trailing */ issue.
>
> Signed-off-by: srishti sharma
> ---
> drivers/staging/bcm2835-audio/bcm2835-ctl.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
You sent two copies of this, which a
Em Thu, 18 May 2017 15:50:09 +0200
Greg Kroah-Hartman escreveu:
> Hi Mauro,
>
> Here's the set of accumulated atomisp staging patches that I had in my
> to-review mailbox. After this, my queue is empty, the driver is all
> yours!
Thanks!
Alan, please let me know if you prefer if I don't apply
On 18-05-17, 15:28, Arnd Bergmann wrote:
> Enabling the arche platform for compile testing showed a harmless
> warning with CONFIG_PM=n:
>
> drivers/staging/greybus/arche-platform.c:632:12: error:
> 'arche_platform_resume' defined but not used [-Werror=unused-function]
> drivers/staging/greybus/a
On May 18, 2017, at 15:53, Greg Kroah-Hartman
wrote:
>
> On Thu, May 04, 2017 at 12:13:38PM -0400, Mathias Rav wrote:
>> Prefer kstrtouint_from_user to copy_from_user+simple_strtoul.
>>
>> The helper function lprocfs_wr_uint() is only used to implement
>> "dump_granted_max" in debugfs.
>>
>> N
On Thu, May 18, 2017 at 4:18 PM, Viresh Kumar wrote:
> On 18-05-17, 15:28, Arnd Bergmann wrote:
>> Enabling the arche platform for compile testing showed a harmless
>> warning with CONFIG_PM=n:
>>
>> drivers/staging/greybus/arche-platform.c:632:12: error:
>> 'arche_platform_resume' defined but no
On Thu, 18 May 2017 14:48:25 +
"Dilger, Andreas" wrote:
> On May 18, 2017, at 15:53, Greg Kroah-Hartman
> wrote:
> >
> > On Thu, May 04, 2017 at 12:13:38PM -0400, Mathias Rav wrote:
> >> Prefer kstrtouint_from_user to copy_from_user+simple_strtoul.
> >>
> >> The helper function lprocfs_
-- Forwarded message --
From: srishti sharma
Date: Thu, May 18, 2017 at 9:06 PM
Subject: Re: [PATCH] Staging: bcm2835-audio: bcm2835_ctl.c: Fixed a
comment coding style issue
To: Greg KH
On Thu, May 18, 2017 at 7:28 PM, Greg KH wrote:
> On Thu, May 18, 2017 at 04:20:15PM +0530,
These are refactoring changes to the Hyper-V scsi driver.
Stephen Hemminger (2):
storvsc: use in place iterator function
storvsc: remove unnecessary channel inbound lock
drivers/hv/channel_mgmt.c | 1 -
drivers/scsi/storvsc_drv.c | 52 --
include
In 4.12-rc1, new functions were added to support iterating
over elements in the vmbus event ring. This patch uses them to
simplify the ring buffer handling in virtual SCSI driver as well.
Signed-off-by: Stephen Hemminger
---
drivers/scsi/storvsc_drv.c | 44 +++
In storvsc driver, inbound messages do not go through inbound lock.
The only effect of this lock was is to provide a barrier for
connect and remove logic.
Signed-off-by: Stephen Hemminger
---
drivers/hv/channel_mgmt.c | 1 -
drivers/scsi/storvsc_drv.c | 8 +++-
include/linux/hyperv.h |
This set of patches cleans up the vmbus infrastructure.
The introduction of the vmbus iterator in 4.12-rc1 allows
for other simplifications. And the lock on ring buffer read
can be replaced by a better algorithm. Unused and duplicated
functions can be eliminated.
Stephen Hemminger (5):
vmbus: si
This function is not used anywhere in current code.
Signed-off-by: Stephen Hemminger
---
drivers/hv/channel.c | 58 +-
include/linux/hyperv.h | 6 --
2 files changed, 1 insertion(+), 63 deletions(-)
diff --git a/drivers/hv/channel.c b/drive
vmbus_sendpacket and vmbus_sendpacket_ctl were just synonyms for
the same function.
Signed-off-by: Stephen Hemminger
---
drivers/hv/channel.c| 29 +++--
drivers/net/hyperv/netvsc.c | 8 +++-
include/linux/hyperv.h | 16
3 files changed,
Use a reservation similar to ftrace to make vmbus ring buffer writes
lock free.
The algorithm uses cmpxchg to atomically reserve an area in the ring
buffer. Then the data is copied into the ring, and the updates to the
head of the ring are ordered.
Other similar implementions are FreeBSD buf_ring
With new iterator functions (and the double mapping) the ring buffer
read function can be greatly simplified.
Signed-off-by: Stephen Hemminger
---
drivers/hv/ring_buffer.c | 118 +++
1 file changed, 17 insertions(+), 101 deletions(-)
diff --git a/driv
There were two instances for vmbus_sendpacket_pagebuffer_ctl
and vmbus_sendpacket_pagebuffer. Go with simpler name and get
rid of wrapper.
Signed-off-by: Stephen Hemminger
---
drivers/hv/channel.c| 28 +---
drivers/net/hyperv/netvsc.c | 14 +++---
include/
On Wed, 2017-05-17 at 21:48 -0400, Manny Vindiola wrote:
> This is a patch to the atomisp_tpg.c file that fixes up a missing
> blank line warning found by the checkpatch.pl tool
>
> Signed-off-by: Manny Vindiola
> ---
> drivers/staging/media/atomisp/pci/atomisp2/atomisp_tpg.c | 1 +
> 1 file cha
On Thu, 2017-05-18 at 11:10 -0300, Mauro Carvalho Chehab wrote:
> Em Thu, 18 May 2017 15:50:09 +0200
> Greg Kroah-Hartman escreveu:
>
> >
> > Hi Mauro,
> >
> > Here's the set of accumulated atomisp staging patches that I had in
> > my
> > to-review mailbox. After this, my queue is empty, the d
From: K. Y. Srinivasan
Some miscellaneous fixes.
Andy Shevchenko (1):
vmbus: Reuse uuid_le_to_bin() helper
K. Y. Srinivasan (2):
Drivers: hv: vmbus: Get the current time from the current clocksource
drivers: hv: vmbus: Increase the time between retries in
vmbus_post_msg()
Michael Kel
From: Michael Kelley
Extend the disabling of preemption to include the hypercall so that
another thread can't get the CPU and corrupt the per-cpu page used
for hypercall arguments.
Cc: #4.11
Signed-off-by: Michael Kelley
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/hv.c |7 ++-
1 f
From: Vitaly Kuznetsov
Turns out that our implementation of .getcrosststamp() never actually
worked. Hyper-V is sending time samples every 5 seconds and this is
too much for get_device_system_crosststamp() as it's interpolation
algorithm (which nobody is currently using in kernel, btw) accounts
f
From: K. Y. Srinivasan
The current code uses the MSR based mechanism to get the current tick.
Use the current clock source as that might be more optimal.
Signed-off-by: K. Y. Srinivasan
---
arch/x86/include/asm/mshyperv.h |1 -
drivers/hv/hv.c |2 +-
2 files changed, 1
From: Vitaly Kuznetsov
It was found that ICTIMESYNCFLAG_SYNC packets are handled incorrectly
on WS2012R2, e.g. after the guest is paused and resumed its time is set
to something different from host's time. The problem is that we call
adj_guesttime() with reftime=0 for these old hosts and we don't
From: K. Y. Srinivasan
Commit c0bb03924f1a ("Drivers: hv: vmbus: Raise retry/wait limits in
vmbus_post_msg()") increased the retry/wait limits of vmbus_post_msg()
to address the new DoS protection policies in WS2016.
Increase the time between retries to make the
code more robust.
Signed-off-by:
From: Andy Shevchenko
Instead of open coded variant use generic helper to convert UUID strings
to binary format.
Signed-off-by: Andy Shevchenko
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/vmbus_drv.c | 51 +--
1 files changed, 10 insertions(+),
Update the Hyper-V vPCI driver to use the Server-2016 version
of the vPCI protocol, fixing MSI creation and retargeting issues.
[PATCH 1/4] Hyper-V vPCI: Minor format and semantic fix
[PATCH 2/4] Hyper-V vPCI: Use page allocation for hbus structure
[PATCH 3/4] Hyper-V vPCI: Add vPCI version protoc
From: Jork Loeser
The hv_pcibus_device structure contains an in-memory hypercall argument
that must not cross a page boundary. Allocate the structure as a page
to ensure that.
Signed-off-by: Jork Loeser
---
drivers/pci/host/pci-hyperv.c | 12 +---
1 files changed, 9 insertions(+), 3
From: Jork Loeser
Fix comment formatting and use proper integer fields.
Signed-off-by: Jork Loeser
---
drivers/pci/host/pci-hyperv.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyperv.c
index 8493638..7b
From: Jork Loeser
Hyper-V vPCI offers different protocol versions. This patch creates the
the infra for negotiating the one to use.
Signed-off-by: Jork Loeser
---
drivers/pci/host/pci-hyperv.c | 72 +---
1 files changed, 52 insertions(+), 20 deletions(-)
From: Jork Loeser
Update the Hyper-V vPCI driver to use the Server-2016 version
of the vPCI protocol, fixing MSI creation and retargeting issues.
Signed-off-by: Jork Loeser
---
arch/x86/include/uapi/asm/hyperv.h |6 +
drivers/pci/host/pci-hyperv.c | 297 ++
Fixes checkpatch warning:
WARNING: please, no space before tabs
Signed-off-by: Branislav Katreniak
---
drivers/staging/ccree/cc_crypto_ctx.h | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/ccree/cc_crypto_ctx.h
b/drivers/staging/ccree/cc_cr
>From c93e5f16090b217d76c0c0b94201a97e0c43c32b Mon Sep 17 00:00:00 2001
From: Lior Pugatch
Date: Thu, 18 May 2017 22:49:12 +0300
Subject: [PATCH] Fixing sparse warning of variable length array
Signed-off-by: Lior Pugatch
---
drivers/staging/lustre/lustre/llite/xattr.c | 4 +++-
1 file changed,
On Thu, May 18, 2017 at 11:39:48PM +0300, Lior Pugatch wrote:
> >From c93e5f16090b217d76c0c0b94201a97e0c43c32b Mon Sep 17 00:00:00 2001
> From: Lior Pugatch
> Date: Thu, 18 May 2017 22:49:12 +0300
> Subject: [PATCH] Fixing sparse warning of variable length array
>
> Signed-off-by: Lior Pugatch
>
On Thu, May 18, 2017 at 11:39:48PM +0300, Lior Pugatch wrote:
> >From c93e5f16090b217d76c0c0b94201a97e0c43c32b Mon Sep 17 00:00:00 2001
> From: Lior Pugatch
> Date: Thu, 18 May 2017 22:49:12 +0300
> Subject: [PATCH] Fixing sparse warning of variable length array
>
> Signed-off-by: Lior Pugatch
>
On Thu, May 18, 2017 at 05:31:20PM +0100, Alan Cox wrote:
> On Wed, 2017-05-17 at 21:48 -0400, Manny Vindiola wrote:
> > This is a patch to the atomisp_tpg.c file that fixes up a missing
> > blank line warning found by the checkpatch.pl tool
> >
> > Signed-off-by: Manny Vindiola
> > ---
> > driv
Fix warning issued by sparse:
symbol 'ion_device_create' was not declared. Should it be static?
Signed-off-by: Paolo Cretaro
---
drivers/staging/android/ion/ion.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/android/ion/ion.c
b/drivers/staging/android/ion/
During initialization, the channel initialization code schedules the
tasklet to scan the VMBUS receive event page (i.e. simulates an
interrupt). The problem was that it invokes the tasklet on a different
CPU from where it normally runs and therefore if an event is present,
it will clear the bit but
On Thu, 18 May 2017 12:14:30 -0700
Jork Loeser wrote:
> From: Jork Loeser
>
> Update the Hyper-V vPCI driver to use the Server-2016 version
> of the vPCI protocol, fixing MSI creation and retargeting issues.
>
> Signed-off-by: Jork Loeser
This patch conflicts with already submitted patch tha
Stephen,
> These are refactoring changes to the Hyper-V scsi driver.
Applied to 4.13/scsi-queue. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/m
Jonathan,
Yes I see it in the testing branch, thank you for letting me know.
Thanks,
Harinath
On Tue, May 16, 2017 at 2:24 PM, Jonathan Cameron wrote:
> On 15/05/17 05:38, harinath Nampally wrote:
>>
>> Jonathan,
>>
>> Thank you for letting me know about correct iio git tree.
>>
>> I checked ou
> -Original Message-
> From: Stephen Hemminger [mailto:step...@networkplumber.org]
> Sent: Thursday, May 18, 2017 16:59
>
> > From: Jork Loeser
> >
> > Update the Hyper-V vPCI driver to use the Server-2016 version of the
> > vPCI protocol, fixing MSI creation and retargeting issues.
> >
>
This patchset fixes two style issues in lprocfs_status.c related to
simple_strtoul and seq_printf (reported by checkpatch).
v1->v2:
- Patch 1/2 now completely removes lprocfs_{rd,wr}_uint
- Patch 2/2 unchanged
Mathias Rav (2):
staging: lustre: Use kstrtouint_from_user in ldlm_rw_uint
Clean up the helper functions used to implement "dump_granted_max" in
debugfs.
Replace the lprocfs_rd_uint() and lprocfs_wr_uint() generic callbacks
with a simpler, more direct implementation of ldlm_rw_uint_fops.
There's a slight change in lustre debugfs write semantics: Using kstrtox
causes EIN
Reported by checkpatch.pl: "WARNING: Prefer seq_puts to seq_printf".
Signed-off-by: Mathias Rav
---
.../staging/lustre/lustre/obdclass/lprocfs_status.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/lustre/lustre/obdclass/lprocfs_statu
On May 18, 2017, at 17:13, Mathias Rav wrote:
>
> On Thu, 18 May 2017 14:48:25 +
> "Dilger, Andreas" wrote:
>
>> On May 18, 2017, at 15:53, Greg Kroah-Hartman
>> wrote:
>>>
>>> On Thu, May 04, 2017 at 12:13:38PM -0400, Mathias Rav wrote:
Prefer kstrtouint_from_user to copy_from_us
On 18-05-17, 16:51, Arnd Bergmann wrote:
> I find that a lot of users get the #ifdef wrong, either using the wrong
> macro (CONFIG_PM vs CONFIG_PM_SLEEP) or not using the right
> set of functions (e.g. calling a function only from the suspend handler).
>
> The __maybe_unused annotation avoids both
On Fri, May 19, 2017 at 5:40 AM, Viresh Kumar wrote:
> On 18-05-17, 16:51, Arnd Bergmann wrote:
>> I find that a lot of users get the #ifdef wrong, either using the wrong
>> macro (CONFIG_PM vs CONFIG_PM_SLEEP) or not using the right
>> set of functions (e.g. calling a function only from the suspe
76 matches
Mail list logo
