Re: [PATCH 3/3] hwrng: mxc-fsl - add support for Freescale RNGC

2017-07-17 Thread PrasannaKumar Muralidharan
Hi Martin, On 18 July 2017 at 02:46, Martin Kaiser wrote: > From: Steffen Trumtrar > > The driver is ported from Freescales Linux git and can be > found in the > > vendor/freescale/imx_2.6.35_maintain > > branch. > > According to that code, the RNGC is found on Freescales i.MX3/5 SoCs. >

Re: [PATCH] arm64: defconfig: enable CONFIG_UNIPHIER_WATCHDOG

2017-07-17 Thread Masahiro Yamada
2017-07-18 14:24 GMT+09:00 Keiji Hayashibara : > Enable the watchdog driver for UniPhier SoC > > Signed-off-by: Keiji Hayashibara > --- > arch/arm64/configs/defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig > index 6c7d1

Re: [PATCH] staging: iio: adc: add space around substraction operation

2017-07-17 Thread Jonathan Cameron
On 18 July 2017 06:38:08 BST, Hari Prasath wrote: >Fix checkpatch warning to add space around the substraction operation > >Signed-off-by: Hari Prasath >--- > drivers/staging/iio/adc/ad7280a.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/drivers/staging/iio/adc/ad7

Re: [RFC v2 5/6] drivers: boot_constraint: Add initial DT bindings

2017-07-17 Thread Viresh Kumar
On 17-07-17, 12:34, Rob Herring wrote: > On Thu, Jul 13, 2017 at 03:06:08PM +0530, Viresh Kumar wrote: > > On 12-07-17, 16:28, Rob Herring wrote: > > > Display is a pretty well known use case here. Do you have other > > > examples in mind? > > > > No, I don't. > > > > @Stephen: Do you have more c

Re: [PATCH 3/3] ghes_edac: add platform check to enable ghes_edac

2017-07-17 Thread Borislav Petkov
On Mon, Jul 17, 2017 at 03:59:12PM -0600, Toshi Kani wrote: > The ghes_edac driver was introduced in 2013 [1], but it has not > been enabled by any distro yet. This driver obtains error info > from firmware interfaces, which are not properly implemented on > many platforms, as the driver always em

Re: Droid 4 boot failure due to 422580c3cea7 (mm/oom_kill.c: add tracepoints for oom reaper-related events)

2017-07-17 Thread Tony Lindgren
* Sebastian Reichel [170717 04:52]: > Hi, > > On Mon, Jul 17, 2017 at 01:17:13PM +0200, Sebastian Reichel wrote: > > [dropping people from Cc] > > > > Hi, > > > > On Mon, Jul 17, 2017 at 03:19:28AM -0700, Tony Lindgren wrote: > > > * Sebastian Reichel [170717 02:52]: > > > > On Sun, Jul 16, 20

[PATCH v5 3/3] dt-bindings: cpufreq: enhance MediaTek cpufreq dt-binding document

2017-07-17 Thread sean.wang
From: Sean Wang Update binding document with adding operating-points-v2 as the required property and the cooling level as the optional properties and adding more examples guiding people how to use MediaTek cpufreq driver for MediaTek SoCs. Signed-off-by: Sean Wang Acked-by: Viresh Kumar Acked-

[PATCH v5 2/3] dt-bindings: cpufreq: move MediaTek cpufreq dt-bindings document to proper place

2017-07-17 Thread sean.wang
From: Sean Wang The old place is Documentation/devicetree/bindings/clock/ that would let people hard to find how to use MediaTek cpufreq driver, so moving it to the appropriate place as other cpufreq drivers done would be better. Signed-off-by: Sean Wang Acked-by: Viresh Kumar Acked-by: Rob He

[PATCH v5 1/3] cpufreq: mediatek: Add support of cpufreq to MT2701/MT7623 SoC

2017-07-17 Thread sean.wang
From: Sean Wang MT2701/MT7623 is a 32-bit ARMv7 based quad-core (4 * Cortex-A7) with single cluster and this hardware is also compatible with the existing driver through enabling CPU frequency feature with operating-points-v2 bindings. Also, this driver actually supports all MediaTek SoCs, the Kc

[PATCH v5 0/3] some fixups for MediaTek cpufreq driver

2017-07-17 Thread sean.wang
From: Sean Wang Changes since v4: - add back the missing Acked-by from v3 Changes since v3: - remove the redundant line depends on ARM || ARM64 || COMPILE_TEST since the whole Kconfig.arm file depends on ARM || ARM64. - renaming from mtk-cpufreq.c to mediatek-cpufreq.c avoiding collide and cause

[PATCH 1/5] serial: arc: Remove __init marking from early write

2017-07-17 Thread Jeffy Chen
The earlycon would be alive outside the init code in these cases: 1/ we have keep_bootcon in cmdline. 2/ we don't have a real console to switch to. So remove the __init marking to avoid invalid memory access. Signed-off-by: Jeffy Chen --- drivers/tty/serial/arc_uart.c | 4 ++-- 1 file changed,

[PATCH 2/5] serial: omap: Remove __init marking from early write

2017-07-17 Thread Jeffy Chen
The earlycon would be alive outside the init code in these cases: 1/ we have keep_bootcon in cmdline. 2/ we don't have a real console to switch to. So remove the __init marking to avoid invalid memory access. Signed-off-by: Jeffy Chen --- drivers/tty/serial/omap-serial.c | 13 ++--- 1

[PATCH 4/5] serial: 8250_ingenic: Remove __init marking from early write

2017-07-17 Thread Jeffy Chen
The earlycon would be alive outside the init code in these cases: 1/ we have keep_bootcon in cmdline. 2/ we don't have a real console to switch to. So remove the __init marking to avoid invalid memory access. Signed-off-by: Jeffy Chen --- drivers/tty/serial/8250/8250_ingenic.c | 8 1

[PATCH 5/5] serial: 8250_early: Remove __init marking from early write

2017-07-17 Thread Jeffy Chen
The earlycon would be alive outside the init code in these cases: 1/ we have keep_bootcon in cmdline. 2/ we don't have a real console to switch to. So remove the __init marking to avoid invalid memory access. Signed-off-by: Jeffy Chen --- drivers/tty/serial/8250/8250_early.c | 8 1 fi

[PATCH 3/5] serial: xuartps: Remove __init marking from early write

2017-07-17 Thread Jeffy Chen
The earlycon would be alive outside the init code in these cases: 1/ we have keep_bootcon in cmdline. 2/ we don't have a real console to switch to. So remove the __init marking to avoid invalid memory access. Signed-off-by: Jeffy Chen --- drivers/tty/serial/xilinx_uartps.c | 2 +- 1 file chang

[PATCH 0/5] earlycon hang under some conditions

2017-07-17 Thread Jeffy Chen
I was testing earlycon with 8250 dw serial console. And it hangs in these cases: 1/ kernel hang when calling early write function after free_initmem: a) the earlycon not disabled after the init code(due to keep_bootcon or not specify a real console to switch to) b) the early write func is marked

Re: [RFC PATCH 1/5] serial: arc: Remove __init marking from early write

2017-07-17 Thread jeffy
Hi Greg, On 07/18/2017 01:08 PM, Greg KH wrote: On Tue, Jul 18, 2017 at 12:29:59PM +0800, Jeffy Chen wrote: The earlycon would be alive outside the init code in these cases: 1/ we have keep_bootcon in cmdline. 2/ we don't have a real console to switch to. So remove the __init marking to avoid

Re: [PATCH v7 12/13] ACPI / init: Invoke early ACPI initialization earlier

2017-07-17 Thread Dou Liyang
Hi, Zheng At 07/18/2017 01:18 PM, Zheng, Lv wrote: Hi, Can the problem be fixed by invoking acpi_put_table() for mapped DMAR table? Invoking acpi_put_table() is my first choice. But it made the kernel *panic* when we try to get the table again in intel_iommu_init() in late stage. I am also c

[PATCH] efifb: allow user to disable write combined mapping.

2017-07-17 Thread Dave Airlie
This patch allows the user to disable write combined mapping of the efifb framebuffer console using an nowc option. A customer noticed major slowdowns while logging to the console with write combining enabled, on other tasks running on the same CPU. (10x or greater slow down on all other cores on

RE: [PATCH] arm64: defconfig: enable CONFIG_UNIPHIER_WATCHDOG

2017-07-17 Thread Keiji Hayashibara
Hello Yamada-san, > From: Masahiro Yamada [mailto:yamada.masah...@socionext.com] > Sent: Tuesday, July 18, 2017 2:51 PM > Subject: Re: [PATCH] arm64: defconfig: enable CONFIG_UNIPHIER_WATCHDOG > > 2017-07-18 14:24 GMT+09:00 Keiji Hayashibara > : > > Enable the watchdog driver for UniPhier SoC >

Re: fscrypt request_module() deadlock

2017-07-17 Thread Herbert Xu
On Fri, Jun 30, 2017 at 05:27:34PM +0200, Richard Weinberger wrote: > Hi! > > David and I faced a deadlock with switch_root when fscrypt was in use. > When /sbin/modprobe is encrypted using fscrypt and no other kernel component > requested an AES cipher before, first access to an encrypted file wi

Re: [PATCH] Stagung: rtl8188eu: core: fixd comment coding style issue in rtw_xmit.c

2017-07-17 Thread Greg KH
On Mon, Jul 17, 2017 at 10:53:21PM +0200, ArminSchoenlieb wrote: > From: Armin Schoenlieb > > This is a patch to the rtw_xmit.c file that fixes up a comment/80 character > warning found by the checkpatch.pl tool It's ironic that your changelog violates this same issue :(

Re: [RFC][PATCH] drm: kirin: Restrict modes to known good mode clocks

2017-07-17 Thread Daniel Vetter
On Mon, Jul 17, 2017 at 04:20:23PM -0700, John Stultz wrote: > On Tue, Jul 11, 2017 at 9:27 AM, Daniel Vetter wrote: > > On Tue, Jul 11, 2017 at 5:44 PM, John Stultz wrote: > >> On Tue, Jul 11, 2017 at 8:12 AM, Daniel Vetter wrote: > >>> On Tue, Jul 11, 2017 at 5:05 PM, John Stultz > >>> wrote

Re: [PATCH] x86/microcode: Document the three loading methods

2017-07-17 Thread Dominik Brodowski
Thanks for the nice write-up! A few comments below: On Mon, Jul 17, 2017 at 11:43:28AM +0200, Borislav Petkov wrote: > new file mode 100644 > index ..6ab130c6ca45 > --- /dev/null > +++ b/Documentation/x86/microcode.txt > @@ -0,0 +1,133 @@ > + The Linux Microcode Loader > + > +Autho

ping/icmp sockets: define "sane".

2017-07-17 Thread Rob Landley
The title is from this comment in net/ipv4: /* * Sane defaults - nobody may create ping sockets. * Boot scripts should set this to distro-specific group. */ So in 2011 you added ICMP sockets, but made it so nobody could use them without root performing a magic incatation at boot time. From

[PATCH V3 1/5] drivers/fmc: remove unused variable

2017-07-17 Thread Alessandro Rubini
From: Federico Vaga Signed-off-by: Federico Vaga Tested-by: Pat Riehecky Acked-by: Alessandro Rubini --- V3 (Alessandro): fixed From line in patch, added alessandro's acked-by V2 (Pat): added Tested-by and incorrect From line V1 (Pat): picked from ohwr.org repo, where most fmc users pick fr

[PATCH V3 5/5] drivers/fmc: carrier can program FPGA on registration

2017-07-17 Thread Alessandro Rubini
From: Federico Vaga The initial FPGA may require programming before it is useful. Signed-off-by: Federico Vaga Tested-by: Pat Riehecky Acked-by: Alessandro Rubini --- V3 (Alessandro): fixed From line in patch, added alessandro's acked-by V2 (Pat): added Tested-by and incorrect From line V1

[PATCH V3 2/5] drivers/fmc: hide fmc operations behind helpers

2017-07-17 Thread Alessandro Rubini
From: Federico Vaga This gave us more freedom to change/add/remove operations without recompiling all device driver. Typically, Carrier board implement the fmc operations, so they will not use these helpers. Signed-off-by: Federico Vaga Tested-by: Pat Riehecky Acked-by: Alessandro Rubini ---

[PATCH V3 3/5] drivers/fmc: The only way to dump the SDB is from debugfs

2017-07-17 Thread Alessandro Rubini
From: Federico Vaga Driver should not call fmc_sdb_dump() anymore. (actually they can but the operation is not supported, so it will print an error message) Signed-off-by: Federico Vaga Tested-by: Pat Riehecky Acked-by: Alessandro Rubini --- V3 (Alessandro): fixed From line in patch, added a

[PATCH V3 4/5] drivers/fmc: change registration prototype

2017-07-17 Thread Alessandro Rubini
From: Federico Vaga Permit use of either fmc_device_register_n or fmc_device_register_n_gw depending on the type of device in use. Signed-off-by: Federico Vaga Tested-by: Pat Riehecky Acked-by: Alessandro Rubini --- V3 (Alessandro): fixed From line in patch, added alessandro's acked-by V2 (

[PATCH V3 0/5] Update kernel to latest FMC bus release

2017-07-17 Thread Alessandro Rubini
Version 3: fixed From: line to set Federico Vaga as author, added my Acked-by: line. removed drivers_other@kernel-bugs as recipient (NXDOMAIN) Version 2 (posted by Pat Riehecky): added Tested-by: and (incorrect) From: Comments by Pat, original poster, for version 1: The following p

Re: [PATCH] kthread: Fix race condition between kthread_parkme() and kthread_unpark()

2017-07-17 Thread Thomas Gleixner
On Mon, 17 Jul 2017, Junaid Shahid wrote: > Hi, > > Has anyone been able to take a look at this? Yes. It's in my pile of stuff to look at.

[PATCH v3] Bluetooth: btusb: Fix memory leak in play_deferred

2017-07-17 Thread Jeffy Chen
Currently we are calling usb_submit_urb directly to submit deferred tx urbs after unanchor them. So the usb_giveback_urb_bh would failed to unref it in usb_unanchor_urb and cause memory leak: unreferenced object 0xffc0ce0fa400 (size 256): ... backtrace: [] __save_stack_trace+0x48/0x6c

[PATCH v2] arm64: defconfig: enable CONFIG_UNIPHIER_WATCHDOG

2017-07-17 Thread Keiji Hayashibara
Enable the watchdog driver for UniPhier SoC Signed-off-by: Keiji Hayashibara --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 6c7d147..629c815 100644 --- a/arch/arm64/configs/defconfig +++ b/arc

[PATCH] audit: fix memleak in auditd_send_unicast_skb.

2017-07-17 Thread shuwang
From: Shu Wang Found this issue by kmemleak report, auditd_send_unicast_skb did not free skb if rcu_dereference(auditd_conn) returns null. unreferenced object 0x88082568ce00 (size 256): comm "auditd", pid 1119, jiffies 4294708499 backtrace: [] kmemleak_alloc+0x4a/0xa0 [] kmem_cache_alloc_nod

Re: [PATCH v2 1/8] exec: Correct comments about "point of no return"

2017-07-17 Thread Kees Cook
On Mon, Jul 10, 2017 at 10:07 AM, Eric W. Biederman wrote: > Kees Cook writes: > >> On Mon, Jul 10, 2017 at 1:46 AM, Eric W. Biederman >> wrote: >>> >>> But you miss it. >>> >>> The "point of no return" is the call to de_thread. Or aguably anything in >>> flush_old_exec. Once anything in the c

Re: [PATCH v2] drm/i915: Consistently use enum pipe for PCH transcoders

2017-07-17 Thread Daniel Vetter
On Mon, Jul 17, 2017 at 11:14:03AM -0700, Matthias Kaehlcke wrote: > The current code uses in some instances enum transcoder for PCH > transcoders and enum pipe in others. This is error prone and clang > raises warnings like this: > > drivers/gpu/drm/i915/intel_dp.c:3546:51: warning: implicit conv

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-17 Thread Thomas Gleixner
On Mon, 17 Jul 2017, dbasehore . wrote: > On Mon, Jul 17, 2017 at 6:33 PM, Rafael J. Wysocki wrote: > I could make a patch to try it out. I would probably add a flag to rtc > timers to indicate whether it wakes the system (default true). We > would have to add a sync with the rtc irq and the rtc i

Re: [RFC PATCH v2] Bluetooth: btusb: Fix memory leak in play_deferred

2017-07-17 Thread Marcel Holtmann
Hi Oliver, > Currently we are calling usb_submit_urb directly to submit deferred tx > urbs after unanchor them. > > So the usb_giveback_urb_bh would failed to unref it in usb_unanchor_urb > and cause memory leak: > unreferenced object 0xffc0ce0fa400 (size 256): > ... > backtrace: >[] __s

Re: [PATCH 6/7] docs: Do not include from include/drm/drm_color_mgmt.h

2017-07-17 Thread Daniel Vetter
On Mon, Jul 17, 2017 at 03:00:47PM -0600, Jonathan Corbet wrote: > Commit 8f2e045ec878 (drm/color: un-inline drm_color_lut_extract()) moved > the only kerneldoc comment out of include/drm/drm_color_mgmt.h, leading to > this warning: > > ./include/drm/drm_color_mgmt.h:1: warning: no structured

Re: [RFC PATCH v1 00/11] Create fast idle path for short idle periods

2017-07-17 Thread Thomas Gleixner
On Mon, 17 Jul 2017, Andi Kleen wrote: > > We need a tradeoff here IMHO. I'll check Daniel's work to understand how/if > > it's better than menu governor. > > I still would like to see how the fast path without the C1 heuristic works. > > Fast pathing is a different concept from a better predict

Re: [PATCH v3] Bluetooth: btusb: Fix memory leak in play_deferred

2017-07-17 Thread Marcel Holtmann
Hi Oliver, > Currently we are calling usb_submit_urb directly to submit deferred tx > urbs after unanchor them. > > So the usb_giveback_urb_bh would failed to unref it in usb_unanchor_urb > and cause memory leak: > unreferenced object 0xffc0ce0fa400 (size 256): > ... > backtrace: >[] __s

Re: [PATCH v2 2/8] exec: Move security_bprm_secureexec() earlier

2017-07-17 Thread Kees Cook
On Mon, Jul 10, 2017 at 7:07 PM, Kees Cook wrote: > On Mon, Jul 10, 2017 at 10:18 AM, Eric W. Biederman > wrote: >> Kees Cook writes: >> >>> On Mon, Jul 10, 2017 at 1:57 AM, Eric W. Biederman >>> wrote: Kees Cook writes: > There are several places where exec needs to know if a pr

Re: WARN_ON_ONCE(work > weight) in napi_poll()

2017-07-17 Thread Ryan Hsu
On 07/11/2017 06:19 PM, Igor Mitsyanko wrote: > On 07/11/2017 10:28 AM, Andrey Ryabinin wrote: >> >> It gave me this: >> >> [118648.825347] #1 quota too big 72 64 16 >> [118648.825351] #2 quota too big 72 64 16 >> [118648.825471] [ cut here ] >> [118648.825484] WARNING: CPU

Re: [PATCH v7 1/1] f2fs: dax: implement direct access

2017-07-17 Thread Chao Yu
On 2017/7/18 11:45, sunqiuyang wrote: > From: Qiuyang Sun > > This patch implements Direct Access (DAX) in F2FS, including: > - a mount option to choose whether to enable DAX or not > - read/write and mmap of regular files in the DAX way > - zero-out of unaligned partial blocks in the DAX way > -

Re: [PATCH v2] staging: unisys: visorbus: Constify attribute_group structures.

2017-07-17 Thread Greg KH
On Tue, Jul 18, 2017 at 10:54:21AM +0530, Arvind Yadav wrote: > attribute_group are not supposed to change at runtime. All functions > working with attribute_group provided by work > with const attribute_group. So mark the non-const structs as const. > > Signed-off-by: Arvind Yadav > --- > Chang

[PATCH v2] PCI: dwc: designware: test PCIE_ATU_ENABLE bit to check enabled or not

2017-07-17 Thread Jisheng Zhang
The ATU CTRL2 register is 32 bit, besides the enable bit, other bits may also be set. To check whether the ATU is enabled or not, we should test the enable bit. Signed-off-by: Jisheng Zhang Acked-by: Joao Pinto --- Since v1: - Add Joao's Ack - Fix typo in commit msg, thank Jingoo drivers/pci

Re: [PATCH V3 1/5] drivers/fmc: remove unused variable

2017-07-17 Thread Greg KH
On Tue, Jul 18, 2017 at 08:32:42AM +0200, Alessandro Rubini wrote: > From: Federico Vaga > > Signed-off-by: Federico Vaga > Tested-by: Pat Riehecky > Acked-by: Alessandro Rubini > --- Again, I do not like to take patches without any changelog text at all :(

Re: [RFC PATCH v1 00/11] Create fast idle path for short idle periods

2017-07-17 Thread Li, Aubrey
On 2017/7/18 14:43, Thomas Gleixner wrote: > On Mon, 17 Jul 2017, Andi Kleen wrote: > >>> We need a tradeoff here IMHO. I'll check Daniel's work to understand how/if >>> it's better than menu governor. >> >> I still would like to see how the fast path without the C1 heuristic works. >> >> Fast pat

Re: [PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

2017-07-17 Thread Greg KH
On Mon, Jul 17, 2017 at 08:19:45PM +0530, Gaurav Pathak wrote: > staging: rts5208: Fix coding style issue caught by checkpatch.pl related to > function name in debug print and shorten few lines over 80 char No, only do one-type-of-thing per patch. It would make sense to maybe fix an 80 char line

Re: [PATCH] staging: wlan-ng: Fix endianness warnings

2017-07-17 Thread Greg KH
On Wed, Jul 12, 2017 at 09:38:50PM +0100, alfonsolimaas...@gmail.com wrote: > From: Alfonso Lima Astor > > These variables were set to u16 and u32 although they always hold a little > endian value. > > This patch fixes multiple sparse warnings like: > drivers/staging/wlan-ng/prism2sta.c:375:46:

Re: [PATCH v3 1/1] Staging: android/ion: declare function static

2017-07-17 Thread Greg KH
On Sat, Jul 15, 2017 at 11:43:05AM +, Joseph Wright wrote: > Declare private function static to fix sparse warning: > > ion_cma_heap.c:109:5: warning: symbol '__ion_add_cma_heaps' \ > was not declared. Should it be static? > > Signed-off-by: Joseph Wright > --- > Changes in v3: > - M

<    7   8   9   10   11   12