Re: add Packet hub driver for Topcliff Platform controller hub

2013-01-08 Thread Arnd Bergmann
On Monday 07 January 2013, Dan Carpenter wrote: Sorry my question is about an old patch. Smatch complains because we only pass a PAGE_SIZE buffer to sysfs files so the test for if (count PCH_PHUB_OROM_SIZE) { makes it think we are overflowing. In fact, count is never more than 4096 so there

[PATCH] cpufreq: exynos: Show list of available frequencies

2013-01-08 Thread Inderpal Singh
Add freq_attr attribute to show list of available frequencies. Signed-off-by: Donggeun Kim dg77@samsung.com Signed-off-by: MyungJoo Ham myungjoo@samsung.com Signed-off-by: KyungMin Park kyungmin.p...@samsung.com Signed-off-by: Inderpal Singh inderpal.si...@linaro.org ---

Re: [PATCH 1/1]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-08 Thread Sebastian Andrzej Siewior
On Sat, Jan 05, 2013 at 10:57:42AM +0800, fangxiaozhi 00110321 wrote: From: fangxiaozhi huana...@huawei.com 1. Optimize the match rules with new macro for Huawei USB storage devices, to avoid to load USB storage driver for the modem interface with Huawei devices. 2. Add to support

[PATCH V4 RESEND] serial: tegra: add serial driver

2013-01-08 Thread Laxman Dewangan
NVIDIA's Tegra has multiple UART controller which supports: - APB DMA based controller fifo read/write. - End Of Data interrupt in incoming data to know whether end of frame achieve or not. - HW controlled RTS and CTS flow control to reduce SW overhead. Add serial driver to use all above

Re: [PATCH 03/11] spi/pxa2xx-pci: switch to use pcim_* interfaces

2013-01-08 Thread Mark Brown
On Mon, Jan 07, 2013 at 12:44:32PM +0200, Mika Westerberg wrote: Instead of open-coding all the error management in the driver we can take advantage of the pcim_* interfaces that release the resources automatically. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] Added NIST CMAC to the CryptoAPI library

2013-01-08 Thread Tom St Denis
Hi all, I agree with the ^ comment ... I'll look into the styles issue, though ironically I would consider that a maintainers job... I'll resend a bit later today. Tom - Original Message - From: Steffen Klassert steffen.klass...@secunet.com To: Tom St Denis tstde...@elliptictech.com

Re: [RFC PATCH 1/2] efi: Make 'efi_enabled' a function to

2013-01-08 Thread Matt Fleming
On Sat, 2013-01-05 at 11:28 +0800, Guo Chao wrote: query EFI facilities Reply-To: 1357219085-4312-2-git-send-email-m...@console-pimps.org In-Reply-To: 1357219085-4312-2-git-send-email-m...@console-pimps.org diff --git a/init/main.c b/init/main.c index e33e09d..e71d924 100644 ---

Re: [PATCH] pinctrl/nomadik: Add ste,config property

2013-01-08 Thread Linus Walleij
On Fri, Jan 4, 2013 at 9:02 PM, Stephen Warren swar...@wwwdotorg.org wrote: -- ste,input : 0/1/2/3 +- ste,config: Handle of pin configuration node (ste,config = in_pu) + +- ste,input : 0/1/2 The changes to ste,input and ste,sleep-output look like some unrelated change. I guess the idea

Re: [PATCH 05/11] spi/pxa2xx: make clock rate configurable from platform data

2013-01-08 Thread Mark Brown
On Mon, Jan 07, 2013 at 12:44:34PM +0200, Mika Westerberg wrote: If the architecture doesn't support clk framework (like x86) we need a way to pass the SSP clock rate to the driver. This patch adds a field in the platform data 'fixed_clk_rate' that allows passing the rate. No, the way to do

[PATCH v2] pinctrl: nomadik: Add ste,config property etc

2013-01-08 Thread Linus Walleij
From: Gabriel Fernandez gabriel.fernan...@stericsson.com The ste,config property will contain the pin config node. It will be easier to define a pin configuration and use it by reference without duplicating lines tedious. Other cleanups to the initial DT patch. This will all be squashed into the

Re: [PATCH] cpufreq: exynos: Show list of available frequencies

2013-01-08 Thread Inderpal Singh
+CC: dg77@samsung.com, myungjoo@samsung.com, kyungmin.p...@samsung.com On 8 January 2013 16:20, Inderpal Singh inderpal.si...@linaro.org wrote: Add freq_attr attribute to show list of available frequencies. Signed-off-by: Donggeun Kim dg77@samsung.com Signed-off-by: MyungJoo Ham

Re: [PATCH RFT] regulator: s5m8767: Convert to regulator_[get|set]_voltage_sel_regmap

2013-01-08 Thread Mark Brown
On Sat, Dec 29, 2012 at 12:58:21AM +0800, Axel Lin wrote: Signed-off-by: Axel Lin axel@ingics.com This doesn't apply against -rc2, can you please regenerate. signature.asc Description: Digital signature

Re: [PATCH] regulator: s5m8767: Remove max_vol parameter from s5m8767_convert_voltage_to_sel

2013-01-08 Thread Mark Brown
On Sat, Dec 29, 2012 at 09:38:50AM +0800, Axel Lin wrote: It looks pointless to pass max_vol to s5m8767_convert_voltage_to_sel(). Compare selected voltage to desc-max is enough to ensure selected voltage is in supported range. This doesn't apply against -rc2, can you please regenerate?

Re: [PATCH 1/3] regulator: max8997: Use uV in voltage_map_desc

2013-01-08 Thread Mark Brown
On Fri, Dec 28, 2012 at 05:09:03PM +0800, Axel Lin wrote: Current code does integer division (min_vol = min_uV / 1000) before pass min_vol to max8997_get_voltage_proper_val(). So it is possible min_vol is truncated to a smaller value. Applied, thanks. signature.asc Description: Digital

Re: [PATCH 1/5] capemgr: Beaglebone DT overlay based cape manager

2013-01-08 Thread Sascha Hauer
On Mon, Jan 07, 2013 at 09:35:04PM +, Arnd Bergmann wrote: (Adding Sascha Hauer, Linus Walleij, Lee Jones to Cc) On Monday 07 January 2013, Tony Lindgren wrote: At the end of the line, some kind of hardware glue is going to be needed. I just feel that drawing from a sample

Re: [PATCH 2/3] regulator: max8998: Use uV in voltage_map_desc

2013-01-08 Thread Mark Brown
On Fri, Dec 28, 2012 at 05:10:20PM +0800, Axel Lin wrote: Integer division may truncate. This happens when pdata-buckx_voltagex setting is not align with 1000 uV. Thus use uV in voltage_map_desc, this ensures the selected voltage won't less than pdata buckx_voltagex settings. Applied, thanks.

Re: [PATCH 3/3] regulator: max8998: Return enough delay time for max8998_set_voltage_buck_time_sel

2013-01-08 Thread Mark Brown
On Fri, Dec 28, 2012 at 05:14:58PM +0800, Axel Lin wrote: Use DIV_ROUND_UP to prevent truncation by integer division issue. This ensures we return enough delay time. This bit is OK. Since the delay is required only if the voltage is increasing, and we know both old_selector and new_selector.

Re: [PATCH 1/1] uprobes: Teach handler_chain() to filter out the probed task

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-29 18:36:14]: Currrently the are 2 problems with pre-filtering: 1. It is not possible to add/remove a task (mm) after uprobe_register() 2. A forked child inherits all breakpoints and uprobe_consumer can not control this. This patch does the

Re: [PATCH 0/2] dma: tegra: add channel wise pause control support and Tegra114 SoCs.

2013-01-08 Thread Vinod Koul
On Sun, Jan 06, 2013 at 09:52:01PM +0530, Laxman Dewangan wrote: This patch series add the channel wise pause control on dma driver and then add support for Tegra114 SoCs. The orginal change was in single patch and based on review comment, trying to split the change to have more meaningful

Re: [PATCH 1/2] uprobes/x86: Change __skip_sstep() to actually skip the whole insn

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-30 16:47:19]: __skip_sstep() doesn't update regs-ip. Currently this is correct but only by accident and it doesn't skip the whole insn. Change it to advance -ip by the length of the detected 0x66*0x90 sequence. Signed-off-by: Oleg Nesterov

[PATCH 1/2] tools: hv: Fix how ifcfg-* file is created

2013-01-08 Thread Tomas Hozza
Fix for the daemon code and for hv_set_ifconfig.sh script, so that the created ifcfg-* file is consistent with initscripts documentation. Signed-off-by: Tomas Hozza tho...@redhat.com --- tools/hv/hv_kvp_daemon.c| 73 ++--- tools/hv/hv_set_ifconfig.sh |

[PATCH 2/2] tools: hv: Use CLOEXEC when opening kvp_pool files

2013-01-08 Thread Tomas Hozza
Use CLOEXEC flag when opening kvp_pool_x files to prevent file descriptor leakage. Not using it was causing a problem when SELinux was enabled. Signed-off-by: Tomas Hozza tho...@redhat.com --- tools/hv/hv_kvp_daemon.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

Re: linux-next: comment in commit in the slave-dma tree (dmaengine: dw_dmac: Enhance device tree support)

2013-01-08 Thread Stephen Rothwell
Hi Vinod, On Tue, 8 Jan 2013 00:17:35 -0800 Vinod Koul vinod.k...@intel.com wrote: I think I was trying to remove it. Anyway I have changed the commit to remove this and also another instance. So you should be happy about it :) Thanks. I would have been even happier if you had just added

Re: [PATCH 0/2] dma: tegra: add channel wise pause control support and Tegra114 SoCs.

2013-01-08 Thread Laxman Dewangan
On Tuesday 08 January 2013 04:24 PM, Vinod Koul wrote: On Sun, Jan 06, 2013 at 09:52:01PM +0530, Laxman Dewangan wrote: This patch series add the channel wise pause control on dma driver and then add support for Tegra114 SoCs. The orginal change was in single patch and based on review comment,

Re: add Packet hub driver for Topcliff Platform controller hub

2013-01-08 Thread Dan Carpenter
On Tue, Jan 08, 2013 at 07:30:34PM +0900, Tomoya MORINAGA wrote: Hi Dan, On Mon, Jan 7, 2013 at 6:02 PM, Dan Carpenter dan.carpen...@oracle.com wrote: The patch cf4ece53460c: add Packet hub driver for Topcliff Platform controller hub from Sep 1, 2010, leads to the following warning:

Re: [PATCH 2/2] uprobes: Change handle_swbp() to expose bp_vaddr to handler_chain()

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-30 16:47:22]: Change handle_swbp() to set regs-ip = bp_vaddr in advance, this is what consumer-handler() needs but uprobe_get_swbp_addr() is not exported. This also simplifies the code and makes it more consistent across the supported architectures.

Re: ata1: COMRESET failed

2013-01-08 Thread Jiri Slaby
On 01/08/2013 02:31 AM, Aaron Lu wrote: On 01/08/2013 01:24 AM, Borislav Petkov wrote: Adding some more people. Aaron, Shane, do you guys have any ideas? I've been seeing something similar on an AMD desktop box recently and Jiri sees it on a laptop with an SSD. I've seen a similar problem

Re: [PATCH 1/7] uprobes: Move alloc_page() from xol_add_vma() to xol_alloc_area()

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-31 18:52:12]: Move alloc_page() from xol_add_vma() to xol_alloc_area() to cleanup the code. This separates the memory allocations and consolidates the -EALREADY cleanups and the error handling. Signed-off-by: Oleg Nesterov o...@redhat.com (One

Re: add Packet hub driver for Topcliff Platform controller hub

2013-01-08 Thread Dan Carpenter
On Tue, Jan 08, 2013 at 10:49:14AM +, Arnd Bergmann wrote: On Monday 07 January 2013, Dan Carpenter wrote: Sorry my question is about an old patch. Smatch complains because we only pass a PAGE_SIZE buffer to sysfs files so the test for if (count PCH_PHUB_OROM_SIZE) { makes it think we

Re: [PATCH 1/7] KVM: Write protect the updated slot only when we start dirty logging

2013-01-08 Thread Gleb Natapov
On Mon, Jan 07, 2013 at 06:11:46PM -0200, Marcelo Tosatti wrote: On Tue, Dec 18, 2012 at 04:26:47PM +0900, Takuya Yoshikawa wrote: This is needed to make kvm_mmu_slot_remove_write_access() rmap based: otherwise we may end up using invalid rmap's. Signed-off-by: Takuya Yoshikawa

Re: [PATCH 2/7] uprobes: Fold xol_alloc_area() into get_xol_area()

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-31 18:52:16]: Currently only xol_get_insn_slot() does get_xol_area() + xol_alloc_area(), but this will have more users and we do not want to copy-and-paste this code. This patch simply moves xol_alloc_area() into get_xol_area() to simplify the current

Re: add Packet hub driver for Topcliff Platform controller hub

2013-01-08 Thread Arnd Bergmann
On Tuesday 08 January 2013, Dan Carpenter wrote: On Tue, Jan 08, 2013 at 10:49:14AM +, Arnd Bergmann wrote: On Monday 07 January 2013, Dan Carpenter wrote: Sorry my question is about an old patch. Smatch complains because we only pass a PAGE_SIZE buffer to sysfs files so the test for

Re: [PATCH 3/7] uprobes: Turn add_utask() into get_utask()

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-31 18:52:20]: Rename add_utask() into get_utask() and change it to allocate on demand to simplify the caller. Like get_xol_area() it will have more users. Signed-off-by: Oleg Nesterov o...@redhat.com Acked-by: Srikar Dronamraju

Re: [PATCH 1/1]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-08 Thread Oliver Neukum
On Tuesday 08 January 2013 11:57:30 Sebastian Andrzej Siewior wrote: +static int usb_stor_huawei_scsi_init(struct us_data *us) +{ + int result = 0; + int act_len = 0; + struct bulk_cb_wrap *bcbw = (struct bulk_cb_wrap *) us-iobuf; + char rewind_cmd[] = {0x11, 0x06,

Re: [PATCH 4/7] uprobes: Do not play with utask in xol_get_insn_slot()

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-31 18:52:23]: pre_ssout()-xol_get_insn_slot() path is confusing and buggy. This patch cleanups the code, the next one fixes the bug. Change xol_get_insn_slot() to only allocate the slot and do nothing more, move the initialization of

Re: [PATCH 1/5] capemgr: Beaglebone DT overlay based cape manager

2013-01-08 Thread Arnd Bergmann
On Tuesday 08 January 2013, Lee Jones wrote: If there is not, there is no way to automatically load the overlays; you can always use the kernel command line, or have the a user space application to request the loading of a specific board's overlay. Unfortunately, there is no way to

Commit 7032a3dd9 (svcrpc: demote some printks to a dprintk) adds warnings

2013-01-08 Thread Russell King
Bruce, Your commit below causes a new build warning in the kernel: net/sunrpc/svc.c: In function 'svc_printk': net/sunrpc/svc.c:1050:7: warning: unused variable 'buf' which affects all ARM builds; it is the only warning in a lot of cases. Please can you resolve this new warning using whatever

Re: [PATCH 5/7] uprobes: Fix utask-xol_vaddr leak in pre_ssout()

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-31 18:52:26]: pre_ssout() should do xol_free_insn_slot() if arch_uprobe_pre_xol() fails, otherwise nobody will free the allocated slot. Signed-off-by: Oleg Nesterov o...@redhat.com Acked-by: Srikar Dronamraju sri...@linux.vnet.ibm.com (one nit

Re: [PATCH 6/7] uprobes: Do not allocate current-utask unnecessary

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-31 18:52:29]: handle_swbp() does get_utask() before can_skip_sstep() for no reason, we do not need -utask if can_skip_sstep() succeeds. Move get_utask() to pre_ssout() who actually starts to use it. Move the initialization of

Re: [PATCH 7/7] uprobes: Kill the bogus IS_ERR_VALUE(xol_vaddr) check

2013-01-08 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-12-31 18:52:32]: utask-xol_vaddr is either zero or valid, remove the bogus IS_ERR_VALUE() check in xol_free_insn_slot(). Signed-off-by: Oleg Nesterov o...@redhat.com Acked-by: Srikar Dronamraju sri...@linux.vnet.ibm.com --- kernel/events/uprobes.c |

Re: [PATCH 0/4] mfd/regulator: tps65090: add DT support and suspend/resume cleanups

2013-01-08 Thread Laxman Dewangan
Hi Mark, On Friday 28 December 2012 02:59 PM, Laxman Dewangan wrote: The patch series add DT support on TPS65090 device. Also remove the suspend/resume implementation as it duplicates with irq_suspend/irq_resume(). Laxman Dewangan (4): mfd: tps65090: add DT support for tps65090

Re: [3.6.9 - 3.7.1 regression] sound: snd_hda_intel codec probing issue?

2013-01-08 Thread Vincent Blut
Le lundi 07 janvier 2013 à 09:32 +0100, Takashi Iwai a écrit : At Sat, 05 Jan 2013 22:24:03 +0100, Vincent Blut wrote: Le jeudi 03 janvier 2013 à 10:19 +0100, Takashi Iwai a écrit : At Fri, 28 Dec 2012 15:25:40 +0100, Vincent Blut wrote: Hi, Since I updated to Linux

Re: [PATCH] drivers: xhci: fix incorrect bit test

2013-01-08 Thread Sergei Shtylyov
Hello. On 08-01-2013 7:39, Nickolai Zeldovich wrote: Fix incorrect bit test that originally showed up in 4ee823b83bc9851743fab756c76b27d6a1e2472b: use '' instead of ''. Please also specify the summary of that commit in parens (or however you like). Signed-off-by: Nickolai Zeldovich

Re: [PATCH 05/11] spi/pxa2xx: make clock rate configurable from platform data

2013-01-08 Thread Mika Westerberg
On Tue, Jan 08, 2013 at 11:02:28AM +, Mark Brown wrote: On Mon, Jan 07, 2013 at 12:44:34PM +0200, Mika Westerberg wrote: If the architecture doesn't support clk framework (like x86) we need a way to pass the SSP clock rate to the driver. This patch adds a field in the platform

Re: [PATCH signal#execve2] syscalls,x86: Add execveat() system call (v3)

2013-01-08 Thread Meredydd Luff
On Sun, Jan 6, 2013 at 4:31 PM, Al Viro v...@zeniv.linux.org.uk wrote: OK, now that sys_execve() unification has settled down, let's get back to this one. The real problem is what you are doing with bprm-filename and bprm-interp; blind use of -d_name is completely wrong. ACK. I've blocked out

Re: [3.6.9 - 3.7.1 regression] sound: snd_hda_intel codec probing issue?

2013-01-08 Thread Takashi Iwai
At Tue, 08 Jan 2013 13:28:55 +0100, Vincent Blut wrote: Le lundi 07 janvier 2013 à 09:32 +0100, Takashi Iwai a écrit : At Sat, 05 Jan 2013 22:24:03 +0100, Vincent Blut wrote: Le jeudi 03 janvier 2013 à 10:19 +0100, Takashi Iwai a écrit : At Fri, 28 Dec 2012 15:25:40 +0100,

Re: [PATCH 1/4] gpiolib: introduce descriptor-based GPIO interface

2013-01-08 Thread Arnd Bergmann
On Tuesday 08 January 2013, Alexandre Courbot wrote: +struct gpio_desc *__must_check devm_gpiod_get(struct device *dev, +const char *con_id) +{ + struct gpio_desc **dr; + struct gpio_desc *desc; + + dr =

[PATCH] pps: hide more configuration symbols behind CONFIG_PPS

2013-01-08 Thread Florian Fainelli
This patch makes CONFIG_PPS_DEBUG and CONFIG_NTP_PPS be hidden if CONFIG_PPS is not selected, such that we are not prompted for these configuration options if CONFIG_PPS is not set. Signed-off-by: Florian Fainelli flor...@openwrt.org --- drivers/pps/Kconfig |4 1 file changed, 4

[PATCH] HID: i2c-hid: add ACPI support

2013-01-08 Thread Mika Westerberg
The HID over I2C protocol specification states that when the device is enumerated from ACPI the HID descriptor address can be obtained by executing _DSM for the device with function 1. Enable this. Signed-off-by: Mika Westerberg mika.westerb...@linux.intel.com --- drivers/hid/i2c-hid/i2c-hid.c |

Re: oops in copy_page_rep()

2013-01-08 Thread Hillf Danton
On Tue, Jan 8, 2013 at 1:34 AM, Linus Torvalds torva...@linux-foundation.org wrote: On Mon, Jan 7, 2013 at 4:24 AM, Hillf Danton dhi...@gmail.com wrote: I take another try with waiting added, take a look please. Hmm. Is there some reason we never need to worry about it for the pmd_numa()

Re: [PATCH 0/4] gpio: introduce descriptor-based interface

2013-01-08 Thread Arnd Bergmann
On Tuesday 08 January 2013, Alexandre Courbot wrote: This series introduce a first take at implementing the RFC for the new GPIO API that I submitted last month. It proposes a new, opaque descriptor-based GPIO API that becomes available when GPIOlib is compiled, and provides a safer, more

Re: [PATCH 2/4] gpiolib: add gpiod_get and gpiod_put functions

2013-01-08 Thread Arnd Bergmann
On Tuesday 08 January 2013, Alexandre Courbot wrote: Adds new GPIO allocation functions that work with the opaque descriptor interface. Signed-off-by: Alexandre Courbot acour...@nvidia.com I think you need to reorder the patches slightly, since the gpiod_get function introduced here is

[PATCH] clk: remove unreachable code

2013-01-08 Thread Rajagopal Venkat
while reparenting a clock, NULL check is done for clock in consideration and its new parent. So re-check is not required. If done, else part becomes unreachable. Signed-off-by: Rajagopal Venkat rajagopal.ven...@linaro.org --- drivers/clk/clk.c |5 + 1 file changed, 1 insertion(+), 4

[PATCH] lockdep: Rename print_unlock_inbalance_bug() to print_unlock_imbalance_bug()

2013-01-08 Thread Srivatsa S. Bhat
Fix the typo in the function name (s/inbalance/imbalance) Signed-off-by: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com --- kernel/lockdep.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/kernel/lockdep.c b/kernel/lockdep.c index 7981e5b..5cf12e7 100644 ---

Re: [PATCH 05/11] spi/pxa2xx: make clock rate configurable from platform data

2013-01-08 Thread Mark Brown
On Tue, Jan 08, 2013 at 02:41:53PM +0200, Mika Westerberg wrote: On Tue, Jan 08, 2013 at 11:02:28AM +, Mark Brown wrote: No, the way to do this is to fix x86 to enable the clock API there. The x86 maintainers couldn't be bothered when I submitted a patch and getting anyone to take a

Re: [PATCH RFT] regulator: s5m8767: Convert to regulator_[get|set]_voltage_sel_regmap

2013-01-08 Thread Mark Brown
On Tue, Jan 08, 2013 at 08:38:52PM +0800, Axel Lin wrote: 2013/1/8 Mark Brown broo...@opensource.wolfsonmicro.com Or do you mean Linus' current tree? ( This looks not a 3.8 material. ) I meant Linus' tree since I was starting a new topic branch (not having seen the fix branch). No need to

Re: [PATCH v4 RESEND] pwm: atmel: add Timer Counter Block PWM driver

2013-01-08 Thread Thierry Reding
On Tue, Jan 08, 2013 at 01:43:56PM +0100, Boris BREZILLON wrote: On 08/01/2013 08:10, Thierry Reding wrote: On Thu, Dec 20, 2012 at 10:12:56AM +0100, Boris BREZILLON wrote: [...] +static void atmel_tcb_pwm_disable(struct pwm_chip *chip, struct pwm_device *pwm) +{ [...] + /* If duty

Re: [PATCH v4 RESEND] pwm: atmel: add Timer Counter Block PWM driver

2013-01-08 Thread Boris BREZILLON
On 08/01/2013 08:10, Thierry Reding wrote: On Thu, Dec 20, 2012 at 10:12:56AM +0100, Boris BREZILLON wrote: Hi, Sorry for resend. The previous version still has alignment issues on atmel_tcb_pwm_set_polarity, atmel_tcb_pwm_request and atmel_tcb_pwm_config function parameters. This patch

Re: [PATCH 1/5] capemgr: Beaglebone DT overlay based cape manager

2013-01-08 Thread Pantelis Antoniou
Hi Arnd, On Jan 8, 2013, at 2:12 PM, Arnd Bergmann wrote: On Tuesday 08 January 2013, Lee Jones wrote: If there is not, there is no way to automatically load the overlays; you can always use the kernel command line, or have the a user space application to request the loading of a

[PATCH] omap: DT node Timer iteration fix

2013-01-08 Thread Pantelis Antoniou
The iterator correctly handles of_node_put() calls. Remove it before continue'ing the loop. Without this patch you get: ERROR: Bad of_node_put() on /ocp/timer@44e31000! [c001329c] (unwind_backtrace+0x0/0xe0) from [c03dd8f0] (of_node_release+0x2c/0xa0)! [c03dd8f0] (of_node_release+0x2c/0xa0) from

[PATCH] omap: Avoid crashes in the case of hwmod misconfiguration

2013-01-08 Thread Pantelis Antoniou
omap hwmod is really sensitive to hwmod misconfiguration. Getting a minor clock wrong always ended up in a crash. Attempt to be more resilient by not assigning variables with error codes and then attempting to use them. Without this patch, missing a clock ends up with something like this:

[PATCH] fix multiple mark_page_accessed called when sequentially writing a file with blocksize less than PAGE_SIZE ,which might pollute the LRU.

2013-01-08 Thread Qiang Gao
sequential write to a file with blocksize less than PAGE_SIZE will call mark_page_accessed multiple times, if (!pagevec_space(pvec)) __pagevec_lru_add(pvec, lru); it seems this trick fix this problem,but not quite thoroughly. there's a chance that when another page was

[patch] bnx2x: NULL dereference on error in debug code

2013-01-08 Thread Dan Carpenter
vfop is NULL here. I've changed the debugging to not use it. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com --- Only needed in linux-next. diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c index 71fcef0..3eef972 100644 ---

Re: [PATCH] HID: i2c-hid: add ACPI support

2013-01-08 Thread Benjamin Tissoires
Hi Mika, On Tue, Jan 8, 2013 at 2:05 PM, Mika Westerberg mika.westerb...@linux.intel.com wrote: The HID over I2C protocol specification states that when the device is enumerated from ACPI the HID descriptor address can be obtained by executing _DSM for the device with function 1. Enable this.

Re: [PATCH] perf bench: Flush stdout before starting bench suite

2013-01-08 Thread Hitoshi Mitake
At Tue, 8 Jan 2013 18:39:26 +0900, Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com perf bench prints header message for bench suite before starting the benchmark. However if the stdout is redirected to a file and bench suite forks child processes this (and possibly other

Re: sched: Consequences of integrating the Per Entity Load Tracking Metric into the Load Balancer

2013-01-08 Thread Vincent Guittot
On 8 January 2013 07:06, Preeti U Murthy pre...@linux.vnet.ibm.com wrote: On 01/07/2013 09:18 PM, Vincent Guittot wrote: On 2 January 2013 05:22, Preeti U Murthy pre...@linux.vnet.ibm.com wrote: Hi everyone, I have been looking at how different workloads react when the per entity load

[PATCH] tools/perf: let it build from perf-*-src-pkg

2013-01-08 Thread Sebastian Andrzej Siewior
Thanks (mostly) to uapi the package created from perf-*-src-pkg FTBFS: |CC perf.o |In file included from util/../perf.h:8:0, | from util/cache.h:7, | from perf.c:12: |arch/x86/include/asm/unistd.h:4:29: fatal error: uapi/asm/unistd.h: No such file or directory

Re: [3.6.9 - 3.7.1 regression] sound: snd_hda_intel codec probing issue?

2013-01-08 Thread Vincent Blut
Le mardi 08 janvier 2013 à 13:56 +0100, Takashi Iwai a écrit : At Tue, 08 Jan 2013 13:28:55 +0100, Vincent Blut wrote: Le lundi 07 janvier 2013 à 09:32 +0100, Takashi Iwai a écrit : At Sat, 05 Jan 2013 22:24:03 +0100, One more thing to test is whether azx_runtime_resume() is

Re: [PATCH v3 09/22] sched: compute runnable load avg in cpu_load and cpu_avg_load_per_task

2013-01-08 Thread Alex Shi
On 01/07/2013 02:31 AM, Linus Torvalds wrote: On Sat, Jan 5, 2013 at 11:54 PM, Alex Shi alex@intel.com wrote: I just looked into the aim9 benchmark, in this case it forks 2000 tasks, after all tasks ready, aim9 give a signal than all tasks burst waking up and run until all finished.

Re: [RFC PATCH 5/6] uprobes: add bp_vaddr argument to consumer handler

2013-01-08 Thread Anton Arapov
On Sun, Dec 23, 2012 at 04:49:10PM +0100, Oleg Nesterov wrote: On 12/22, Oleg Nesterov wrote: Forgot to ask... On 12/22, Oleg Nesterov wrote: On 12/21, Anton Arapov wrote: struct uprobe_consumer { - int (*handler)(struct uprobe_consumer *self, struct pt_regs

Re: [PATCH 02/25] ab8500_charger: don't use [delayed_]work_pending()

2013-01-08 Thread Linus Walleij
On Sat, Dec 22, 2012 at 2:56 AM, Tejun Heo t...@kernel.org wrote: There's no need to test whether a (delayed) work item in pending before queueing, flushing or cancelling it. Most uses are unnecessary and quite a few of them are buggy. Remove unnecessary pending tests from ab8500_charger.

Re: [PATCH v3 2/4] videobuf2-dma-streaming: new videobuf2 memory allocator

2013-01-08 Thread Jonathan Corbet
On Tue, 08 Jan 2013 07:50:41 +0100 Marek Szyprowski m.szyprow...@samsung.com wrote: Couldn't this performance difference be due to the usage of GFP_DMA inside the VB2 code, like Federico's new patch series is proposing? If not, why are there a so large performance penalty? Nope, this

Re: [alsa-devel] [3.6.9 - 3.7.1 regression] sound: snd_hda_intel codec probing issue?

2013-01-08 Thread David Henningsson
On 01/08/2013 01:56 PM, Takashi Iwai wrote: I'll test with enable_msi=0 later. Thanks. FWIW, below is a patch I'm considering to merge (after testing, of course). Could you buys check it? Takashi --- From: Takashi Iwai ti...@suse.de Subject: [PATCH] ALSA: hda - Disable runtime D3 for

[PATCH] PCI: Make pci_find_upstream_pcie_bridge() handle non PCIE VFs well

2013-01-08 Thread tadeusz . struk
pci_find_upstream_pcie_bridge() doesn't handle well non PCIE VFs that are part of a PCIE PF device. Signed-off-by: Tadeusz Struk tadeusz.st...@intel.com --- drivers/pci/search.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/pci/search.c

Re: kernel BUG at kernel/sched_rt.c:493!

2013-01-08 Thread Steven Rostedt
I've also managed to reproduce this on 3.8.0-rc2 so it appears the bug is still present in the latest kernel. Shawn, Can you send me your .config file. Thanks, -- Steve -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: Hangcheck timer elapsed... GPU hung in 3.8.0-rc2

2013-01-08 Thread J. Bruce Fields
On Sun, Jan 06, 2013 at 07:06:52PM +0100, Daniel Vetter wrote: On Thu, Jan 03, 2013 at 06:11:23PM -0500, J. Bruce Fields wrote: On Thu, Jan 03, 2013 at 04:16:24PM -0500, Josh Boyer wrote: On Thu, Jan 3, 2013 at 3:46 PM, J. Bruce Fields bfie...@fieldses.org wrote: I got a crash after

[PATCH] drivers: i2c: muxes: fix sizeof(ptr)

2013-01-08 Thread Laurent Navet
sizeof when applied to a pointer typed expression gives the size of the pointer The semantic patch that makes this output is available in scripts/coccinelle/misc/noderef.cocci. More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Laurent Navet

Re: [alsa-devel] [3.6.9 - 3.7.1 regression] sound: snd_hda_intel codec probing issue?

2013-01-08 Thread Takashi Iwai
At Tue, 08 Jan 2013 15:32:13 +0100, David Henningsson wrote: On 01/08/2013 01:56 PM, Takashi Iwai wrote: I'll test with enable_msi=0 later. Thanks. FWIW, below is a patch I'm considering to merge (after testing, of course). Could you buys check it? Takashi --- From:

Re: [PATCH UPDATED 15/24] cfq-iosched: enable full blkcg hierarchy support

2013-01-08 Thread Vivek Goyal
On Mon, Jan 07, 2013 at 08:34:05AM -0800, Tejun Heo wrote: [..] +weight leaf_weight + root : 125125 + A: 500750 + B: 250500 + AA : 500500 + AB : 1000500 + +root never has a parent making its weight is meaningless. For backward +compatibility,

Re: [PATCH] drivers: i2c: muxes: fix sizeof(ptr)

2013-01-08 Thread Laurent Navet
This is not really a bug as mux-busses is a multilevel pointer, so the result of sizeof(ptr) is the same as sizeof(*ptr). But I think this is more logical and according to CodingStyle rules like this. This is also what is done for mux-states a few lines above. Regards, 2013/1/8, Laurent Navet

Re: [PATCH] sunrpc: verbs: Avoid 1kb stack

2013-01-08 Thread J. Bruce Fields
On Mon, Jan 07, 2013 at 11:41:36AM -0800, Joe Perches wrote: 16 * 64 is a bit much. Use kmalloc_array instead. I thought there was some reason we didn't do this. Grepping up through the callers It looks like the result is xprt_rdma_send_request returns -EIO, and as far as I can tell that

RE: [patch] bnx2x: NULL dereference on error in debug code

2013-01-08 Thread Ariel Elior
-Original Message- From: netdev-ow...@vger.kernel.org [mailto:netdev- ow...@vger.kernel.org] On Behalf Of Dan Carpenter Sent: Tuesday, January 08, 2013 3:42 PM To: Eilon Greenstein Cc: net...@vger.kernel.org; linux-kernel@vger.kernel.org; kernel- janit...@vger.kernel.org Subject:

Re: [Consult] our latest kernel and latest Android under arm Samsung S5PV210 with yaffs2 file system

2013-01-08 Thread Theodore Ts'o
On Tue, Jan 08, 2013 at 06:25:06PM +0800, Chen Gang F T wrote: use yaffs2 file system for android it seems yaffs2 is a standard file system for android. for my android HTC phone is also yaffs2 file system. yaffs2 is not a _standard_ file system for Android. There may be some

Re: [PATCH v4 1/3] videobuf2-dma-contig: user can specify GFP flags

2013-01-08 Thread Federico Vaga
Ok, then I would simply pass the flags from the driver without any alternation in the allocator itself, so drivers can pass 'GFP_KERNEL' or 'GFP_KERNEL | GFP_DMA' depending on their preference. Please also update all the existing clients of vb2_dma_dc allocator. I taked a look at drivers

RE: [patch] bnx2x: NULL dereference on error in debug code

2013-01-08 Thread Ariel Elior
-Original Message- From: netdev-ow...@vger.kernel.org [mailto:netdev- ow...@vger.kernel.org] On Behalf Of Dan Carpenter Sent: Tuesday, January 08, 2013 3:42 PM To: Eilon Greenstein Cc: net...@vger.kernel.org; linux-kernel@vger.kernel.org; kernel- janit...@vger.kernel.org Subject:

[PATCH 0/5] kfifo cleanup and log based kfifo API

2013-01-08 Thread Yuanhan Liu
The current kfifo API take the kfifo size as input, while it rounds _down_ the size to power of 2 at __kfifo_alloc/init. This may introduce several potential issues. First, the kfifo size is not what we want. Say, if we want to allocate a kfifo with size of 127 elements. Then in the end, we can

[PATCH 4/5] kfifo: remove kfifo_init

2013-01-08 Thread Yuanhan Liu
kfifo_init is buggy, we should never use that. Instead, we should use kfifo_alloc. Cc: Stefani Seibold stef...@seibold.net Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Yuanhan Liu yuanhan@linux.intel.com --- include/linux/kfifo.h | 27 ++-

[PATCH 5/5] kfifo: log based kfifo API

2013-01-08 Thread Yuanhan Liu
The current kfifo API take the kfifo size as input, while it rounds _down_ the size to power of 2 at __kfifo_alloc. This may introduce potential issue. Take the code at drivers/hid/hid-logitech-dj.c as example: if (kfifo_alloc(djrcv_dev-notif_fifo,

[PATCH 2/5] libsrp: replace kfifo_init with kfifo_alloc

2013-01-08 Thread Yuanhan Liu
kfifo_init will use a pre-allocated buffer as fifo buffer; the buffer size is determinted at caller side. While, kfifo will maintain a real kfifo buffer size(rounddown power of 2 aligned). So, the two size may not be equal. So, if max is not power of 2, this code will not work. As it assume the

[PATCH 3/5] libiscsi: replace kfifo_init with kfifo_alloc

2013-01-08 Thread Yuanhan Liu
kfifo_init will use a pre-allocated buffer as fifo buffer; the buffer size is determinted at caller side. While, kfifo will maintain a real kfifo buffer size(rounddown power of 2 aligned). So, the two size may not be equal. So, if max is not power of 2, this code will not work. As it assume the

[PATCH 1/5] kfifo: remove unnecessary type check

2013-01-08 Thread Yuanhan Liu
Firstly, this kind of type check doesn't work. It does something similar as following: void * __dummy = NULL; __buf = __dummy; __dummy is defined as void *. Thus it will not trigger warnings as expected. Second, we don't need that kind of check. Since the prototype of __kfifo_out

Re: [PATCH v2 0/5] staging/fwserial: Address reviewer comments

2013-01-08 Thread Peter Hurley
On Mon, 2013-01-07 at 15:22 -0800, Greg Kroah-Hartman wrote: On Sat, Dec 15, 2012 at 01:03:14AM -0500, Peter Hurley wrote: Overdue respin. v2 changes: Don't use 'card' when referring to firewire node Removed lower clamp in link_speed_to_max_payload() Ripped out the bandwidth

Re: kernel BUG at kernel/sched_rt.c:493!

2013-01-08 Thread Shawn Bohrer
On Tue, Jan 08, 2013 at 09:36:05AM -0500, Steven Rostedt wrote: I've also managed to reproduce this on 3.8.0-rc2 so it appears the bug is still present in the latest kernel. Shawn, Can you send me your .config file. I've attached the 3.8.0-rc2 config that I used to reproduce this in

[PATCH v2] efi, x86: Pass a proper identity mapping in efi_call_phys_prelog

2013-01-08 Thread Nathan Zimmer
Update efi_call_phys_prelog to install an identity mapping of all available memory. This corrects a bug on very large systems with more then 512 GB in which bios would not be able to access addresses above not in the mapping. The result is a crash that looks much like this. BUG: unable to

Antw: [PATCH 0/5] kfifo cleanup and log based kfifo API

2013-01-08 Thread Ulrich Windl
Yuanhan Liu yuanhan@linux.intel.com schrieb am 08.01.2013 um 15:57 in Nachricht 1357657073-27352-1-git-send-email-yuanhan@linux.intel.com: [...] My proposal is to replace kfifo_init with kfifo_alloc, where it allocate buffer and maintain fifo size inside kfifo. Then we can remove

Re: [PATCH] nfsd: Remove write permission from file content

2013-01-08 Thread J. Bruce Fields
On Sat, Jan 05, 2013 at 12:29:05PM +0800, Yanchuan Nian wrote: 2013/1/5 J. Bruce Fields bfie...@fieldses.org On Fri, Jan 04, 2013 at 07:45:35PM +0800, ycn...@gmail.com wrote: From: Yanchuan Nian ycn...@gmail.com The write function doesn't be implemented in file content, and it's

Re: [PATCH v6 3/4] block: implement runtime pm strategy

2013-01-08 Thread Alan Stern
On Tue, 8 Jan 2013, Aaron Lu wrote: --- a/include/linux/blkdev.h +++ b/include/linux/blkdev.h @@ -974,6 +974,40 @@ extern int blk_pre_runtime_suspend(struct request_queue *q); extern void blk_post_runtime_suspend(struct request_queue *q, int err); extern void

Re: [PATCH v4 RESEND] pwm: atmel: add Timer Counter Block PWM driver

2013-01-08 Thread Russell King - ARM Linux
On Tue, Jan 08, 2013 at 04:21:59PM +0100, Boris BREZILLON wrote: Do I have to break the error string so that the line does not exceed 80 characters ? No. Checkpath script does not complain about it, and the CodingStyle file specify that visible strings should not be broken... Correct.

Re: ALSA: Conexant CX20585 (thinkpad t510) speaker powersave regression

2013-01-08 Thread Takashi Iwai
At Sat, 22 Dec 2012 14:23:24 -0700, Kevin Fenzi wrote: Greetings. I've got an issue with sound on my t510. It started in the late 3.4.x kernels. Sound works on boot and for 5-10min after, then the speakers stop working at all. I posted a while back on the alsa-users list, and someone

Re: [PATCH v6 0/4] block layer runtime pm

2013-01-08 Thread Alan Stern
On Tue, 8 Jan 2013, Aaron Lu wrote: So this also reminds me that as long as CONFIG_PM_RUNTIME is selected, the blk_pm_add/put/peek_request functions will be in the block IO path. Shall we introduce a new config option to selectively build block runtime PM functionality? something like

<    3   4   5   6   7   8   9   10   11   12   >