Re: [PATCH] mfd: cpcap: Add minimal support

2016-11-21 Thread Rob Herring
On Mon, Nov 21, 2016 at 11:45:58AM +, Lee Jones wrote: > On Fri, 18 Nov 2016, Tony Lindgren wrote: > > > Many Motorola phones like droid 4 are using a custom PMIC called CPCAP > > or 6556002. We can support it's core features quite easily with regmap_spi > > and regmap_irq. > > > > The childr

Re: [PATCH 2/2] tools lib traceevent: Use USECS_PER_SEC instead of hardcoded number

2016-11-21 Thread Steven Rostedt
On Fri, 18 Nov 2016 20:40:22 -0300 Arnaldo Carvalho de Melo wrote: > Em Fri, Nov 18, 2016 at 05:54:01PM -0500, Steven Rostedt escreveu: > > From: "Steven Rostedt (Red Hat)" > > > > Instead of using 100, define a USECS_PER_SEC macro and use that > > instead. > > We already have it in too

Re: [PATCH 9/9] MODSIGN: Allow the "db" UEFI variable to be suppressed

2016-11-21 Thread Ard Biesheuvel
On 21 November 2016 at 16:26, Josh Boyer wrote: > On Mon, Nov 21, 2016 at 11:18 AM, Ard Biesheuvel > wrote: >> On 16 November 2016 at 18:11, David Howells wrote: >>> From: Josh Boyer >>> >>> If a user tells shim to not use the certs/hashes in the UEFI db variable >>> for verification purposes,

Re: [RFC PATCH v2 2/7] misc: minimal mux subsystem and gpio-based mux controller

2016-11-21 Thread Peter Rosin
On 2016-11-19 16:34, Jonathan Cameron wrote: > On 17/11/16 21:48, Peter Rosin wrote: >> When both the iio subsystem and the i2c subsystem wants to update >> the same mux, there needs to be some coordination. Invent a new >> minimal "mux" subsystem that handles this. > I'd probably put something mor

Re: [PATCH v2 1/5] ARM: memory: da8xx-ddrctl: new driver

2016-11-21 Thread Sekhar Nori
On Monday 31 October 2016 08:15 PM, Bartosz Golaszewski wrote: > +static int da8xx_ddrctl_probe(struct platform_device *pdev) > +{ > + const struct da8xx_ddrctl_config_knob *knob; > + const struct da8xx_ddrctl_setting *setting; > + struct device_node *node; > + struct resource *res;

[PATCH] kernel/watchdog.c: Only output hw-PMU message once

2016-11-21 Thread Prarit Bhargava
When CONFIG_BOOTPARAM_HOTPLUG_CPU0 is enabled, the socket containing the boot cpu can be replaced. During the hot add event, the message NMI watchdog: enabled on all CPUs, permanently consumes one hw-PMU counter. is output implying that the NMI was disabled at some point. This is not the case a

Re: [PATCH 4.8 00/49] 4.8.10-stable review

2016-11-21 Thread Shuah Khan
On 11/19/2016 02:22 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.8.10 release. > There are 49 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses sh

Re: [PATCH] cpufreq: schedutil: add up/down frequency transition rate limits

2016-11-21 Thread Peter Zijlstra
On Mon, Nov 21, 2016 at 04:24:24PM +, Patrick Bellasi wrote: > On 21-Nov 16:26, Peter Zijlstra wrote: > > In any case, worth trying, see what happens. > > Are you saying that you would like to see the code which implements a > more generic version of the peak_util "filter" on top of PELT? No

Re: [PATCH] PCI: Add information about describing PCI in ACPI

2016-11-21 Thread Bjorn Helgaas
On Mon, Nov 21, 2016 at 08:52:52AM +, Gabriele Paoloni wrote: > Hi Bjorn > > > -Original Message- > > From: Bjorn Helgaas [mailto:helg...@kernel.org] > > Sent: 18 November 2016 17:54 > > To: Gabriele Paoloni > > Cc: Bjorn Helgaas; linux-...@vger.kernel.org; linux- > > a...@vger.kernel.

Re: [RFC PATCH net v2 2/3] dt: bindings: add ethernet phy eee-disable-advert option documentation

2016-11-21 Thread Andrew Lunn
> What I did not realize when doing this patch for the realtek driver is > that there is already 6 valid modes defined in the kernel > > #define MDIO_EEE_100TXMDIO_AN_EEE_ADV_100TX /* > 100TX EEE cap */ > #define MDIO_EEE_1000TMDIO_AN_EEE_ADV_1000T /* > 1000T EE

[PATCH] staging: slicoss: change some macros into inline functions

2016-11-21 Thread Sergio Paracuellos
This patch changes macros into inline functions to avoid this checkpatch check: Macro argument 'x' may be better as '(x)' to avoid precedence issues It also removes two macros because they aren't being used. Signed-off-by: Sergio Paracuellos --- drivers/staging/slicoss/slic.h| 20 ++

Re: [PATCH v2 1/5] ARM: memory: da8xx-ddrctl: new driver

2016-11-21 Thread Bartosz Golaszewski
2016-11-21 17:33 GMT+01:00 Sekhar Nori : > On Monday 31 October 2016 08:15 PM, Bartosz Golaszewski wrote: >> +static int da8xx_ddrctl_probe(struct platform_device *pdev) >> +{ >> + const struct da8xx_ddrctl_config_knob *knob; >> + const struct da8xx_ddrctl_setting *setting; >> + struct

Re: [PATCH 4.4 00/37] 4.4.34-stable review

2016-11-21 Thread Greg Kroah-Hartman
On Mon, Nov 21, 2016 at 09:32:17AM -0700, Shuah Khan wrote: > On 11/19/2016 02:20 AM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 4.4.34 release. > > There are 37 patches in this series, all will be posted as a response > > to this one. If anyone has any issu

Re: Linux 4.9-rc6

2016-11-21 Thread Eric Dumazet
On Mon, 2016-11-21 at 05:51 -0800, Eric Dumazet wrote: > + while (chunk_order) { > + if (node == NUMA_NO_NODE) > + page = alloc_pages(multi_alloc_mask, > chunk_order); > + else > + page = a

Re: [PATCH v6] USB hub_probe: rework ugly goto-into-compound-statement

2016-11-21 Thread Greg Kroah-Hartman
On Sat, Nov 19, 2016 at 09:55:45AM +0300, Eugene Korenevsky wrote: > Rework smelling code (goto inside compound statement). Perhaps this is > legacy. Anyway such code is not appropriate for Linux kernel. > > Signed-off-by: Eugene Korenevsky > --- > Changes in v6: more pedantic conversion from `in

Re: [v5,3/5] USB: ohci: da8xx: Allow a regulator to handle VBUS

2016-11-21 Thread David Lechner
On 11/21/2016 04:22 AM, Axel Haslam wrote: Hi David, Thanks for the review, You're welcome. @@ -160,15 +212,41 @@ static void ohci_da8xx_ocic_handler(struct da8xx_ohci_root_hub *hub, hub->set_power(port, 0); } +static int ohci_da8xx_regulator_event(struct notifier_block *

Re: [RFC,2/2] hwmon: adt7411: add min, max and alarm attributes

2016-11-21 Thread Michael Walle
Am 2016-11-19 19:05, schrieb Guenter Roeck: Hi Michael, On Fri, Oct 14, 2016 at 11:43:35AM +0200, Michael Walle wrote: This patch adds support for the min, max and alarm attributes of the voltage and temperature channels. Additionally, the temp2_fault attribute is supported which indicates a f

Re: [PATCH v5 2/9] IB/core: Replace semaphore sm_sem with an atomic wait

2016-11-21 Thread Arnd Bergmann
On Monday, November 21, 2016 7:57:53 AM CET Linus Torvalds wrote: > Don't do this. > > Never ever do your own locking primitives. You will get the memory ordering > wrong. And even if you get it right, why do it? > > If you want to get rid of semaphores, and replace them with a mutex, that's > OK

Re: [PATCH v2] infiniband: remove WARN that is not kernel bug

2016-11-21 Thread Jason Gunthorpe
On Mon, Nov 21, 2016 at 02:14:08PM +0200, Leon Romanovsky wrote: > > > > In ib_ucm_write function there is a wrong prefix: > > > > + pr_err_once("ucm_write: process %d (%s) tried to do something hinky\n", > > I did it intentionally to have the same errors for all flows. Lets actually use a good m

Re: [PATCH 2/2] pci: Don't set RCB bit in LNKCTL if the upstream bridge hasn't

2016-11-21 Thread Bjorn Helgaas
On Wed, Nov 16, 2016 at 12:11:58PM -0600, Bjorn Helgaas wrote: > Hi Johannes, > > On Wed, Nov 02, 2016 at 04:35:52PM -0600, Johannes Thumshirn wrote: > > The Read Completion Boundary (RCB) bit must only be set on a device or > > endpoint if it is set on the root complex. > > I propose the followi

[RESEND PATCH 1/3] ARM: davinci: da830: Handle vbus with a regulator

2016-11-21 Thread Axel Haslam
The usb driver can now take a regulator instead of the platform callbacks for vbus handling. Lets use a regulator so we can remove the callbacks in a later patch. Signed-off-by: Axel Haslam --- arch/arm/mach-davinci/board-da830-evm.c | 108 +++- 1 file changed, 38 ins

[RESEND PATCH 0/3] ARM: davinci: OHCI: Use a regulator instead of callbacks

2016-11-21 Thread Axel Haslam
With the ultimate goal to able to probe the ohci driver form DT, convert users of OHCI pdata to use a regulator instead of passing platform function pointers. This will help to remove the platform callbacks in a future series. These patches were included in previous series[1], im sending them agai

Re: [PATCH reworded] x86/kbuild: enable modversions for symbols exported from asm

2016-11-21 Thread Oliver Hartkopp
On 11/21/2016 03:52 PM, Adam Borowski wrote: Commit 4efca4ed ("kbuild: modversions for EXPORT_SYMBOL() for asm") adds modversion support for symbols exported from asm files. Architectures must include C-style declarations for those symbols in asm/asm-prototypes.h in order for them to be version

[PATCH] net: phy: micrel: fix KSZ8041FTL supported value

2016-11-21 Thread Kirill Esipov
Fix setting of SUPPORTED_FIBRE bit as it was not present in features of KSZ8041. Signed-off-by: Kirill Esipov --- drivers/net/phy/micrel.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/phy/micrel.c b/drivers/net/phy/micrel.c index 081df68..ea92d52 100644

Re: [PATCH] ext4: remove unused function ext4_aligned_io()

2016-11-21 Thread Theodore Ts'o
On Sun, Nov 20, 2016 at 10:02:17PM -0700, Ross Zwisler wrote: > The last user of ext4_aligned_io() was the DAX path in > ext4_direct_IO_write(). This usage was removed by Jan Kara's patch > entitled "ext4: Rip out DAX handling from direct IO path". > > Signed-off-by: Ross Zwisler Applied, thank

[PATCH] ARM: dts: sunxi: Enable UEXT related nodes for Olimex A20 SOM EVB

2016-11-21 Thread Emmanuel Vadot
UEXT are Universal EXTension connector from Olimex. They embed i2c, spi and uart pins along power in one connector and are found on most, if not all, Olimex boards. The Olimex A20 SOM EVB have two UEXT connector so enable the nodes found on those two connectors. Signed-off-by: Emmanuel Vadot ---

[RESEND PATCH 2/3] ARM: davinci: hawk: Remove vbus and over current gpios

2016-11-21 Thread Axel Haslam
The hawk board VBUS is fixed to a 5v source, and the over current pin is actually not connected to the SoC. Do not reseve these gpios for OHCI as they are not related to usb. Signed-off-by: Axel Haslam --- arch/arm/mach-davinci/board-omapl138-hawk.c | 99 ++--- 1 file ch

[RESEND PATCH 3/3] ARM: davinci: remove ohci platform usage

2016-11-21 Thread Axel Haslam
As all users of ohci platform data have been converted to use a regulator, we dont need to pass platform data to register the ohci device anymore. Signed-off-by: Axel Haslam --- arch/arm/mach-davinci/board-da830-evm.c | 2 +- arch/arm/mach-davinci/board-omapl138-hawk.c | 2 +- arch/arm/mach-

Re: [PATCH v5 2/9] IB/core: Replace semaphore sm_sem with an atomic wait

2016-11-21 Thread Christoph Hellwig
On Mon, Nov 21, 2016 at 05:52:27PM +0100, Arnd Bergmann wrote: > I think a mutex would't work here, since fops->open() and fops->close() > are not called from the same context and lockdep will complain > about that. > > Version of the series had replaced the semaphore with a completion > here, whi

Re: Formal description of system call interface

2016-11-21 Thread Cyril Hrubis
Hi! > > Looking ahead into the future, I was also thinking that if this becomes > > robust, we could also start an integration specification, that could > > describe how different system calls interact with each other. Like > > open() to read(), write() and close(). > > > > But this is just an idea

[PATCH 1/2] ARM: dts: da850: Add usb device node

2016-11-21 Thread Axel Haslam
Add the usb1 device node for the da850 soc. This will allow boards to use the usb1 port when booting through DT. Signed-off-by: Axel Haslam --- arch/arm/boot/dts/da850.dtsi | 8 1 file changed, 8 insertions(+) diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi ind

[PATCH 2/2] ARM: dts: da850-lcdk: Enable ohci for omapl138 lcdk

2016-11-21 Thread Axel Haslam
Enable the usb1 controller (ohci) and phy for the lcdk board Signed-off-by: Axel Haslam --- arch/arm/boot/dts/da850-lcdk.dts | 8 1 file changed, 8 insertions(+) diff --git a/arch/arm/boot/dts/da850-lcdk.dts b/arch/arm/boot/dts/da850-lcdk.dts index 7b8ab21..a739603 100644 --- a/arch/ar

[PATCH 0/2] ARM: davinvi: da850 add ohci DT nodes

2016-11-21 Thread Axel Haslam
This adds the DT node for the ohci controller and enables it for the omapl138-lckd platform. DEPENDENCIES: 1. [PATCH v6 0/5] USB: ohci-da8xx: Add device tree support https://lkml.org/lkml/2016/11/21/558 2. [PATCH v3 0/2] regulator: handling of error conditions for usb drivers https://lkml.org/lk

[PATCH] ARM: davinci_all_defconfig: Enable OHCI as module

2016-11-21 Thread Axel Haslam
Enable the davinci ohci driver as a module for usb tested with the omap138-lcdk hawk board. Signed-off-by: Axel Haslam --- arch/arm/configs/davinci_all_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/configs/davinci_all_defconfig b/arch/arm/configs/davinci_all_defconfig i

Re: [PATCH 1/2] ARM64: dts: Add support for Meson GXM

2016-11-21 Thread Kevin Hilman
Neil Armstrong writes: > Following the Amlogic Linux kernel, it seem the only differences > between the GXL and GXM SoCs are the CPU Clusters. > > This commit renames the gxl-s905d-p23x DTSI in a common file for > S905D p20x and S912 q20x boards. > > Then adds a meson-gxm dtsi and reproduce the P

Re: [PATCH v2] Add a "nosymlinks" mount option.

2016-11-21 Thread James Bottomley
On Wed, 2016-11-16 at 13:18 -0800, Mattias Nissler wrote: > I understand that silence suggests there's little interest, but > here's some new information I discovered today that may justify to > reconsider the patch: > > The BSDs already have exactly what I propose, the mount option is > called "

Re: [PATCH 0/2] ARM64: dts: Add support for Meson GXM

2016-11-21 Thread Kevin Hilman
Neil Armstrong writes: > The new Amlogic GXM SoC (S912) is part of the Meson GX family and is nearly > identical to GXM but with a second Quad-A53 core cluster. > > The GXM dtsi includes the GXL dtsi and the p20x dtsi is refactored in a > common p20x/q20x to support the GXM Q200 and Q201 board th

Re: [PATCH v2 3/3] usb: ohci-da8xx: rename driver to ohci-da8xx

2016-11-21 Thread Axel Haslam
Hi Greg, On Thu, Nov 3, 2016 at 5:03 PM, Axel Haslam wrote: > The davinci ohci driver name (currently "ohci") is too generic. > To be consistent with other usb dirvers, append the "-da8xx" postfix > to the name. > if there are no objections, would it be possible to pick up this patch? the corres

Re: [PATCH v4 1/3] Documentation: DT: add dma compatible for sun50i A64 SOC

2016-11-21 Thread Rob Herring
On Sun, Nov 20, 2016 at 06:45:38PM +0800, Hao Zhang wrote: > This add the property of Allwinner sun50i A64 dma. > > Signed-off-by: Hao Zhang > --- > Documentation/devicetree/bindings/dma/sun6i-dma.txt | 1 + > 1 file changed, 1 insertion(+) Acked-by: Rob Herring

Re: [PATCH 0/2] ARM: davinvi: da850 add ohci DT nodes

2016-11-21 Thread David Lechner
On 11/21/2016 10:59 AM, Axel Haslam wrote: This adds the DT node for the ohci controller and enables it for the omapl138-lckd platform. DEPENDENCIES: 1. [PATCH v6 0/5] USB: ohci-da8xx: Add device tree support https://lkml.org/lkml/2016/11/21/558 2. [PATCH v3 0/2] regulator: handling of error c

Re: [PATCH] ARM: dts: exynos: remove the cd-gpios property for eMMC of odroid-xu3/4

2016-11-21 Thread Krzysztof Kozlowski
On Mon, Nov 21, 2016 at 04:10:32PM +0900, Jaehoon Chung wrote: > Odroid-xu3/4 didn't need to use the cd-gpios for detecting card. > Because Host controller has the CDETECT register through SDx_CDN line. > Host controller can know whether card is inserted or not with this > register. > > When i hav

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Andi Kleen
> > it got away with attached change.. but this rcu logic > > is far beyond me, so it's just wild guess.. ;-) > > I think I prefer something like the below, that only annotates the one > RDMSR in question, instead of all of them. It would be far better to just fix trace points that they always wo

Re: [RFC PATCH] scsi: libsas: fix WARN on device removal

2016-11-21 Thread Dan Williams
On Mon, Nov 21, 2016 at 7:16 AM, John Garry wrote: > @Maintainers, would you be willing to accept this patch as an interim > fix > for the dastardly WARN while we try to fix the flutter issue? To me this adds a bug to quiet a benign, albeit noisy, warning. >>>

Re: [PATCH 00/11] trace-cmd: make libtracecmd a linkable object

2016-11-21 Thread Steven Rostedt
On Fri, 20 Nov 2015 15:57:22 -0500 Josef Bacik wrote: > We use trace-cmd pretty extensively inside of Facebook for various things, > such > as live gathering of statistics. Currently this involves launching trace-cmd > to > record data and then processing the information afterwards. This make

Re: [PATCH for-next 03/11] IB/hns: Optimize the logic of allocating memory using APIs

2016-11-21 Thread Leon Romanovsky
On Mon, Nov 21, 2016 at 04:12:38PM +, Salil Mehta wrote: > > -Original Message- > > From: Leon Romanovsky [mailto:l...@kernel.org] > > Sent: Wednesday, November 16, 2016 8:36 AM > > To: Salil Mehta > > Cc: dledf...@redhat.com; Huwei (Xavier); oulijun; > > mehta.salil@gmail.com; linu

[PATCH] drm: tilcdc: reduce max_width for revision 1

2016-11-21 Thread Bartosz Golaszewski
It has been determined that the highest resolution supported correctly by LCDC rev1 is 800x600. Reduce the max_width value for rev1 to 800 in crtc_max_width(). Signed-off-by: Bartosz Golaszewski --- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH] coresight: perf: Add a missing call to etm_free_aux

2016-11-21 Thread Mathieu Poirier
On Sat, Nov 19, 2016 at 06:41:24PM +0100, Quentin Lambert wrote: > Most error branches following the call to alloc_event_data contain a call to > etm_free_aux. This patch add a call to etm_free_aux to an error branch > that does not call it. > > This issue was found with Hector. > > Signed-off-by

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2016-11-21 Thread Mark Brown
On Thu, Nov 17, 2016 at 05:46:13PM +1100, NeilBrown wrote: > On Thu, Nov 17 2016, Mark Brown wrote: > > To me that's pretty much what's being done here, the code just happens > > to sit in USB instead but fundamentally it's just a blob of helper code, > > you could replace the notifier with a call

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Peter Zijlstra
On Mon, Nov 21, 2016 at 09:06:13AM -0800, Andi Kleen wrote: > > > it got away with attached change.. but this rcu logic > > > is far beyond me, so it's just wild guess.. ;-) > > > > I think I prefer something like the below, that only annotates the one > > RDMSR in question, instead of all of them

Re: [PATCHv2] arm64: dts: exynos: add the mshc_2 node for supporting T-Flash

2016-11-21 Thread Krzysztof Kozlowski
On Mon, Nov 21, 2016 at 01:58:39PM +0900, Jaehoon Chung wrote: > Add the mshc_2 node for supporting T-flash. > > And it needs to add the "mshc*" aliases. Because dwmmc driver should be > assigned to "ctrl_id" after parsing to "mshc". > If there is no aliases for mshc, then it might be set to the w

RE: [PATCH] PCI: Add information about describing PCI in ACPI

2016-11-21 Thread Gabriele Paoloni
Hi Bjorn > -Original Message- > From: linux-pci-ow...@vger.kernel.org [mailto:linux-pci- > ow...@vger.kernel.org] On Behalf Of Bjorn Helgaas > Sent: 21 November 2016 16:47 > To: Gabriele Paoloni > Cc: Bjorn Helgaas; linux-...@vger.kernel.org; linux- > a...@vger.kernel.org; linux-kernel@vge

[PATCH] drm: tilcdc: reduce max_width for revision 1

2016-11-21 Thread Bartosz Golaszewski
While debugging the drm_bridge support for revision 1 I noticed the driver was selecting the 1024x768 resolution as default from the set retrieved from EDID. The following patch reduces the max_width for rev1 in tilcdc. Bartosz Golaszewski (1): drm: tilcdc: reduce max_width for revision 1 driv

Re: [PATCH 2/3] locking/percpu-rwsem: Replace bulky wait-queues with swait

2016-11-21 Thread Davidlohr Bueso
On Mon, 21 Nov 2016, Oleg Nesterov wrote: On 11/18, Davidlohr Bueso wrote: @@ -12,7 +12,7 @@ struct percpu_rw_semaphore { struct rcu_sync rss; unsigned int __percpu *read_count; struct rw_semaphore rw_sem; - wait_queue_head_t writer; + st

Re: [PATCH] drm: tilcdc: reduce max_width for revision 1

2016-11-21 Thread Jyri Sarha
On 11/21/16 19:16, Bartosz Golaszewski wrote: > It has been determined that the highest resolution supported correctly > by LCDC rev1 is 800x600. Reduce the max_width value for rev1 to 800 in > crtc_max_width(). > I don't think this is the right way to limit the supported video modes. There is te

Re: [PATCH] drm: tilcdc: reduce max_width for revision 1

2016-11-21 Thread Bartosz Golaszewski
2016-11-21 18:26 GMT+01:00 Jyri Sarha : > On 11/21/16 19:16, Bartosz Golaszewski wrote: >> It has been determined that the highest resolution supported correctly >> by LCDC rev1 is 800x600. Reduce the max_width value for rev1 to 800 in >> crtc_max_width(). >> > > I don't think this is the right way

Re: [PATCH 0/2] ARM: davinvi: da850 add ohci DT nodes

2016-11-21 Thread Axel Haslam
On Mon, Nov 21, 2016 at 6:04 PM, David Lechner wrote: > On 11/21/2016 10:59 AM, Axel Haslam wrote: >> >> This adds the DT node for the ohci controller and >> enables it for the omapl138-lckd platform. >> >> DEPENDENCIES: >> >> 1. [PATCH v6 0/5] USB: ohci-da8xx: Add device tree support >> https://l

Re: [PATCH v2] infiniband: remove WARN that is not kernel bug

2016-11-21 Thread Leon Romanovsky
On Mon, Nov 21, 2016 at 09:52:53AM -0700, Jason Gunthorpe wrote: > On Mon, Nov 21, 2016 at 02:14:08PM +0200, Leon Romanovsky wrote: > > > > > > In ib_ucm_write function there is a wrong prefix: > > > > > > + pr_err_once("ucm_write: process %d (%s) tried to do something hinky\n", > > > > I did it in

Re: [PATCH v5 1/2] dt-bindings: display: Add Sharp LQ150X1LG11 panel binding

2016-11-21 Thread Rob Herring
On Mon, Nov 21, 2016 at 04:00:48PM +0100, Peter Rosin wrote: > The Sharp 15" LQ150X1LG11 panel is an XGA TFT LCD panel. > > Signed-off-by: Peter Rosin > --- > .../bindings/display/panel/sharp,lq150x1lg11.txt | 36 > ++ > 1 file changed, 36 insertions(+) > create mode 1006

Re: [PATCHv3 5/6] arm64: Use __pa_symbol for kernel symbols

2016-11-21 Thread Laura Abbott
On 11/18/2016 06:35 AM, Mark Rutland wrote: > Hi Laura, > > On Thu, Nov 17, 2016 at 05:16:55PM -0800, Laura Abbott wrote: >> >> __pa_symbol is technically the marco that should be used for kernel >> symbols. Switch to this as a pre-requisite for DEBUG_VIRTUAL which >> will do bounds checking. >> >

Re: [PATCH 0/9] Get rid of bitmap images

2016-11-21 Thread Jonathan Corbet
On Sun, 20 Nov 2016 14:08:31 -0200 Mauro Carvalho Chehab wrote: > The goal of this patch series is to get rid of PNG images, using either > graphviz > or SVG for images. > > For old images generated with xfig, stored inside PDF, just convert them to > SVG > and cleanup the images using inkscap

Re: [PATCH 0/2] ARM: davinvi: da850 add ohci DT nodes

2016-11-21 Thread David Lechner
On 11/21/2016 11:29 AM, Axel Haslam wrote: On Mon, Nov 21, 2016 at 6:04 PM, David Lechner wrote: On 11/21/2016 10:59 AM, Axel Haslam wrote: This adds the DT node for the ohci controller and enables it for the omapl138-lckd platform. DEPENDENCIES: 1. [PATCH v6 0/5] USB: ohci-da8xx: Add devic

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Andi Kleen
On Mon, Nov 21, 2016 at 06:18:53PM +0100, Peter Zijlstra wrote: > On Mon, Nov 21, 2016 at 09:06:13AM -0800, Andi Kleen wrote: > > > > it got away with attached change.. but this rcu logic > > > > is far beyond me, so it's just wild guess.. ;-) > > > > > > I think I prefer something like the below,

Re: [PATCH v2 1/5] ARM: memory: da8xx-ddrctl: new driver

2016-11-21 Thread Robin Murphy
Hi Bartosz, Sekhar, On 21/11/16 16:48, Bartosz Golaszewski wrote: > 2016-11-21 17:33 GMT+01:00 Sekhar Nori : >> On Monday 31 October 2016 08:15 PM, Bartosz Golaszewski wrote: >>> +static int da8xx_ddrctl_probe(struct platform_device *pdev) >>> +{ >>> + const struct da8xx_ddrctl_config_knob *kn

Re: [PATCH] kernel/watchdog.c: Only output hw-PMU message once

2016-11-21 Thread Don Zickus
On Mon, Nov 21, 2016 at 11:35:19AM -0500, Prarit Bhargava wrote: > When CONFIG_BOOTPARAM_HOTPLUG_CPU0 is enabled, the socket containing the > boot cpu can be replaced. During the hot add event, the message > > NMI watchdog: enabled on all CPUs, permanently consumes one hw-PMU counter. > > is out

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Steven Rostedt
On Mon, 21 Nov 2016 18:18:53 +0100 Peter Zijlstra wrote: > Its not ftrace as such though, its RCU, ftrace simply uses RCU to avoid > locking, as one does. Just to be clear, as ftrace in the kernel mostly represents function tracing, which doesn't use RCU. This is a tracepoint feature. > > Bigg

Re: [PATCH v2 1/5] ARM: memory: da8xx-ddrctl: new driver

2016-11-21 Thread Sudeep Holla
Hi Robin, On 21/11/16 17:47, Robin Murphy wrote: Hi Bartosz, Sekhar, On 21/11/16 16:48, Bartosz Golaszewski wrote: [...] Hi Sekhar, thanks for spotting that. I think we should introduce this function right away, rather than having two static functions doing the same thing. If you don't mi

Re: [PATCH v5 2/9] IB/core: Replace semaphore sm_sem with an atomic wait

2016-11-21 Thread Linus Torvalds
On Mon, Nov 21, 2016 at 8:52 AM, Arnd Bergmann wrote: > > Version of the series had replaced the semaphore with a completion > here, which worked correctly, but one reviewer suggested using > the wait_event() instead since it's confusing to have a completion > starting out in 'completed' state. Q

Re: What exactly do 32-bit x86 exceptions push on the stack in the CS slot?

2016-11-21 Thread Linus Torvalds
On Sun, Nov 20, 2016 at 11:13 PM, Ingo Molnar wrote: > > So I have applied your fix that addresses the worst fallout directly: > > fc0e81b2bea0 x86/traps: Ignore high word of regs->cs in > early_fixup_exception() > > ... but otherwise we might be better off zeroing out the high bits of segment

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Steven Rostedt
On Mon, 21 Nov 2016 09:45:04 -0800 Andi Kleen wrote: > On Mon, Nov 21, 2016 at 06:18:53PM +0100, Peter Zijlstra wrote: > > On Mon, Nov 21, 2016 at 09:06:13AM -0800, Andi Kleen wrote: > > > > > it got away with attached change.. but this rcu logic > > > > > is far beyond me, so it's just wild gu

Re: [PATCH] kernel/watchdog.c: Only output hw-PMU message once

2016-11-21 Thread Prarit Bhargava
On 11/21/2016 12:48 PM, Don Zickus wrote: > On Mon, Nov 21, 2016 at 11:35:19AM -0500, Prarit Bhargava wrote: >> When CONFIG_BOOTPARAM_HOTPLUG_CPU0 is enabled, the socket containing the >> boot cpu can be replaced. During the hot add event, the message >> >> NMI watchdog: enabled on all CPUs, per

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Andi Kleen
> And a popf can be much more expensive than any of these. You should > know, not all instructions are equal. > > Using perf, I've seen popf take up almst 30% of a function the size of > this. In any case it's a small fraction of the 600+ instructions which are currently executed for every enable

Re: vmalloced stacks and scatterwalk_map_and_copy()

2016-11-21 Thread Eric Biggers
On Mon, Nov 21, 2016 at 04:26:19PM +0800, Herbert Xu wrote: > crypto: scatterwalk - Remove unnecessary aliasing check in map_and_copy > > The aliasing check in map_and_copy is no longer necessary because > the IPsec ESP code no longer provides an IV that points into the > actual request data. As

[PATCH 2/2] power: supply: add sbs-charger driver

2016-11-21 Thread Nicola Saenz Julienne
This adds support for sbs-charger compilant chips as defined here: http://sbs-forum.org/specs/sbc110.pdf This was tested on a arm board connected to an LTC41000 battery charger chip. Signed-off-by: Nicola Saenz Julienne --- drivers/power/supply/Kconfig | 6 + drivers/power/supply/Makefi

[PATCH 1/2] power: supply: sbs-battery: use fixed device name

2016-11-21 Thread Nicola Saenz Julienne
The current device name for sbs-battery is derived from it's i2c address. This is not acceptable if we want to be able to trigger the "external_power_changed()" routine from a charger driver. Signed-off-by: Nicola Saenz Julienne --- drivers/power/supply/sbs-battery.c | 6 +- 1 file changed,

Re: [PATCH 2/2] Add basic support for DW CSI-2 Host IPK

2016-11-21 Thread Ramiro Oliveira
Hi Sakari, Thank you for your feedback. On 11/17/2016 4:34 PM, Sakari Ailus wrote: > Hi Ramiro, > > Thank you for the patchset. Please see my comments below. > > If you've got a CSI-2 receiver, why do you use the DV timing presets? What > do you use those for? > Our CSI-2 Host has two differe

[PATCH 0/2] power: supply: add sbs-charger driver

2016-11-21 Thread Nicola Saenz Julienne
This series adds support for all SBS compatible battery chargers, as defined here: http://sbs-forum.org/specs/sbc110.pdf. The first patch changes the sbs-battery device name in order to be able to create a proper supplier/supplied relation between the two of them. The second introduces the driver

Re: What exactly do 32-bit x86 exceptions push on the stack in the CS slot?

2016-11-21 Thread Linus Torvalds
On Mon, Nov 21, 2016 at 7:58 AM, H. Peter Anvin wrote: > On 11/20/16 20:54, h...@zytor.com wrote: >> >> I believe i686+ writes zero, older CPUs leave unchanged. > > I should point out that, at least from my memory, the same applies to > instructions like "movl ". I can't even remember for sure ho

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Steven Rostedt
On Mon, 21 Nov 2016 10:06:54 -0800 Andi Kleen wrote: > > And a popf can be much more expensive than any of these. You should > > know, not all instructions are equal. > > > > Using perf, I've seen popf take up almst 30% of a function the size of > > this. > > In any case it's a small fraction

Re: [PATCH] drm/atomic: cleanup debugfs entries on un-registering the driver.

2016-11-21 Thread Sean Paul
On Thu, Nov 17, 2016 at 7:26 AM, Brian Starkey wrote: > On Thu, Nov 17, 2016 at 11:41:29AM +, Liviu Dudau wrote: >> >> Cleanup the debugfs entries created by commit 6559c901cb48 when >> the driver's minor gets un-registered. Without it, DRM drivers >> compiled as modules cannot be rmmod-ed and

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Steven Rostedt
Paul, On Mon, 21 Nov 2016 12:55:01 -0500 Steven Rostedt wrote: > On Mon, 21 Nov 2016 18:18:53 +0100 > Peter Zijlstra wrote: > > > Its not ftrace as such though, its RCU, ftrace simply uses RCU to avoid > > locking, as one does. > > Just to be clear, as ftrace in the kernel mostly represen

Re: [PATCH v6 0/9] tpm: cleanup/fixes in existing event log support

2016-11-21 Thread Jason Gunthorpe
On Sun, Nov 20, 2016 at 09:58:48AM +, Jarkko Sakkinen wrote: > On Sat, Nov 19, 2016 at 11:36:27AM -0700, Jason Gunthorpe wrote: > > On Tue, Nov 15, 2016 at 10:54:53AM -0800, Jarkko Sakkinen wrote: > > > On Tue, Nov 15, 2016 at 11:06:10AM -0700, Jason Gunthorpe wrote: > > > > On Tue, Nov 15, 201

Re: What exactly do 32-bit x86 exceptions push on the stack in the CS slot?

2016-11-21 Thread H. Peter Anvin
On 11/21/16 10:00, Linus Torvalds wrote: > > Ugh. > > I'd much rather we go back to just making the "cs" entry explicitly > 16-bit, and have a separate padding entry, the way we used to long > long ago. > I would agree 100% with this. -hpa

Re: [PATCH] ufs: qcom: Properly clear hba priv on failure

2016-11-21 Thread Subhash Jadavani
On 2016-11-19 22:15, Bjorn Andersson wrote: On Sat 19 Nov 12:30 PST 2016, Subhash Jadavani wrote: On 2016-11-18 12:55, Bjorn Andersson wrote: >In the case where we fail to acquire the phy the hba priv will be set >already, so during cleanup ufs_qcom_setup_clocks() will dereference the >now free

Re: [PATCH v2] ufs: qcom: Properly clear hba priv on failure

2016-11-21 Thread Subhash Jadavani
On 2016-11-19 22:34, Bjorn Andersson wrote: ufs_qcom_init() sets the hba priv data before attempting to acquire the phy handle, so make sure to clear this in the case of an error. Failing to do this will make ufs_qcom_setup_clocks() operate on the uninitalized host object. Signed-off-by: Bjorn

[GIT] Networking

2016-11-21 Thread David Miller
1) Clear congestion control state when changing algorithms on an existing socket, from Florian Westphal. 2) Fix register bit values in altr_tse_pcs portion of stmmac driver, from Jia Jie Ho. 3) Fix PTP handling in stammc driver for GMAC4, from Giuseppe CAVALLARO. 4) Fix udplite multica

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Andi Kleen
> It tries to be optimized. I "unoptimized" it a while back to pull out > all the inlines that were done in the tracepoint itself. That is, the > trace_() function is inlined in the code itself. By > breaking that up a bit, I was able to save a bunch of text because the > tracepoints were bloating

[RFC/PATCH] powernv: Handle wakeup from stop due to SRESET

2016-11-21 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" The existing code doesn't handle the case when CPU which was in a hardware-idle state (nap,sleep,winkle on POWER8 and various stop states on POWER9) gets woken up due to a System Reset interrupt. This patch checks if the CPU was woken up due to System Reset, in which ca

[GIT] Sparc

2016-11-21 Thread David Miller
1) With modern networking cards we can run out of 32-bit DMA space, so support 64-bit DMA addressing when possible on sparc64. From Dave Tushar. 2) Some signal frame validation checks are inverted on sparc32, fix from Andreas Larsson. 3) Lockdep tables can get too large in some circums

Re: [PATCH v2 0/3] Clean up watchdog handlers

2016-11-21 Thread David Miller
From: Babu Moger Date: Tue, 1 Nov 2016 14:13:43 -0700 > This is an attempt to cleanup watchdog handlers. Right now, > kernel/watchdog.c implements both softlockup and hardlockup detectors. > Softlockup code is generic. Hardlockup code is arch specific. Some > architectures don't use hardlockup d

Re: [PATCH] scripts/decode_stacktrace.sh: canonicalize basepath and modpath

2016-11-21 Thread Kirill A. Shutemov
On Mon, Nov 21, 2016 at 06:57:26PM +0300, Konstantin Khlebnikov wrote: > On Mon, Nov 21, 2016 at 2:29 PM, Kirill A. Shutemov > wrote: > > I use decode_stacktrace.sh as > > > > $ ./scripts/decode_stacktrace.sh vmlinux . . > > > > It means basepath is equal to "." and decode_stacktrace stips dot fro

Re: [PATCH] vfio: fix vfio_info_cap_add/shift

2016-11-21 Thread David Hildenbrand
Am 21.11.2016 um 07:21 schrieb Eric Auger: Capability header next field is an offset relative to the start of the INFO buffer. tmp->next is assigned the proper value but iterations implemented in vfio_info_cap_add and vfio_info_cap_shift use next as an offset between the headers. When coping with

Re: [1/2] kbuild: provide include/asm/asm-prototypes.h for ARM

2016-11-21 Thread Uwe Kleine-König
Hello, On Mon, Oct 24, 2016 at 05:05:26PM +0200, Arnd Bergmann wrote: > This adds an asm/asm-prototypes.h header for ARM to fix the > broken symbol versioning for symbols exported from assembler > files. > > In addition to the header, we have to do these other small > changes: > > - move the exp

Re: [PATCH 0/2] ARM: davinvi: da850 add ohci DT nodes

2016-11-21 Thread Axel Haslam
On Mon, Nov 21, 2016 at 6:45 PM, David Lechner wrote: > On 11/21/2016 11:29 AM, Axel Haslam wrote: >> >> On Mon, Nov 21, 2016 at 6:04 PM, David Lechner >> wrote: >>> >>> On 11/21/2016 10:59 AM, Axel Haslam wrote: This adds the DT node for the ohci controller and enables it for

Re: [PATCH 4/4] timekeeping: clocksource_cyc2ns: Document intended range limitation

2016-11-21 Thread Chris Metcalf
On 11/21/2016 3:54 AM, Ingo Molnar wrote: * John Stultz wrote: From: Chris Metcalf The "cycles" argument should not be an absolute clocksource cycle value, as the implementation's arithmetic will overflow relatively easily with wide (64 bit) clocksource counters. For performance, the implem

Re: [PATCH 9/9] MODSIGN: Allow the "db" UEFI variable to be suppressed

2016-11-21 Thread Peter Jones
On Mon, Nov 21, 2016 at 04:42:45PM +, Ard Biesheuvel wrote: > On 21 November 2016 at 16:26, Josh Boyer wrote: > > On Mon, Nov 21, 2016 at 11:18 AM, Ard Biesheuvel > > wrote: > >> On 16 November 2016 at 18:11, David Howells wrote: > >>> From: Josh Boyer > >>> > >>> If a user tells shim to no

Re: [PATCH 9/9] MODSIGN: Allow the "db" UEFI variable to be suppressed

2016-11-21 Thread Ard Biesheuvel
On 21 November 2016 at 20:05, Peter Jones wrote: > On Mon, Nov 21, 2016 at 04:42:45PM +, Ard Biesheuvel wrote: >> On 21 November 2016 at 16:26, Josh Boyer wrote: >> > On Mon, Nov 21, 2016 at 11:18 AM, Ard Biesheuvel >> > wrote: >> >> On 16 November 2016 at 18:11, David Howells wrote: >> >>>

Re: [PATCH 2/2] tools lib traceevent: Use USECS_PER_SEC instead of hardcoded number

2016-11-21 Thread Arnaldo Carvalho de Melo
Em Mon, Nov 21, 2016 at 11:41:49AM -0500, Steven Rostedt escreveu: > On Fri, 18 Nov 2016 20:40:22 -0300 > Arnaldo Carvalho de Melo wrote: > > > Em Fri, Nov 18, 2016 at 05:54:01PM -0500, Steven Rostedt escreveu: > > > From: "Steven Rostedt (Red Hat)" > > > > > > Instead of using 100, define

Re: [1/2] kbuild: provide include/asm/asm-prototypes.h for ARM

2016-11-21 Thread Russell King - ARM Linux
On Mon, Nov 21, 2016 at 07:46:44PM +0100, Uwe Kleine-König wrote: > Hello, > > On Mon, Oct 24, 2016 at 05:05:26PM +0200, Arnd Bergmann wrote: > > This adds an asm/asm-prototypes.h header for ARM to fix the > > broken symbol versioning for symbols exported from assembler > > files. > > > > In addi

Re: [BUG] msr-trace.h:42 suspicious rcu_dereference_check() usage!

2016-11-21 Thread Steven Rostedt
On Mon, 21 Nov 2016 10:37:00 -0800 Andi Kleen wrote: > Ok so how should tracing in idle code work then in your opinion? As I suggested already. If we can get a light weight rcu_is_watching() then we can do the rcu_idle work when needed, and not when we don't need it. Sure this will add more bran

Re: [PATCH 0/9] Get rid of bitmap images

2016-11-21 Thread Mauro Carvalho Chehab
Em Mon, 21 Nov 2016 10:44:44 -0700 Jonathan Corbet escreveu: > On Sun, 20 Nov 2016 14:08:31 -0200 > Mauro Carvalho Chehab wrote: > > > The goal of this patch series is to get rid of PNG images, using either > > graphviz > > or SVG for images. > > > > For old images generated with xfig, stored

<    2   3   4   5   6   7   8   9   >