Re: [PATCH] cpufreq: schedutil: Update last_update from sugov_set_iowait_boost()

2017-07-18 Thread Juri Lelli
On 18/07/17 16:55, Viresh Kumar wrote: > On 18-07-17, 12:20, Juri Lelli wrote: > > Hi Viresh, > > > > On 18/07/17 10:24, Viresh Kumar wrote: [...] > > > > It actually belongs here, IMHO. We update other fields (util, max, > > flags) > > Yeah, because they have bigger roles and aren't specific

Re: [PATCH v5 4/4]: perf/core: complete replace of lists by rb trees for pinned and flexible groups at perf_event_context

2017-07-18 Thread Alexander Shishkin
Alexander Shishkin writes: > Alexey Budankov writes: > >> Are there any new comments so far? Could you please suggest further steps >> forward? > > Apparently the patches are not threaded, so one needs to fish them out > one by one in order to review. And I can't seem to find the cover letter

Re: [PATCH] drm/rockchip: cdn-dp: send audio infoframe to sink

2017-07-18 Thread Chris Zhong
Hi Sean Thanks for your replying. On Tuesday, July 18, 2017 04:23 AM, Sean Paul wrote: On Sat, Jul 15, 2017 at 07:00:18PM +0800, Chris Zhong wrote: Some DP/HDMI sink need to receive the audio infoframe to play sound, especially some multi-channel AV receiver, they need the channel_allocation

Re: [PATCH] osq_lock: avoid live-lock issue for RT task

2017-07-18 Thread Peter Zijlstra
On Fri, Jul 14, 2017 at 07:19:09PM +0530, Prateek Sood wrote: > Live Lock due to task spinning while unqueue of CPU osq_node > from optimistic_spin_queue. Task T1 had decremented mutex count to > acquire the lock on CPU0. Before setting owner it got preempted. You've been working on ancient kerne

Re: [PATCH 6/7] staging: fsl-mc: rewrite mc command send/receive to work on 32-bits

2017-07-18 Thread Arnd Bergmann
On Tue, Jul 18, 2017 at 1:08 PM, Laurentiu Tudor wrote: > On 07/17/2017 06:00 PM, Arnd Bergmann wrote: >> Strictly speaking the __raw_writel() won't guarantee that the >> data is written as a single word, the compiler might decide to >> split it up into byte-sized writes if it believes the destin

Re: Session 152 terminated -- received PADT from peer

2017-07-18 Thread Sekar D
Please let me know if someone can help here. On Mon, Jul 17, 2017 at 12:07 PM, Sekar D wrote: > Hello All, > > I am getting the following error messages through PPP connection > without any manual intervention. > > Session 152 terminated -- received PADT from peer > PADT: Generic-Error: RP-PPPoE:

[PATCH v1 1/2] drm/bridge/synopsys: dsi: Constify funcs structures

2017-07-18 Thread Philippe CORNU
Signed-off-by: Philippe CORNU --- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c index 36f5ccb..63c7a01 100644 --- a/drivers/gpu/drm/br

[PATCH v1 0/2] drm/bridge/synopsys: dsi: Various cleanups

2017-07-18 Thread Philippe CORNU
Version 1: - Initial commit The purpose of this set of patches is to clean up the mipi dsi dw Synopsys drm bridge. Philippe CORNU (2): drm/bridge/synopsys: dsi: Constify funcs structures drm/bridge/synopsys: dsi: Register list clean up drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 79

[PATCH v1 2/2] drm/bridge/synopsys: dsi: Register list clean up

2017-07-18 Thread Philippe CORNU
This patch cleans up the Synopsys mipi dsi register list: - remove unused registers - rename registers according to the Synopsys documentation (1.30 & 1.31) - fix typos - re-order registers for a better coherency Signed-off-by: Philippe CORNU --- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c |

Re: [PATCH v10 00/15] Replace PCI pool by DMA pool API

2017-07-18 Thread Romain Perier
Hello, Le 09/07/2017 à 23:30, Doug Ledford a écrit : > On Thu, 2017-07-06 at 10:12 +0200, Romain Perier wrote: >> The current PCI pool API are simple macro functions direct expanded >> to >> the appropriate dma pool functions. The prototypes are almost the >> same >> and semantically, they are ve

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Vivek Goyal
On Mon, Jul 17, 2017 at 04:50:22PM -0400, Stefan Berger wrote: > On 07/17/2017 02:58 PM, Vivek Goyal wrote: > > On Tue, Jul 11, 2017 at 11:05:11AM -0400, Stefan Berger wrote: > > > > [..] > > > +/* > > > + * xattr_list_userns_rewrite - Rewrite list of xattr names for user > > > namespaces > > > +

[PATCH v2 3/7] ALSA: ad1848: fix format string overflow warning

2017-07-18 Thread Arnd Bergmann
The snd_pcm name is too long to fit into the card shortname or a part of the longname: sound/isa/ad1848/ad1848.c: In function 'snd_ad1848_probe': sound/isa/ad1848/ad1848.c:116:26: error: ' at 0x' directive writing 6 bytes into a region of size between 1 and 80 [-Werror=format-overflow=] sprintf

[PATCH v2 1/7] ALSA: als100: fix format string overflow warning

2017-07-18 Thread Arnd Bergmann
The compiler sees that the format string might overflow for the longname: sound/isa/als100.c: In function 'snd_als100_pnp_detect': sound/isa/als100.c:225:27: error: ', dma ' directive writing 6 bytes into a region of size between 0 and 64 [-Werror=format-overflow=] sprintf(card->longname, "%s,

[PATCH v2 2/7] ALSA: cs423x: fix format string overflow warning

2017-07-18 Thread Arnd Bergmann
The snd_pcm name may overflow the card->longname total size: sound/isa/cs423x/cs4231.c: In function 'snd_cs4231_probe': sound/isa/cs423x/cs4231.c:115:26: error: ' at 0x' directive writing 6 bytes into a region of size between 1 and 80 [-Werror=format-overflow=] 0x%lx, irq %d, dma %d", sprintf(

[PATCH v2 5/7] ALSA: mixart: fix string overflow warning

2017-07-18 Thread Arnd Bergmann
Using a temporary string produces warnings about a possible overflow in sprintf: sound/pci/mixart/mixart.c: In function 'snd_mixart_probe': sound/pci/mixart/mixart.c:1353:28: error: ' [PCM #' directive writing 7 bytes into a region of size between 1 and 32 [-Werror=format-overflow=] sprintf(ca

Re: [PATCH V1 15/15] spmi: pmic-arb: instantiate spmi_devices at arch_initcall

2017-07-18 Thread kgunda
On 2017-06-01 03:37, Stephen Boyd wrote: On 05/30, Kiran Gunda wrote: @@ -1384,7 +1384,12 @@ static int spmi_pmic_arb_remove(struct platform_device *pdev) .of_match_table = spmi_pmic_arb_match_table, }, }; -module_platform_driver(spmi_pmic_arb_driver); + +int __init spm

[PATCH v2 4/7] ALSA: opti9xx: fix format string overflow warning

2017-07-18 Thread Arnd Bergmann
We pass a long name from "codec->pcm->name" into the longname string of the same length: sound/isa/opti9xx/miro.c: In function 'snd_miro_probe': sound/isa/opti9xx/miro.c:1356:39: error: '%s' directive writing up to 79 bytes into a region of size between 35 and 72 [-Werror=format-overflow=] spri

[PATCH v2 6/7] ALSA: rme9652: fix format overflow warnings

2017-07-18 Thread Arnd Bergmann
gcc-7 warns about a possible sprintf format string overflow with a temporary buffer that is used to print from another buffer of the same size: sound/pci/rme9652/hdspm.c: In function 'snd_hdspm_create_alsa_devices': sound/pci/rme9652/hdspm.c:2123:17: error: ' MIDIoverMADI' directive writing 13 by

[PATCH v2 7/7] ALSA: pcxhr: fix string overflow warnings

2017-07-18 Thread Arnd Bergmann
With gcc-7, we get a warning about a possible string overflow: sound/pci/pcxhr/pcxhr.c: In function 'pcxhr_probe': sound/pci/pcxhr/pcxhr.c:1647:28: error: ' [PCM #' directive writing 7 bytes into a region of size between 1 and 32 [-Werror=format-overflow=] The shortname can simply be removed, an

Re: [PATCH 20/22] sound: pci: avoid string overflow warnings

2017-07-18 Thread Arnd Bergmann
On Fri, Jul 14, 2017 at 2:28 PM, Takashi Iwai wrote: > On Fri, 14 Jul 2017 14:07:12 +0200, > > Thanks for the patch. I have seen it but ignored, so far, as not sure > which action is the best. An alternative solution is to use > snprintf() blindly, for example. > > For mixart, it's even better t

RE: [PATCH] Coccinelle report script for refcounters

2017-07-18 Thread Julia Lawall
On Tue, 18 Jul 2017, Reshetova, Elena wrote: > > On Tue, 18 Jul 2017, Elena Reshetova wrote: > > > > > The below script can be used to detect potential misusage > > > of atomic_t type and API for reference counting purposes. > > > Now when we have a dedicated refcount_t type and API with > > > s

Re: [PATCH V1 14/15] spmi: pmic-arb: do not ack and clear peripheral interrupts in cleanup_irq

2017-07-18 Thread kgunda
On 2017-06-13 07:40, Stephen Boyd wrote: On 06/06, kgu...@codeaurora.org wrote: On 2017-06-02 12:56, kgu...@codeaurora.org wrote: >On 2017-05-31 23:23, Stephen Boyd wrote: >>On 05/30, Kiran Gunda wrote: >>>From: Subbaraman Narayanamurthy >>> >>>Currently, cleanup_irq() is invoked when a periphe

Re: [PATCH RFC 2/2] dt-bindings: add binding documentation for Allwinner CSI

2017-07-18 Thread Baruch Siach
Hi Yong, I am trying to get this driver working on the Olimex A33 OLinuXino. I didn't get it working yet, but I had some progress. See the comment below on one issue I encountered. On Tue, Jun 27, 2017 at 07:07:34PM +0800, Yong Deng wrote: > Add binding documentation for Allwinner CSI. > > Sig

Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera

2017-07-18 Thread Hans Verkuil
On 12/07/17 22:01, Sylwester Nawrocki wrote: > Hi Hugues, > > On 07/03/2017 11:16 AM, Hugues Fruchet wrote: >> This patchset enables OV9655 camera support. >> >> OV9655 support has been tested using STM32F4DIS-CAM extension board >> plugged on connector P1 of STM32F746G-DISCO board. >> Due to lack

Re: [PATCH 0/9] make clk_get_rate implementations behavior more consistent

2017-07-18 Thread Geert Uytterhoeven
Hi Jonas, On Tue, Jul 18, 2017 at 12:17 PM, Jonas Gorski wrote: > The common clock and several other clock API implementations allow > calling clk_get_rate with a NULL pointer. While not specified as > expected behavior of the API, device drivers have come to rely on that, > causing them to OOPS

Re: [PATCH v5 1/4]: perf/core: use rb trees for pinned/flexible groups

2017-07-18 Thread Alexander Shishkin
Alexey Budankov writes: > +static void > +perf_event_groups_rotate(struct perf_event_groups *groups, int cpu) > +{ > + struct rb_node *node; > + struct perf_event *node_event; > + > + WARN_ON_ONCE(!groups); This seems redundant. > + > + list_rotate_left(&groups->list); > + > +

Re: [PATCH 4/9] m68k: allow NULL clock for clk_get_rate

2017-07-18 Thread Greg Ungerer
Hi Jonas, On 18/07/17 20:17, Jonas Gorski wrote: Make the behaviour of clk_get_rate consistent with common clk's clk_get_rate by accepting NULL clocks as parameter. Some device drivers rely on this, and will cause an OOPS otherwise. Fixes: facdf0ed4f59 ("m68knommu: introduce basic clk infrastru

Re: [PATCH v10 00/38] x86: Secure Memory Encryption (AMD)

2017-07-18 Thread Thomas Gleixner
On Mon, 17 Jul 2017, Tom Lendacky wrote: > This patch series provides support for AMD's new Secure Memory Encryption > (SME) > feature. > > SME can be used to mark individual pages of memory as encrypted through the > page tables. A page of memory that is marked encrypted will be automatically >

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Stefan Berger
On 07/18/2017 07:48 AM, Vivek Goyal wrote: On Mon, Jul 17, 2017 at 04:50:22PM -0400, Stefan Berger wrote: On 07/17/2017 02:58 PM, Vivek Goyal wrote: On Tue, Jul 11, 2017 at 11:05:11AM -0400, Stefan Berger wrote: [..] +/* + * xattr_list_userns_rewrite - Rewrite list of xattr names for user nam

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Daniel Vetter
On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: > Hi Maxime, > > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > > On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > > > On Thursday 13 Jul 2017 16:41:13 Maxime Ripard wrote: > > >> The current drm_atomic_he

Re: [PATCH] drm/i915: Synchronize connectors states when switching from poll to irq

2017-07-18 Thread Paul Kocialkowski
On Mon, 2017-06-26 at 15:32 +0300, Paul Kocialkowski wrote: > After detecting an IRQ storm, hotplug detection will switch from > irq-based detection to poll-based detection. After a short delay or > when resetting storm detection from debugfs, detection will switch > back to being irq-based. > > H

Re: [BUG] x86/mm: Found insecure W+X mapping at address ffff88000005f000/0xffff88000005f000

2017-07-18 Thread Thomas Gleixner
On Tue, 18 Jul 2017, Piotr Gregor wrote: > > Dmesg reports insecure W+X mapping found at address > 8805f000/0x8805f000 > > on 4.4.70 kernel patched with -rt83 patch: Does the same problem happen with a plain 4.4.70? as well? Thanks, tglx

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Stefan Berger
On 07/18/2017 03:01 AM, James Morris wrote: On Thu, 13 Jul 2017, Stefan Berger wrote: A file shared by 2 containers, one mapping root to uid=1000, the other mapping root to uid=2000, will show these two xattrs on the host (init_user_ns) once these containers set xattrs on that file. I may be m

Re: perf: bisected sampling bug in Linux 4.11-rc1

2017-07-18 Thread Ingo Molnar
* Alexander Shishkin wrote: > Vince Weaver writes: > > > I was tracking down some regressions in my perf_event_test testsuite. > > Some of the tests broke in the 4.11-rc1 timeframe. > > > > I've bisected one of them, this report is about > > tests/overflow/simul_oneshot_group_overflow > >

Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera

2017-07-18 Thread H. Nikolaus Schaller
Hi, > Am 18.07.2017 um 13:59 schrieb Hans Verkuil : > > On 12/07/17 22:01, Sylwester Nawrocki wrote: >> Hi Hugues, >> >> On 07/03/2017 11:16 AM, Hugues Fruchet wrote: >>> This patchset enables OV9655 camera support. >>> >>> OV9655 support has been tested using STM32F4DIS-CAM extension board >>>

[tip:perf/urgent] perf/x86/intel: Add Goldmont Plus CPU PMU support

2017-07-18 Thread tip-bot for Kan Liang
Commit-ID: dd0b06b551f6b14da19582e301814746d838965a Gitweb: http://git.kernel.org/tip/dd0b06b551f6b14da19582e301814746d838965a Author: Kan Liang AuthorDate: Wed, 12 Jul 2017 09:44:23 -0400 Committer: Ingo Molnar CommitDate: Tue, 18 Jul 2017 14:13:40 +0200 perf/x86/intel: Add Goldmont P

[tip:perf/urgent] perf/x86/intel: Fix debug_store reset field for freq events

2017-07-18 Thread tip-bot for Jiri Olsa
Commit-ID: dc853e26f73e903e0c87e24f2695b5dcf33b3bc1 Gitweb: http://git.kernel.org/tip/dc853e26f73e903e0c87e24f2695b5dcf33b3bc1 Author: Jiri Olsa AuthorDate: Fri, 14 Jul 2017 18:35:51 +0200 Committer: Ingo Molnar CommitDate: Tue, 18 Jul 2017 14:13:41 +0200 perf/x86/intel: Fix debug_stor

[tip:perf/urgent] perf/x86/intel: Enable C-state residency events for Apollo Lake

2017-07-18 Thread tip-bot for Harry Pan
Commit-ID: 5c10b048c37cc08a21fa97a0575eccf4948948ca Gitweb: http://git.kernel.org/tip/5c10b048c37cc08a21fa97a0575eccf4948948ca Author: Harry Pan AuthorDate: Mon, 17 Jul 2017 18:37:49 +0800 Committer: Ingo Molnar CommitDate: Tue, 18 Jul 2017 14:13:40 +0200 perf/x86/intel: Enable C-state

Re: [PATCH resend] checkpatch: Replace __init_refok by __refconst and __refdata

2017-07-18 Thread Geert Uytterhoeven
Hi Joe, On Mon, Jul 17, 2017 at 5:01 PM, Joe Perches wrote: > On Mon, 2017-07-17 at 15:42 +0200, Geert Uytterhoeven wrote: >> As of commit bd721ea73e1f9655 ("treewide: replace obsolete _refok by >> __ref"), __init_refok no longer exists, so it can be removed. >> While at it, add the modern varian

Re: [PATCH] audit: fix memleak in auditd_send_unicast_skb.

2017-07-18 Thread Paul Moore
On Tue, Jul 18, 2017 at 2:37 AM, wrote: > 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 > backtr

Re: [PATCH] Free skb at error context in auditd_send_unicast_skb().

2017-07-18 Thread Paul Moore
On Mon, Jul 17, 2017 at 11:50 AM, Masami Ichikawa wrote: > I got following memory leak reports by kmemleak. > > unreferenced object 0x965962fa0600 (size 256): > comm "auditd", pid 401, jiffies 4294671604 (age 62.331s) > hex dump (first 32 bytes): > 00 00 00 00 00 00 00 00 00 00 00 00 0

RE: [PATCH] Coccinelle report script for refcounters

2017-07-18 Thread Reshetova, Elena
> On Tue, 18 Jul 2017, Reshetova, Elena wrote: > > > > On Tue, 18 Jul 2017, Elena Reshetova wrote: > > > > > > > The below script can be used to detect potential misusage > > > > of atomic_t type and API for reference counting purposes. > > > > Now when we have a dedicated refcount_t type and API

Re: [patch 0/3] Re: tty contention resulting from tty_open_by_device export

2017-07-18 Thread Dan Carpenter
On Tue, Jul 18, 2017 at 12:29:52PM +0100, Okash Khawaja wrote: > +struct tty_struct *tty_kopen(dev_t device) > +{ > + struct tty_struct *tty; > + struct tty_driver *driver = NULL; > + int index = -1; > + > + mutex_lock(&tty_mutex); > + driver = tty_lookup_driver(device

[PATCH v3 3/3] Staging: iio: adc: ad7280a.c: Fixed Macro argument reuse

2017-07-18 Thread Jaya Durga
CHECK: Macro argument reuse 'addr' - possible side-effects? convert AD7280A_DEVADDR to ad7280a_devaddr static function to fix checkpath check v3: small style changes Signed-off-by: Jaya Durga --- drivers/staging/iio/adc/ad7280a.c | 21 + 1 file changed, 13 insertions(+), 8

Re: [PATCH v5 1/4]: perf/core: use rb trees for pinned/flexible groups

2017-07-18 Thread Alexander Shishkin
Alexey Budankov writes: > +/* > + * Helper function to test if event groups are empty; > + */ > +static int > +perf_event_groups_empty(struct perf_event_groups *groups) > +{ > + return list_empty(&groups->list); > +} This doesn't seem useful, it's only used once. Also, it's not clear how acc

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Vivek Goyal
On Tue, Jul 18, 2017 at 08:05:18AM -0400, Stefan Berger wrote: > On 07/18/2017 07:48 AM, Vivek Goyal wrote: > > On Mon, Jul 17, 2017 at 04:50:22PM -0400, Stefan Berger wrote: > > > On 07/17/2017 02:58 PM, Vivek Goyal wrote: > > > > On Tue, Jul 11, 2017 at 11:05:11AM -0400, Stefan Berger wrote: > >

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

2017-07-18 Thread Oliver Neukum
Am Dienstag, den 18.07.2017, 17:56 +0800 schrieb jeffy: > Hi Oliver, > > > I think that as soon as one URB fails, you should not even try > > to submit any other deferred URBs. You are taking one out from > > the middle of a sequence. That cannot be right. > ok, that make sense. > > new patch sen

[PATCH] perf report: Enable finding the kernel inline function

2017-07-18 Thread Jin Yao
Currently perf has supported a mode to query inline stack. It works well for finding the user space inline function but it doesn't work for finding kernel inline function due to some unnecessary checking. This patch removes these unnecessary checking. Now the kernel inline function can be reported

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Vivek Goyal
On Tue, Jul 18, 2017 at 08:30:09AM -0400, Vivek Goyal wrote: > On Tue, Jul 18, 2017 at 08:05:18AM -0400, Stefan Berger wrote: > > On 07/18/2017 07:48 AM, Vivek Goyal wrote: > > > On Mon, Jul 17, 2017 at 04:50:22PM -0400, Stefan Berger wrote: > > > > On 07/17/2017 02:58 PM, Vivek Goyal wrote: > > >

[PATCH 1/3] iio: adc: stm32: fix common clock rate

2017-07-18 Thread Fabrice Gasnier
Fixes commit 95e339b6e85d ("iio: adc: stm32: add support for STM32H7") Fix common clock rate used then by stm32-adc sub-devices: take common prescaler into account. Fix ADC max clock rate on STM32H7 (fADC from datasheet) Signed-off-by: Fabrice Gasnier --- drivers/iio/adc/stm32-adc-core.c | 10 +

[PATCH 2/3] dt-bindings: iio: adc: stm32: add optional min-sample-time

2017-07-18 Thread Fabrice Gasnier
STM32 ADC allows each channel to be sampled with a different sampling time. There's an application note that deals with this: 'How to get the best ADC accuracy in STM32...' It basically depends on analog input signal electrical properties (depends on board). Add optional 'min-sample-time' property

[PATCH 3/3] iio: adc: stm32: add optional min-sample-time

2017-07-18 Thread Fabrice Gasnier
STM32 ADC allows each channel to be sampled with a different sampling time, by setting SMPR registers. Basically, value depends on local electrical properties. Selecting correct value for sampling time highly depends on analog source impedance. There is a manual that may help in this process: 'How

Re: [PATCH] perf/x86/intel: Add proper condition to run sched_task callbacks

2017-07-18 Thread Peter Zijlstra
On Tue, Jul 18, 2017 at 11:29:32AM +0200, Jiri Olsa wrote: > because we have 2 places using the same callback > - PEBS drain for free running counters > - LBR save/store > > both of them called from intel_pmu_sched_task > > so let's say PEBS drain setup the callback for the event, > but in t

[PATCH 0/3] Allow to tune sampling time on STM32 ADC

2017-07-18 Thread Fabrice Gasnier
STM32 ADC allows each channel to be sampled with a different sampling time. This series fixes common clock rate and adds optional device tree property, so minimum sampling time can be tuned. This is done via device tree as it's tightly coupled with hardware (analog source). Fabrice Gasnier (3):

Re: Session 152 terminated -- received PADT from peer

2017-07-18 Thread James Carlson
On 07/18/17 07:42, Sekar D wrote: > Please let me know if someone can help here. Probably not, but I'll give a try. >> I am getting the following error messages through PPP connection >> without any manual intervention. >> >> Session 152 terminated -- received PADT from peer PADT is the PPPoE te

[PATCH] Staging: pi433: Coding style fixes

2017-07-18 Thread Simo Koskinen
Fixes if-statement related warning and errors reported by checkpatch.pl Signed-off-by: Simo Koskinen --- drivers/staging/pi433/pi433_if.c | 253 +-- drivers/staging/pi433/rf69.c | 69 ++- 2 files changed, 148 insertions(+), 174 deletions(-) diff

[PATCH][next] spi: loopback-test: make several module parameters static

2017-07-18 Thread Colin King
From: Colin Ian King Several module parameters are local to the source, so make them static. Cleans up several sparse warnings such as: "symbol 'loop_req' was not declared. Should it be static?" Signed-off-by: Colin Ian King --- drivers/spi/spi-loopback-test.c | 14 +++--- 1 file chan

Re: [PATCH v2 1/2] livepatch: introduce shadow variable API

2017-07-18 Thread Petr Mladek
On Wed 2017-06-28 11:37:26, Joe Lawrence wrote: > diff --git a/Documentation/livepatch/shadow-vars.txt > b/Documentation/livepatch/shadow-vars.txt > new file mode 100644 > index ..7f28982e6b1c > --- /dev/null > +++ b/Documentation/livepatch/shadow-vars.txt > +Use cases > +- > +

Re: [PATCH v5 4/4]: perf/core: complete replace of lists by rb trees for pinned and flexible groups at perf_event_context

2017-07-18 Thread Alexander Shishkin
Alexey Budankov writes: > diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h > index 7b2cddf..8e1967f 100644 > --- a/include/linux/perf_event.h > +++ b/include/linux/perf_event.h > @@ -603,13 +603,6 @@ struct perf_event { >*/ > struct list_headgroup

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Laurent Pinchart
Hi Daniel, On Tuesday 18 Jul 2017 14:08:39 Daniel Vetter wrote: > On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: > > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > >> On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > >>> On Thursday 13 Jul 2017 16:41:13

Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera

2017-07-18 Thread Hugues FRUCHET
On 07/18/2017 02:17 PM, H. Nikolaus Schaller wrote: > Hi, > >> Am 18.07.2017 um 13:59 schrieb Hans Verkuil : >> >> On 12/07/17 22:01, Sylwester Nawrocki wrote: >>> Hi Hugues, >>> >>> On 07/03/2017 11:16 AM, Hugues Fruchet wrote: This patchset enables OV9655 camera support. OV9655

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 12:36:29PM +0200, Johan Hovold wrote: > On Tue, Jul 18, 2017 at 11:06:55AM +0100, Mark Brown wrote: > > Reverts shouldn't be special - they're just regular patches and should > > have sensible changelogs like any others. > Stating that you're reverting a commit and which c

Re: [PATCH v2 1/2] livepatch: introduce shadow variable API

2017-07-18 Thread Petr Mladek
On Mon 2017-07-17 17:35:38, Miroslav Benes wrote: > On Thu, 13 Jul 2017, Josh Poimboeuf wrote: > > > On Wed, Jun 28, 2017 at 11:37:26AM -0400, Joe Lawrence wrote: > > > > > +Brief API summary > > > +- > > > + > > > +See the full API usage docbook notes in the livepatch/shadow.c >

[PATCH 2/3][staging-next] staging: pi433: Make functions rf69_set_bandwidth_intern static

2017-07-18 Thread Colin King
From: Colin Ian King The function rf69_set_bandwidth_intern is local to the source and do not need to be in global scope, so make it static. Also break overly wide line. Cleans up sparse warning: symbol 'update_share_count' was not declared. Should it be static? Signed-off-by: Colin Ian King -

[PATCH 1/3][staging-next] staging: pi433: Make a couple of functions static

2017-07-18 Thread Colin King
From: Colin Ian King The functions pi433_receive and pi433_tx_thread are local to the source and do not need to be in global scope, so make them static Cleans up sparse warnings: symbol 'pi433_receive' was not declared. Should it be static? symbol 'pi433_tx_thread' was not declared. Should it be

[PATCH 3/3][staging-next] staging: pi433: fix thread_run failure check

2017-07-18 Thread Colin King
From: Colin Ian King The check for thread_run failure is incorrect, use IS_ERR instead. Cleans up sparse error message: "error: incompatible types for operation (<)" Fixes: 874bcba65f9a ("staging: pi433: New driver") Signed-off-by: Colin Ian King --- drivers/staging/pi433/pi433_if.c | 3 +--

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Daniel Vetter
On Tue, Jul 18, 2017 at 2:47 PM, Laurent Pinchart wrote: > On Tuesday 18 Jul 2017 14:08:39 Daniel Vetter wrote: >> On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: >> > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: >> >> On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pin

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

2017-07-18 Thread Jonathan Corbet
On Tue, 18 Jul 2017 08:42:42 +0200 Daniel Vetter 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 structur

Re: [PATCH][next] spi: loopback-test: make several module parameters static

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 01:42:32PM +0100, Colin King wrote: > From: Colin Ian King It would be very helpful if you could ensure your commit author and e-mail From: line up so that git doesn't make your patches look like non-author signoffs. The [next] isn't needed in the subject line either, it

[RESEND PATCH v4] Bluetooth: btusb: Fix memory leak in play_deferred

2017-07-18 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

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

2017-07-18 Thread jeffy
Hi Oliver, On 07/18/2017 08:29 PM, Oliver Neukum wrote: Am Dienstag, den 18.07.2017, 17:56 +0800 schrieb jeffy: Hi Oliver, I think that as soon as one URB fails, you should not even try to submit any other deferred URBs. You are taking one out from the middle of a sequence. That cannot be rig

Re: [PATCH][next] spi: loopback-test: make several module parameters static

2017-07-18 Thread Colin Ian King
On 18/07/17 14:09, Mark Brown wrote: > On Tue, Jul 18, 2017 at 01:42:32PM +0100, Colin King wrote: >> From: Colin Ian King > > It would be very helpful if you could ensure your commit author and > e-mail From: line up so that git doesn't make your patches look like > non-author signoffs. > > The

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

2017-07-18 Thread Eric W. Biederman
Kees Cook writes: > 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_

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Stefan Berger
On 07/18/2017 08:30 AM, Vivek Goyal wrote: On Tue, Jul 18, 2017 at 08:05:18AM -0400, Stefan Berger wrote: On 07/18/2017 07:48 AM, Vivek Goyal wrote: On Mon, Jul 17, 2017 at 04:50:22PM -0400, Stefan Berger wrote: On 07/17/2017 02:58 PM, Vivek Goyal wrote: On Tue, Jul 11, 2017 at 11:05:11AM -04

Re: [PATCH 01/18] regmap: mmio: Add function to attach a clock

2017-07-18 Thread Mark Brown
On Mon, Jul 17, 2017 at 11:01:07AM +0200, Maxime Ripard wrote: > On Thu, Jul 13, 2017 at 05:01:42PM +0100, Mark Brown wrote: > > > This might be problematic if the clock to enable is stored in another > > > node. > > > Let's add a function that allows to attach a clock that has already been > > >

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

2017-07-18 Thread Peter Zijlstra
On Tue, Jul 18, 2017 at 11:14:57AM +0800, Li, Aubrey wrote: > On 2017/7/18 3:23, Peter Zijlstra wrote: > > On Fri, Jul 14, 2017 at 09:26:19AM -0700, Andi Kleen wrote: > >>> And as said; Daniel has been working on a better predictor -- now he's > >>> probably not used it on the network workload you'

[PATCH v2 1/3] mfd: syscon: update Atmel SMC binding doc

2017-07-18 Thread Ludovic Desroches
A new compatible string is introduced for SMC on sama5d2 to manage a different layout of the registers. Signed-off-by: Ludovic Desroches Acked-by: Rob Herring --- Documentation/devicetree/bindings/mfd/atmel-smc.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bin

[PATCH v2 2/3] mfd: syscon: atmel-smc: add helper to retrieve register layout

2017-07-18 Thread Ludovic Desroches
For HSMC controller, the register layout depends on the device i.e. the offset of setup, pulse, cycle, mode and timings registers is not the same. An helper is added to provide the correct register layout. Fixes: fe9d7cb22ef3 ("mfd: syscon: atmel-smc: Add new helpers to ease SMC regs manipulation"

[PATCH v2 0/3] Fix SMC layout register for sama5d2

2017-07-18 Thread Ludovic Desroches
Hi, The offset of some registers of the SMC is different between sama5d3/sama5d4 and sama5d2. An helper function has been added to retrieve the correct layout with the help of a new compatible string for sama5d2. The atmel-ebi device will store the register layout to allow the nand controller to s

[PATCH v2 3/3] ARM: dts: at91: sama5d2: use sama5d2 compatible string for SMC

2017-07-18 Thread Ludovic Desroches
A new compatible string has been introduced for sama5d2 SMC to allow to manage the registers mapping change. Signed-off-by: Ludovic Desroches --- arch/arm/boot/dts/sama5d2.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/

[PATCH 1/2] MIPS: ralink: Fix build error due to missing header

2017-07-18 Thread Harvey Hunt
Previously, was included before ralink_regs.h in all ralink files - leading to being implicitly included. After commit 26dd3e4ff9ac ("MIPS: Audit and remove any unnecessary uses of module.h") removed the inclusion of module.h from multiple places, some ralink platforms failed to build with the f

Re: [PATCH v1 1/2] drm/bridge/synopsys: dsi: Constify funcs structures

2017-07-18 Thread Laurent Pinchart
Hi Philippe, Thank you for the patch. On Tuesday 18 Jul 2017 13:43:51 Philippe CORNU wrote: > Signed-off-by: Philippe CORNU Reviewed-by: Laurent Pinchart > --- > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gp

[PATCH 2/2] MIPS: ralink: mt7620: Add missing header

2017-07-18 Thread Harvey Hunt
Fix a build error caused by not including . The following compilation errors are caused by the missing header: arch/mips/ralink/mt7620.c: In function ‘mt7620_get_cpu_pll_rate’: arch/mips/ralink/mt7620.c:431:2: error: implicit declaration of function ‘WARN_ON’ [-Werror=implicit-function-declarati

Re: [PATCH][next] spi: loopback-test: make several module parameters static

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 02:13:35PM +0100, Colin Ian King wrote: > On 18/07/17 14:09, Mark Brown wrote: > > It would be very helpful if you could ensure your commit author and > > e-mail From: line up so that git doesn't make your patches look like > > non-author signoffs. > ACK, will do. Want me

Re: [PATCH v2 3/9] regulator: mt6380: Add support for MT6380

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 05:49:23PM +0800, sean.w...@mediatek.com wrote: > + if (!info->modeset_mask) { > + dev_err(&rdev->dev, "regulator %s doesn't support set_mode\n", > + info->desc.name); > + return -EINVAL; > + } This is mostly fine but the

Re: [PATCH] Staging: pi433: Coding style fixes

2017-07-18 Thread Greg KH
On Tue, Jul 18, 2017 at 02:41:33PM +0200, Simo Koskinen wrote: > Fixes if-statement related warning and errors reported > by checkpatch.pl > > Signed-off-by: Simo Koskinen > --- > drivers/staging/pi433/pi433_if.c | 253 > +-- > drivers/staging/pi433/rf69.c

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Eric W. Biederman
Stefan Berger writes: > On 07/18/2017 03:01 AM, James Morris wrote: >> On Thu, 13 Jul 2017, Stefan Berger wrote: >> >>> A file shared by 2 containers, one mapping root to uid=1000, the other >>> mapping >>> root to uid=2000, will show these two xattrs on the host (init_user_ns) once >>> these co

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-18 Thread Eric W. Biederman
Vivek Goyal writes: > On Tue, Jul 18, 2017 at 08:30:09AM -0400, Vivek Goyal wrote: >> On Tue, Jul 18, 2017 at 08:05:18AM -0400, Stefan Berger wrote: >> > On 07/18/2017 07:48 AM, Vivek Goyal wrote: >> > > On Mon, Jul 17, 2017 at 04:50:22PM -0400, Stefan Berger wrote: >> > > > On 07/17/2017 02:58 P

[PATCH v2 4/8] staging: fsl-mc: fix compilation with non-generic msi domain ops

2017-07-18 Thread laurentiu.tudor
From: Laurentiu Tudor The bus driver relies on generic msi domain ops. Fix compilation for architectures that don't provide it (e.g. x86_64). Signed-off-by: Laurentiu Tudor --- Notes: -v2 -no changes drivers/staging/fsl-mc/bus/fsl-mc-msi.c| 4 drivers/staging/fs

[PATCH v2 0/7] staging: fsl-mc: make the driver compile on other architectures

2017-07-18 Thread laurentiu.tudor
From: Laurentiu Tudor Apart from a small change (first patch) which adds a missing comment, this series make the bus driver compile on other architectures, as per GregKH comment [1]. Compiled tested on: - booke powerpc (corenet{32,64}_smp_defconfig) with this ppc patch [2] - x86 (i386_defconfig

[PATCH v2 6/8] staging: fsl-mc: don't use raw device io functions

2017-07-18 Thread laurentiu.tudor
From: Laurentiu Tudor As raw device io functions are not portable and don't handle byte-order (triggering suspicion that endianness isn't handled well) switch to using the standard api. Since MC expects LE byte-order and the upper layers already take care of that, we need to trick the device io a

[PATCH v2 8/8] staging: fsl-mc: allow the driver compile multi-arch

2017-07-18 Thread laurentiu.tudor
From: Laurentiu Tudor Drop dependency on ARCH_LAYERSCAPE (which in turn depends on ARM64), thus leaving this driver compile on other architectures. Also, other drivers depending on the bus are updated to depend on ARCH_LAYERSCAPE until they'll also be made multi-arch. This was compiled tested on:

Re: [PATCH v5 4/4]: perf/core: complete replace of lists by rb trees for pinned and flexible groups at perf_event_context

2017-07-18 Thread Alexey Budankov
Hi, On 18.07.2017 14:33, Alexander Shishkin wrote: > Alexey Budankov writes: > >> Hi, > > Hi, > >> Are there any new comments so far? Could you please suggest further steps >> forward? > > Apparently the patches are not threaded, so one needs to fish them out > one by one in order to review.

Re: [PATCH v5 4/4]: perf/core: complete replace of lists by rb trees for pinned and flexible groups at perf_event_context

2017-07-18 Thread Alexey Budankov
Hi, On 18.07.2017 15:40, Alexander Shishkin wrote: > Alexey Budankov writes: > >> diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h >> index 7b2cddf..8e1967f 100644 >> --- a/include/linux/perf_event.h >> +++ b/include/linux/perf_event.h >> @@ -603,13 +603,6 @@ struct perf_even

Re: [PATCH v5 1/4]: perf/core: use rb trees for pinned/flexible groups

2017-07-18 Thread Alexey Budankov
Hi, On 18.07.2017 15:02, Alexander Shishkin wrote: > Alexey Budankov writes: > >> +static void >> +perf_event_groups_rotate(struct perf_event_groups *groups, int cpu) >> +{ >> +struct rb_node *node; >> +struct perf_event *node_event; >> + >> +WARN_ON_ONCE(!groups); > > This seems re

[PATCH v2 7/8] staging: fsl-mc: make the driver compile on 32-bit

2017-07-18 Thread laurentiu.tudor
From: Laurentiu Tudor Since there's no real constrain in MC to do only atomic 64-bit we can enable this driver on 32-bit platforms too. Include linux/io-64-nonatomic-hi-lo.h to make quad device io apis used in the driver available on 32-bit platforms. Signed-off-by: Laurentiu Tudor --- Notes:

Re: [PATCH v2 3/3] ARM: dts: at91: sama5d2: use sama5d2 compatible string for SMC

2017-07-18 Thread Nicolas Ferre
On 18/07/2017 at 15:22, Ludovic Desroches wrote: > A new compatible string has been introduced for sama5d2 SMC to allow to > manage the registers mapping change. > > Signed-off-by: Ludovic Desroches Acked-by: Nicolas Ferre > --- > arch/arm/boot/dts/sama5d2.dtsi | 2 +- > 1 file changed, 1 ins

[PATCH v2 5/8] staging: fsl-mc: fix formating of phys_addr_t on 32 bits

2017-07-18 Thread laurentiu.tudor
From: Laurentiu Tudor Use correct format specifier for phys_addr_t variables (%pa) instead of %llx. This fixes these warnings on 32 bit targets: "format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'phys_addr_t' [-Wformat=]" Signed-off-by: Laurentiu Tudor

Re: [PATCH v5 1/4]: perf/core: use rb trees for pinned/flexible groups

2017-07-18 Thread Alexey Budankov
Hi, On 18.07.2017 15:29, Alexander Shishkin wrote: > Alexey Budankov writes: > >> +/* >> + * Helper function to test if event groups are empty; >> + */ >> +static int >> +perf_event_groups_empty(struct perf_event_groups *groups) >> +{ >> +return list_empty(&groups->list); >> +} > > This doe

Re: [PATCH v1 2/2] drm/bridge/synopsys: dsi: Register list clean up

2017-07-18 Thread Laurent Pinchart
Hi Philippe, Thank you for the patch. On Tuesday 18 Jul 2017 13:43:52 Philippe CORNU wrote: > This patch cleans up the Synopsys mipi dsi register list: > - remove unused registers Is the documentation for the DSI transmitter core public ? If not, it could be useful to keep unused registers for

<    1   2   3   4   5   6   7   8   9   10   >