[PATCH 0/2] usbnet: Adjustments for usbnet_write_cmd_async()

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 22 May 2017 07:04:03 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination

Re: gpio-exar: Why filtering out Commtech devices?

2017-05-21 Thread Jan Kiszka
On 2017-05-21 22:08, Sudip Mukherjee wrote: > Hi Jan, > > On 21/05/17 12:46, Jan Kiszka wrote: >> Hi Sudip, >> >> why do we carry >> >> if (pcidev->vendor != PCI_VENDOR_ID_EXAR) >> return -ENODEV; >> >> in gpio_exar_probe? This effectively prevents that >> >> EXAR_DEVICE(COMMTECH,

Re: [PATCH 1/3] mfd: lp87565: Add lp87565 PMIC support

2017-05-21 Thread Keerthy
On Friday 19 May 2017 09:12 PM, Andrew F. Davis wrote: > On 05/19/2017 07:42 AM, Keerthy wrote: >> The LP87565 chip is a power management IC for Portable Navigation Systems >> and Tablet Computing devices. It contains the following components: >> >> - Configurable Bucks(Single and

Re: [PATCH v2 0/4] ARM: dts: keystone: Add support for new K2G evm

2017-05-21 Thread santosh.shilim...@oracle.com
On 5/10/17 9:08 AM, Santosh Shilimkar wrote: On 4/24/2017 1:22 PM, Franklin S Cooper Jr wrote: This patchset adds support for new K2G Industrial Communication Engine evm. For now only a bare minimal dts which will allow ram boot. Additional peripherals will be added when base K2G SoC patches

Re: [PATCH v6] iio: adc: Add support for TI ADC108S102 and ADC128S102

2017-05-21 Thread Jan Kiszka
On 2017-05-21 16:13, Jonathan Cameron wrote: > On 21/05/17 15:06, Jonathan Cameron wrote: >> On 21/05/17 13:44, Jan Kiszka wrote: >>> On 2017-05-21 14:18, Jonathan Cameron wrote: On 21/05/17 13:17, Jonathan Cameron wrote: > On 21/05/17 12:59, Jan Kiszka wrote: >> On 2017-05-21 12:47,

Re: linux-next: build failure after merge of the net-next tree

2017-05-21 Thread David Miller
From: Stephen Rothwell Date: Mon, 22 May 2017 11:16:05 +1000 > After merging the net-next tree, today's linux-next build (powerpc > ppc64_defconfig) failed like this: > > net/socket.c: In function 'put_ts_pktinfo': > net/socket.c:695:28: error: 'SCM_TIMESTAMPING_PKTINFO'

Re: [PATCH v3 1/2] PCI: mediatek: Add Mediatek PCIe host controller support

2017-05-21 Thread Ryder Lee
On Sat, 2017-05-20 at 15:46 -0400, Paul Gortmaker wrote: > On Tue, May 9, 2017 at 10:06 PM, Ryder Lee wrote: > > Add support for the Mediatek PCIe Gen2 controller which can > > be found on MT7623 series SoCs. > > > > Signed-off-by: Ryder Lee > >

Re: [PATCH 1/2] libsas: Don't process sas events in static works

2017-05-21 Thread wangyijing
Hi Dan, thanks for your review and comments! 在 2017/5/21 11:44, Dan Williams 写道: > On Fri, May 19, 2017 at 11:39 PM, Yijing Wang wrote: >> Now libsas hotplug work is static, LLDD driver queue >> the hotplug work into shost->work_q. If LLDD driver >> burst post lots hotplug

Re: linux-next: build failure after merge of the net-next tree

2017-05-21 Thread Willem de Bruijn
On Sun, May 21, 2017 at 9:16 PM, Stephen Rothwell wrote: > Hi all, > > After merging the net-next tree, today's linux-next build (powerpc > ppc64_defconfig) failed like this: > > net/socket.c: In function 'put_ts_pktinfo': > net/socket.c:695:28: error:

Re: linux-next: build failure after merge of the net-next tree

2017-05-21 Thread Stephen Rothwell
Hi Dave, On Sun, 21 May 2017 23:14:10 -0400 (EDT) David Miller wrote: > > From: Stephen Rothwell > Date: Mon, 22 May 2017 11:16:05 +1000 > > > After merging the net-next tree, today's linux-next build (powerpc > > ppc64_defconfig) failed like this:

Re: [PATCH 4/6] ubifs: Maintain a parent pointer

2017-05-21 Thread Hyunchul Lee
Hi Richard, On Sun, May 21, 2017 at 10:20:49PM +0200, Richard Weinberger wrote: > The new feature UBIFS_FLG_PARENTPOINTER allows looking > up the parent. Usually the Linux VFS walks down the filesystem > and no parent pointers are needed. But when a filesystem > is exportable via NFS such a

[PATCH v3 0/2] Hi3660: enable sp804 timer

2017-05-21 Thread Leo Yan
This patch set is to enable sp804 timer on Hi3660 for Hikey960 platform. On Hi3660, the sp804 timer co-exists with CPUs' architecture timer; but sp804 timer is located in SoC level but CPU's architecture timer is in CPU power domain. sp804 timer is used as broadcast timer when CPU enters idle

Re: CHIPPro NAND issue with 4.12 rc1

2017-05-21 Thread Angus Ainslie
On 2017-05-20 23:45, Boris Brezillon wrote: Le Sat, 20 May 2017 15:24:06 -0600, Angus Ainslie a écrit : On 2017-05-20 09:14, Boris Brezillon wrote: > Le Sat, 20 May 2017 08:49:04 -0600, > Angus Ainslie a écrit : > >> Hi All, >> >> I'm trying to boot a CHIPPro

[PATCH linux-next 1/1] spi: imx: only allow dynamic burst in PIO mode

2017-05-21 Thread Jiada Wang
Currently only PIO mode supports dynamic burst length adjust, in DMA mode, bpw (bytes per word) value still has to be used as burst length, other wise transfer issue will be caused. This patch avoid using dynamic burst in DMA mode by set reset dynamic_burst when DMA mode is used. Signed-off-by:

Re: [PATCH 3/4] mtd: mchp23k256: add partitioning support

2017-05-21 Thread Chris Packham
On 18/05/17 03:29, Boris Brezillon wrote: > Hi Chris, > > On Wed, 17 May 2017 17:39:07 +1200 > Chris Packham wrote: > >> Setting the of_node for the mtd device allows the generic mtd code to >> setup the partitions. Additionally we must specify a non-zero

[PATCH v3 1/2] clk: Hi3660: register fixed_rate_clks with CLK_OF_DECLARE_DRIVER

2017-05-21 Thread Leo Yan
The timer will register into system at very early phase at kernel boot; if timer needs to use clock, the clock should be get ready in function of_clk_init() so later the timer driver probe can retrieve clock successfully. This is finished in below flow on arm64: start_kernel() `->

[PATCH] ocfs2: fix a static checker warning

2017-05-21 Thread Gang He
This patch will fix a static checker warning, this warning was caused by commit d56a8f32e4c662509ce50a37e78fa66c777977d3. after apply this patch, the error return value will not be NULL(zero). Signed-off-by: Gang He --- fs/ocfs2/inode.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v3 2/2] arm64: dts: add sp804 timer node for Hi3660

2017-05-21 Thread Leo Yan
The Hi3660 SoC comes with the sp804 timer in addition to the architecture timers. These ones are shutdown when reaching a deep idle states and a backup timer is needed. The sp804 belongs to another power domain and can fulfill the purpose of replacing temporarily an architecture timer when the CPU

[PATCH v2] nvme/pci: remap BAR0 to cover admin CQ doorbell for large stride

2017-05-21 Thread Xu Yu
The existing driver initially maps 8192 bytes of BAR0 which is intended to cover doorbells of admin SQ and CQ. However, if a large stride, e.g. 10, is used, the doorbell of admin CQ will be out of 8192 bytes. Consequently, a page fault will be raised when the admin CQ doorbell is accessed in

Re: [PATCH] ALSA: sound/isa: constify snd_kcontrol_new structures

2017-05-21 Thread Bhumika Goyal
On Sun, May 21, 2017 at 12:39 PM, Takashi Iwai wrote: > On Wed, 17 May 2017 14:33:45 +0200, > Bhumika Goyal wrote: >> >> On Wed, May 17, 2017 at 5:50 PM, Takashi Iwai wrote: >> > On Wed, 17 May 2017 13:38:56 +0200, >> > Bhumika Goyal wrote: >> >> >> >> Declare

Re: is alpha jensen support dead?

2017-05-21 Thread Michael Ellerman
Christoph Hellwig writes: > Hi all, > > it seems like the Alpha Jense build (the only one using pci-noop.c) > and thus being a different build than all the later PCI capable > system has been broken since at least: > > commit 6aca0503847f6329460b15b3ab2b0e30bb752793 > Author:

Re: gpio-exar: Why filtering out Commtech devices?

2017-05-21 Thread Jan Kiszka
On 2017-05-22 07:48, Jan Kiszka wrote: > On 2017-05-21 22:08, Sudip Mukherjee wrote: >> Hi Jan, >> >> On 21/05/17 12:46, Jan Kiszka wrote: >>> Hi Sudip, >>> >>> why do we carry >>> >>> if (pcidev->vendor != PCI_VENDOR_ID_EXAR) >>> return -ENODEV; >>> >>> in gpio_exar_probe? This

linux-next: manual merge of the tip tree with Linus' tree

2017-05-21 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the tip tree got a conflict in: arch/arm64/include/asm/cpufeature.h between commit: 63a1e1c95e60 ("arm64/cpufeature: don't use mutex in bringup path") from Linus' tree and commit: d54bb72551b9 ("arm64/cpufeature: Use

[PATCH v2 1/4] mtd: mchp23k256: Add OF device ID table

2017-05-21 Thread Chris Packham
This allows registering of this device via a Device Tree. Signed-off-by: Chris Packham Reviewed-by: Andrew Lunn Tested-by: Andrew Lunn --- Changes in v2 - collect review/test from Andrew

[PATCH v2 0/4] mtd: mchp23k256: device tree and mchp23lcv1024

2017-05-21 Thread Chris Packham
This series adds device tree support to the mchp23k256 driver and support for the mchp23lcv1024 chip. I suspect there are more compatible variants that we could now enumerate if desired. Chris Packham (4): mtd: mchp23k256: Add OF device ID table mtd: mchp23k256: switch to

[PATCH v2 3/4] mtd: mchp23k256: add partitioning support

2017-05-21 Thread Chris Packham
Setting the of_node for the mtd device allows the generic mtd code to setup the partitions. Additionally we must specify a non-zero erasesize for the partitions to be writeable. Signed-off-by: Chris Packham Reviewed-by: Andrew Lunn Tested-by:

[PATCH v2 4/4] mtd: mchp23k256: Add support for mchp23lcv1024

2017-05-21 Thread Chris Packham
The mchp23lcv1024 is software compatible with the mchp23k256, the only difference (from a software point of view) is the size. There is no way to detect the size so we must be told via a Device Tree. Signed-off-by: Chris Packham --- Changes in v2: - fix

[PATCH v2 2/4] mtd: mchp23k256: switch to mtd_device_register()

2017-05-21 Thread Chris Packham
Use mtd_device_register() instead of mtd_device_parse_register() to eliminate two unused parameters. Signed-off-by: Chris Packham Reviewed-by: Andrew Lunn Tested-by: Andrew Lunn --- Changes in v2 - collect review/test from

Re: [PATCH 0/5] hwmon: move include files out of include/linux/i2c

2017-05-21 Thread Guenter Roeck
On 05/21/2017 01:34 PM, Wolfram Sang wrote: It doesn't make sense to use include/linux/i2c for client drivers which may in fact rather be hwmon or input or whatever devices. As a result, I want to deprecate include/linux/i2c for good. This series moves the include files to a better location,

Re: [for-next][PATCH 09/33] selftests: ftrace: Add test to test reading of set_ftrace_file

2017-05-21 Thread Masami Hiramatsu
Hi Steve, On Fri, 21 Apr 2017 17:30:29 -0400 Steven Rostedt wrote: > From: "Steven Rostedt (VMware)" > > The set_ftrace_file lists both functions that are filtered, as well as > function probes (triggers) that are attached to a function, like traceon

Re: [PATCH 2/9] timers: provide a "modern" variant of timers

2017-05-21 Thread 'Christoph Hellwig'
On Fri, May 19, 2017 at 10:48:51AM +, David Laight wrote: > From: Christoph Hellwig > > Sent: 16 May 2017 12:48 > > > > The new callback gets a pointer to the timer_list itself, which can > > then be used to get the containing structure using container_of > > instead of casting from and to

Re: [PATCH 03/23] uuid: remove uuid_be defintions from the uapi header

2017-05-21 Thread Christoph Hellwig
On Fri, May 19, 2017 at 11:58:41AM +0100, David Howells wrote: > Christoph Hellwig wrote: > > > It's not a userspace ABI, so by defintion it does not break an > > existing user program. > > That's an invalid assumption. It is a de facto userspace ABI as it has been > exposed in

Re: [PATCH] input: move gpio mouse to managed api

2017-05-21 Thread Bram Vlerick
Hi Dmitry, I've updated the patch with your comments, unregistering first before freeing the gpios. In regards of your comment for the gpio-keys-polled update, I've just started with kernel dev and thought moving old drivers to the new managed api was a good start. So I don't really know how to

Re: [PATCH 18/24] thunderbolt: Store Thunderbolt generation in the switch structure

2017-05-21 Thread Mika Westerberg
On Sun, May 21, 2017 at 07:35:21AM +0200, Lukas Wunner wrote: > On Sun, May 21, 2017 at 05:29:47AM +, Levy, Amir (Jer) wrote: > > On Sun, May 21 2017, 07:47 AM, Lukas Wunner wrote: > > > On Thu, May 18, 2017 at 05:39:08PM +0300, Mika Westerberg wrote: > > > > + > > > > + default: > > > >

Re: [RFC] KVM: SVM: do not drop VMCB CPL to 0 if SS is not present

2017-05-21 Thread Roman Penyaev
On Sun, May 21, 2017 at 5:31 AM, Andy Lutomirski wrote: > On 05/19/2017 09:14 AM, Roman Penyaev wrote: >> >> Hi folks, >> >> After experiencing guest double faults (sometimes triple faults) on >> 3.16 guest kernels with the following common pattern: >> [cut] >> >> Further

Re: [patch v2 1/6] tty: export tty_open_by_driver

2017-05-21 Thread Christoph Hellwig
On Mon, May 15, 2017 at 06:45:32PM +0100, Okash Khawaja wrote: > This exports tty_open_by_driver so that it can be called from other places > inside the kernel. The checks for null file pointer are based on Alan Cox's > patch here: >

Re: [PATCH v2] Use ctlr directly in rdac_failover_get()

2017-05-21 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [patch v2 1/6] tty: export tty_open_by_driver

2017-05-21 Thread Greg Kroah-Hartman
On Sat, May 20, 2017 at 11:09:02PM -0700, Christoph Hellwig wrote: > On Mon, May 15, 2017 at 06:45:32PM +0100, Okash Khawaja wrote: > > This exports tty_open_by_driver so that it can be called from other places > > inside the kernel. The checks for null file pointer are based on Alan Cox's > >

Re: [PATCH] ALSA: sound/isa: constify snd_kcontrol_new structures

2017-05-21 Thread Takashi Iwai
On Wed, 17 May 2017 14:33:45 +0200, Bhumika Goyal wrote: > > On Wed, May 17, 2017 at 5:50 PM, Takashi Iwai wrote: > > On Wed, 17 May 2017 13:38:56 +0200, > > Bhumika Goyal wrote: > >> > >> Declare snd_kcontrol_new structures as const as they are only passed an > >> argument to the

Re: [PATCH 10/24] thunderbolt: Read vendor and device name from DROM

2017-05-21 Thread Mika Westerberg
On Sun, May 21, 2017 at 07:31:14AM +0200, Lukas Wunner wrote: > On Fri, May 19, 2017 at 01:28:36PM +0300, Mika Westerberg wrote: > > On Fri, May 19, 2017 at 12:07:10PM +0200, Lukas Wunner wrote: > > > Apple uses 0x30 to store a > > > serial number. Is this attribute number assigned by Intel to

Re: [PATCH] goldfish_pipe: use GFP_ATOMIC under spin lock

2017-05-21 Thread Michal Hocko
On Sun 21-05-17 00:45:46, Wei Yongjun wrote: > From: Wei Yongjun > > The function get_free_pipe_id_locked() is called from > goldfish_pipe_open() with a lock is held, so we should > use GFP_ATOMIC instead of GFP_KERNEL. Why is GFP_NOWAIT insufficient? Does this path

Re: [lkp-robot] [waitid()] 75f64d68f9: Kernel_panic-not_syncing:Attempted_to_kill_init!exitcode=

2017-05-21 Thread Al Viro
On Fri, May 19, 2017 at 02:08:20PM +0800, kernel test robot wrote: > > FYI, we noticed the following commit: > > commit: 75f64d68f9816a1c244b8685f056389b24d97e98 ("waitid(): switch copyout > of siginfo to unsafe_put_user()") > url: >

Re: [PATCH 0/8] CaitSith LSM module

2017-05-21 Thread Tetsuo Handa
John Johansen wrote: > On 05/20/2017 09:59 PM, Tetsuo Handa wrote: > > John Johansen wrote: > >> On 11/22/2016 10:31 PM, Tetsuo Handa wrote: > >>> Tetsuo Handa wrote: > John Johansen wrote: > >> In order to minimize the burden of reviewing, this patchset implements > >> only

Re: [PATCH] nvme: pci: Fix NULL dereference when resetting NVMe SSD

2017-05-21 Thread Christoph Hellwig
On Sat, May 20, 2017 at 08:59:54PM +0300, Rakesh Pandit wrote: > While doing IO if I reset NVMe SSD (model :Samsung MZVPV512HDGL-0) > it doesn't work as expected also results in NULL point dereference and > system becomes unstable. > > Device's access is successfully disabled and reset

Re: [PATCH 2/2] zram: do not count duplicated pages as compressed

2017-05-21 Thread Christoph Hellwig
On Wed, May 17, 2017 at 05:32:12PM +0900, Minchan Kim wrote: > Is block device(esp, zram which is compressed ram block device) okay to > return garbage when ongoing overwrite IO fails? > > O_DIRECT write 4 block "aaa.." -> success > read 4 block "aaa.." -> success > O_DIRECT write 4 block

Re: [PATCH 18/24] thunderbolt: Store Thunderbolt generation in the switch structure

2017-05-21 Thread Mika Westerberg
On Sun, May 21, 2017 at 10:40:41AM +0300, Mika Westerberg wrote: > On Sun, May 21, 2017 at 07:35:21AM +0200, Lukas Wunner wrote: > > On Sun, May 21, 2017 at 05:29:47AM +, Levy, Amir (Jer) wrote: > > > On Sun, May 21 2017, 07:47 AM, Lukas Wunner wrote: > > > > On Thu, May 18, 2017 at 05:39:08PM

RE: [PATCH 18/24] thunderbolt: Store Thunderbolt generation in the switch structure

2017-05-21 Thread Levy, Amir (Jer)
On Sun, May 21 2017, 11:00 AM, Mika Westerberg wrote: > On Sun, May 21, 2017 at 10:40:41AM +0300, Mika Westerberg wrote: > > On Sun, May 21, 2017 at 07:35:21AM +0200, Lukas Wunner wrote: > > > On Sun, May 21, 2017 at 05:29:47AM +, Levy, Amir (Jer) wrote: > > > > On Sun, May 21 2017, 07:47 AM,

Re: [PATCH 2/3] regulator: lp87565: Add support for lp87565 PMIC regulators

2017-05-21 Thread Keerthy
On Friday 19 May 2017 09:23 PM, Andrew F. Davis wrote: > On 05/19/2017 07:42 AM, Keerthy wrote: >> The regulators set consists of 4 BUCKs. The output >> voltages are configurable and are meant to supply power to the >> main processor and other components. The ramp delay is configurable >> for

[PATCH 2/2] usbnet: Improve a size determination in usbnet_write_cmd_async()

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 22 May 2017 06:42:33 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH] cpufreq: dt: Set default policy->transition_delay_ns

2017-05-21 Thread Viresh Kumar
The rate_limit_us for the schedutil governor is getting set to 500 ms by default for the ARM64 hikey board. And its way too much, even for the default value. Lets set the default transition_delay_ns to something more realistic (10 ms), while the userspace always have a chance to set something it

[PATCH 1/2] usbnet: Delete an error message for a failed memory allocation in usbnet_write_cmd_async()

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 22 May 2017 06:33:48 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Link:

Re: is alpha jensen support dead?

2017-05-21 Thread Matt Turner
On Sun, May 21, 2017 at 1:58 AM, Christoph Hellwig wrote: > Hi all, > > it seems like the Alpha Jense build (the only one using pci-noop.c) > and thus being a different build than all the later PCI capable > system has been broken since at least: > > commit

[PATCH] i2c: designware: Fix bogus sda_hold_time due to uninitialized vars

2017-05-21 Thread Jan Kiszka
We need to initializes those variables to 0 for platforms that do not provide ACPI parameters. Otherwise, we set sda_hold_time to random values, breaking e.g. Galileo and IOT2000 boards. Fixes: 9d6408433019 ("i2c: designware: don't infer timings described by ACPI from clock rate") Signed-off-by:

Re: [PATCH v2] drm: mediatek: change the variable type of rdma threshold

2017-05-21 Thread CK Hu
Hi, Bibby: One comment inline. On Fri, 2017-05-19 at 17:57 +0800, Bibby Hsieh wrote: > For some greater resolution, the rdma threshold > variable will overflow. > > Signed-off-by: Bibby Hsieh > --- > drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 7 --- > 1 file

Re: [PATCH 2/9] timers: provide a "modern" variant of timers

2017-05-21 Thread Christoph Hellwig
On Thu, May 18, 2017 at 10:57:31AM +0200, Arnd Bergmann wrote: > How expensive would it be to add another field to timer_list and > just have both pointers? That would add 4/8 bytes to every structure containing a timer, so I'd rather avoid it if possible. But one option might be to inflict this

Re: [PATCH 2/2] zram: do not count duplicated pages as compressed

2017-05-21 Thread Minchan Kim
On Sun, May 21, 2017 at 12:04:27AM -0700, Christoph Hellwig wrote: > On Wed, May 17, 2017 at 05:32:12PM +0900, Minchan Kim wrote: > > Is block device(esp, zram which is compressed ram block device) okay to > > return garbage when ongoing overwrite IO fails? > > > > O_DIRECT write 4 block "aaa.."

Re: [PATCH v5 17/32] x86/mm: Add support to access boot related data in the clear

2017-05-21 Thread Borislav Petkov
On Fri, May 19, 2017 at 03:50:32PM -0500, Tom Lendacky wrote: > The "worker" function would be doing the loop through the setup data, > but since the setup data is mapped inside the loop I can't do the __init > calling the non-init function and still hope to consolidate the code. > Maybe I'm

[PATCH] drm/i915/gvt: remove redundant -Wall

2017-05-21 Thread Nick Desaulniers
This flag is already set in the top level Makefile of the kernel. Also, by having set CONFIG_DRM_I915_GVT, thereby appending -Wall to ccflags, you undo all the -Wno-* cflags previously set in the Make variable KBUILD_CFLAGS. For example: cc foo.c -Wall -Wno-format -Wall resets -Wformat.

[PATCH 1/3] zswap: Delete an error message for a failed memory allocation in zswap_pool_create()

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 20 May 2017 22:33:21 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Link:

[PATCH 2/3] zswap: Improve a size determination in zswap_frontswap_init()

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 20 May 2017 22:44:03 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 3/3] zswap: Delete an error message for a failed memory allocation in zswap_dstmem_prepare()

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 21 May 2017 09:29:25 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Link:

gpio-exar: Why filtering out Commtech devices?

2017-05-21 Thread Jan Kiszka
Hi Sudip, why do we carry if (pcidev->vendor != PCI_VENDOR_ID_EXAR) return -ENODEV; in gpio_exar_probe? This effectively prevents that EXAR_DEVICE(COMMTECH, COMMTECH_4222PCIE, pbn_exar_XR17V35x), EXAR_DEVICE(COMMTECH, COMMTECH_4224PCIE,

Re: [PATCH v2] iio: adc: bcm_iproc_adc: swap primary and secondary isr handler's

2017-05-21 Thread Jonathan Cameron
On 16/05/17 07:52, Raveendra Padasalagi wrote: The third argument of devm_request_threaded_irq() is the primary handler. It is called in hardirq context and checks whether the interrupt is relevant to the device. If the primary handler returns IRQ_WAKE_THREAD, the secondary handler (a.k.a.

Re: [PATCH 2/9] timers: provide a "modern" variant of timers

2017-05-21 Thread Arnd Bergmann
On Sun, May 21, 2017 at 9:00 AM, Christoph Hellwig wrote: > On Thu, May 18, 2017 at 10:57:31AM +0200, Arnd Bergmann wrote: >> How expensive would it be to add another field to timer_list and >> just have both pointers? > > That would add 4/8 bytes to every structure containing a

[PATCH] PCI: fix whitespace in printk() to align log entries

2017-05-21 Thread Vincent Legoll
Remove the space character between "PCI" and the colon. This was printed before the patch: PCI : PCI BIOS area is rw and x. Use pci=nobios if you want it NX. Whereas other PCI printks are like that: $ dmesg | grep 'PCI:' [0.649178] PCI: MMCONFIG for domain [bus 00-7f] at [mem

Re: [PATCH] kconfig: Check for libncurses before menuconfig

2017-05-21 Thread Masahiro Yamada
Hi Borislav, 2017-05-21 5:33 GMT+09:00 Borislav Petkov : > On Sun, May 21, 2017 at 04:38:35AM +0900, Masahiro Yamada wrote: >> Strange, I could not reproduce this. > > Remove libncurses5-dev or whatever it is called on your system and do: > > $ make menuconfig > HOSTCC

[PATCH v2] kconfig: Check for libncurses before menuconfig

2017-05-21 Thread Borislav Petkov
On Sun, May 21, 2017 at 06:15:53PM +0900, Masahiro Yamada wrote: > Could you change line 199 > > $(addprefix $(obj)/,$(lxdialog)): $(obj)/dochecklxdialog > > to > > $(addprefix $(obj)/, mconf.o $(lxdialog)): $(obj)/dochecklxdialog > > and send v2, please? Here it is: --- From: Borislav

Re: [PATCH v3] staging: iio: light: Replace symbolic permissions as per coding style

2017-05-21 Thread Jonathan Cameron
On 20/05/17 19:36, Brian Masney wrote: On Sat, May 20, 2017 at 06:55:02PM +0100, Jonathan Cameron wrote: On 19/05/17 10:37, surenderpols...@gmail.com wrote: From: Surender Polsani Fixed the following checkpatch.pl warnings: octal permissions are more preferable

[PATCH] LSM: Make security_hook_heads a local variable.

2017-05-21 Thread Tetsuo Handa
A sealable memory allocator patch was proposed at http://lkml.kernel.org/r/20170519103811.2183-1-igor.sto...@huawei.com , and is waiting for a follow-on patch showing how any of the kernel can be changed to use this new subsystem. So, here it is for LSM hooks. The LSM hooks ("struct

Re: [PATCH v2 2/3] iio: adc: at91-sama5d2_adc: add hw trigger and buffer support

2017-05-21 Thread Jonathan Cameron
On 17/05/17 12:35, Eugen Hristev wrote: On 17.05.2017 10:47, Peter Rosin wrote: On 2017-05-16 20:03, Jonathan Cameron wrote: As we are only left with one area of questions. +static const struct at91_adc_trigger at91_adc_trigger_list[] = { +{ +.name = "external-rising", +

Re: [PATCH v6] iio: adc: Add support for TI ADC108S102 and ADC128S102

2017-05-21 Thread Jan Kiszka
On 2017-05-21 12:47, Jonathan Cameron wrote: > On 17/05/17 16:28, Jan Kiszka wrote: >> This is an upstream port of an IIO driver for the TI ADC108S102 and >> ADC128S102. The former can be found on the Intel Galileo Gen2 and the >> Siemens SIMATIC IOT2000. For those boards, ACPI-based enumeration

Re: [PATCH -next] drm/vgem: Fix return value check in vgem_init()

2017-05-21 Thread Chris Wilson
On Sun, May 21, 2017 at 01:19:39AM +, Wei Yongjun wrote: > From: Wei Yongjun > > In case of error, the function platform_device_register_simple() returns > ERR_PTR() and never returns NULL. The NULL test in the return value > check should be replaced with IS_ERR(). >

Re: [PATCH v6] iio: adc: Add support for TI ADC108S102 and ADC128S102

2017-05-21 Thread Jonathan Cameron
On 21/05/17 13:17, Jonathan Cameron wrote: On 21/05/17 12:59, Jan Kiszka wrote: On 2017-05-21 12:47, Jonathan Cameron wrote: On 17/05/17 16:28, Jan Kiszka wrote: This is an upstream port of an IIO driver for the TI ADC108S102 and ADC128S102. The former can be found on the Intel Galileo Gen2

Re: [PATCH 00/24] Thunderbolt security levels and NVM firmware upgrade

2017-05-21 Thread mika.westerb...@linux.intel.com
On Sat, May 20, 2017 at 09:15:17AM +, Levy, Amir (Jer) wrote: > > I created a udev rule that will automatically authorize the dock and cable. > > #dell cable > > ACTION=="add", SUBSYSTEM=="thunderbolt", ATTR{authorized}=="0", > > ATTR{vendor}=="0xd4", ATTR{device}=="0xb051",

Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error

2017-05-21 Thread gengdongjiu
Hi James, sorry for the late response due to recently verify and debug the RAS solution. 2017-05-13 1:24 GMT+08:00, James Morse : > Hi gengdongjiu, > > On 05/05/17 13:31, gengdongjiu wrote: >> when guest OS happen an SEA, My current solution is shown below: >> >> (1) host

Re: [PATCH] rcu: mark debug_lockdep_rcu_enabled() as pure

2017-05-21 Thread Konstantin Khlebnikov
On 20.05.2017 19:42, Paul E. McKenney wrote: On Fri, May 19, 2017 at 10:03:59AM +0300, Konstantin Khlebnikov wrote: This allows to get rid of unneeded invocations. Function debug_lockdep_rcu_enabled() becomes really hot if several debug options are enabled together with CONFIG_PROVE_RCU.

Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error

2017-05-21 Thread gengdongjiu
2017-05-13 1:25 GMT+08:00, James Morse : > Hi gengdongjiu, > > On 10/05/17 09:44, gengdongjiu wrote: >> On 2017/5/9 1:28, James Morse wrote: > (hwpoison for KVM is a corner case as Qemu's memory effectively has two > users, > Qemu and KVM. This isn't the example of

Re: zswap: Delete an error message for a failed memory allocation in zswap_dstmem_prepare()

2017-05-21 Thread SF Markus Elfring
> Markus, can you please stop CCing me on every of those patches? Yes, of course. >> Link: >> http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf Did I interpret any information from your presentation slides in an inappropriate way? > And why do

Re: zswap: Delete an error message for a failed memory allocation in zswap_dstmem_prepare()

2017-05-21 Thread Wolfram Sang
Markus, > > Markus, can you please stop CCing me on every of those patches? > > Yes, of course. Thanks! > >> Link: > >> http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf > > Did I interpret any information from your presentation slides in an >

RE: [PATCH 18/24] thunderbolt: Store Thunderbolt generation in the switch structure

2017-05-21 Thread Bernat, Yehezkel
> -Original Message- > From: Levy, Amir (Jer) > Sent: Sunday, May 21, 2017 11:07 > To: Mika Westerberg ; Lukas Wunner > > Cc: Greg Kroah-Hartman ; Andreas Noever > ; Jamet, Michael

[GIT pull] scheduler fix for 4.12

2017-05-21 Thread Thomas Gleixner
Linus, please pull the latest sched-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git sched-urgent-for-linus A single scheduler fix: Prevent idle task from ever being preempted. That makes sure that synchronize_rcu_tasks() which is ignoring idle

Re: Using best practices for big software change possibilities

2017-05-21 Thread Wolfram Sang
> How do you think about to resolve them by additional means besides mail > exchange? That can work. E.g. meeting at conferences often solved mail communication problems. For now, I still wonder why you were unsure about grouping the changes into one patch? Maybe there is something to be

Re: Using best practices for big software change possibilities

2017-05-21 Thread SF Markus Elfring
>> How do you think about to resolve them by additional means besides mail >> exchange? > > That can work. I am curious to find out which other communication means could really help here. > E.g. meeting at conferences often solved mail communication problems. I find my resources too limited

Re: [PATCH 18/24] thunderbolt: Store Thunderbolt generation in the switch structure

2017-05-21 Thread Mika Westerberg
On Sun, May 21, 2017 at 09:55:55AM +, Bernat, Yehezkel wrote: > > > > -Original Message- > > From: Levy, Amir (Jer) > > Sent: Sunday, May 21, 2017 11:07 > > To: Mika Westerberg ; Lukas Wunner > > > > Cc: Greg Kroah-Hartman

Re: [PATCH v6] iio: adc: Add support for TI ADC108S102 and ADC128S102

2017-05-21 Thread Jonathan Cameron
On 17/05/17 16:28, Jan Kiszka wrote: This is an upstream port of an IIO driver for the TI ADC108S102 and ADC128S102. The former can be found on the Intel Galileo Gen2 and the Siemens SIMATIC IOT2000. For those boards, ACPI-based enumeration is included. Due to the lack of regulators under ACPI,

[PATCH 0/3] zswap: Adjustments for three function implementations

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 21 May 2017 10:20:03 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in zswap_pool_create() Improve a size

Re: [PATCH v3 1/2] PCI/portdrv: add support for different MSI interrupts for PCIe port services

2017-05-21 Thread Christoph Hellwig
> + * > + * pci_irq_vector() below is able to handle entry differently > + * depending on MSI vs MSI-x case > + * >*/ One more instance of this comment left :) > pcie_capability_read_word(dev, PCI_EXP_FLAGS, ); >

Re: [PATCH 3/3] zswap: Delete an error message for a failed memory allocation in zswap_dstmem_prepare()

2017-05-21 Thread Wolfram Sang
Markus, can you please stop CCing me on every of those patches? > Omit an extra message for a memory allocation failure in this function. > > This issue was detected by using the Coccinelle software. > > Link: >

Re: [PATCH 4.4-only] openvswitch: clear sender cpu before forwarding packets

2017-05-21 Thread kbuild test robot
Hi Anoob, [auto build test ERROR on v4.9-rc8] [also build test ERROR on next-20170519] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH v5 0/3] More fixes for twl4030 charger

2017-05-21 Thread H. Nikolaus Schaller
Changes V5: * reworked max_current removal patch (comments by Sebastian Reichel) * resubmit missing madc connection for AC power detection * resubmit patch for irq allocation and -EPROBE_DEFER * rebased on 4.12-rc1 Changes V4: * resent commit (original one did contain material not upstream)

[PATCH v5 3/3] drivers:power:twl4030-charger: add deferred probing for phy and iio

2017-05-21 Thread H. Nikolaus Schaller
This fixes an issue if both this twl4030_charger driver and phy-twl4030-usb are compiled as modules and loaded in random order. It has been observed on GTA04 and OpenPandora devices that in worst case the boot process hangs and in best case the AC detection fails with a warning. Therefore we add

[PATCH v5 2/3] ARM: dts: twl4030: Add missing madc reference for bci subnode

2017-05-21 Thread H. Nikolaus Schaller
From: Marek Belisko The twl4030_charger driver expects an iio channel to detect the presence of an AC charger by looking at VAC (madc channel 11). This definition is missing in the device tree. Signed-off-by: Marek Belisko Signed-off-by: Sebastian

[PATCH v5 1/3] drivers:power:twl4030-charger: remove nonstandard max_current sysfs attribute

2017-05-21 Thread H. Nikolaus Schaller
Since we now support the standard 'input_current_limit' property by commit 3fb319c2cdcd ("power: supply: twl4030-charger: add writable INPUT_CURRENT_LIMIT property") we can now remove the nonstandard 'max_current' sysfs attribute. See Documentation/power/power_supply_class.txt line 125 Both

[RFC 2/3] misc: Add w2sg0004 (gps receiver) power control driver

2017-05-21 Thread H. Nikolaus Schaller
Add driver for Wi2Wi W2SG0004/84 GPS module connected through uart. Use serdev API hooks to monitor and forward the UART traffic to /dev/BTn and turn on/off the module. It also detects if the module is turned on (sends data) but should be off, e.g. if it was already turned on during boot or

[RFC 1/3] DTS: gta04: add serdev nodes for w2sg00x4, w2cbw etc.

2017-05-21 Thread H. Nikolaus Schaller
Signed-off-by: H. Nikolaus Schaller --- arch/arm/boot/dts/omap3-gta04.dtsi | 18 ++ 1 file changed, 18 insertions(+) diff --git a/arch/arm/boot/dts/omap3-gta04.dtsi b/arch/arm/boot/dts/omap3-gta04.dtsi index 430b438e0ee9..45f0c109960f 100644 ---

[RFC 3/3] misc: Add w2cbw003 (wifi/bluetooth) power control driver

2017-05-21 Thread H. Nikolaus Schaller
Add driver for Wi2Wi W2CBW003 WiFi and Bluetooth module where the Bluetooth interface is connected through uart. Uses the new serdev API to glue with tty and turn on/off the module if the tty port (/dev/ttyBTn) is opened. Note that this is only for the Bluetooth side. The WLAN (libertas) sdio

[RFC 0/3] misc: new serdev based drivers for w2sg00x4 GPS module and w2cbw003 wifi/bluetooth

2017-05-21 Thread H. Nikolaus Schaller
Since our proposed API was not acceptable and the new serdev API has arrived in 4.11 kernels, we finally took the challenge to update the w2sg and w2cbw drivers to use the serdev API. The approach is to write a "man in the middle" driver which is on one side a serdev client which directly

Re: [PATCH 18/24] thunderbolt: Store Thunderbolt generation in the switch structure

2017-05-21 Thread Mika Westerberg
On Sun, May 21, 2017 at 08:07:08AM +, Levy, Amir (Jer) wrote: > On Sun, May 21 2017, 11:00 AM, Mika Westerberg wrote: > > On Sun, May 21, 2017 at 10:40:41AM +0300, Mika Westerberg wrote: > > > On Sun, May 21, 2017 at 07:35:21AM +0200, Lukas Wunner wrote: > > > > On Sun, May 21, 2017 at

[PATCH] KEYS: Delete an error message for a failed memory allocation in get_derived_key()

2017-05-21 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 21 May 2017 11:23:55 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Link:

Re: [PATCH 18/24] thunderbolt: Store Thunderbolt generation in the switch structure

2017-05-21 Thread Mika Westerberg
On Sun, May 21, 2017 at 11:18:01AM +, Bernat, Yehezkel wrote: > Nothing that I'm aware of. OK. > Still, I like the idea of having a table-like construct somewhere in > the code to centralize the handling of various controller-specific > info (generation, DMA port, maybe more things in the

  1   2   3   4   5   6   >