Hi Krzysztof,
2016-09-04 20:04 GMT+09:00 Krzysztof Kozlowski :
>
> Hi,
>
> Changes since v2
>
> 1. Combine separate patchsets into one. Previously I sent separately the fixes
>and changes for S3C platforms.
> 2. Fix issues pointed during review.
> 3. Add review tags.
>
> Chang
ping...
Hi Tyhicks,
We observed a ecryptFS crash occasionally in Linux kernel 4.1.18. The call
trace is attached below. Is it a known issue? Look forward to hearing from you.
Thanks in advance!
[19314.529479s][pid:2694,cpu3,GAC_Executor[0]]Call trace:
[19314.529510s][pid:2694,cpu3,GAC_Execu
From: Hongtao Jia
SoC compatible string and endianness property are added according to the
new bindings.
Signed-off-by: Jia Hongtao
---
Changes for V2:
* Rebase on latest linux-next tree (next-20161006).
arch/powerpc/boot/dts/fsl/t1023si-post.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2
From: Hongtao Jia
SoC compatible string and endianness property are added according to the
new bindings.
Signed-off-by: Jia Hongtao
---
Changes for V2:
* Rebase on latest linux-next tree (next-20161006).
arch/powerpc/boot/dts/fsl/t1040si-post.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2
From: Hongtao Jia
Also add nodes and properties for thermal management support.
Signed-off-by: Jia Hongtao
---
Changes for V2:
* Rebase on latest linux-next tree (next-20161006).
arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 2 +-
arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts | 2
> This patch series has fixes for compatibility, reliability and
> performance issues and some cleanup. It also includes a new version
> of Ondrej Zary's patch that merges g_NCR5380_mmio into g_NCR5380.
>
> I've tested this patch series on a Powerbook 180. If someone would
> test some of the other
Hi,
I open the Transparent huge page and run the system and hit the bug
in huge_memory.c:
static void __split_huge_page_refcount(struct page *page)
.
.
.
/* tail_page->_mapcount cannot change */
> On 08.10.2016 23:52, Meelis Roos wrote:
> > Just tried 4.8.0-11288-gb66484c on three of my parsic machines (enabled
> > strict usercopy checking or somethinng like that in make oldconfig).
>
> It's not related to the usercopy checks, instead it's most likely a
> parisc-specific problem I just n
On Mon, Oct 03, 2016 at 10:32:04AM +0800, Xin Long wrote:
> On Fri, Sep 30, 2016 at 3:05 PM, Aaron Lu wrote:
> > On 08/23/2016 05:44 AM, Marcelo Ricardo Leitner wrote:
> >> Em 19-08-2016 04:24, Aaron Lu escreveu:
> >>> On Fri, Aug 19, 2016 at 04:19:39AM -0300, Marcelo Ricardo Leitner wrote:
>
On 05/10/16 10:04, Mugunthan V N wrote:
> This patch adds the required pieces to ti_am335x_adc driver for
> DMA support
>
> Signed-off-by: Mugunthan V N
Hi,
Just the one question inline. I'll also need an Ack from Lee as
this touches code in mfd (as does the previous patch).
We have obviously m
From: Hongtao Jia
Also add nodes and properties for thermal management support.
Signed-off-by: Jia Hongtao
---
Changes for V2:
* Rebase on latest linux-next tree (next-20161006).
arch/arm/boot/dts/ls1021a.dtsi | 84 +-
1 file changed, 82 insertions(+),
From: Hongtao Jia
Also add nodes and properties for thermal management support.
Signed-off-by: Jia Hongtao
---
Changes for V2:
* Rebase on latest linux-next tree (next-20161006).
arch/arm64/boot/dts/freescale/fsl-ls2080a-qds.dts | 2 +-
arch/arm64/boot/dts/freescale/fsl-ls2080a-rdb.dts |
On 05/10/16 10:04, Mugunthan V N wrote:
> Add DMA properties for tscadc
>
> Signed-off-by: Mugunthan V N
Do the binding docs need updating to reflect this? I can't immediately
find the relevant doc to check!
Jonathan
> ---
> arch/arm/boot/dts/am33xx.dtsi | 2 ++
> 1 file changed, 2 insertions(
On 09.10.2016 11:03, Meelis Roos wrote:
>> On 08.10.2016 23:52, Meelis Roos wrote:
>>> rp3410 crashed on boot with the following:
>>>
>>> Linux version 4.8.0-11288-gb66484c (mroos@rp3410) (gcc version 5.4.0
>>> (Gentoo 5.4.0 p1.0) ) #81 Sat Oct 8 20:40:24 EEST 2016
>>> unwind_init: start = 0x4076e
Rich,
On Sat, 8 Oct 2016, Rich Felker wrote:
> On Sat, Oct 08, 2016 at 07:03:30PM +0200, Thomas Gleixner wrote:
> > Because you drop out the idle spin due to an interrupt, but no interrupt is
> > handled according to the trace. You just go back to sleep and the trace is
> > full of this behaviour.
On Sun, Oct 09, 2016 at 06:35:28AM +, Winkler, Tomas wrote:
>
>
> >
> > Request and relinquish locality for the driver use in order to be a better
> > citizen
> > in a multi locality environment like with TXT as it uses locality 2.
>
> >
> > Signed-off-by: Jarkko Sakkinen
> > ---
On Sat, Oct 08, 2016 at 07:42:56PM -0600, Jason Gunthorpe wrote:
> On Sun, Oct 09, 2016 at 03:15:09AM +0300, Jarkko Sakkinen wrote:
> > + ctrl = crb_map_res(dev, priv, &io_res, buf->control_address,
> > + sizeof(struct crb_regs) -
> > + offsetof(struct cr
> >
> > >
> > > Request and relinquish locality for the driver use in order to be a better
> citizen
> > > in a multi locality environment like with TXT as it uses locality 2.
> >
> > >
> > > Signed-off-by: Jarkko Sakkinen
> > > ---
> > > drivers/char/tpm/tpm_crb.c | 36
> > > +++
> -Original Message-
> From: Zhang Rui [mailto:rui.zh...@intel.com]
> Sent: Monday, September 19, 2016 10:44 AM
> To: Leo Li ; Rob Herring
> Cc: Troy Jia ; devicet...@vger.kernel.org; linux-
> p...@vger.kernel.org; linuxppc-dev ; lkml
> ker...@vger.kernel.org>; Scott Wood ; Eduardo Valen
Hi,
> Do we have any resolution of this?
I got the exact same problem if CONFIG_MODVERSIONS is enabled.
Here is my understanding.
All the symbols exported using assembler macro EXPORT_SYMBOL will need a
__crc_ to be defined later on (include/asm-generic/export.h).
This is what genksyms should
Hello,
While running syzkaller fuzzer on commit
b66484cd74706fa8681d051840fe4b18a3da40ff (Oct 7), I am getting:
[ BUG: syz-executor/15138 still has locks held! ]
4.8.0+ #29 Not tainted
-
1 lock held by syz-executor/15138:
#0: (&pipe->mutex/1){+.+.+.}, at: [<
Hello,
While running syzkaller fuzzer on commit
b66484cd74706fa8681d051840fe4b18a3da40ff (Oct 7), I am getting:
[ BUG: syz-executor/15138 still has locks held! ]
4.8.0+ #29 Not tainted
-
1 lock held by syz-executor/15138:
#0: (&pipe->mutex/1){+.+.+.}, at: [<
On Sep 25 Alexey Khoroshilov wrote:
> There is no check if ioremap_nocache() returns a valid pointer.
> Potentially it can lead to null pointer dereference.
>
> Found by Linux Driver Verification project (linuxtesting.org).
>
> Signed-off-by: Alexey Khoroshilov
> ---
> drivers/firewire/nosy.c |
We need to generalize tpm2_get_tpm_pt to tpm2_get_cap in order to it be
suitable for grabbing handles, PCR banks and so forth.
Jarkko Sakkinen (2):
tpm: move TPM 2.0 command and response constants to tpm2-cmd.c
tpm: refactor tpm2_get_tpm_pt to tpm2_getcap_cmd
drivers/char/tpm/tpm.h
The constans used to construct TPM 2.0 messages are only needed inside
tpm2-cmd.c so it is better to move them there.
Signed-off-by: Jarkko Sakkinen
---
drivers/char/tpm/tpm.h | 42 --
drivers/char/tpm/tpm2-cmd.c | 42 +
Refactored tpm2_get_tpm_pt to tpm2_getcap_cmd, which means that it also
takes capability ID as input. This is required to access
TPM_CAP_HANDLES, which contains metadata needed for swapping transient
data.
Signed-off-by: Jarkko Sakkinen
---
drivers/char/tpm/tpm.h | 6 +++-
drivers/char
From: Tang Yuantian
Ls1046a is a new introduced soc which supports ATA3.0.
Signed-off-by: Tang Yuantian
---
drivers/ata/ahci_qoriq.c | 16 +---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/drivers/ata/ahci_qoriq.c b/drivers/ata/ahci_qoriq.c
index 1eba8df..9884c8c
On Sun, Oct 09, 2016 at 09:43:59AM +, Winkler, Tomas wrote:
>
> > >
> > > >
> > > > Request and relinquish locality for the driver use in order to be a
> > > > better
> > citizen
> > > > in a multi locality environment like with TXT as it uses locality 2.
> > >
> > > >
> > > > Signed-off-by:
On Sun, 9 Oct 2016, Rich Felker wrote:
> Ideas for improvement are welcome -- for example the
> irq_is_percpu(irq_desc_get_irq(desc)) thing looks rather silly but I
See the other mail.
> didn't see a better way without poking through abstractions -- but
> overall I think this both solves the time
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
Signed-off-by: Philippe Reynes
---
drivers/net/usb/lan78xx.c | 70 +---
1 files changed, 40 insertions(+), 30 deletions(-)
diff --git a/drivers/ne
Good morning LKML
http://jyt.org.uk/valuable.php?theres=khya20h4t8hkzymt2
Mark
On Fri, 7 Oct 2016, Peter Zijlstra wrote:
> top_waiter = futex_top_waiter(hb, &key);
> if (top_waiter) {
> - ret = wake_futex_pi(uaddr, uval, top_waiter, hb);
> + struct futex_pi_state *pi_state = top_waiter->pi_state;
> +
> + ret = -EINVAL;
> +
On Fri, Oct 07, 2016 at 07:45:52PM -0700, Fenghua Yu wrote:
> From: Fenghua Yu
>
> Some Haswell generation CPUs support RDT, but they don't enumerate this
> using CPUID. Use rdmsr_safe() and wrmsr_safe() to probe the MSRs on
> cpu model 63 (INTEL_FAM6_HASWELL_X)
>
> Signed-off-by: Fenghua Yu
>
On Sat, Oct 08, 2016 at 08:43:51PM -0700, Joel Fernandes wrote:
> On Thu, Sep 29, 2016 at 1:18 AM, Chris Wilson
> wrote:
> > On Thu, Sep 29, 2016 at 03:34:11PM +0800, Jisheng Zhang wrote:
> >> On Marvell berlin arm64 platforms, I see the preemptoff tracer report
> >> a max 26543 us latency at __p
在 2016/10/8 17:28, Ziji Hu 写道:
Hi Shawn,
On 2016/10/8 10:44, Shawn Lin wrote:
在 2016/10/7 23:22, Gregory CLEMENT 写道:
From: Ziji Hu
Marvell Xenon eMMC/SD/SDIO Host Controller contains PHY.
Three types of PHYs are supported.
Add support to multiple types of PHYs init and configuration.
Add re
On Fri, Oct 07, 2016 at 04:27:49PM +0800, YiPing Xu wrote:
>
>
> On 2016/10/5 2:02, Laura Abbott wrote:
> > On 09/30/2016 01:18 AM, Xu YiPing wrote:
> > > ion_alloc may get into slow path to get free page,
> > > the call stack:
> > >
> > > __alloc_pages_slowpath
> > > ion_page_pool_alloc_pages
>
On Sun, Oct 09, 2016 at 01:41:16PM +0200, Borislav Petkov wrote:
> On Fri, Oct 07, 2016 at 07:45:52PM -0700, Fenghua Yu wrote:
> > From: Fenghua Yu
> >
> > Some Haswell generation CPUs support RDT, but they don't enumerate this
> > using CPUID. Use rdmsr_safe() and wrmsr_safe() to probe the MSRs
On 2016-10-09, at 5:03 AM, Meelis Roos wrote:
>>> Linux version 4.8.0-11288-gb66484c (mroos@rp3410) (gcc version 5.4.0
>>> (Gentoo 5.4.0 p1.0) ) #81 Sat Oct 8 20:40:24 EEST 2016
>>> unwind_init: start = 0x4076e980, end = 0x407a7060, entries = 14446
>>> The 64-bit Kernel has started...
>>> Kernel
On Fri, Oct 07, 2016 at 02:05:13PM -0700, Nadim Almas wrote:
> Fixed coding style issue
Your subject is huge, please don't make it so long.
thanks,
greg k-h
On Fri, Oct 07, 2016 at 05:04:16PM -0700, Chase Metzger wrote:
> Added braces to else statements where checkpatch complained.
>
> Signed-off-by: Chase Metzger
You used the same subject here as your last patch, which implies it did
the same thing, yet it did not :(
So please make the subject uni
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
Signed-off-by: Philippe Reynes
---
drivers/staging/netlogic/xlr_net.c | 14 --
1 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/netlogic/xlr_net.
On Sun, Oct 02, 2016 at 10:28:28PM -0400, James Simmons wrote:
> From: Bobi Jam
>
> If normal IO got short read/write, we'd restart the IO from where
> we've accomplished until we meet EOF or error happens.
>
> Signed-off-by: Bobi Jam
> Signed-off-by: Jinshan Xiong
> Intel-bug-id: https://jira
On Sun, Oct 02, 2016 at 10:28:31PM -0400, James Simmons wrote:
> From: Henri Doreau
>
> Implement non-blocking migration based on exclusive open instead of
> group lock. Implemented exclusive close operation to atomically put
> a lease, swap two layouts and close a file. This allows race-free
> m
This patch replaces calls to the "printk" with appropriate "pr_*"
function thus addressing the following warning generated by the
checkpatch script:
Prefer [subsystem eg: netdev]_warn([subsystem]dev, ... then
dev_warn(dev, ... then pr_warn(... to printk(KERN_WARNING ...
Signed-off-by: H
On Mon, Oct 03, 2016 at 06:04:00PM +0200, Sergio Paracuellos wrote:
> This patch series fix some of the following checkpatch.pl reported warnings:
> WARNING: line over 80 characters
> WARNING: Block comments should align the * on each line
> WARNING: Block comments use a trailing */ on a separate l
On Mon, Oct 03, 2016 at 06:07:03PM +0200, Sergio Paracuellos wrote:
> This patch fixes the following checkpatch.pl warning in hfa384x.h:
> WARNING: do not add new typedefs
>
> It applies for typedef CTLX_STATE
>
> Signed-off-by: Sergio Paracuellos
> ---
> drivers/staging/wlan-ng/hfa384x.h |
On Mon, Oct 03, 2016 at 06:05:44PM +0200, Sergio Paracuellos wrote:
> This patch fix the following sparse warnings in prism2fw.c:
> warning: memset with byte count of 12
>
> Signed-off-by: Sergio Paracuellos
> ---
> drivers/staging/wlan-ng/prism2fw.c | 11 ---
> 1 file changed, 8 ins
(Second, rebased submission, since first submission yielded no replies.)
This adds a new per-task hidepid= flag that is honored by procfs when
presenting /proc to the user, in addition to the existing hidepid= mount
option. So far, hidepid= was exclusively a per-pidns setting. Locking
down a set o
(Second, rebased submission, since first submission yielded no replies.)
Previously, the hidepid parameter was checked by comparing literal
integers 0, 1, 2. Let's add a proper enum for this, to make the checking
more expressive:
0 → HIDEPID_OFF
1 → HIDEPID_NO_ACCESS
2 → H
On Sun, Oct 09, 2016 at 11:14:20AM +0200, Thomas Gleixner wrote:
> Rich,
>
> On Sat, 8 Oct 2016, Rich Felker wrote:
> > On Sat, Oct 08, 2016 at 07:03:30PM +0200, Thomas Gleixner wrote:
> > > Because you drop out the idle spin due to an interrupt, but no interrupt
> > > is
> > > handled according
On Sun, Oct 09, 2016 at 07:58:00PM +0530, Harman Kalra wrote:
> This patch replaces calls to the "printk" with appropriate "pr_*"
> function thus addressing the following warning generated by the
> checkpatch script:
>
> Prefer [subsystem eg: netdev]_warn([subsystem]dev, ... then
> dev_warn
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: b66484cd74706fa8681d051840fe4b18a3da40ff
commit: 51a021244b9d579be6b4f8c15c493a76deb2a79e atomic64: no need for
CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE
date: 2 days ago
config: frv-allmodconfig (attached
This patch resolves all the following CHECKs caught by checkpatch.pl
CHECK: Alignment should match open parenthesis
Signed-off-by: Harman Kalra
---
drivers/staging/fbtft/fb_ili9325.c |9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/fbtft/fb_ili9325.
On Sun, Oct 09, 2016 at 01:03:10PM +0200, Thomas Gleixner wrote:
> On Sun, 9 Oct 2016, Rich Felker wrote:
> > Ideas for improvement are welcome -- for example the
> > irq_is_percpu(irq_desc_get_irq(desc)) thing looks rather silly but I
>
> See the other mail.
>
> > didn't see a better way without
Move the ohci-exynos system PM callbacks within #ifdef CONFIG_PM_SLEEP
as to avoid them being build when not used. This also allows us to use the
SET_SYSTEM_SLEEP_PM_OPS macro which simplifies the code.
Signed-off-by: Anand Moon
---
drivers/usb/host/ohci-exynos.c | 20 +---
1 fil
Move the ehci-exynos system PM callbacks within #ifdef CONFIG_PM_SLEEP
as to avoid them being build when not used. This also allows us to use the
SET_SYSTEM_SLEEP_PM_OPS macro which simplifies the code.
Signed-off-by: Anand Moon
---
drivers/usb/host/ehci-exynos.c | 14 ++
1 file chan
On Sun, Oct 09, 2016 at 11:23:25AM +0530, Sabitha George wrote:
> Fixes checkpatch warning on printk usage in ks_hostif.c
>
> Signed-off-by: Sabitha George
> ---
> drivers/staging/ks7010/ks_hostif.c | 31 +++
> 1 file changed, 15 insertions(+), 16 deletions(-)
Due to
On Sun, Oct 09, 2016 at 11:22:32AM +0530, Sabitha George wrote:
> This patch fixes the below checkpatch warning in ks_hostif.c:
> __packed is preferred over __attribute__((packed))
>
> Signed-off-by: Sabitha George
> ---
> drivers/staging/ks7010/ks_hostif.c | 8
> 1 file changed, 4 ins
On Fri, Oct 07, 2016 at 10:27:54PM +0530, Harman Kalra wrote:
> Warning "Prefer "pr_debug over printk(KERN_DEBUG... " and "few line more than
> 80 character" fixed
That is two different things again :(
I can't take this as is...
greg k-h
This patch replaces calls to the "printk" with appropriate "pr_*"
function thus addressing the following warning generated by the
checkpatch script:
Prefer [subsystem eg: netdev]_warn([subsystem]dev, ... then
dev_warn(dev, ... then pr_warn(... to printk(KERN_WARNING ...:w
Signed-off-by:
On Fri, Oct 07, 2016 at 10:40:42PM +0300, Andy Shevchenko wrote:
> On Fri, Oct 7, 2016 at 9:17 PM, Bacchewar, Nilesh
> wrote:
> > Here, Its marking GPIO controller IRQ line as wake capable not GPIO lines.
>
> AFAIU this is called per (GPIO) pin which is IRQ enabled.
>
> Imagine a case when you
On Fri, Oct 07, 2016 at 10:27:56PM +0530, Harman Kalra wrote:
> Error "open brace { should be on the previous line" caught by checkpatch.pl
> fixed.
> Signed-off-by: Harman Kalra
Please put a blank line before the signed-off-by: line. As examples,
look at all other commits in the tree :)
thank
On Fri, Oct 07, 2016 at 10:27:55PM +0530, Harman Kalra wrote:
> Errors "open brace { should be on the previous line" caught by checkpatch.pl
> fixed.
Why two ' ' characters?
> Signed-off-by: Harman Kalra
Same blank line needed.
thanks,
greg k-h
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
Signed-off-by: Philippe Reynes
---
net/dsa/slave.c | 14 --
1 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/net/dsa/slave.c b/net/dsa/slave.c
index 6b1282c..687
On Mon, Oct 03, 2016 at 11:16:22PM +, Wayne Porter wrote:
> Convert CamelCase to recommended style.
>
> Signed-off-by: Wayne Porter
> ---
> drivers/staging/rts5208/xd.c | 8
> drivers/staging/rts5208/xd.h | 8
> 2 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --g
On Fri, Oct 07, 2016 at 08:05:14PM +0300, Andy Shevchenko wrote:
> On Thu, Sep 29, 2016 at 4:39 PM, Mika Westerberg
> wrote:
> > DT has property 'gpio-line-names' to name GPIO lines the controller has if
> > present. Use this very same property in ACPI as well to provide nice names
> > for the GPI
Fixed coding style issue by replacing ENOSYS with EINVAL.
This patch fix the following checkpatch.pl warnings in p80211netdev.c:
WARNING: ENOSYS means 'invalid syscall nr' and nothing else
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211netdev.c | 6 +++---
1 file changed, 3
This patch fix the following checkpatch.pl warnings in p80211netdev.c:
WARNING: line over 80 characters
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211netdev.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211netdev.c
This patch series fix some of the following checkpatch.pl reported warnings:
WARNING: line over 80 characters
WARNING: Block comments should align the * on each line
WARNING: Block comments use a trailing */ on a separate line
WARNING: ENOSYS means 'invalid syscall nr' and nothing else
Sergio Para
This patch fix the following checkpatch.pl warnings in p80211req.c:
WARNING: line over 80 characters
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211req.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211req.c
b/drivers/
This patch fixes the following checkpatch.pl warning in hfa384x.h:
WARNING: do not add new typedefs
It applies for typedef CTLX_STATE
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/hfa384x.h | 3 +--
drivers/staging/wlan-ng/hfa384x_usb.c | 2 +-
2 files changed, 2 insertions(
On Sat, Oct 08, 2016 at 07:20:08PM -0400, Dave Jones wrote:
> On Sat, Oct 08, 2016 at 07:29:03PM +0100, Al Viro wrote:
> > On Sat, Oct 08, 2016 at 02:08:06PM -0400, Dave Jones wrote:
> > > That code: matches this dissembly:
> > >
> > > for (i = seg + 1; i < iter->nr_segs;
This patch fix the following checkpatch.pl warnings in p80211wep.c:
WARNING: line over 80 characters
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211wep.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlan-ng/p80211wep.c
b/drivers/st
This patch fix the following checkpatch.pl warnings in p80211conv.c:
WARNING: Block comments should align the * on each line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211conv.c | 318 +--
1 file changed, 159 insertions(+), 159 deletions(-)
This patch fix the following checkpatch.pl warnings in prism2sta.c:
WARNING: line over 80 characters
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/prism2sta.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wlan-ng/prism2sta.c
b/drive
This patch fix the following checkpatch.pl warnings in prism2mib.c:
WARNING: Block comments should align the * on each line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/prism2mib.c | 100 ++--
1 file changed, 50 insertions(+), 50 deletions(-)
dif
This patch fix the following checkpatch.pl warnings in hfa384x_usb.c:
WARNING: line over 80 characters
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/hfa384x_usb.c | 71 +++
1 file changed, 38 insertions(+), 33 deletions(-)
diff --git a/drivers/sta
This patch fix the following checkpatch.pl warnings in prism2mgmt.c:
WARNING: Block comments should align the * on each line
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/prism2mgmt.c | 518 ++---
This patch fix the following checkpatch.pl warnings in p80211req.c:
WARNING: Block comments should align the * on each line
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211req.c | 166 ++-
This patch fix the following checkpatch.pl warnings in prism2sta.c:
WARNING: Block comments should align the * on each line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/prism2sta.c | 36 ++--
1 file changed, 18 insertions(+), 18 deletions(-)
diff
This patch fix the following sparse warnings in prism2fw.c:
warning: memset with byte count of 12
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/prism2fw.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wlan-ng/prism2fw.c
b/dri
This patch fix the following checkpatch.pl warnings in p80211wep.c:
WARNING: Block comments should align the * on each line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211wep.c | 90 ++---
1 file changed, 45 insertions(+), 45 deletions(-)
dif
This patch fix the following checkpatch.pl warnings in p80211netdev.c:
WARNING: Block comments should align the * on each line
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211netdev.c | 613 +
This patch fix the following checkpatch.pl warnings in prism2fw.c
WARNING: Block comments should align the * on each line
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/prism2fw.c | 485 +++---
This patch fix the following checkpatch.pl warnings in hfa384x_usb.c:
WARNING: Block comments should align the * on each line
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/hfa384x_usb.c | 2370 +--
On Fri, Oct 07, 2016 at 08:47:51AM +1100, Dave Chinner wrote:
> Except that it's DAX, and in 4.7-rc1 that used shared locking at the
> XFS level and never took exclusive locks.
>
> *However*, the DAX IO path locking in XFS has changed in 4.9-rc1 to
> match the buffered IO single writer POSIX sema
Looks fine:
Reviewed-by: Christoph Hellwig
On Sun, Oct 09, 2016 at 08:29:11PM +0530, Harman Kalra wrote:
> This patch replaces calls to the "printk" with appropriate "pr_*"
> function thus addressing the following warning generated by the
> checkpatch script:
>
> Prefer [subsystem eg: netdev]_warn([subsystem]dev, ... then
> dev_wa
Nicholas,
some of my qemu tests for ppc64 started failing on mainline (and -next).
You can find a test log at
http://kerneltests.org/builders/qemu-ppc64-master/builds/580/steps/qemubuildcommand/logs/stdio
The scripts to run the test are available at
https://github.com/groeck/linux-build-test/tree
On Sun, Oct 09, 2016 at 05:10:31PM +0200, Sergio Paracuellos wrote:
> This patch fix the following checkpatch.pl warnings in prism2sta.c:
> WARNING: line over 80 characters
>
> Signed-off-by: Sergio Paracuellos
> ---
> drivers/staging/wlan-ng/prism2sta.c | 9 ++---
> 1 file changed, 6 insert
Can you send this one to Dave for 4.9? It would be silly to rename
something one merge window after it's just been introduced.
This patch fix the following sparse warnings in prism2fw.c:
warning: memset with byte count of 12
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/prism2fw.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/wlan-ng/prism2fw.c
b/dri
This patch replaces call to printk with appropriate pr_*
function thus addressing the following warning generated by the
checkpatch script:
Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then
dev_dbg(dev, ... then pr_debug(... to printk(KERN_DEBUG ...
Signed-off-by: Harman Kalra
This patch resolves the following error caught by checkpatch.pl:
ERROR: that open brace { should be on the previous line
Signed-off-by: Harman Kalra
---
drivers/staging/i4l/icn/icn.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/i4l/icn/icn.c b/drivers/
This patch resolves the following error caught by checkpatch.pl:
ERROR: that open brace { should be on the previous line
Signed-off-by: Harman Kalra
---
drivers/staging/i4l/icn/icn.h |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/i4l/icn/icn.h b/drivers/
On Sun, Oct 09, 2016 at 05:14:52PM +0200, Sergio Paracuellos wrote:
> This patch fix the following sparse warnings in prism2fw.c:
> warning: memset with byte count of 12
>
> Signed-off-by: Sergio Paracuellos
> ---
> drivers/staging/wlan-ng/prism2fw.c | 11 ---
> 1 file changed, 8 ins
On Sun, Oct 09, 2016 at 05:29:22PM +0200, Sergio Paracuellos wrote:
> This patch fix the following sparse warnings in prism2fw.c:
> warning: memset with byte count of 12
>
> Signed-off-by: Sergio Paracuellos
> ---
> drivers/staging/wlan-ng/prism2fw.c | 11 ---
> 1 file changed, 8 ins
Just sitting in front of my laptop :) Thanks to you for code review!
Cheers,
Sergio Paracuello
El 2016年10月09日 a las 17:36, Greg KH escribió:
On Sun, Oct 09, 2016 at 05:29:22PM +0200, Sergio Paracuellos wrote:
This patch fix the following sparse warnings in prism2fw.c:
warning: memset w
This warnings seems to be fixed in the new tree, so I don't resend anything.
El 2016年10月09日 a las 17:23, Greg KH escribió:
On Sun, Oct 09, 2016 at 05:10:31PM +0200, Sergio Paracuellos wrote:
This patch fix the following checkpatch.pl warnings in prism2sta.c:
WARNING: line over 80 characters
Si
This patch fixes the following checkpatch.pl warning in p80211netdev.h
Block comments should align the * on each line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/p80211netdev.h | 100 -
1 file changed, 50 insertions(+), 50 deletions(-)
diff --gi
1 - 100 of 281 matches
Mail list logo