The qtree_fmt_operations structures are never modified, so declare them as
const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
fs/ocfs2/quota.h|2 +-
fs/ocfs2/quota_global.c |2 +-
fs/quota/quota_v2.c |4 ++--
include/linux/dqblk_qtree.h
On Tue, Dec 22, 2015 at 8:23 PM, wrote:
> Hi,
>
> following is a revised set of the CGroup Namespace patchset which Aditya
> Kali has previously sent. The code can also be found in the cgroupns.v8
> branch of
>
> https://git.kernel.org/cgit/linux/kernel/git/sergeh/linux-security.git/
Happy New
On Mon, Dec 28, 2015 at 08:35:12AM +0900, Minchan Kim wrote:
> During my compaction-related stuff, I encountered a bug
> with ballooning.
>
> With repeated inflating and deflating cycle, guest memory(
> ie, cat /proc/meminfo | grep MemTotal) is decreased and
> couldn't be recovered.
>
> The reaso
Even if an invalid gpio number is sent to gpio_to_desc() we try to find
it in the list before checking if it is invalid. We can check it first
so that it will save us from traversing the list incase of invalid gpio.
Signed-off-by: Sudip Mukherjee
---
drivers/gpio/gpiolib.c | 8 +---
1 file c
Hi all,
Same here on rockchip.
See "[PATCH] stmmac: Don't exit mdio registration when mdio subnode is
not found in the DTS"
Regards,
Romain
2015-12-18 18:45 GMT+01:00 Dinh Nguyen :
> Hi,
>
> It appears that commit e34d65696d2e 'stmmac: create of compatible mdio
> bus for
> stmmac driver' is caus
These lu_device_operations and cl_device_operations structures are never
modified, so declare them as const. Other structures of these types are
already const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
drivers/staging/lustre/lustre/obdecho/echo_client.c |4 ++--
1
On Fri, Jan 01, 2016 at 12:19:44AM -0800, Dan Williams wrote:
> On Tue, Dec 22, 2015 at 8:23 PM, wrote:
> > Hi,
> >
> > following is a revised set of the CGroup Namespace patchset which Aditya
> > Kali has previously sent. The code can also be found in the cgroupns.v8
> > branch of
> >
> > https
This kset_uevent_ops structure is never modified, so declare it as const.
Other structures of this type are already const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
fs/dlm/lockspace.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/dlm/lockspac
On Fri, 1 Jan 2016, Mike Galbraith wrote:
> On Thu, 2015-12-31 at 10:24 -0600, Clark Williams wrote:
> > I pulled this update and tried it on my laptop (i7 quad-core with HT)
> > and an Atom testbox. I'm seeing a change in the cpu utilization of
> > ksoftirqd between 4.1.15-rt17 and 4.4-rc2-rt1, wh
This export_operations structure is never modified, so declare it as const.
Most other structures of this type are already const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
drivers/staging/lustre/lustre/llite/llite_internal.h |2 +-
drivers/staging/lustre/lustre/llit
This export_operations structure is never modified, so declare it as const.
Most other structures of this type are already const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall
---
Mike, I saw your name on all of the patches for this driver, but you aren't
in the MAINTAINERS fil
Thanks for your response. I didn't ask tool to analyse. I asked about call flow.
Like how they are communicating between User space to kernel, cfg80211 to
mac80211,
mac80211 to driver and driver to hardware.
Thanks,
Ramesh
- Original Message -
From: "Paul G. Allen"
To: "Ramesh"
Cc:
On Mon, Dec 28, 2015 at 08:35:13AM +0900, Minchan Kim wrote:
> In balloon_page_dequeue, pages_lock should cover the loop
> (ie, list_for_each_entry_safe). Otherwise, the cursor page could
> be isolated by compaction and then list_del by isolation could
> poison the page->lru.{prev,next} so the loop
virtio ring uses smp_wmb on SMP and wmb on !SMP,
the reason for the later being that it might be
talking to another kernel on the same SMP machine.
This is exactly what __smp_XXX barriers do,
so switch to these instead of homegrown ifdef hacks.
Cc: Peter Zijlstra
Cc: Alexander Duyck
Signed-off-
On Fri, Jan 1, 2016 at 12:59 AM, Serge E. Hallyn
wrote:
> On Fri, Jan 01, 2016 at 12:19:44AM -0800, Dan Williams wrote:
>> On Tue, Dec 22, 2015 at 8:23 PM, wrote:
>> > Hi,
>> >
>> > following is a revised set of the CGroup Namespace patchset which Aditya
>> > Kali has previously sent. The code
On Fri, 2016-01-01 at 10:14 +0100, Thomas Gleixner wrote:
> On Fri, 1 Jan 2016, Mike Galbraith wrote:
> > On Thu, 2015-12-31 at 10:24 -0600, Clark Williams wrote:
> > > I pulled this update and tried it on my laptop (i7 quad-core with HT)
> > > and an Atom testbox. I'm seeing a change in the cpu ut
Minchan Kim noticed that balloon_page_dequeue walks the pages list
without holding the pages_lock. This can race e.g. with isolation, which
has been reported to cause list corruption and crashes in leak_balloon.
Page can also in theory get freed before it's locked, corrupting memory.
To fix, make
On Thu, 31 Dec 2015, Roman Volkov wrote:
> From: Roman Volkov
>
> Since vt8500 and PXA timers are identical, use MIN_OSCR_DELTA from PXA,
> which is bigger than existing value. It is required to determine the
> minimum delay which hardware can generate.
This changelog makes no sense at all.
>
On Fri, Dec 04, 2015 at 02:37:50PM +0100, Petr Mladek wrote:
> fill_balloon() and leak_balloon() manipulate only a limited number
> of pages in one call. This is the reason why remove_common() calls
> leak_balloon() in a while cycle.
>
> remove_common() is called also when the system is being froz
On Fri, Jan 01, 2016 at 12:11:02PM +0200, Michael S. Tsirkin wrote:
> On Fri, Dec 04, 2015 at 02:37:50PM +0100, Petr Mladek wrote:
> > fill_balloon() and leak_balloon() manipulate only a limited number
> > of pages in one call. This is the reason why remove_common() calls
> > leak_balloon() in a wh
Peter Mladek reported that balloon might never refill completely after
restore. This is because fill_balloon is only called once there.
Calling fill_balloon repeatedly seems too aggressive,
especially in light of the fact that it sleeps on failure: let's
wake the config change handler and fill it
I am Ms.Golan I am getting in touch with you regarding an extremely important
and urgent matter.If you would oblige me the opportunity,shall provide you with
details upon your response.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord
On Fri, Dec 04, 2015 at 02:37:51PM +0100, Petr Mladek wrote:
> From: Petr Mladek
>
> This patch moves the deferred work from the "vballoon" kthread into a
> system freezable workqueue.
>
> We do not need to maintain and run a dedicated kthread. Also the event
> driven workqueues API makes the lo
On Fri, Jan 01, 2016 at 11:39:40AM +0200, Michael S. Tsirkin wrote:
> virtio ring uses smp_wmb on SMP and wmb on !SMP,
> the reason for the later being that it might be
> talking to another kernel on the same SMP machine.
>
> This is exactly what __smp_XXX barriers do,
> so switch to these instead
On Sunday 27 December 2015 06:53 PM, Noam Camus wrote:
> From: Noam Camus
>
> Add internal tick generator which is shared by all cores.
> Each cluster of cores view it through dedicated address.
> This is used for SMP system where all CPUs synced by same
> clock source.
>
> Signed-off-by: Noam C
All calls to isp1704_write() are using parameter sequence of
isp1704_write(isp, reg, val) but the function is defined as
isp1704_write(isp, val, reg). Fix isp1704_write function definition so that
the driver to be functional.
Signed-off-by: Ivaylo Dimitrov
---
drivers/power/isp1704_charger.c | 2
On Friday 01 January 2016 12:03:29 Ivaylo Dimitrov wrote:
> All calls to isp1704_write() are using parameter sequence of
> isp1704_write(isp, reg, val) but the function is defined as
> isp1704_write(isp, val, reg). Fix isp1704_write function definition
> so that the driver to be functional.
>
> Si
Hi Pali,
On 1.01.2016 13:26, Pali Rohár wrote:
On Friday 01 January 2016 12:03:29 Ivaylo Dimitrov wrote:
All calls to isp1704_write() are using parameter sequence of
isp1704_write(isp, reg, val) but the function is defined as
isp1704_write(isp, val, reg). Fix isp1704_write function definition
Hi,
On 24.12.2015 20:56, Tony Lindgren wrote:
Maybe update the description to say "This fixes a regression with
device tree based booting compared to legacy booting for n900 to
make the n900 legacy user space to also work with device tree based
booting".
OK, will do.
It would be nice to ge
Use list_for_each_entry_safe() instead of list_for_each_safe() to
simplify the code.
Signed-off-by: Geliang Tang
---
virt/kvm/kvm_main.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
index 484079e..5b6df4f 100644
--- a/virt
Use list_for_each_entry*() instead of list_for_each*() to simplify
the code.
Signed-off-by: Geliang Tang
---
arch/x86/kvm/assigned-dev.c | 14 --
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/arch/x86/kvm/assigned-dev.c b/arch/x86/kvm/assigned-dev.c
index 9dc091a..30
To make the intention clearer, use list_first_entry instead of
list_entry.
Signed-off-by: Geliang Tang
---
virt/kvm/async_pf.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/virt/kvm/async_pf.c b/virt/kvm/async_pf.c
index 77d42be..62c4857 100644
--- a/virt/kvm/async_
To make the intention clearer, use list_last_entry instead of
list_entry.
Signed-off-by: Geliang Tang
---
arch/x86/kvm/mmu.c | 4 ++--
arch/x86/kvm/vmx.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c
index e7c2c14..5041ef0 100644
Hi Tomi! Can you review this patch? It is waiting here for two years!
On Thursday 26 December 2013 00:12:39 Ivaylo Dimitrov wrote:
> From: Ivaylo Dimitrov
>
> On memory limited devices, CMA fails easily when asked to allocate
> big chunks of memory like framebuffer memory needed for video
> play
Use to_pci_dev() instead of open-coding it.
Signed-off-by: Geliang Tang
---
This patch is against net-next tree.
Changes in v2:
- fix wrong title.
---
drivers/net/wireless/intel/ipw2x00/ipw2100.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/intel/ipw
Commit be4c9dd7aee5 ("cpuset: enable onlined cpu/node in effective masks")
leverages cpuset's cpus_allowed and its parent's effective_cpus to calculate
the new_cpus by:
cpumask_and(&new_cpus, cs->cpus_allowed, parent_cs(cs)->effective_cpus);
However cpus_allowed will also be updated after the CPU
Previous one seems to be dropped by maillist so I resend it,
sorry for any inconvenience and happy new year : - )
thanks,
Yu
> -Original Message-
> From: Chen, Yu C
> Sent: Friday, January 01, 2016 8:09 PM
> To: cgro...@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org; Chen, Yu C; Vlas
Hi Jaegeuk,
On 1/1/16 11:50 AM, Jaegeuk Kim wrote:
> Hi Chao,
>
> ...
>
> On Tue, Dec 29, 2015 at 11:12:36AM +0800, Chao Yu wrote:
>> f2fs support atomic write with following semantics:
>> 1. open db file
>> 2. ioctl start atomic write
>> 3. (write db file) * n
>> 4. ioct
From: Markus Elfring
Date: Fri, 1 Jan 2016 13:15:34 +0100
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (3):
Less function calls after error detection
Delete unnecessary variable initialisations
Extend an initialisation clause of a for lo
From: Markus Elfring
Date: Fri, 1 Jan 2016 11:16:04 +0100
The kfree() function was called in one case by the
gfar_ethflow_to_filer_table() function during error handling
even if a passed variable contained a null pointer.
* Return directly if a memory allocation failed at the beginning.
* Adjus
From: Markus Elfring
Date: Fri, 1 Jan 2016 12:56:23 +0100
Omit explicit initialisation at the beginning for four local variables
which are redefined before their first use.
Signed-off-by: Markus Elfring
---
drivers/net/ethernet/freescale/gianfar_ethtool.c | 6 +++---
1 file changed, 3 insertio
From: Markus Elfring
Date: Fri, 1 Jan 2016 13:00:06 +0100
Move the assignment for the variable "j" from the beginning
into an initialisation clause of a for loop.
Signed-off-by: Markus Elfring
---
drivers/net/ethernet/freescale/gianfar_ethtool.c | 5 ++---
1 file changed, 2 insertions(+), 3 de
On 29.12.2015 09:46, Tomi Valkeinen wrote:
Oh, I'm sorry, I must have forgotten about that. Please, send a new patch.
Tomi
Actually it is me to be sorry for making noise, I've missed
0eb0dafb674cd6bfac2e3204b2f8b907e26b1138 with all those patches moving
files around.
Ivo
--
To unsubs
Use to_platform_device() instead of open-coding it.
Signed-off-by: Geliang Tang
Reviewed-by: Moritz Fischer
---
Changes in v2:
- split from 'spi: cadence, zynq: use to_platform_device()'
---
drivers/spi/spi-cadence.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/dri
Use to_platform_device() instead of open-coding it.
Signed-off-by: Geliang Tang
Reviewed-by: Moritz Fischer
---
Changes in v2:
- split from 'spi: cadence, zynq: use to_platform_device()'
---
drivers/spi/spi-zynqmp-gqspi.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --gi
On Fri, 1 Jan 2016, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Fri, 1 Jan 2016 11:16:04 +0100
>
> The kfree() function was called in one case by the
> gfar_ethflow_to_filer_table() function during error handling
> even if a passed variable contained a null pointer.
>
> * Return d
The normalization pass in the sorting routine of the relative exception
table serves two purposes:
- it ensures that the address fields of the exception table entries are
fully ordered, so that no ambiguities arise between entries with
identical instruction offsets (i.e., when two instructions
A new warning about a real bug has come up from a recent cleanup:
fs/udf/inode.c: In function 'udf_setup_indirect_aext':
fs/udf/inode.c:1927:28: warning: 'adsize' may be used uninitialized in this
function [-Wmaybe-uninitialized]
If the alloc_type is neither ICBTAG_FLAG_AD_SHORT nor ICBTAG_FLAG_
>> +++ b/drivers/net/ethernet/freescale/gianfar_ethtool.c
>> @@ -778,11 +778,13 @@ static int gfar_ethflow_to_filer_table(struct
>> gfar_private *priv, u64 ethflow,
>>
>> local_rqfpr = kmalloc_array(MAX_FILER_IDX + 1, sizeof(unsigned int),
>> GFP_KERNEL);
>>
On Tuesday 29 December 2015 14:59:59 Florian Fainelli wrote:
> On December 27, 2015 11:22:20 PM PST, Xing Zheng
> wrote:
> >The RK3036's GRFs offset are different with RK3066/RK3188, and need to
> >set
> >mac TX/RX clock before probe emac.
> >
> >Signed-off-by: Xing Zheng
> >---
>
> > };
> >
>
On Fri, 1 Jan 2016, SF Markus Elfring wrote:
> >> +++ b/drivers/net/ethernet/freescale/gianfar_ethtool.c
> >> @@ -778,11 +778,13 @@ static int gfar_ethflow_to_filer_table(struct
> >> gfar_private *priv, u64 ethflow,
> >>
> >>local_rqfpr = kmalloc_array(MAX_FILER_IDX + 1, sizeof(unsigned int
From: Markus Elfring
Date: Fri, 1 Jan 2016 13:56:09 +0100
The kfree() function was called in one case by the
gfar_ethflow_to_filer_table() function during error handling
even if a passed variable contained a null pointer.
* Return directly if a memory allocation failed at the beginning.
* Adjus
The newly added code for Fiji creates a correct compiler warning
about invalid use of the do_div macro:
In file included from powerplay/hwmgr/ppatomctrl.c:31:0:
drivers/gpu/drm/amd/amdgpu/../powerplay/hwmgr/ppevvmath.h: In function
'fDivide':
drivers/gpu/drm/amd/amdgpu/../powerplay/hwmgr/ppevvmat
Hi Chris,
thanks for your answer.
I will try the latest intel driver, but in case it helps or
you get an idea, I found that it has to do with the
Antialiasing settings:
I am using an OTF font (Lucida Sans OT Regular) with cinnamon.
Sometimes when I wake the computer up from suspend to ram the
f
Hi Maxime,
Maxime Ripard schrieb am 31.12.2015 17:00:
> Hi Arnd,
>
> On Thu, Dec 31, 2015 at 04:46:01PM +0100, Arnd Bergmann wrote:
>> On Tuesday 29 December 2015 02:47:32 Timo Sigurdsson wrote:
>> > Here is a late patch aimed for 4.4-rc8 (or 4.4). It would be nice if this
>> >
>> > could ma
Hi Arnd,
Arnd Bergmann schrieb am 31.12.2015 16:46:
> On Tuesday 29 December 2015 02:47:32 Timo Sigurdsson wrote:
>> Here is a late patch aimed for 4.4-rc8 (or 4.4). It would be nice if this
>> could make it into 4.4 in order to avoid unpleasent surprises (and
>> subsequent questions) with bui
From: Roman Volkov
vt8500 hangs in nanosleep() function, starting from commit
c6eb3f70d4482806dc2d3e1e3c7736f497b1d418, making the system unusable.
Per investigation, looks like set_next_event() now receives too small
delta and fails with -ETIME.
Google group discussion:
https://groups.google.co
From: Roman Volkov
The vt8500 clocksource driver declares itself as capable to handle the
minimum delay of 4 cycles by passing the value into
clockevents_config_and_register(). The vt8500_timer_set_next_event()
requires the passed cycles value to be at least 16. The impact is that
userspace hangs
From: Roman Volkov
vt8500 timer requires special synchronization for accessing some of its
registers. Define special read and write functions to handle this process
transparently.
Use relaxed read/write, according to the following:
http://permalink.gmane.org/gmane.linux.ports.arm.kernel/117658:
From: Roman Volkov
The purpose of the 'loops' variable is unclear. vt8500 hardware does not
require any protections, in case if these variables intended for preventing
infinite loops (identical PXA timer works perfectly without these ones).
Signed-off-by: Roman Volkov
Acked-by: Alexey Charkov
Commit 69fb4dcada77 ("power: Add an axp20x-usb-power driver") introduced a
new driver for the USB power supply used on various Allwinner based SBCs.
However, the driver was not added to multi_v7_defconfig which breaks USB
support for some boards (e.g. LeMaker BananaPi) as the kernel will now
turn o
The latest changes to the rockchip clk implementation cause
tons of warnings and/or errors (depending on the configuration):
WARNING: vmlinux.o(.data+0x125394): Section mismatch in reference from the
variable __compound_literal.125 to the (unknown reference)
.init.rodata:(unknown)
The reason is
From: Roman Volkov
Since WM8650 has the same 'WMT' SDHC controller as WM8505, and the driver
is already in the kernel, this node enables the controller support for
WM8650
Signed-off-by: Roman Volkov
Reviewed-by: Alexey Charkov
---
arch/arm/boot/dts/wm8650.dtsi | 9 +
1 file changed, 9
From: Roman Volkov
According to datasheet, the registers space of SDHC controller is 1Kb,
not '0x1000', the correct value should be '0x400'. Bracket interrupt
numbers individually per recommendations.
Signed-off-by: Roman Volkov
---
arch/arm/boot/dts/wm8505.dtsi | 4 ++--
1 file changed, 2 ins
From: Roman Volkov
This patch set enables SD controller support for WM8650 and
fixes minor errors in WM8505 Device Tree file.
Changes in v3:
1. Add minor fixes for WM8505 SDHC node
Tested on both WM8505 and WM8650.
Roman Volkov (2):
dts: vt8500: Add SDHC node to DTS file for WM8650
dts: vt
>From 433df301cf49624871346fa63f3fc65033caeda3 Mon Sep 17 00:00:00 2001
From: Arnd Bergmann
Date: Fri, 1 Jan 2016 13:18:48 +0100
Subject: [PATCH] net: make ip6tunnel_xmit definition conditional
Moving the caller of iptunnel_xmit_stats causes a build error in
randconfig builds that disable CONFIG_
ARM allmodconfig fails because of the addition of the FMAN driver:
drivers/built-in.o: In function `dtsec_restart_autoneg':
binder.c:(.text+0x173328): undefined reference to `mdiobus_read'
binder.c:(.text+0x173348): undefined reference to `mdiobus_write'
drivers/built-in.o: In function `dtsec_conf
Hello,
The following program triggers GPF in llcp_sock_getname:
// autogenerated by syzkaller (http://github.com/google/syzkaller)
#include
#include
#include
#include
#include
#include
#include
#include
int fd;
void *thr(void *arg)
{
struct sockaddr_nfc_llcp sa;
sa.sa_fa
On Friday 01 January 2016 14:25:01 Timo Sigurdsson wrote:
> Hi Arnd,
>
> Arnd Bergmann schrieb am 31.12.2015 16:46:
>
> > On Tuesday 29 December 2015 02:47:32 Timo Sigurdsson wrote:
> >> Here is a late patch aimed for 4.4-rc8 (or 4.4). It would be nice if this
> >>
> >> could make it into 4.4
Global init can have sub-threads, so ->pid check is not reliable enough
for is_global_init(), we need to check tgid instead. This has been spotted
by Oleg and a fix was proposed by Richard a long time ago (see the link
below).
Oleg wrote:
:Because is_global_init() is only true for the main thread
On Friday 01 January 2016 14:38:22 Timo Sigurdsson wrote:
> Commit 69fb4dcada77 ("power: Add an axp20x-usb-power driver") introduced a
> new driver for the USB power supply used on various Allwinner based SBCs.
> However, the driver was not added to multi_v7_defconfig which breaks USB
> support for
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
fs/nfsd/nfs4state.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index 6ef240c..c484a2b 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
fs/lockd/svc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/fs/lockd/svc.c b/fs/lockd/svc.c
index 44d18ad..b4006c72 100644
--- a/fs/lockd/svc.c
+++ b/fs/lockd/svc.c
@@ -94,8 +94,7 @@ static
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
fs/jffs2/wbuf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/jffs2/wbuf.c b/fs/jffs2/wbuf.c
index f3a4857..5a3da3f 100644
--- a/fs/jffs2/wbuf.c
+++ b/fs/jffs2/wbuf.c
@@ -1153,7 +1153,7 @@ s
The newly added DRM_HDLCD driver tries to select DMA_CMA, but that is
not necessarily possible, as not all configurations contain HAVE_DMA_CONTIGUOUS:
warning: (DRM_HDLCD) selects DMA_CMA which has unmet direct dependencies
(HAVE_DMA_CONTIGUOUS && CMA)
drivers/built-in.o: In function `dma_alloc_f
A new warning about a real bug has come up from a recent cleanup:
fs/udf/inode.c: In function 'udf_setup_indirect_aext':
fs/udf/inode.c:1927:28: warning: 'adsize' may be used uninitialized in this
function [-Wmaybe-uninitialized]
If the alloc_type is neither ICBTAG_FLAG_AD_SHORT nor ICBTAG_FLAG_
On January 1, 2016 3:56:01 AM EET, Daniel Kurtz wrote:
>Hi Mark,
>
>Thanks for responding.
>
>On Fri, Jan 1, 2016 at 6:07 AM, Mark Brown wrote:
>> On Thu, Dec 31, 2015 at 09:45:51PM +0800, Daniel Kurtz wrote:
>>> On Thu, Dec 31, 2015 at 1:22 AM, Mark Brown
>wrote:
>>
>>> > I really don't think
On Friday 01 January 2016 16:38:10 Roman Volkov wrote:
> From: Roman Volkov
>
> This patch set enables SD controller support for WM8650 and
> fixes minor errors in WM8505 Device Tree file.
>
> Changes in v3:
> 1. Add minor fixes for WM8505 SDHC node
>
> Tested on both WM8505 and WM8650.
>
> Ro
From: Markus Elfring
Date: Fri, 1 Jan 2016 15:11:09 +0100
Replace explicit initialisations for four local variables at the beginning
by assignments that will only be performed if the corresponding code
will really be executed.
Signed-off-by: Markus Elfring
---
drivers/net/ethernet/intel/i40e/i
The hdlcd driver has no build-time dependency on the SCPI clock
and the bogus 'select' causes a warning when SCPI is disabled:
warning: (DRM_HDLCD) selects COMMON_CLK_SCPI which has unmet direct
dependencies (COMMON_CLK && (ARM_SCPI_PROTOCOL || COMPILE_TEST))
This removes the select statement.
The hdlcd_pm_suspend and hdlcd_pm_resume are intentionally unused
when CONFIG_PM is not set in this driver, and we get a compiler warning
for this:
drivers/gpu/drm/arm/hdlcd_drv.c:521:12: warning: 'hdlcd_pm_suspend' defined but
not used [-Wunused-function]
drivers/gpu/drm/arm/hdlcd_drv.c:536:12:
Julia Lawall :
> On Fri, 1 Jan 2016, SF Markus Elfring wrote:
[...]
> > > Normally, one returns -ENOMEM for this case, but it looks like this
> > > function is returning 0 on failure.
> >
> > Should a symbol like "false" be used instead of such a special number?
>
> Maybe it's better than 0 and
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/input/touchscreen/pcap_ts.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/touchscreen/pcap_ts.c
b/drivers/input/touchscreen/pcap_ts.c
index 23a354a..0e3fc41 100644
--- a/
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/char/xillybus/xillybus_core.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/char/xillybus/xillybus_core.c
b/drivers/char/xillybus/xillybus_core.c
index 77d6c12..3bd36e0 10064
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/mmc/host/sh_mmcif.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mmc/host/sh_mmcif.c b/drivers/mmc/host/sh_mmcif.c
index 1ca8a13..e0c076a 100644
--- a/drivers/mmc/host/sh_mmcif
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/power/generic-adc-battery.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/generic-adc-battery.c
b/drivers/power/generic-adc-battery.c
index fedc581..edb36bf 100644
--- a/
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/mmc/host/usdhi6rol0.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mmc/host/usdhi6rol0.c b/drivers/mmc/host/usdhi6rol0.c
index b47122d..b2752fe 100644
--- a/drivers/mmc/host/us
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/scsi/mvsas/mv_sas.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
index e712fe7..78bf557 100644
--- a/drivers/scsi/mvsas/mv_sas
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/edac/i5100_edac.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/edac/i5100_edac.c b/drivers/edac/i5100_edac.c
index 4091777..c655162 100644
--- a/drivers/edac/i5100_edac.c
++
Hi Eduardo,
thanks for your review,
> -Original Message-
> From: Eduardo Valentin [mailto:edubez...@gmail.com]
> Sent: Friday, January 01, 2016 3:04 AM
> To: Chen, Yu C
> Cc: Zhang, Rui; javi.mer...@arm.com; linux...@vger.kernel.org; linux-
> ker...@vger.kernel.org; sta...@vger.kernel.org
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/net/wireless/ti/wlcore/main.c | 10 +-
drivers/net/wireless/ti/wlcore/ps.c | 2 +-
drivers/net/wireless/ti/wlcore/scan.c | 2 +-
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/dri
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/net/wireless/ti/wl1251/ps.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ti/wl1251/ps.c
b/drivers/net/wireless/ti/wl1251/ps.c
index b9e27b9..fa01b0a 100644
--- a/
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
net/mac80211/mlme.c | 3 +--
net/wireless/mlme.c | 2 +-
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
index 1c342e2..ea70917 100644
--- a/net/mac80211/mlme
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/net/wireless/marvell/mwifiex/11h.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/wireless/marvell/mwifiex/11h.c
b/drivers/net/wireless/marvell/mwifiex/11h.c
index 71a
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/net/wireless/realtek/rtlwifi/wifi.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/realtek/rtlwifi/wifi.h
b/drivers/net/wireless/realtek/rtlwifi/wifi.h
index 454475
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang
---
drivers/net/ethernet/tile/tilepro.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/tile/tilepro.c
b/drivers/net/ethernet/tile/tilepro.c
index 6f0a449..298e059 100644
---
From: Markus Elfring
Date: Fri, 1 Jan 2016 16:54:13 +0100
Omit explicit initialisation at the beginning for one local variable
that is redefined before its first use.
Signed-off-by: Markus Elfring
---
drivers/net/usb/huawei_cdc_ncm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
fixed spelling mistakes which reffered to OUTPUT as OUPUT
Signed-off-by: Saatvik Arya
---
drivers/staging/media/davinci_vpfe/dm365_resizer.c | 22 +++---
drivers/staging/media/davinci_vpfe/dm365_resizer.h | 2 +-
2 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/d
Hi,
I started to run my kernels with KASan enabled and found this after resume from
ram on a 4.3.3 kernel:
Jan 01 15:16:59 localhost.localdomain kernel:
==
Jan 01 15:16:59 localhost.localdomain kernel: BUG: KASan: out of bounds on
From: Markus Elfring
Date: Fri, 1 Jan 2016 17:47:46 +0100
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (2):
Refactoring for qmi_wwan_bind()
Delete an unnecessary variable initialisation in qmi_wwan_register_subdriver()
drivers/net/usb/qm
1 - 100 of 187 matches
Mail list logo