[PATCH 1/1] add-ZTE-pid

2015-08-02 Thread Liu.Zhao
Signed-off-by: Liu.Zhao lzsos...@163.com --- drivers/usb/serial/option.c | 29 + 1 file changed, 29 insertions(+) diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c index f0c0c53..6996308 100644 --- a/drivers/usb/serial/option.c +++

[PATCH] igb: Fix a deadlock in igb_sriov_reinit

2015-08-02 Thread Jia-Ju Bai
When igb_init_interrupt_scheme in igb_sriov_reinit is failed, the lock acquired by rtnl_lock() is not released, which causes a deadlock. This patch adds rtnl_unlock() in error handling to fix it. Signed-off-by: Jia-Ju Bai baijiaju1...@163.com --- drivers/net/ethernet/intel/igb/igb_main.c |1

[PATCH 1/1] add-ZTE-pid

2015-08-02 Thread Liu.Zhao
Signed-off-by: Liu.Zhao lzsos...@163.com --- drivers/usb/serial/option.c | 29 + 1 file changed, 29 insertions(+) diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c index f0c0c53..6996308 100644 --- a/drivers/usb/serial/option.c +++

Re: [PATCH v4 2/2] dt: power: st: Provide bindings for ST's OPPs

2015-08-02 Thread Viresh Kumar
On 31-07-15, 09:37, Stephen Boyd wrote: For qcom platforms, the frequency is almost always constant. There may be some tables where we have a couple higher frequencies than others because the speed bin is different. Otherwise the voltage/current is changing based on the silicon

Re: [PATCH 2/2] media: atmel-isi: move configure_geometry() to start_streaming()

2015-08-02 Thread Josh Wu
HI, Laurent On 7/31/2015 10:37 PM, Laurent Pinchart wrote: Hi Josh, Thank you for the patch. On Wednesday 17 June 2015 18:39:39 Josh Wu wrote: As in set_fmt() function we only need to know which format is been set, we don't need to access the ISI hardware in this moment. So move the

[PATCH v1 1/2] clk: rockchip: add pclk_pd_pmu to the list of rk3288 critical clocks

2015-08-02 Thread huang lin
pclk_pd_pmu needs to keep running and with the upcoming gpio clock handling this is not always the case anymore. So add it to the list of critical clocks for now. Signed-off-by: Heiko Stuebner he...@sntech.de Signed-off-by: huang lin h...@rock-chips.com --- drivers/clk/rockchip/clk-rk3288.c | 1

[PATCH v1 2/2] pinctrl: rockchip: only enable gpio clock when it setting

2015-08-02 Thread huang lin
gpio can keep state even the clock disable, for save power consumption, only enable gpio clock when it setting Signed-off-by: Heiko Stuebner he...@sntech.de Signed-off-by: huang lin h...@rock-chips.com Signed-off-by: huang lin h...@rock-chips.com --- drivers/pinctrl/pinctrl-rockchip.c | 60

Re: [PATCH] mm: add the block to the tail of the list in expand()

2015-08-02 Thread Dave Hansen
On 08/02/2015 07:05 PM, Xishi Qiu wrote: Also, this might not do very much good in practice. If you are splitting a high-order page, you are doing the split because the lower-order lists are empty. So won't that list_add() be to an empty I made a mistake, you are right, all the

Re: [PATCH] pci/pciehp: bail on bogus pcie reads from removed devices

2015-08-02 Thread Bjorn Helgaas
On Tue, Jul 21, 2015 at 12:25:30PM -0400, Jarod Wilson wrote: https://bugzilla.kernel.org/show_bug.cgi?id=99841 Seems like a read of all 1's from a register of a device that has gone away should be taken as a sign that the device has gone away. Section 6.2.10 of the PCIE spec (v4.0, rev 0.3,

Re: [linux-sunxi] [PATCH] ARM: dts: sunxi: Raise minimum CPU voltage for sun7i-a20 to a level all boards can supply

2015-08-02 Thread Julian Calaby
Hi Chen-Yu, On Mon, Aug 3, 2015 at 12:37 PM, Chen-Yu Tsai w...@csie.org wrote: Hi, On Mon, Aug 3, 2015 at 7:35 AM, Julian Calaby julian.cal...@gmail.com wrote: Hi Timo, On Mon, Aug 3, 2015 at 5:23 AM, Timo Sigurdsson public_tim...@silentcreek.de wrote: sun7i-a20.dtsi contains an cpufreq

Re: [PATCH 3.2 000/164] 3.2.70-rc1 review

2015-08-02 Thread Guenter Roeck
On 08/02/2015 02:49 PM, Ben Hutchings wrote: On Sat, 2015-08-01 at 19:23 -0700, Guenter Roeck wrote: On 08/01/2015 05:02 PM, Ben Hutchings wrote: This is the start of the stable review cycle for the 3.2.70 release. There are 164 patches in this series, which will be posted as responses to this

Re: [PATCH v2 0/4] mtd: pxa3xx_nand: rework the timing setup

2015-08-02 Thread Robert Jarzmik
Ezequiel Garcia ezequ...@vanguardiasur.com.ar writes: Just tested linux-next (hence *without* the patchset) and I see the same Wait time out. In other words, pxa3xx-nand is broken on PXA :/ Interestingly, the culprit doesn't seem to be in pxa3xx-nand itself. Reverting the recent commits on

Re: [PATCH v2] HID: hid-input: Fix accessing freed memory during device disconnect

2015-08-02 Thread Krzysztof Kozlowski
On 03.08.2015 09:01, Krzysztof Kozlowski wrote: During unbinding the driver was dereferencing a pointer to memory already freed by power_supply_unregister(). Driver was freeing its internal description of battery through pointers stored in power_supply structure. However, because the core

Re: [RFC PATCH 4/5] ARM: dts: DRA7: Add memory map region entries for qspi

2015-08-02 Thread Vignesh R
On 07/31/2015 11:49 PM, Mark Brown wrote: On Tue, Jul 28, 2015 at 02:11:15PM +0530, Vignesh R wrote: Add qspi memory mapped region entries for DRA7xx based SoCs. Signed-off-by: Vignesh R vigne...@ti.com qspi: qspi@4790 { compatible = ti,am4372-qspi;

[PATCH] 3c59x: Fix resource leaks in vortex_open

2015-08-02 Thread Jia-Ju Bai
When vortex_up is failed, the skb buffers allocated by __netdev_alloc_skb in vortex_open are not released, which may cause resource leaks. This bug has been submitted before. This patch modifies the error handling code to fix it. Signed-off-by: Jia-Ju Bai baijiaju1...@163.com ---

[PATCH v2] mmc: dw_mmc: Add external dma interface support

2015-08-02 Thread Shawn Lin
DesignWare MMC Controller can support two types of DMA mode: external dma and internal dma. We get a RK312x platform integrated dw_mmc and ARM pl330 dma controller. This patch add edmac ops to suuport these platforms. I've tested it on RK312x platform with edmac mode and RK3288 platform with idmac

[PATCH] usb_gadget: fix spin_lock in pch_udc

2015-08-02 Thread Pengyu Ma
When remove module g_serial on quark platform, the following Warning on: Modules linked in: usb_f_acm u_serial g_serial(-) pch_udc libcomposite configfs udc_core ad7298 industrialio_triggered_buffer kfifo_buf tpm_i2c_infineon indus CPU: 0 PID: 369 Comm: modprobe Not tainted

Re: [PATCH] mmc: dw_mmc: Fix coding style issues

2015-08-02 Thread Jaehoon Chung
Hi, Shawn. On 07/28/2015 12:06 PM, Shawn Lin wrote: This patch fixes the following issues reported by checkpatch.pl: - use -EINVAL instead of -ENOSYS, to fix warning message: ENOSYS means 'invalid syscall nr' and nothing else - split lines whose length is greater than 80 characters - avoid

[PATCH] fs: orangefs: remove execute priviliges from module params

2015-08-02 Thread Sasha Levin
This makes no sense and causes warnings on boot. Signed-off-by: Sasha Levin sasha.le...@oracle.com --- fs/orangefs/pvfs2-mod.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/orangefs/pvfs2-mod.c b/fs/orangefs/pvfs2-mod.c index 69289c5..d80537d 100644 ---

Re: [linux-sunxi] [PATCH] ARM: dts: sunxi: Raise minimum CPU voltage for sun7i-a20 to a level all boards can supply

2015-08-02 Thread Chen-Yu Tsai
On Mon, Aug 3, 2015 at 12:22 PM, Julian Calaby julian.cal...@gmail.com wrote: Hi Chen-Yu, On Mon, Aug 3, 2015 at 12:37 PM, Chen-Yu Tsai w...@csie.org wrote: Hi, On Mon, Aug 3, 2015 at 7:35 AM, Julian Calaby julian.cal...@gmail.com wrote: Hi Timo, On Mon, Aug 3, 2015 at 5:23 AM, Timo

Re: [PATCH 9/15 V2] drivers: rtc: Drop (un)likely before IS_ERR(_OR_NULL)

2015-08-02 Thread Hans Ulli Kroll
Hi On Fri, 31 Jul 2015, Viresh Kumar wrote: IS_ERR(_OR_NULL) already contain an 'unlikely' compiler flag and there is no need to do that again from its callers. Drop it. gemini driver was using likely() for a failure case while the rtc driver is getting registered. That looks wrong and it

Re: Re: [PATCH] PM / devfreq: event: Remove incorrect property in exynos-ppmu DT binding

2015-08-02 Thread MyungJoo Ham
Hello Myungjoo, On 07/23/2015 10:30 AM, Javier Martinez Canillas wrote: Hello Chanwoo, On 07/23/2015 10:19 AM, Chanwoo Choi wrote: Hi Javier, On 07/13/2015 03:58 PM, Javier Martinez Canillas wrote: The exynos-ppmu driver is only a clock consumer and not a clock provider but its

Re: samples/kdbus/kdbus-workers.c and cross compiling MIPS

2015-08-02 Thread Michael Ellerman
On Fri, 2015-07-31 at 09:29 -0700, Greg Kroah-Hartman wrote: On Fri, Jul 31, 2015 at 05:01:01PM +1000, Michael Ellerman wrote: It's obviously possible that some samples build with that configuration, but building against another arch'es kernel headers just seems like it's asking for

Re: [RFC PATCH 1/5] spi: introduce flag for memory mapped read

2015-08-02 Thread Vignesh R
Hi, On 7/31/2015 11:47 PM, Mark Brown wrote: On Tue, Jul 28, 2015 at 02:11:12PM +0530, Vignesh R wrote: Introduce use_mmap_read field in spi_message struct. This can be set by mtd devices (m25p80) to indicate to spi-master (ti-qspi) to perform memory mapped read. This helps to distinguish

Re: [PATCH v2] acpi: Use kstrtoul() instead of strtoul()/simple_strtoul()

2015-08-02 Thread Pavel Machek
On Mon 2015-07-27 17:32:22, Guenter Roeck wrote: simple_strtoul() is deprecated; replace with kstrtoul() and kstrtouint(). Return an error if the value passed to the sysfs attribute is not a number. Drop the definition of strtoul() since it is no longer needed. Signed-off-by: Guenter

Re: [PATCH] rx51-battery: Set name to rx51-battery

2015-08-02 Thread Pavel Machek
On Tue 2015-07-28 20:44:41, Pali Rohár wrote: For compatibility between board code and DT, set battery name to same value. Signed-off-by: Pali Rohár pali.ro...@gmail.com Acked-by: Pavel Machek pa...@ucw.cz --- drivers/power/rx51_battery.c |2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH v2 6/6] power: bq27xxx_battery: Cleanup health checking

2015-08-02 Thread Pavel Machek
On Tue 2015-07-28 14:39:13, Andrew F. Davis wrote: Reorganize the logic checking battery health and add under temperature condition checking. Signed-off-by: Andrew F. Davis a...@ti.com Acked-by: Pavel Machek pa...@ucw.cz -- (english) http://www.livejournal.com/~pavelmachek (cesky,

Re: [PATCH v6 5/5] nvme: LightNVM support

2015-08-02 Thread Pavel Machek
Hi! +struct nvme_nvm_hb_rw { + __u8opcode; + __u8flags; + __u16 command_id; + __le32 nsid; + __u64 rsvd2; + __le64 metadata; + __le64

Re: [PATCH] Adding YAMA hooks also when YAMA is not stacked.

2015-08-02 Thread Salvatore Mesoraca
Ah! Good catch, thank you! CONFIG based stacking for Yama will be removed 4.3, but this is appropriate for fixing in 4.2. Thank you very much for your time! Salvatore Mesoraca -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [PATCH v2] acpi: Use kstrtoul() instead of strtoul()/simple_strtoul()

2015-08-02 Thread Guenter Roeck
On 08/02/2015 12:16 AM, Pavel Machek wrote: On Mon 2015-07-27 17:32:22, Guenter Roeck wrote: simple_strtoul() is deprecated; replace with kstrtoul() and kstrtouint(). Return an error if the value passed to the sysfs attribute is not a number. Drop the definition of strtoul() since it is no

Re: [PATCH 03/10] nbd: Remove 'harderror' and propagate error properly

2015-08-02 Thread Pavel Machek
On Mon 2015-07-27 09:12:49, Markus Pargmann wrote: Instead of a variable 'harderror' we can simply try to correctly propagate errors to the userspace. This patch removes the harderror variable and passes errors through error pointers and nbd_do_it back to the userspace. Signed-off-by:

Re: [PATCH] HID: use generic driver for ThingM blink(1) if !CONFIG_HID_THINGM

2015-08-02 Thread Pavel Machek
On Mon 2015-07-27 12:17:06, Reilly Grant wrote: In commit 30ba2fbde1840db44 an LED class driver for this device was added and at the same time it was blacklisted by the hid-generic driver. This patch removes the device from the hid-generic driver's blacklist if the LED class driver is not

Re: May close() return any error code?

2015-08-02 Thread Pavel Machek
On Wed 2015-07-29 12:46:59, Takashi Iwai wrote: Hi, while debugging a problem of X and gdm with the old systemd-210, we encountered a sudden death of systemd-logind, and this turned out to be an unexpected errno from close(). The close() call for input devices returns ENODEV error. The

Re: 4.0.8-4.1.3 : after resume from s2ram both internal and external display of a docked ThinkPad ate black

2015-08-02 Thread Pavel Machek
On Wed 2015-07-29 15:54:00, Toralf Förster wrote: Undocking helps, and then I can dock again. This happens at a hardened 64 bit Gentoo with i915, but I think, it is not hardened related, or ? Any chance to bisect it? -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures)

Re: [RFC][PATCH] ecryptfs: Allow only one instance per lower path

2015-08-02 Thread Richard Weinberger
Am 02.08.2015 um 03:03 schrieb Tyler Hicks: Thanks for the report and for the patch, Richard! On 2015-07-31 12:23:10, Richard Weinberger wrote: Mounting the same lower path multiple times should not result into multiple ecryptfs instances, otherwise ecryptfs gets confused. A command

Re: [PATCH 4/4] drivers:pci:hv: New paravirtual PCI front-end for Hyper-V VMs

2015-08-02 Thread Thomas Gleixner
On Sun, 2 Aug 2015, ja...@microsoft.com wrote: +#include linux/kernel.h +#include linux/jiffies.h +#include linux/mman.h +#include linux/delay.h +#include linux/init.h +#include linux/module.h +#include linux/slab.h +#include linux/kthread.h +#include linux/completion.h +#include

Re: [PATCH] power_supply: Adjust devm usage

2015-08-02 Thread Pavel Machek
On Fri 2015-07-24 17:28:13, Vaishali Thakkar wrote: Use devm_kasprintf instead of kasprintf. Also, remove various gotos by direct returns and drop unneeded label err_free_name. What happens if some /sys file is still open when the device is removed? Signed-off-by: Vaishali Thakkar

Re: [PATCH v6 0/4] Add Mediatek MT8173 cpufreq driver

2015-08-02 Thread Viresh Kumar
On 9 July 2015 at 15:57, Pi-Cheng Chen pi-cheng.c...@linaro.org wrote: MT8173 is a ARMv8 based SoC with 2 clusters. All CPUs in a single cluster share the same power and clock domain. This series tries to add cpufreq support for MT8173 SoC. changes in v6: - Move clock and regulator

Re: [PATCH v2 5/6] power: bq27xxx_battery: Add support for additional bq27xxx family devices

2015-08-02 Thread Pavel Machek
+#define INVALID_REG_ADDR 0xff +static u8 bq27000_regs[] = { + 0x00, /* CONTROL */ + 0x06, /* TEMP */ + 0xff, /* INT TEMP - NA*/ I'd use the INVALID_REG_ADDR macro here.. Makes it more clear what is going on (and you may change it to -1 in future, for

Re: [PATCH v2 4/6] power: bq27xxx_battery: Fix typos and change naming for state of charge functions

2015-08-02 Thread Pavel Machek
On Tue 2015-07-28 14:39:11, Andrew F. Davis wrote: Fix typos and change relative state of charge to state of charge as not all supported devices use relative state of charge. Signed-off-by: Andrew F. Davis a...@ti.com Acked-by: Pavel Machek pa...@ucw.cz ---

[PATCH v2 00/11] gpio: Allow compile test of GPIO consumers if !GPIOLIB

2015-08-02 Thread Geert Uytterhoeven
Hi all, The GPIO subsystem provides dummy GPIO consumer functions if GPIOLIB is not enabled. Hence drivers that depend on GPIOLIB, but use GPIO consumer functionality only, can still be compiled if GPIOLIB is not enabled. This patch series relaxes the dependencies on GPIOLIB if

Re: linux: sata_nv: adma support

2015-08-02 Thread Pali Rohár
On Sunday 02 August 2015 03:45:32 Robert Hancock wrote: On Sat, Aug 1, 2015 at 2:09 PM, Pali Rohár pali.ro...@gmail.com wrote: On Thursday 25 December 2014 07:22:13 Robert Hancock wrote: On Tue, Dec 23, 2014 at 1:51 PM, Pali Rohár pali.ro...@gmail.com wrote: Hello, I have

Re: [PATCH 01/10] nbd: sock_shutdown, remove conditional lock

2015-08-02 Thread Pavel Machek
On Mon 2015-07-27 09:12:47, Markus Pargmann wrote: Move the conditional lock from sock_shutdown into the surrounding code. Signed-off-by: Markus Pargmann m...@pengutronix.de Acked-by: Pavel Machek pa...@ucw.cz -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures)

Re: [PATCH 02/10] nbd: restructure sock_shutdown

2015-08-02 Thread Pavel Machek
On Mon 2015-07-27 09:12:48, Markus Pargmann wrote: This patch restructures sock_shutdown to avoid having the main code path in an if block. Signed-off-by: Markus Pargmann m...@pengutronix.de Acked-by: Pavel Machek pa...@ucw.cz --- drivers/block/nbd.c | 13 +++-- 1 file changed,

Re: [PATCH] Fix logical-not-parentheses warning

2015-08-02 Thread Takashi Iwai
On Sun, 02 Aug 2015 07:43:22 +0200, Tomer Barletz wrote: Signed-off-by: Tomer Barletz barl...@gmail.com This is rather a gcc5 problem that doesn't recognize the expression correctly. Though, this change make things worse, so I'm willing to take it. But please give the warning message itself

Re: [PATCH v2 2/6] power: bq27x00_battery: Renaming for consistency

2015-08-02 Thread Pavel Machek
On Tue 2015-07-28 14:39:09, Andrew F. Davis wrote: Rename functions that are used by multiple devices. New devices have been added and the function names and driver name are no longer general enough for the functionality they provide. Signed-off-by: Andrew F. Davis a...@ti.com ---

[GIT PULL] ARM: SoC fixes

2015-08-02 Thread Olof Johansson
Hi Linus, Here's this week's pull request for arm-soc. Please merge. Thanks! -Olof The following changes since commit 3eae03daa538303e0d28f56412c6c51f6452e493: Merge tag 'pxa-fixes-v4.2-rc2' of https://github.com/rjarzmik/linux into fixesD (2015-07-18 21:06:10 -0700) are available in

Re: [PATCH 2/2] drm/panel: Add Sharp LS043T1LE01 MIPI DSI panel

2015-08-02 Thread Pavel Machek
On Tue 2015-07-28 18:37:41, Bjorn Andersson wrote: From: Werner Johansson werner.johans...@sonymobile.com This adds support for the Sharp panel found on the Qualcomm Snapdragon 800 Dragonboard (APQ8074) Signed-off-by: Werner Johansson werner.johans...@sonymobile.com Signed-off-by: Bjorn

[PATCH] Fix logical-not-parentheses warning

2015-08-02 Thread Tomer Barletz
This fixes the following warning, that is seen with gcc 5.1: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]. Signed-off-by: Tomer Barletz barl...@gmail.com --- sound/pci/oxygen/oxygen_mixer.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH 1/3] clocksource: atmel-st: Remove irq handler when clock event is unused

2015-08-02 Thread Thomas Gleixner
On Mon, 20 Jul 2015, Thomas Gleixner wrote: On Mon, 20 Jul 2015, Alexandre Belloni wrote: On 20/07/2015 at 11:04:30 +0200, Thomas Gleixner wrote : That's why I used setup/remove_irq() in the stale RT changes. Hum, actually, you hit the same thing with setup/remove_irq (and I'm

Re: [PATCH 0/4] watchdog: avoid races in watchdog_nmi_{en|disable} functions

2015-08-02 Thread Aaron Tomlin
On Sat 2015-08-01 14:49 +0200, Ulrich Obergfell wrote: Originally watchdog_nmi_enable(cpu) and watchdog_nmi_disable(cpu) were only called in watchdog thread context. However, the following commits utilize these functions outside of watchdog thread context too. commit

[PATCH] Staging: wilc1000 :Insert blank line after declaration

2015-08-02 Thread Shraddha Barke
This patch fixes the following checkpatch.pl warning: WARNING: Missing a blank line after declarations Signed-off-by: Shraddha Barke shraddha.6...@gmail.com --- drivers/staging/wilc1000/coreconfigurator.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH] Staging : lustre: Struct file_operations should be const

2015-08-02 Thread Shraddha Barke
This patch fixes the following checkpatch.pl warning: WARNING: struct file_operations should normally be const Signed-off-by: Shraddha Barke shraddha.6...@gmail.com --- drivers/staging/lustre/lustre/llite/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] Staging : lustre : Insert blank line after declaration

2015-08-02 Thread Shraddha Barke
This patch fixes the following checkpatch.pl warning: WARNING: Missing a blank line after declarations Signed-off-by: Shraddha Barke shraddha.6...@gmail.com --- drivers/staging/lustre/lustre/llite/llite_internal.h | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [PATCH] Fix logical-not-parentheses warning

2015-08-02 Thread Joe Perches
On Sun, 2015-08-02 at 02:08 -0700, Tomer Barletz wrote: This fixes the following warning, that is seen with gcc 5.1: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]. [] diff --git a/sound/pci/oxygen/oxygen_mixer.c [] @@ -88,7 +88,7 @@

Re: [PATCH 05/15] drivers: input: Drop unlikely before IS_ERR(_OR_NULL)

2015-08-02 Thread Pavel Machek
On Sat 2015-08-01 13:44:59, Pali Rohár wrote: On Saturday 01 August 2015 13:22:51 Viresh Kumar wrote: On 31-07-15, 09:58, Dmitry Torokhov wrote: On Fri, Jul 31, 2015 at 02:08:25PM +0530, Viresh Kumar wrote: IS_ERR(_OR_NULL) already contain an 'unlikely' compiler flag and there is no

Re: [PATCH] x86, acpi: Handle lapic/x2apic entries in MADT

2015-08-02 Thread Thomas Gleixner
On Thu, 30 Jul 2015, Lukasz Anaczkowski wrote: Also, order in which MADT LAPIC/X2APIC handlers are passed is reversed to achieve correct CPU enumeration. Ok. That is an understandable changelog and the patch itself looks about correct, but I leave that judgement to the ACPI experts. This also

[PATCH] Adding return value to securityfs_remove.

2015-08-02 Thread Salvatore Mesoraca
securityfs_remove can fail if it tries to remove a non-empty directory. This can happen, for example, if it tries to remove a file and its parent directory while the file is busy: the file removal will be delayed and the directory removal will fail. This patch adds a return value to

RE: [PATCH v4] powerpc/mpc85xx: Add MDIO bus muxing support to the board device tree(s)

2015-08-02 Thread Liberman Igal
Regards, Igal Liberman. -Original Message- From: Wood Scott-B07421 Sent: Thursday, July 30, 2015 11:16 PM To: Liberman Igal-B31950 Cc: devicet...@vger.kernel.org; linuxppc-...@lists.ozlabs.org; Bucur Madalin-Cristian-B32716; Xie Shaohui-B21989; linux-kernel@vger.kernel.org

Re: May close() return any error code?

2015-08-02 Thread Al Viro
On Wed, Jul 29, 2015 at 07:45:11AM -0700, Dmitry Torokhov wrote: This seems coming from evdev_flush(). As there is no fd leak, it's no big problem per se. But, now the question is whether returning such an error code is correct behavior at all. At least, it doesn't seem defined in

[PATCH v2] Staging: fbtft: Use a struct to describe each LCD controller

2015-08-02 Thread Fabio Falzoi
Use a struct flexfb_lcd_controller to holds chip properties, instead of relying on a long 'if - else if' chain. This allows to: - use a simple linear search to verify if a certain LCD controller model is supported or not. - add support for a new LCD chip controller simply defining a new

Re: [PATCH RESEND] nohz: Affining unpinned timers

2015-08-02 Thread Vatika Harlalka
If everyone thinks its alright, could this patch be applied please? :) Thanks Vatika -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the

Re: [RFC PATCH v2 0/3] genirq, serial: 8250: Workaround to avoid irq=0 for console

2015-08-02 Thread Thomas Gleixner
On Fri, 31 Jul 2015, Taichi Kageyama wrote: On 2015/07/31 8:43, Peter Hurley wrote: If you look at the actual autoprobe users aside of 8250. That's really all ancient ISA hardware and hardly interesting. So all we really care about are the 8250 serial ports. In this case, I think [patch

[PATCH] Avoid compiler warning by storing the result of rq_data_dir() in an int variable

2015-08-02 Thread Tomer Barletz
With gcc 5.1 I get: warning: switch condition has boolean value [-Wswitch-bool] Signed-off-by: Tomer Barletz barl...@gmail.com --- drivers/mtd/mtd_blkdevs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/mtd_blkdevs.c b/drivers/mtd/mtd_blkdevs.c index

Re: [PATCH] x86, acpi: Handle lapic/x2apic entries in MADT

2015-08-02 Thread Marc Zyngier
On Thu, 30 Jul 2015 19:43:39 +0200 Lukasz Anaczkowski lukasz.anaczkow...@intel.com wrote: From the ACPI spec: Logical processors with APIC ID values less than 0xFF (whether in XAPIC or X2APIC mode) must use the Processor LAPIC structure [...]. Logical processors with APIC ID values 0xFF and

Re: cgroup/loop Bad page state oops in Linux v4.2-rc3-136-g45b4b782e848

2015-08-02 Thread Josh Boyer
On Fri, Jul 31, 2015 at 2:58 PM, Josh Boyer jwbo...@fedoraproject.org wrote: On Thu, Jul 30, 2015 at 8:19 PM, Mike Snitzer snit...@redhat.com wrote: On Thu, Jul 30 2015 at 7:14pm -0400, Josh Boyer jwbo...@fedoraproject.org wrote: On Thu, Jul 30, 2015 at 7:27 AM, Josh Boyer

Re: [PATCH 1/3] clocksource: atmel-st: Remove irq handler when clock event is unused

2015-08-02 Thread Peter Zijlstra
On Sun, Aug 02, 2015 at 11:10:21AM +0200, Thomas Gleixner wrote: I think Boris Brezillon had implemented it at some point, but it was shot down for reasons I can't remember. You weren't around at the time.. DT people didn't like it, said they didn't like having to make up fake hardware in

Re: [PATCH] x86, pat: Add comments to cachemode translation tables

2015-08-02 Thread Thomas Gleixner
On Thu, 23 Jul 2015, Toshi Kani wrote: On Thu, 2015-07-23 at 09:36 -0600, Jan Beulich wrote: On 23.07.15 at 17:25, toshi.k...@hp.com wrote: Yes, I agree with you. But such risk is very low -- 1) the regular case (no fallback) is used most of the cases, 2) the code using WT

Re: [PATCH] watchdog: Fix race on single cpu boot

2015-08-02 Thread Ulrich Obergfell
Peter, I posted the patch set here: https://lkml.org/lkml/2015/8/1/64 https://lkml.org/lkml/2015/8/1/65 https://lkml.org/lkml/2015/8/1/66 https://lkml.org/lkml/2015/8/1/67 https://lkml.org/lkml/2015/8/1/68 If only you didn't use lkml.org links, that site is flaky beyond

Re: [PATCH] Check return code from pdc20621_i2c_read()

2015-08-02 Thread Sergei Shtylyov
Hello. On 8/2/2015 1:12 PM, Tomer Barletz wrote: The variable spd0 might be used uninitialized when pdc20621_i2c_read() fails. This also generates a compilation warning with gcc 5.1. Signed-off-by: Tomer Barletz barl...@gmail.com --- drivers/ata/sata_sx4.c | 14 ++ 1 file

Re: May close() return any error code?

2015-08-02 Thread Al Viro
On Sun, Aug 02, 2015 at 09:42:20AM +0200, Pavel Machek wrote: This seems coming from evdev_flush(). As there is no fd leak, it's no big problem per se. But, now the question is whether returning such an error code is correct behavior at all. At least, it doesn't seem defined in POSIX:

Re: [PATCH 03/27] iio: Export I2C module alias information in missing drivers

2015-08-02 Thread Jonathan Cameron
On 30/07/15 17:18, Javier Martinez Canillas wrote: The I2C core always reports the MODALIAS uevent as i2c:client name regardless if the driver was matched using the I2C id_table or the of_match_table. So the driver needs to export the I2C table and this be built into the module or udev won't

Re: [PATCH 3/9] x86/intel_rdt: Cache Allocation documentation and cgroup usage guide

2015-08-02 Thread Martin Kletzander
On Thu, Jul 30, 2015 at 05:08:13PM -0300, Marcelo Tosatti wrote: On Thu, Jul 30, 2015 at 10:47:23AM -0700, Vikas Shivappa wrote: Marcello, On Wed, 29 Jul 2015, Marcelo Tosatti wrote: How about this: desiredclos (closid p1 p2 p3 p4) 1 1 0 0 0 2 0 0

Re: [PATCH 05/15] drivers: input: Drop unlikely before IS_ERR(_OR_NULL)

2015-08-02 Thread Pali Rohár
On Sunday 02 August 2015 17:43:52 Pavel Machek wrote: On Sat 2015-08-01 13:44:59, Pali Rohár wrote: On Saturday 01 August 2015 13:22:51 Viresh Kumar wrote: On 31-07-15, 09:58, Dmitry Torokhov wrote: On Fri, Jul 31, 2015 at 02:08:25PM +0530, Viresh Kumar wrote: IS_ERR(_OR_NULL)

[GIT PULL] SCSI fixes for 4.2-rc4

2015-08-02 Thread James Bottomley
A set of three fixes for the ipr driver and one fairly major one for memory leaks in the mq path of SCSI. The patch is available here: git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git scsi-fixes The short changelog is: Brian King (3): ipr: Fix invalid array indexing for HRRQ

Re: [PATCH 5/9] x86/intel_rdt: Add new cgroup and Class of service management

2015-08-02 Thread Tejun Heo
Hello, On Fri, Jul 31, 2015 at 12:12:18PM -0300, Marcelo Tosatti wrote: I don't really think it makes sense to implement a fully hierarchical cgroup solution when there isn't the basic affinity-adjusting interface What is an affinity adjusting interface ? Can you give an example

[PATCH v2] MIPS: Remove all the uses of custom gpio.h

2015-08-02 Thread Alban Bedel
Currently CONFIG_ARCH_HAVE_CUSTOM_GPIO_H is defined for all MIPS machines, and each machine type provides its own gpio.h. However only a handful really implement the GPIO API, most just forward everythings to gpiolib. The Alchemy machine is notable as it provides a system to allow implementing

Re: [PATCH 1/2] iio: magn: bmc150_magn: add locking comment for runtime resume

2015-08-02 Thread Jonathan Cameron
On 23/07/15 18:15, Irina Tirdea wrote: Runtime resume function is called with the data-mutex lock held. Add a comment to indicate this. Signed-off-by: Irina Tirdea irina.tir...@intel.com Sensible follow up to Hartmut's patches. Applied to the togreg branch of iio.git Thanks, Jonathan

Re: [PATCH 2/2] iio: magn: bmc150_magn: do not set power state twice when setting trigger state

2015-08-02 Thread Jonathan Cameron
On 23/07/15 18:15, Irina Tirdea wrote: When setting the trigger state, the device power state is set through buffer preenable and postdisable hooks. There is no need to also set it in the trigger set state call. Remove duplicate set power state from the trigger set state call.

Re: [PATCH] iio: inv-mpu: make inv_write_raw_get_fmt() more readable

2015-08-02 Thread Jonathan Cameron
On 27/07/15 22:48, Luis Henriques wrote: Replace nested switch statements by a single if statement, making function inv_write_raw_get_fmt() more readable. Signed-off-by: Luis Henriques luis.henriq...@canonical.com Hi Luis, I can see where you are coming from here, but my gut feeling is to

[PATCH 4/4] Plain integer instead of NULL pointer fix

2015-08-02 Thread Adrian Remonda
This patch fix the using plain integer as NULL pointer Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/most/hdm-dim2/dim2_hal.c

[PATCH 2/4] Plain integer instead of NULL pointer fix

2015-08-02 Thread Adrian Remonda
This patch fix the using plain integer as NULL pointer Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/most/mostcore/core.c b/drivers/staging/most/mostcore/core.c

[PATCH 0/4] Staging: most: several warnings fix reported by sparse

2015-08-02 Thread Adrian Remonda
This patch series fix several warnings reported by the Sparse tool Adrian Remonda (4): This is a patch to the mostcore/core.c file. It makes several local functions static to prevent global visibility. This patch fix the using plain integer as NULL pointer This is a patch to the

[PATCH 1/4] Staging: most: static keyword missing

2015-08-02 Thread Adrian Remonda
This is a patch to the mostcore/core.c file. It makes several local functions static to prevent global visibility. Signed-off-by: Adrian Remonda adrianremo...@gmail.com --- drivers/staging/most/mostcore/core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

Re: [PATCH v2 2/2] iio: magnetometer: add mmc34160 magnetometer driver

2015-08-02 Thread Jonathan Cameron
On 02/08/15 17:37, Dan Carpenter wrote: On Sun, Aug 02, 2015 at 05:32:30PM +0100, Jonathan Cameron wrote: On 20/07/15 15:14, Dan Carpenter wrote: On Mon, Jul 20, 2015 at 04:52:30PM +0300, Teodora Baluta wrote: +static int mmc34160_raw_to_mgauss(int raw[3], int sens[3], int nfo, +

Re: [PATCH 02/10] nbd: restructure sock_shutdown

2015-08-02 Thread Markus Pargmann
Hi, On Sun, Aug 02, 2015 at 09:08:44AM +0200, Pavel Machek wrote: On Mon 2015-07-27 09:12:48, Markus Pargmann wrote: This patch restructures sock_shutdown to avoid having the main code path in an if block. Signed-off-by: Markus Pargmann m...@pengutronix.de Acked-by: Pavel Machek

Re: [PATCH] x86: serialize LVTT and TSC_DEADLINE write

2015-08-02 Thread Shaohua Li
On Sat, Aug 01, 2015 at 12:10:41PM +0200, Thomas Gleixner wrote: On Fri, 31 Jul 2015, Shaohua Li wrote: @@ -336,6 +336,22 @@ static void __setup_APIC_LVTT(unsigned int clocks, int oneshot, int irqen) apic_write(APIC_LVTT, lvtt_value); if (lvtt_value

Re: unpinning an unpinned lock. (pidns/scheduler)

2015-08-02 Thread Peter Zijlstra
On Sun, Aug 02, 2015 at 11:38:07AM -0400, Dave Jones wrote: On Fri, Jul 31, 2015 at 01:43:53PM -0400, Dave Jones wrote: Just found a machine with this on 4.2-rc4 WARNING: CPU: 0 PID: 11787 at kernel/locking/lockdep.c:3497 lock_unpin_lock+0x109/0x110() unpinning an unpinned lock

Re: [PATCH 1/1] x86: replace RDRAND forced-reseed with simple sanity check

2015-08-02 Thread Pavel Machek
On Fri 2015-07-31 11:27:39, Len Brown wrote: From: Len Brown len.br...@intel.com x86_init_rdrand() was added with 2 goals: 1. Sanity check that the built-in-self-test circuit on the Digital Random Number Generator (DRNG) is not complaining. As RDRAND HW self-checks on every

Re: [PATCH v3 1/2] iio: mmc35240: minor change to improve code readibility

2015-08-02 Thread Jonathan Cameron
On 31/07/15 15:27, Teodora Baluta wrote: This patch changes two variables to arrays to improve code readibility. Signed-off-by: Teodora Baluta teodora.bal...@intel.com After a brief argument with myself (see inline). Applied to the togreg branch of iio.git Thanks, Jonathan ---

[PATCH] staging: lustre: echo_client: fix sparse declaration warnings

2015-08-02 Thread Zoltán Lajos Kis
Fixes the following sparse warnings: drivers/staging/lustre/lustre/obdecho/echo_client.c:2142:5: warning: symbol 'echo_client_init' was not declared. Should it be static? drivers/staging/lustre/lustre/obdecho/echo_client.c:2157:6: warning: symbol 'echo_client_exit' was not declared. Should it be

Re: [PATCH 2/7] iio: bmg160: Use i2c regmap instead of direct i2c access

2015-08-02 Thread Markus Pargmann
Hi, On Fri, Jul 31, 2015 at 12:29:58PM -0700, Srinivas Pandruvada wrote: On Wed, 2015-07-29 at 15:46 +0200, Markus Pargmann wrote: This patch introduces regmap usage into the driver. This is done to later easily support the SPI interface of this chip. Signed-off-by: Markus Pargmann

Re: [PATCH 3/6] iio: trigger: Add missing fields in kernel docs

2015-08-02 Thread Jonathan Cameron
On 24/07/15 14:20, Cristina Opriceana wrote: Fix kernel docs warnings by adding the missing description for each of the existing function parameters. Signed-off-by: Cristina Opriceana cristina.oprice...@gmail.com A few comments inline. --- drivers/iio/industrialio-trigger.c | 27

Re: [PATCH 5/6] include: linux: iio: Fix function parameter name in kernel doc

2015-08-02 Thread Jonathan Cameron
On 24/07/15 14:23, Cristina Opriceana wrote: Fix buffer name from kernel doc according to the function parameter. Signed-off-by: Cristina Opriceana cristina.oprice...@gmail.com Applied Jonathan --- include/linux/iio/consumer.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v2] Revert Staging : lustre: Struct file_operations should be const

2015-08-02 Thread Shraddha Barke
Declare the file_operations structure ll_file_operations as const, as done elsewhere in the kernel, as there are no modifications to its fields. Problem found using checkpatch. Signed-off-by: Shraddha Barke shraddha.6...@gmail.com --- Changes in v2: - Make the commit message more clearer.

Re: [PATCH 4/6] iio: event: Add missing fields in kernel docs

2015-08-02 Thread Jonathan Cameron
On 24/07/15 14:21, Cristina Opriceana wrote: Fix kernel docs warnings by adding the missing fields, each with its associated description. Signed-off-by: Cristina Opriceana cristina.oprice...@gmail.com Applied. Thanks Jonathan --- drivers/iio/industrialio-event.c | 6 ++ 1 file

Re: [PATCH 6/6] include: linux: iio: Add missing kernel doc field

2015-08-02 Thread Jonathan Cameron
On 24/07/15 14:26, Cristina Opriceana wrote: Fix kernel doc for the iio_dev_attr structure by adding its missing field. Signed-off-by: Cristina Opriceana cristina.oprice...@gmail.com Applied to the togreg branch fo iio.git with one small change. --- include/linux/iio/sysfs.h | 3 ++- 1

Re: [PATCH 5/6] ASoC: fsl_ssi: instantiate AC'97 CODEC

2015-08-02 Thread Markus Pargmann
On Fri, Jul 31, 2015 at 04:39:20PM +0200, Maciej S. Szmigiero wrote: On 31.07.2015 07:46, Markus Pargmann wrote: On Thu, Jul 30, 2015 at 04:35:23PM +0200, Maciej S. Szmigiero wrote: Instantiate AC'97 CODEC in fsl_ssi driver AC'97 mode. Signed-off-by: Maciej Szmigiero

Re: [PATCH v2] Staging: fbtft: Use a struct to describe each LCD controller

2015-08-02 Thread Noralf Trønnes
Den 02.08.2015 16:57, skrev Fabio Falzoi: Use a struct flexfb_lcd_controller to holds chip properties, instead of relying on a long 'if - else if' chain. This allows to: - use a simple linear search to verify if a certain LCD controller model is supported or not. - add support for a new LCD

Re: [PATCH] Check return code from pdc20621_i2c_read()

2015-08-02 Thread Tomer Barletz
Re-sending in plain-text. On Sun, Aug 2, 2015 at 4:09 AM, Sergei Shtylyov sergei.shtyl...@cogentembedded.com wrote: Please use pr_err() instead. And 0x%d makes no sense at all, please use %#x instead. Yeah, not sure what I was drinking before writing this 0x%d thing... Regarding the

  1   2   3   4   5   6   >