Re: [PATCH v2 12/15] [media] lirc_dev: fix error return value

2016-07-04 Thread Hans Verkuil
On 07/01/2016 10:01 AM, Andi Shyti wrote: > If ioctl is called, it cannot be a case of invalid system call > number (ENOSYS), that is an operation not permitted (EPERM). > Replace ENOSYS with EPERM. I'd say it is ENOTTY, i.e. this hardware does not support this ioctl. Regards, Hans > >

Re: [BUG] Regression, usb-dwc3 not working, Odroid XU3, usb 3.0

2016-07-04 Thread Felipe Balbi
Hi, Krzysztof Kozlowski writes: >>> Since three recent next releases (first on 20160630), on Odroid XU3 board >>> the USB 3.0 host port (the only one) stopped working. >>> >>> This is a "samsung,exynos5250-dwusb3" (drivers/usb/dwc3/dwc3-exynos.c). >>> >>> Two issues here: >>> 1. The port does no

Re: [PATCH] netfilter: nf_conntrack_sip: CSeq 0 is a valid CSeq

2016-07-04 Thread Liping Zhang
2016-07-04 14:14 GMT+08:00 Christophe Leroy : >> I think there is no need to convert simple_strtoul to kstrtouint, add >> a further check seems better? >> Like this: >> - if (!cseq) { >> + if (!cseq && *(*dptr + matchoff) != '0') { >> > > And what about an invalid CSeq that would look

Re: [PATCH 01/15 RESEND] staging: dgnc: remove redundant NULL checks in

2016-07-04 Thread DaeSeok Youn
2016-07-04 20:36 GMT+09:00 Daeseok Youn : > The dgnc_block_til_ready() is only used in dgnc_tty_open(). > The unit data(struct un_t) was stored into tty->driver_data in > dgnc_tty_open(). > And also tty and un were tested about NULL so these variables doesn't > need to check for NULL in dgnc_block

RE: [RFC PATCH v2 2/4] net: ethernet: xilinx: Add gmii2rgmii converter support

2016-07-04 Thread Appana Durga Kedareswara Rao
Hi Nicolas, Thanks for the review... > > diff --git a/include/linux/xilinx_gmii2rgmii.h > > b/include/linux/xilinx_gmii2rgmii.h > > new file mode 100644 > > index 000..b328ee7 > > --- /dev/null > > +++ b/include/linux/xilinx_gmii2rgmii.h > > @@ -0,0 +1,24 @@ > > > Here, header of th

[PATCH] mmc: core: Extend sysfs with OCR register

2016-07-04 Thread Bojan Prtvar
Make operation conditions register (OCR) easily accessible from user space. Signed-off-by: Bojan Prtvar --- Documentation/mmc/mmc-dev-attrs.txt | 1 + drivers/mmc/core/mmc.c | 2 ++ 2 files changed, 3 insertions(+) diff --git a/Documentation/mmc/mmc-dev-attrs.txt b/Documentation/m

Re: [PATCH v5 6/8] efi: load SSTDs from EFI variables

2016-07-04 Thread Matt Fleming
(Sorry, didn't get chance to reply before you sent this version) On Fri, 01 Jul, at 11:19:10PM, Octavian Purdila wrote: > + > +static __init int efivar_ssdt_load(void) > +{ > + struct efivar_entry *entry; > + /* We need a temporary empty list to be able to set duplicates > + * to true

Re: [PATCH v3] ARM: at91: Document new TCB bindings

2016-07-04 Thread Boris Brezillon
On Mon, 4 Jul 2016 12:36:31 +0200 Alexandre Belloni wrote: > On 04/07/2016 at 12:24:52 +0200, Boris Brezillon wrote : > > On Fri, 1 Jul 2016 23:52:05 +0200 > > Alexandre Belloni wrote: > > > +One interrupt per TC block: > > > + tcb0: timer@fff7c000 { > > > + compatible = "atmel,at91rm

[PATCH 1/2] drivers/media/dvb-core/en50221: use kref to manage struct dvb_ca_private

2016-07-04 Thread Max Kellermann
Don't free the object until the file handle has been closed. Fixes use-after-free bug which occurs when I disconnect my DVB-S received while VDR is running. This is a crash dump of such a use-after-free: general protection fault: [#1] SMP CPU: 0 PID: 2541 Comm: CI adapter on d Not t

[PATCH 2/2] dvb_frontend: eliminate blocking wait in dvb_unregister_frontend()

2016-07-04 Thread Max Kellermann
The wait_event() call in dvb_unregister_frontend() waits synchronously for other tasks to free a file descriptor, but it does that while holding several mutexes. That alone is a bad idea, but if one user process happens to keep a (defunct) file descriptor open indefinitely, the kernel will correct

Re: [PATCH v3] ARM: at91: Document new TCB bindings

2016-07-04 Thread Alexandre Belloni
On 04/07/2016 at 14:03:58 +0200, Boris Brezillon wrote : > On Mon, 4 Jul 2016 12:36:31 +0200 > Alexandre Belloni wrote: > > > On 04/07/2016 at 12:24:52 +0200, Boris Brezillon wrote : > > > On Fri, 1 Jul 2016 23:52:05 +0200 > > > Alexandre Belloni wrote: > > > > +One interrupt per TC block: >

Re: [RFC PATCH v1] irqchip: add support for SMP irq router

2016-07-04 Thread Mason
On 30/06/2016 18:03, Sebastian Frias wrote: > This adds support for a second-gen irq router/controller present > on some Sigma Designs chips. In the patch subject, do you mean SMP as in Symmetric Multi Processor? > Signed-off-by: Sebastian Frias Is that the address you intend to submit with?

[PATCH 4/4] perf tools: Call unwind__prepare_access for forked thread

2016-07-04 Thread Jiri Olsa
Currently we call unwind__prepare_access for map event. In case we report fork event the thread inherits its parent's maps and unwind__prepare_access is never called for the thread. This causes unwind__get_entries seeing uninitialized unwind_libunwind_ops and thus returning no callchain. Adding u

[PATCHv2 0/4] perf tools: Various fixes

2016-07-04 Thread Jiri Olsa
hi, this is v2 for original post: http://marc.info/?t=14671134543&r=1&w=2 v2 changes: - keep nr_numa_nodes in patch 1 - added patch 2,3,4 thanks, jirka --- Jiri Olsa (4): perf tools: Transform nodes string info to struct perf tests: Fix hist accumulation test perf to

[PATCH 2/4] perf tests: Fix hist accumulation test

2016-07-04 Thread Jiri Olsa
User's values from .perfconfig could overload the default callchain setup and cause this test to fail. Making sure the test is using default callchain_param values. Link: http://lkml.kernel.org/n/tip-7jyzfrdoglqfmht1xt2f83...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/tests/hists_cu

[PATCH 1/4] perf tools: Transform nodes string info to struct

2016-07-04 Thread Jiri Olsa
Storing NUMA info within struct numa_node instead of strings. This way it's usable in future patches. Also it turned out it's slightly less code involved than using strings. Link: http://lkml.kernel.org/n/tip-ka37sax3gfaxwvytfxi0y...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/e

[PATCH 3/4] perf tools: Add initialized arg into unwind__prepare_access

2016-07-04 Thread Jiri Olsa
Adding initialized arg into unwind__prepare_access to get feedback about the initialization state. It's not possible to get it from error code, because we return 0 even in case we don't recognize dso, which is valid. The 'initialized' value is used in following patch to speedup unwind__prepare_ac

Re: [PATCH v3] ARM: at91: Document new TCB bindings

2016-07-04 Thread Boris Brezillon
On Mon, 4 Jul 2016 14:11:27 +0200 Alexandre Belloni wrote: > On 04/07/2016 at 14:03:58 +0200, Boris Brezillon wrote : > > On Mon, 4 Jul 2016 12:36:31 +0200 > > Alexandre Belloni wrote: > > > > > On 04/07/2016 at 12:24:52 +0200, Boris Brezillon wrote : > > > > On Fri, 1 Jul 2016 23:52:05 +0

Re: [PATCH 03/11] efi: Refactor efi_memmap_init_early() into arch-neutral code

2016-07-04 Thread Matt Fleming
On Fri, 24 Jun, at 01:44:48PM, Ard Biesheuvel wrote: > > This assignment breaks the calculation of mapsize in > arm_enable_runtime_services(), so you should probably fold the > following hunk into this patch. > > diff --git a/drivers/firmware/efi/arm-runtime.c > b/drivers/firmware/efi/arm-runtime

Re: [PATCH v1] drivers:iio:accel:mma8452: added cleanup provision in case of failure.

2016-07-04 Thread Martin Kepplinger
Am 2016-07-04 um 12:08 schrieb Bijosh Thykkoottathil: > mma8452_set_freefall_mode can return -ve value in case if > i2c_smbus_read_byte_data fails. This function is called from mma8452_probe, > and returning -ve value from probe indicates probe failure. Need to call > iio_triggered_buffer_cleanup &

Re: [PATCH RFC] sched: Make wake_up_nohz_cpu() handle CPUs going offline

2016-07-04 Thread Frederic Weisbecker
On Fri, Jul 01, 2016 at 05:15:06PM -0700, Paul E. McKenney wrote: > On Sat, Jul 02, 2016 at 01:49:56AM +0200, Frederic Weisbecker wrote: > > On Fri, Jul 01, 2016 at 11:40:54AM -0700, Paul E. McKenney wrote: > > > On Fri, Jul 01, 2016 at 01:29:59AM +0200, Frederic Weisbecker wrote: > > > > > +/* > >

Re: checkkconfigsymbols.py: add --no-color option

2016-07-04 Thread Josh Triplett
On Mon, Jul 04, 2016 at 06:39:30PM +1000, Andrew Donnellan wrote: > Add a new option, --no-color, to get rid of ANSI colour escape codes in the > output. Useful if redirecting output to a file or piping to another script. > > (It should really be --no-colour, but I'll accept US spelling in the nam

Re: [PATCH v1] drivers:iio:accel:mma8452: removed unwanted return statements

2016-07-04 Thread Martin Kepplinger
Am 2016-07-04 um 13:15 schrieb Bijosh Thykkoottathil: > Removed unwanted return statements from the function > mma8452_set_freefall_mode. You could've said "Don't unnecessarily save a return value.", but whatever :) Thanks for your review! > > Signed-off-by: Bijosh Thykkoottathil Reviewed-by: M

Re: [RFC PATCH v2 2/4] net: ethernet: xilinx: Add gmii2rgmii converter support

2016-07-04 Thread Nicolas Ferre
Le 04/07/2016 13:47, Appana Durga Kedareswara Rao a écrit : > Hi Nicolas, > > Thanks for the review... > >>> diff --git a/include/linux/xilinx_gmii2rgmii.h >>> b/include/linux/xilinx_gmii2rgmii.h >>> new file mode 100644 >>> index 000..b328ee7 >>> --- /dev/null >>> +++ b/include/linux/x

[PATCH v3] arm64: dts: rockchip: add the power domain node for rk3399

2016-07-04 Thread Caesar Wang
From: Elaine Zhang In order to meet low power requirements, a power management unit (PMU) is designed for controlling power resources in RK3399. The RK3399 PMU is dedicated for managing the power of the whole chip. 1. add pd node for RK3399 Soc 2. create power domain tree 3. add qos node for dom

RE: [RFC PATCH v2 2/4] net: ethernet: xilinx: Add gmii2rgmii converter support

2016-07-04 Thread Appana Durga Kedareswara Rao
Hi Nicolas, > > > > #ifdef CONFIG_NET_VENDOR_XILINX > > You may need to have: > #if defined(CONFIG_NET_VENDOR_XILINX) && > defined(CONFIG_XILINX_GMII2RGMII) > > > extern int gmii2rgmii_phyprobe(struct gmii2rgmii *xphy); #else extern > > void gmii2rgmii_phyprobe(struct gmii2rgmii *xphy); > > N

[PATCH v2 1/1] platform/x86/intel_pmc_core: Convert to DEFINE_DEBUGFS_ATTRIBUTE

2016-07-04 Thread Andy Shevchenko
This patch does the following: - refactors code to use recently introduced DEFINE_DEBUGFS_ATTRIBUTE() macro - makes absence of DEBUG_FS non-fatal error Signed-off-by: Andy Shevchenko --- In v2: - address Rajneesh's comments drivers/platform/x86/intel_pmc_core.c | 45 -

Re: [PATCH] mtd: nand: jz4780: Update MODULE_AUTHOR email address

2016-07-04 Thread Boris Brezillon
On Thu, 30 Jun 2016 16:49:49 +0100 Harvey Hunt wrote: > Emails will bounce from my imgtec address, so update it to a new one. > > Signed-off-by: Harvey Hunt Applied. BTW, would you mind adding an entry for this driver in MAINTAINERS :)? Thanks, Boris > Cc: Harvey Hunt > Cc: linux-...@list

Re: [PATCH 2/2] arm64: dts: r8a7795: add FDP1 device nodes

2016-07-04 Thread Simon Horman
On Fri, Jul 01, 2016 at 03:37:23PM +0100, Kieran Bingham wrote: > Hi Simon, > > The bindings for this have now been acked by RobH. Thanks, I have queued these up for v4.9. There was a small amount of fuzz when applying the first patch, please check to make sure it has been applied correctly once

Re: [PATCH] ARM: dts: da850: Add new ECAP and EPWM bindings

2016-07-04 Thread Sekhar Nori
On Monday 27 June 2016 07:44 PM, Franklin S Cooper Jr wrote: > From: "Cooper Jr., Franklin" > > Switch to a new ECAP and EPWM bindings that doesn't depend on hwmod to > provide the various required clocks. there is nothing called hwmod on da850, so this need rewording to make sense for that plat

Re: [PATCH v5 3/7] phy: Add set_mode callback

2016-07-04 Thread Kishon Vijay Abraham I
Hi Bin, On Monday 27 June 2016 06:17 PM, Kishon Vijay Abraham I wrote: > Hi, > > On Tuesday 10 May 2016 05:09 AM, David Lechner wrote: >> The initial use for this is for PHYs that have a mode related to USB OTG. >> There are several SoCs (e.g. TI OMAP and DA8xx) that have a mode setting >> in the

Re: [PATCH v6 00/10] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-07-04 Thread Rafael J. Wysocki
On Fri, Jul 1, 2016 at 11:04 PM, Rafael J. Wysocki wrote: > On Friday, July 01, 2016 04:23:40 PM Will Deacon wrote: >> On Thu, Jun 30, 2016 at 09:48:02PM +0800, Hanjun Guo wrote: >> > On 2016/6/30 21:27, Rafael J. Wysocki wrote: >> > >On Thursday, June 30, 2016 10:10:02 AM Hanjun Guo wrote: >> > >

Re: [PATCH] acpi, apei: Add Boot Error Record Table (BERT) support

2016-07-04 Thread Rafael J. Wysocki
On Wednesday, June 29, 2016 09:17:38 PM Luck, Tony wrote: > >> but then it looks like it was forgotten again :-( > > > > Do you want me to take it? > > Yes please. OK, applied. Thanks!

Re: [PATCH v2] ACPI: Execute the _PTS method when system reboot

2016-07-04 Thread Rafael J. Wysocki
On Monday, June 27, 2016 02:50:16 PM Ocean HY1 He wrote: > The _PTS control method is defined in the section 7.4.1 of acpi 6.0 > spec. The _PTS control method is executed by the OS during the sleep > transition process for S1, S2, S3, S4, and for orderly S5 shutdown. > > The _PTS control method pr

Re: [PATCH] ACPI : Dummy acpi_video_register should return error code

2016-07-04 Thread Rafael J. Wysocki
On Thursday, June 23, 2016 09:35:01 AM Aaron Lu wrote: > On 06/23/2016 05:05 AM, Rafael J. Wysocki wrote: > > On Thursday, June 23, 2016 12:26:01 AM Arvind Yadav wrote: > >> The inline acpi_video_register stub simply allows compilation on systems > >> with CONFIG_ACPI_VIDEO disabled. the dummy acpi

[PATCH] RDMA/cxgb3: Use AF_INET for sin_family field

2016-07-04 Thread Amitoj Kaur Chawla
Elsewhere the sin_family field holds a value with a name of the form AF_..., so it seems reasonable to do so here as well. Also the values of PF_INET and AF_INET are the same. The Coccinelle semantic patch that makes this change is as follows: // @@ struct sockaddr_in sip; @@ ( sip.sin_family

Re: [PATCH v2 0/3] cpufreq: intel_pstate: trivial improvements

2016-07-04 Thread Rafael J. Wysocki
On Monday, June 27, 2016 06:07:15 PM Jisheng Zhang wrote: > The first patch fixes incorrect placement of __initdata > > The second patch is to add __init/__initdata marker to some functions > or varaiables so that we can discard them. > > The third patch declares pid_params/pstate_funcs/hwp_activ

Re: [PATCH v7 2/6] ACPI / processor_idle: Add support for Low Power Idle(LPI) states

2016-07-04 Thread Sudeep Holla
On 01/07/16 14:07, Daniel Lezcano wrote: On 06/28/2016 03:55 PM, Sudeep Holla wrote: ACPI 6.0 introduced an optional object _LPI that provides an alternate method to describe Low Power Idle states. It defines the local power states for each node in a hierarchical processor topology. The OSPM c

[PATCH] crypto: bfin_crc: Simplify use of devm_ioremap_resource

2016-07-04 Thread Amitoj Kaur Chawla
Remove unneeded error handling on the result of a call to platform_get_resource when the value is passed to devm_ioremap_resource. The Coccinelle semantic patch that makes this change is as follows: // @@ expression pdev,res,n,e,e1; expression ret != 0; identifier l; @@ - res = platform_get_res

Re: [PATCH] PM / clk: Add support for adding a specific clock from device-tree

2016-07-04 Thread Rafael J. Wysocki
On Tuesday, June 21, 2016 11:33:25 AM Jon Hunter wrote: > Some drivers using the PM clocks framework need to add specific clocks > from device-tree using a name by calling the functions > of_clk_get_by_name() and then pm_clk_add_clk(). Rather than having > drivers call both functions, add a helper

Re: [PATCH] relay: Add global mode support for buffer-only channels

2016-07-04 Thread Goel, Akash
On 7/4/2016 1:30 PM, Chris Wilson wrote: On Sun, Jul 03, 2016 at 09:45:27PM +0530, akash.g...@intel.com wrote: From: Akash Goel The following patch added support to use channels with no associated files. relay: add buffer-only channels; useful for early logging This is useful when th

[PATCH] ASOC: Atmel: ClassD: Simplify use of devm_ioremap_resource

2016-07-04 Thread Amitoj Kaur Chawla
Remove unneeded error handling on the result of a call to platform_get_resource when the value is passed to devm_ioremap_resource. The Coccinelle semantic patch that makes this change is as follows: // @@ expression pdev,res,n,e,e1; expression ret != 0; identifier l; @@ - res = platform_get_res

Re: [PATCH v2] PM / sleep: make pm notifier called symmetrically

2016-07-04 Thread Rafael J. Wysocki
On Sunday, June 19, 2016 11:52:27 PM Lianwei Wang wrote: > This makes pm notifier PREPARE/POST symmetrical: if PREPARE > fails, we will only undo what ever happened on PREPARE. > > It fixes the unbalanced cpu hotplug enable in cpu pm notifier. > > Signed-off-by: Lianwei Wang Applied, thanks!

Re: [PATCH v1 1/1] tools/turbostat: allow user to alter DESTDIR and PREFIX

2016-07-04 Thread Rafael J. Wysocki
On Friday, June 17, 2016 03:59:33 PM Andy Shevchenko wrote: > When run > make -C tools DESTDIR=/my/nice/dir turbostat_install > get a message > install: cannot create regular file '/usr/bin/turbostat': Permission > denied > > Allow user to alter DESTDIR and PREFIX variables. > > Sign

[PATCH] ASOC: atmel-pdmic: Simplify use of devm_ioremap_resource

2016-07-04 Thread Amitoj Kaur Chawla
Remove unneeded error handling on the result of a call to platform_get_resource when the value is passed to devm_ioremap_resource. The Coccinelle semantic patch that makes this change is as follows: // @@ expression pdev,res,n,e,e1; expression ret != 0; identifier l; @@ - res = platform_get_res

[PATCH] ftrace: move function_stats out of __initdata

2016-07-04 Thread Arnd Bergmann
Not marking ftrace_init_tracefs() as __init causes a link time warning: WARNING: vmlinux.o(.text+0xf5fcc): Section mismatch in reference from the function ftrace_init_tracefs() to the (unknown reference) .init.data:(unknown) The problem here is that ftrace_init_tracefs() now gets called by init_

Re: [PATCH v2] ARM: at91: Document new TCB bindings

2016-07-04 Thread Boris Brezillon
On Thu, 30 Jun 2016 20:27:43 -0500 Rob Herring wrote: > On Mon, Jun 27, 2016 at 05:05:01PM +0200, Alexandre Belloni wrote: > > The current binding for the TCB is not flexible enough for some use cases > > and prevents proper utilization of all the channels. > > > > Cc: Daniel Lezcano > > Cc: Th

[PATCH] thermal: qcom: tsens-8916: mark PM functions __maybe_unused

2016-07-04 Thread Arnd Bergmann
The newly added tsens-8916 driver produces warnings when CONFIG_PM is disabled: drivers/thermal/qcom/tsens.c:53:12: error: 'tsens_resume' defined but not used [-Werror=unused-function] static int tsens_resume(struct device *dev) ^~~~ drivers/thermal/qcom/tsens.c:43:12: error:

Re: 4.7-rc6, ext4, sparc64: Unable to handle kernel paging request at ...

2016-07-04 Thread Anatoly Pugachev
On Mon, Jul 4, 2016 at 2:36 PM, Meelis Roos wrote: > Just got this on bootup of my Sun T2000: >... > I have not seen it before, this includes 4.6.0 4.6.0-08907-g7639dad > 4.7.0-rc1-00094-g6b15d66 4.7.0-rc4-00014-g67016f6. > > It is not reproducible, did not appear on next reboot of the same > kern

[PATCH] dmaengine: qcom-bam-dma: add __maybe_unused annotations for PM

2016-07-04 Thread Arnd Bergmann
The bam_dma driver gained runtime PM support, but that causes build warnings whenever CONFIG_PM is disabled: drivers/dma/qcom/bam_dma.c:1324:12: error: 'bam_dma_runtime_resume' defined but not used [-Werror=unused-function] static int bam_dma_runtime_resume(struct device *dev) ^~

Re: [PATCH v7 2/6] ACPI / processor_idle: Add support for Low Power Idle(LPI) states

2016-07-04 Thread Rafael J. Wysocki
On Monday, July 04, 2016 02:00:03 PM Sudeep Holla wrote: > > On 01/07/16 14:07, Daniel Lezcano wrote: > > On 06/28/2016 03:55 PM, Sudeep Holla wrote: > >> ACPI 6.0 introduced an optional object _LPI that provides an alternate > >> method to describe Low Power Idle states. It defines the local powe

[PATCH] drm: sti: fix prototypes after API change

2016-07-04 Thread Arnd Bergmann
Like the mediatek hdmi driver, the sti driver now suffers from an API change in linux-next: drivers/gpu/drm/sti/sti_hdmi.c:1250:15: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types] .hw_params = hdmi_audio_hw_params, ^~~~ dr

Re: GPF in __mark_inode_dirty due to locked_inode_to_wb_and_lock_list returning NULL

2016-07-04 Thread Nikolay Borisov
On 07/01/2016 08:38 PM, Tejun Heo wrote: > On Fri, Jul 01, 2016 at 12:00:50PM +0200, Jan Kara wrote: >> Hello, >> >> On Thu 30-06-16 14:18:14, Nikolay Borisov wrote: >>> In light of the discussion in https://patchwork.kernel.org/patch/9187411/ >>> and >>> the discussion at >>> https://groups.g

[PATCH] drm/amd/powerplay: fix integer overflow warning

2016-07-04 Thread Arnd Bergmann
A late bugfix for v4.7 introduced a build-time regression, producing a (probably harmless) warning: powerplay/hwmgr/polaris10_hwmgr.c: In function 'polaris10_populate_clock_stretcher_data_table': powerplay/hwmgr/polaris10_hwmgr.c:1817:4: error: this decimal constant is unsigned only in ISO C90 [

Re: [PATCH] Revert "gpiolib: Split GPIO flags parsing and GPIO configuration"

2016-07-04 Thread Linus Walleij
On Sun, Jul 3, 2016 at 6:32 PM, Johan Hovold wrote: > This reverts commit 923b93e451db876d1479d3e4458fce14fec31d1c. Laurent, can we have you comment on this? Getting close to v4.7 final so we need to quickly see if this is the solution or if it will cause further regressions on your side if I r

Re: [PATCH 2/7] efi: Document #define FOO_PROTOCOL_GUID layout

2016-07-04 Thread Matt Fleming
On Mon, 27 Jun, at 12:49:20PM, Ingo Molnar wrote: > The other weirdness is the misalignment of the '0xe' portion here: > > #define LINUX_EFI_ARM_SCREEN_INFO_TABLE_GUID EFI_GUID(0xe03fc20a, 0x85dc, > 0x406e, 0xb9, 0xe, 0x4a, 0xb5, 0x02, 0x37, 0x1d, 0x95) > #define LINUX_EFI_LOADER_ENTRY_GUID

[PATCH net-next] lpc_eth: remove unused variable

2016-07-04 Thread Arnd Bergmann
A cleanup patch removed the only user of a local variable, as indicated by a gcc warning. drivers/net/ethernet/nxp/lpc_eth.c: In function 'lpc_eth_ioctl': drivers/net/ethernet/nxp/lpc_eth.c:1183:24: error: unused variable 'pldat' [-Werror=unused-variable] This removes the variable as well. Sign

Re: [PATCH 4/7] efi: Convert efi_call_virt to efi_call_virt_pointer

2016-07-04 Thread Matt Fleming
On Mon, 27 Jun, at 01:00:50PM, Ingo Molnar wrote: > > Note that while at it I renamed 'flags' to '__flags' because 'flags' is > a commonly used variable name and the 'efi_status_t __s' variable was > macro-prefixed already. > > Any objections? Nope, that's fine.

[PATCH] [media] gspca: avoid unused variable warnings

2016-07-04 Thread Arnd Bergmann
When CONFIG_INPUT is disabled, multiple gspca backend drivers print compile-time warnings about unused variables: media/usb/gspca/cpia1.c: In function 'sd_stopN': media/usb/gspca/cpia1.c:1627:13: error: unused variable 'sd' [-Werror=unused-variable] media/usb/gspca/konica.c: In function 'sd_stopN

[PATCH alternative] [media] gspca: avoid unused variable warnings

2016-07-04 Thread Arnd Bergmann
When CONFIG_INPUT is disabled, multiple gspca backend drivers print compile-time warnings about unused variables: media/usb/gspca/cpia1.c: In function 'sd_stopN': media/usb/gspca/cpia1.c:1627:13: error: unused variable 'sd' [-Werror=unused-variable] media/usb/gspca/konica.c: In function 'sd_stopN

Re: [PATCH] drm: sti: fix prototypes after API change

2016-07-04 Thread Mark Brown
On Mon, Jul 04, 2016 at 03:16:51PM +0200, Arnd Bergmann wrote: > This follows the API change, adding the extra function arguments. > I suppose we want this to be part of the ASoC merge in linux-next > for now too. I sent a pull request for the API change to the DRM people which might be easier gi

Re: [PATCH] drm: sti: fix prototypes after API change

2016-07-04 Thread Arnd Bergmann
On Monday, July 4, 2016 3:22:30 PM CEST Mark Brown wrote: > On Mon, Jul 04, 2016 at 03:16:51PM +0200, Arnd Bergmann wrote: > > > This follows the API change, adding the extra function arguments. > > I suppose we want this to be part of the ASoC merge in linux-next > > for now too. > > I sent a pu

Re: [PATCH RESEND 2/3] pwm: kona: Add support for Broadcom iproc pwm controller

2016-07-04 Thread Boris Brezillon
Hi Thierry, This is my last answer to this thread, after that I'll stop bothering you: after all, you're the PWM subsystem maintainer, and I have nothing to say about how you want to maintain your subsytem ;-). On Tue, 28 Jun 2016 14:52:23 +0200 Thierry Reding wrote: > On Thu, Jun 23, 2016 at 1

[PATCH] net-next: mediatek: remove superfluous free_irq() call

2016-07-04 Thread John Crispin
Commit 8067302973a1 ("net-next: mediatek: add support for IRQ grouping") adds handling for irq 1 and 2 to the uninit function but did not remove irq 0 which is not used since irq grouping was introduced. Fix this by removing the superfluous call to free_irq(). Reported-by: Arnd Bergmann Signed-of

Re: [PATCH] drm: sti: fix prototypes after API change

2016-07-04 Thread Mark Brown
On Mon, Jul 04, 2016 at 03:36:57PM +0200, Arnd Bergmann wrote: > On Monday, July 4, 2016 3:22:30 PM CEST Mark Brown wrote: > > I sent a pull request for the API change to the DRM people which might > > be easier given that it's a single commit topic branch rather than the > > whole of DRM. > Agre

Re: [PATCH 3/4] bus: tegra-aconnect: make it explicitly non-modular

2016-07-04 Thread Paul Gortmaker
[Re: [PATCH 3/4] bus: tegra-aconnect: make it explicitly non-modular] On 04/07/2016 (Mon 10:17) Jon Hunter wrote: > Hi Paul, > > On 03/07/16 18:30, Paul Gortmaker wrote: > > The Kconfig currently controlling compilation of this code is: > > > > drivers/bus/Kconfig:config TEGRA_ACONNECT > > driv

Re: [PATCH v7 2/6] ACPI / processor_idle: Add support for Low Power Idle(LPI) states

2016-07-04 Thread Sudeep Holla
On 04/07/16 14:17, Rafael J. Wysocki wrote: On Monday, July 04, 2016 02:00:03 PM Sudeep Holla wrote: On 01/07/16 14:07, Daniel Lezcano wrote: On 06/28/2016 03:55 PM, Sudeep Holla wrote: ACPI 6.0 introduced an optional object _LPI that provides an alternate method to describe Low Power Idle

Re: [PATCH v6 00/10] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-07-04 Thread Daniel Lezcano
On 07/01/2016 11:01 PM, Rafael J. Wysocki wrote: > On Friday, July 01, 2016 04:00:34 PM Daniel Lezcano wrote: >> On 06/30/2016 03:27 PM, Rafael J. Wysocki wrote: >> [ ... ] >> clocksource-probe which is DT based with different drivers using it in >> drivers/clocksource with a pletore of differen

[GIT PULL] ARM: at91: defconfig for 4.8 #2

2016-07-04 Thread Alexandre Belloni
Hi, A late defconfig addition for the sama5d2 Xplained PMIC. The following changes since commit 837f27a2c1b4093cda301aead20fcf7d04acc12a: ARM: multi_v5_defconfig: enable Atmel platforms (2016-06-11 00:58:51 +0200) are available in the git repository at: git://git.kernel.org/pub/scm/linux/k

Re: [PATCH] net-next: mediatek: remove superfluous free_irq() call

2016-07-04 Thread Arnd Bergmann
On Monday, July 4, 2016 3:37:10 PM CEST John Crispin wrote: > Commit 8067302973a1 ("net-next: mediatek: add support for IRQ grouping") > adds handling for irq 1 and 2 to the uninit function but did not remove > irq 0 which is not used since irq grouping was introduced. Fix this by > removing the su

[PATCH 0/4] Add STM32 Reset Driver

2016-07-04 Thread gabriel.fernandez
From: Gabriel Fernandez The STM32 MCUs family IPs can be reset by accessing some registers from the RCC block. Gabriel Fernandez (2): drivers: reset: Add STM32 reset driver ARM: dts: stm32f429: add missing #reset-cells of rcc Maxime Coquelin (2): dt-bindings: mfd: Add STM32F4 RCC numeric

Re: [PATCH 3/4] bus: tegra-aconnect: make it explicitly non-modular

2016-07-04 Thread Jon Hunter
On 04/07/16 14:41, Paul Gortmaker wrote: > [Re: [PATCH 3/4] bus: tegra-aconnect: make it explicitly non-modular] On > 04/07/2016 (Mon 10:17) Jon Hunter wrote: > >> Hi Paul, >> >> On 03/07/16 18:30, Paul Gortmaker wrote: >>> The Kconfig currently controlling compilation of this code is: >>> >>> d

[PATCH 1/4] dt-bindings: mfd: Add STM32F4 RCC numeric constants into DT include file

2016-07-04 Thread gabriel.fernandez
From: Maxime Coquelin Ths patch lists STM32F4's RCC numeric constants. It will be used by clock and reset drivers, and DT bindings. Signed-off-by: Maxime Coquelin --- include/dt-bindings/mfd/stm32f4-rcc.h | 92 +++ 1 file changed, 92 insertions(+) create mode 1

[PATCH 3/4] drivers: reset: Add STM32 reset driver

2016-07-04 Thread gabriel.fernandez
From: Gabriel Fernandez The STM32 MCUs family IPs can be reset by accessing some registers from the RCC block. The list of available reset lines is documented in the DT bindings. Signed-off-by: Maxime Coquelin Signed-off-by: Gabriel Fernandez --- drivers/reset/Makefile | 1 + drivers/

[PATCH 4/4] ARM: dts: stm32f429: add missing #reset-cells of rcc

2016-07-04 Thread gabriel.fernandez
From: Gabriel Fernandez This patch adds #reset-cells property to rcc node. Signed-off-by: Gabriel Fernandez --- arch/arm/boot/dts/stm32f429.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/stm32f429.dtsi b/arch/arm/boot/dts/stm32f429.dtsi index 35df462..fe89236 100644

[PATCH 2/4] dt-bindings: Document the STM32 reset bindings

2016-07-04 Thread gabriel.fernandez
From: Maxime Coquelin This adds documentation of device tree bindings for the STM32 reset controller. Signed-off-by: Maxime Coquelin --- .../devicetree/bindings/reset/st,stm32-rcc.txt | 50 ++ 1 file changed, 50 insertions(+) create mode 100644 Documentation/devicetree

Re: [PATCH] 8250/fintek: rename IRQ_MODE macro

2016-07-04 Thread Ricardo Ribalda Delgado
Acked-by: Ricardo Ribalda Delgado On Mon, Jun 27, 2016 at 11:21 AM, Arnd Bergmann wrote: > A bugfix for the fintek driver required defining some macros, but > one of them clashes with a system header on ARM: > > drivers/tty/serial/8250/8250_fintek.c:34:0: error: "IRQ_MODE" redefined > [-Werror]

[PATCH] Syscall arguments are unsigned long (full registers)

2016-07-04 Thread Tautschnig, Michael
All syscall arguments are passed in as types of the same byte size as unsigned long (width of full registers). Using a smaller type without a cast may result in losing bits of information. In all other instances apart from the ones fixed by the patch the code explicitly introduces type casts (using

Re: [UPDATE PATCH v2 1/3] ACPICA: Events: Introduce acpi_mask_gpe() to implement GPE masking mechanism

2016-07-04 Thread Rafael J. Wysocki
On Thursday, June 23, 2016 03:05:47 PM Lv Zheng wrote: > (remove acpi_unmask_gpe() from the patch description) > > There is a facility in Linux, developers can control the enabling/disabling > of a GPE via /sys/firmware/acpi/interrupts/gpexx. This is mainly for > debugging purposes. > > But many

[PATCH] scripts: Fix size mismatch of kexec_purgatory_size

2016-07-04 Thread Tautschnig, Michael
bin2c is used to create a valid C file out of a binary file where two symbols will be globally defined: and _size. is passed as the first parameter of the host binary. Building using goto-cc reported that the purgatory binary code (the only current user of this utility) declares kexec_purgatory_

Re: [PATCH v3 4/6] mtd: nand: gpmi: correct bitflip for erased NAND page

2016-07-04 Thread Boris Brezillon
On Fri, 24 Jun 2016 16:40:09 -0500 Han Xu wrote: > i.MX6QP and i.MX7D BCH module integrated a new feature to detect the > bitflip number for erased NAND page. So for these two platform, set the > erase threshold to ecc_strength and if bitflip detected, GPMI driver > will correct the data to all 0

[PATCH 1/4] perf tools: Introduce hist_entry__init function

2016-07-04 Thread Jiri Olsa
Move hist_entry initialization code into separate function. It'll be useful and more clear for following patches that introduce allocation callbacks. Link: http://lkml.kernel.org/n/tip-j8dz6ytir91x8qd1zk4vf...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/hist.c | 142 +

[PATCH 2/4] perf tools: Do the error path hist_entry release in hist_entry__new

2016-07-04 Thread Jiri Olsa
It's better to release the hist_entry object in hist_entry__new function (where it's allocated) rather than in hist_entry__init. Link: http://lkml.kernel.org/n/tip-uoatzgsbdk3ebaeu56kdb...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/hist.c | 13 ++--- 1 file changed, 6 in

[PATCH 4/4] perf tools: Introduce hists__add_entry_ops function

2016-07-04 Thread Jiri Olsa
Introducing hists__add_entry_ops function to allow using the allocation callbacks externally. Link: http://lkml.kernel.org/n/tip-r4bumbbg5st7p38hjm2z1...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/hist.c | 42 +++--- tools/perf/util/hist.h | 1

[PATCH 3/4] perf tools: Introduce hist_entry_ops

2016-07-04 Thread Jiri Olsa
Introducing allocation callbacks, that allows to extend current hist_entry object into objects with special needs without polluting the current hist_entry object. Link: http://lkml.kernel.org/n/tip-yvapb3gmmn01qo7qn9lzl...@git.kernel.org Signed-off-by: Jiri Olsa --- tools/perf/util/hist.c | 31 +

[RFC 0/4] perf tools: Add hist_entry allocation callbacks

2016-07-04 Thread Jiri Olsa
hi, this patchset tries to add support provide own allocation zalloc/free methods for hist_entry object. The reason is to provide a way to be able to store more data within hist_entry object in a transparent way to its current usage by allocating its own hist_entry sub object. The user/app which

Re: [PATCH v2 3/3] ACPI / sysfs: Provide quirk mechanism to prevent GPE flooding

2016-07-04 Thread Rafael J. Wysocki
On Thursday, June 23, 2016 02:20:30 PM Lv Zheng wrote: > Sometimes, the users may require a quirk to be provided from ACPI subsystem > core to prevent a GPE from flooding. Normally, if a GPE cannot be > dispatched, ACPICA core automatically prevents the GPE from firing. But > there are cases the GP

Re: [RFC PATCH v2 1/4] Documentation: DT: net: Add Xilinx gmiitorgmii converter device tree binding documentation

2016-07-04 Thread Andrew Lunn
On Mon, Jul 04, 2016 at 02:34:41PM +0530, Kedareswara rao Appana wrote: > Device-tree binding documentation for xilinx gmiitorgmii converter. > > Signed-off-by: Kedareswara rao Appana > --- > Changes for v2: > --> New patch. > > .../devicetree/bindings/net/xilinx_gmii2rgmii.txt | 31 > +++

Re: [PATCH v2 1/8] Input: synaptics-rmi4: Increment the refcount of the transport dev's of_node

2016-07-04 Thread Nick Dyer
On 25/06/2016 01:17, Andrew Duggan wrote: > Increment the refcount for the transport device's of_node before calling > of_find_node_by_name(). Since of_find_node_by_name() assumes the refcount > was incremented by the caller and calls of_node_put() when it is done. > > Signed-off-by: Andrew Duggan

Re: [PATCH v7 2/6] ACPI / processor_idle: Add support for Low Power Idle(LPI) states

2016-07-04 Thread Rafael J. Wysocki
On Mon, Jul 4, 2016 at 3:42 PM, Sudeep Holla wrote: > > > On 04/07/16 14:17, Rafael J. Wysocki wrote: >> >> On Monday, July 04, 2016 02:00:03 PM Sudeep Holla wrote: >>> >>> >>> On 01/07/16 14:07, Daniel Lezcano wrote: On 06/28/2016 03:55 PM, Sudeep Holla wrote: > > ACPI 6.0 intro

Re: [PATCH] mmc: core: Extend sysfs with OCR register

2016-07-04 Thread Wolfram Sang
Bojan, On Mon, Jul 04, 2016 at 01:56:55PM +0200, Bojan Prtvar wrote: > Make operation conditions register (OCR) easily accessible from user space. > > Signed-off-by: Bojan Prtvar You described "what" above. Can you add the "why", too? Regards, Wolfram signature.asc Description: PGP signa

Re: [PATCH v3] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-04 Thread Andrey Ryabinin
On 07/04/2016 07:36 AM, Joonsoo Kim wrote: > On Fri, Jul 01, 2016 at 05:17:10PM +0300, Andrey Ryabinin wrote: >> >> >> On 07/01/2016 05:02 PM, js1...@gmail.com wrote: >>> From: Joonsoo Kim >>> >>> There are two bugs on qlist_move_cache(). One is that qlist's tail >>> isn't set properly. curr->ne

Re: [PATCH] ASOC: Atmel: ClassD: Simplify use of devm_ioremap_resource

2016-07-04 Thread Mark Brown
On Mon, Jul 04, 2016 at 06:33:52PM +0530, Amitoj Kaur Chawla wrote: > Remove unneeded error handling on the result of a call to > platform_get_resource when the value is passed to > devm_ioremap_resource. Please use subject lines matching the style for the subsystem. This makes it easier for peop

[GIT PULL] ARM: at91: dt for 4.8 #2 bis

2016-07-04 Thread Alexandre Belloni
Hi, Please disregard the previous DT pull request, it had an issue... I'm including the corrected change in this one with two fixes for the sam9_l9260 and on fix for the sama5d2 xplained. Also, adding the PMU node to the sama5d2 doesn't seem too risky. The following changes since commit 64c0703e2

kmem_cache_alloc fail with unable to handle paging request after pci hotplug remove.

2016-07-04 Thread Mathias Nyman
Hi AceLan Kao can get his DELL XPS 13 laptop to hang by plugging/un-plugging a USB 3.1 key via thunderbolt port. Allocating memory fails after this, always pointing to NULL pointer or page request failing in get_freepointer() called by kmalloc/kmem_cache_alloc. Unplugging a usb type-c device fr

Re: [PATCH 4/4] bus: simple-pm-bus: convert bool SIMPLE_PM_BUS to tristate

2016-07-04 Thread Geert Uytterhoeven
Hi Paul, On Sun, Jul 3, 2016 at 7:30 PM, Paul Gortmaker wrote: > The Kconfig currently controlling compilation of this code is: > > config SIMPLE_PM_BUS > bool "Simple Power-Managed Bus Driver" > > ...meaning that it currently is not being built as a module by anyone. > > In removing the

Re: [PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-04 Thread Andrey Ryabinin
On 07/04/2016 07:31 AM, js1...@gmail.com wrote: > From: Joonsoo Kim > > There are two bugs on qlist_move_cache(). One is that qlist's tail > isn't set properly. curr->next can be NULL since it is singly linked > list and NULL value on tail is invalid if there is one item on qlist. > Another one

Re: [PATCH v6 00/10] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer

2016-07-04 Thread Rafael J. Wysocki
On Mon, Jul 4, 2016 at 3:43 PM, Daniel Lezcano wrote: > On 07/01/2016 11:01 PM, Rafael J. Wysocki wrote: >> On Friday, July 01, 2016 04:00:34 PM Daniel Lezcano wrote: >>> On 06/30/2016 03:27 PM, Rafael J. Wysocki wrote: >>> > > [ ... ] > >>> clocksource-probe which is DT based with different drive

Re: [PATCH] ftrace: move function_stats out of __initdata

2016-07-04 Thread Steven Rostedt
On Mon, 4 Jul 2016 15:09:44 +0200 Arnd Bergmann wrote: > Not marking ftrace_init_tracefs() as __init causes a link time warning: > > WARNING: vmlinux.o(.text+0xf5fcc): Section mismatch in reference from the > function ftrace_init_tracefs() to the (unknown reference) .init.data:(unknown) > > T

[PATCH] [RFC] Kbuild: avoid "make tinyconfig" warnings

2016-07-04 Thread Arnd Bergmann
The introduction of "make *.config" as a shorthand for merging configuration files unfortunately introduced some build warnings that we see in every single run of the various build bots testing tinyconfig: .config:966:warning: override: NOHIGHMEM changes choice state .config:965:warning: override:

<    1   2   3   4   5   6   7   >