Re: [PATCH] gpio: em: Fix checking return value of irq_alloc_descs

2012-08-30 Thread Linus Walleij
On Tue, Aug 28, 2012 at 4:30 AM, Axel Lin axel@gmail.com wrote: irq_alloc_descs() returns negative error code on failure. Signed-off-by: Axel Lin axel@gmail.com Magnuis can I have your ACK on this? Yours, Linus Walleij -- To unsubscribe from this list: send the line unsubscribe

[PATCH v3] linux/kernel.h: Fix DIV_ROUND_CLOSEST to support negative operands

2012-08-30 Thread Guenter Roeck
DIV_ROUND_CLOSEST returns a bad result for dividends with different sign: DIV_ROUND_CLOSEST(-2, 2) = 0 Most of the time this does not matter. However, in the hardware monitoring subsystem, DIV_ROUND_CLOSEST is sometimes used on integers which can be negative (such as temperatures).

Re: [PATCH] dma/ste_dma40: Fixup clock usage during probe

2012-08-30 Thread Vinod Koul
On Thu, 2012-08-23 at 13:41 +0200, Ulf Hansson wrote: From: Ulf Hansson ulf.hans...@linaro.org Fixup some errorhandling for clocks during probe and make sure to use clk_prepare as well as clk_enable. Signed-off-by: Ulf Hansson ulf.hans...@linaro.org Acked-by: Linus Walleij

Re: [PATCH] ioat: Adding Ivy Bridge IOATDMA PCI device IDs

2012-08-30 Thread Vinod Koul
On Fri, 2012-08-24 at 16:36 -0700, Dave Jiang wrote: Signed-off-by: Dave Jiang dave.ji...@intel.com Sounds okay to me, I can carry it once Dan acks it --- drivers/dma/ioat/pci.c | 22 ++ 1 files changed, 22 insertions(+), 0 deletions(-) diff --git

Re: [PATCH v3] linux/kernel.h: Fix DIV_ROUND_CLOSEST to support negative operands

2012-08-30 Thread Andrew Morton
On Thu, 30 Aug 2012 17:10:47 -0700 Guenter Roeck li...@roeck-us.net wrote: DIV_ROUND_CLOSEST returns a bad result for dividends with different sign: DIV_ROUND_CLOSEST(-2, 2) = 0 Most of the time this does not matter. However, in the hardware monitoring subsystem, DIV_ROUND_CLOSEST is

Re: [PATCH -v12 02/15] resources: Add probe_resource()

2012-08-30 Thread Bjorn Helgaas
On Wed, Aug 29, 2012 at 10:36 AM, Yinghai Lu ying...@kernel.org wrote: On Wed, Aug 29, 2012 at 8:57 AM, Yinghai Lu ying...@kernel.org wrote: also have another version for probe_resource, please check attached version -v8. sorry, v8 forget removing two lines. please -v9 instead. -v8:

Re: [RFC,PATCH] efi: Add support for a UEFI variable filesystem

2012-08-30 Thread Jeremy Kerr
Hi hpa, Thanks for the review! However, I have a question... rather than putting the attributes as the first data bytes, would it be better to make it either part of the filename (assuming there is at least one character other than / which can be reasonably relied upon to not be part of the

Re: [PATCH 0/2] ARM: EXYNOS: Set the capability of pdm0 and pdm1 as DMA_PRIVATE

2012-08-30 Thread Vinod Koul
On Wed, 2012-08-29 at 10:16 +0530, Tushar Behera wrote: DMA clients pdma0 and pdma1 are internal to the SoC and are used only by dedicated peripherals. Since they cannot be used for generic purpose, their capability should be set as DMA_PRIVATE. The patches are rebased on top of v3.6-rc3.

Re: [PATCH v3 3/3] spi: spi-davinci: convert to DMA engine API

2012-08-30 Thread Vinod Koul
On Thu, 2012-08-30 at 10:43 -0400, Matt Porter wrote: On Thu, Aug 30, 2012 at 07:46:32PM +0530, Sekhar Nori wrote: Hi Matt, On 8/23/2012 6:39 AM, Matt Porter wrote: Removes use of the DaVinci EDMA private DMA API and replaces it with use of the DMA engine API. Signed-off-by:

Re: [PATCH 4/9 V3] workqueue: add non_manager_role_manager_mutex_unlock()

2012-08-30 Thread Lai Jiangshan
On 08/30/2012 05:17 PM, Tejun Heo wrote: Hello, Lai. On Thu, Aug 30, 2012 at 05:16:01PM +0800, Lai Jiangshan wrote: gcwq_unbind_fn() is unsafe even it is called from a work item. so we need non_manager_role_manager_mutex_unlock(). If rebind_workers() is called from a work item, it is safe

Re: [PATCH v3] linux/kernel.h: Fix DIV_ROUND_CLOSEST to support negative operands

2012-08-30 Thread Guenter Roeck
On Thu, Aug 30, 2012 at 05:35:31PM -0700, Andrew Morton wrote: On Thu, 30 Aug 2012 17:10:47 -0700 Guenter Roeck li...@roeck-us.net wrote: DIV_ROUND_CLOSEST returns a bad result for dividends with different sign: DIV_ROUND_CLOSEST(-2, 2) = 0 Most of the time this does not matter.

Re: [PATCH] userns: Add basic quota support v4

2012-08-30 Thread Dave Chinner
On Wed, Aug 29, 2012 at 02:31:26AM -0700, Eric W. Biederman wrote: Dave thanks for taking the time to take a detailed look at this code. Dave Chinner da...@fromorbit.com writes: On Tue, Aug 28, 2012 at 12:09:56PM -0700, Eric W. Biederman wrote: Add the data type struct kqid which

Re: [PATCH] gpio: em: Fix checking return value of irq_alloc_descs

2012-08-30 Thread Magnus Damm
On Fri, Aug 31, 2012 at 9:05 AM, Linus Walleij linus.wall...@linaro.org wrote: On Tue, Aug 28, 2012 at 4:30 AM, Axel Lin axel@gmail.com wrote: irq_alloc_descs() returns negative error code on failure. Signed-off-by: Axel Lin axel@gmail.com Magnuis can I have your ACK on this? Yes,

[PATCH v3] hwmon: add Maxim MAX197 support

2012-08-30 Thread Vivien Didelot
The MAX197 is an A/D converter, made by Maxim. This driver currently supports the MAX197, and MAX199. They are both 8-Channel, Multi-Range, 5V, 12-Bit DAS with 8+4 Bus Interface and Fault Protection. The available ranges for the MAX197 are {0,-5V} to 5V, and {0,-10V} to 10V, while they are

Re: [PATCH v7 9/9] block: Avoid deadlocks with bio allocation by stacking drivers

2012-08-30 Thread Kent Overstreet
On Thu, Aug 30, 2012 at 06:07:45PM -0400, Vivek Goyal wrote: On Wed, Aug 29, 2012 at 10:13:45AM -0700, Kent Overstreet wrote: [..] Performance aside, punting submission to per device worker in case of deep stack usage sounds cleaner solution to me. Agreed, but performance tends to

[PATCH] hwmon: (sht15) remove multiple driver registration

2012-08-30 Thread Vivien Didelot
Declare an array of platform_device_id, instead of registering a driver for each supported chip. This makes the code cleaner. Also add a module description. Signed-off-by: Vivien Didelot vivien.dide...@savoirfairelinux.com --- drivers/hwmon/sht15.c | 94

Re: [PATCH v7 9/9] block: Avoid deadlocks with bio allocation by stacking drivers

2012-08-30 Thread Kent Overstreet
On Thu, Aug 30, 2012 at 6:43 PM, Kent Overstreet koverstr...@google.com wrote: On Thu, Aug 30, 2012 at 06:07:45PM -0400, Vivek Goyal wrote: On Wed, Aug 29, 2012 at 10:13:45AM -0700, Kent Overstreet wrote: [..] Performance aside, punting submission to per device worker in case of deep

Re: i2c-eg20t: regression since i2c_add_numbered_adapter change

2012-08-30 Thread Feng Tang
Hi Alexander, On Thu, 30 Aug 2012 13:08:09 +0200 Alexander Stein alexander.st...@systec-electronic.com wrote: On Thursday 30 August 2012 17:19:15, Feng Tang wrote: IMO the i2c_register_board_info only works in quite static setups. Especially with I2C-Busses attached to hotplugable PCI

Re: [RFC,PATCH] efi: Add support for a UEFI variable filesystem

2012-08-30 Thread H. Peter Anvin
Wouldn't that be better handled by O_APPEND? Jeremy Kerr jeremy.k...@canonical.com wrote: Hi hpa, Thanks for the review! However, I have a question... rather than putting the attributes as the first data bytes, would it be better to make it either part of the filename (assuming there is

Re: [PATCH] of: add devres version of of_iomap

2012-08-30 Thread Rob Herring
On 08/30/2012 05:09 PM, Karicheri, Muralidharan wrote: -Original Message- From: Rob Herring [mailto:robherri...@gmail.com] Sent: Thursday, August 30, 2012 2:27 PM To: Karicheri, Muralidharan Cc: grant.lik...@secretlab.ca; devicetree-disc...@lists.ozlabs.org; linux-

RE: [PATCH v7 1/8] Talitos: Support for async_tx XOR offload

2012-08-30 Thread Liu Qiang-B32616
-Original Message- From: Geanta Neag Horia Ioan-B05471 Sent: Thursday, August 30, 2012 10:23 PM To: Liu Qiang-B32616; linux-cry...@vger.kernel.org; dan.j.willi...@gmail.com; herb...@gondor.hengli.com.au; da...@davemloft.net; linux-kernel@vger.kernel.org; linuxppc-

[BUG REPORT] Wrong deaklock warning!

2012-08-30 Thread Stanley.Miao
Hi, All, I used two spinlocks in my code, and I enabled the following CONFIGs for debugging. CONFIG_DEBUG_SPINLOCK=y CONFIG_DEBUG_LOCK_ALLOC=y CONFIG_PROVE_LOCKING=y CONFIG_DEBUG_LOCKDEP=y void abc_init(struct abc_dev *dev) { spin_lock_init(dev-locka); spin_lock_init(dev-lockb); } void

[PATCH] ASoC: correct the check for NULL dma_buffer pointer

2012-08-30 Thread Prasad Joshi
The if condition if (!buf !buf-area) checks if the buf pointer is NULL and then dereferences it again to check if the buffer area is NULL, resulting in possible NULL dereference. Signed-off-by: Prasad Joshi prasadjoshi.li...@gmail.com --- sound/soc/spear/spear_pcm.c |2 +- 1 files

Re: [PATCH 0/5] dev_level and dynamic_debug cleanups

2012-08-30 Thread Jim Cromie
On Thu, Aug 30, 2012 at 11:43 AM, Jim Cromie jim.cro...@gmail.com wrote: On Sun, Aug 26, 2012 at 5:25 AM, Joe Perches j...@perches.com wrote: The recent commit to fix dynamic_debug was a bit unclean. Neaten the style for dynamic_debug. Reduce the stack use of message logging that uses

[REGRESSION] Xorg doesn't like 4e8b14526 time: Improve sanity checking of timekeeping inputs

2012-08-30 Thread Andreas Bombe
I have recently started to get problems with X simply shutting itself down and returning to the login screen. In the X logs I find: [ 1492.936] Fatal server error: [ 1492.936] WaitForSomething(): select: Invalid argument No messages whatsoever is found in the kernel logs. This error

Re: [REGRESSION] Xorg doesn't like 4e8b14526 time: Improve sanity checking of timekeeping inputs

2012-08-30 Thread Linus Torvalds
On Thu, Aug 30, 2012 at 9:05 PM, Andreas Bombe a...@debian.org wrote: With that somewhat easy test I bisected it down to 4e8b14526 time: Improve sanity checking of timekeeping inputs. The latest Linus git (155e36d40) with a revert of the bisected commit does not show the problem. Ok, I guess

Re: [PATCH v2 0/2] dw_dmac: repair driver for use with AVR32 (AP7000)

2012-08-30 Thread Viresh Kumar
On 30 August 2012 22:44, Hein Tibosch hein_tibo...@yahoo.es wrote: After some recent changes to dw_dmac, the driver got broken for the AVR32 platform for two reasons: The accessors to i/o memory had become little-endian. The maximum transfer width on the memory side was increased from 32 to

Re: WARNING: at fs/inode.c:280 drop_nlink+0x31/0x33()

2012-08-30 Thread Nick Pasich
On Wed, Aug 29, 2012 at 03:16:41PM -0700, Jeff Layton wrote: On Wed, 29 Aug 2012 09:25:27 -0700 Nick Pasich n...@nickandbarb.net wrote: I'm using kernel 3.5.3 ... It happens on 3.5.1 and 3.5.2 also. I know that Nick Bowler has already reported this... I'm experiencing the

[PATCH] block/throttle: Call throtl_update_dispatch_stats with throtl_grp directly.

2012-08-30 Thread Tao Ma
From: Tao Ma boyu...@taobao.com All callers of throtl_update_dispatch_stats uses tg_to_blkg and then in this function we use blkg_to_tg again to change it back. So remove all these useless conversion and use throtl_grp directly to call throtl_update_dispatch_stats. Cc: Tejun Heo t...@kernel.org

RE: [PATCH] MAINTAINERS: fix TXT maintainer list and source repo path

2012-08-30 Thread Wei, Gang
Thanks for pointing it out. Jimmy On Thu, 2012-08-30 at 16:34 +0800, j...@perches.com wrote: On Thu, 2012-08-30 at 13:19 +0800, gang@intel.com wrote: diff --git a/MAINTAINERS b/MAINTAINERS [] INTEL(R) TRUSTED EXECUTION TECHNOLOGY (TXT) [] -T: Mercurial

[PATCH V2] block/throttle: Add IO throttled information in blkio.throttle.

2012-08-30 Thread Tao Ma
From: Tao Ma boyu...@taobao.com Currently, if the IO is throttled by io-throttle, the SA has no idea of the situation and can't report it to the real application user about that he/she has to do something. So this patch adds a new interface named blkio.throttle.io_queued which indicates how many

RE: [PATCH] x86/kernel: remove tboot 1:1 page table creation code

2012-08-30 Thread Wei, Gang
Acked-by: Gang Wei gang@intel.com From: Xiaoyan Zhang xiaoyan.zh...@intel.com For TXT boot, while Linux kernel trys to shutdown/S3/S4/reboot, it need to jump back to tboot code and do TXT teardown work. Previously kernel zapped all mem page identity mapping (va=pa) after booting, so

[PATCH] hv: vmbus_drv: detect hyperv through x86_hyper

2012-08-30 Thread Jason Wang
There are two reasons we need to use x86_hyper instead of query_hypervisor_presence(): - Not only hyperv but also other hypervisors such as kvm would set X86_FEATURE_HYTPERVISOR, so query_hypervisor_presence() will return true even in kvm. This may cause extra delay of 5 seconds before

Re: [RFC PATCH 1/5] ACPI: Add acpi_lookup_driver() function

2012-08-30 Thread Bjorn Helgaas
On Thu, Aug 30, 2012 at 1:16 PM, Toshi Kani toshi.k...@hp.com wrote: Added acpi_lookup_driver(), which looks up an associated driver for the notified ACPI device object by walking through the list of ACPI drivers. Signed-off-by: Toshi Kani toshi.k...@hp.com --- drivers/acpi/scan.c |

Re: [PATCH v2 0/2] dw_dmac: repair driver for use with AVR32 (AP7000)

2012-08-30 Thread Viresh Kumar
On 31 August 2012 10:45, Hein Tibosch hein_tibo...@yahoo.es wrote: On 8/31/2012 12:26 PM, Viresh Kumar wrote: BTW, Ideally speaking the fix for AVR32 which will enable 32bit mem support and enable BIG endian support should have been part of this patchset. That code can go through DMA tree as

Re: [RFC v2 PATCH 1/7] thp: remove assumptions on pgtable_t type

2012-08-30 Thread Aneesh Kumar K.V
Gerald Schaefer gerald.schae...@de.ibm.com writes: The thp page table pre-allocation code currently assumes that pgtable_t is of type struct page *. This may not be true for all architectures, so this patch removes that assumption by replacing the functions prepare_pmd_huge_pte() and

<    3   4   5   6   7   8