Because trans_support be used when check i/o accelerator mode support
before its value assigned, it caused following build warning:
drivers/scsi/hpsa.c: In function ‘hpsa_init_one’:
drivers/scsi/hpsa.c:7468:20: warning: ‘trans_support’
may be used uninitialized in this function
Hi, Sylwester
On 3/20/2014 10:44 PM, Sylwester Nawrocki wrote:
Hi Josh,
On 19/03/14 10:17, Josh Wu wrote:
On 3/15/2014 5:17 AM, Sylwester Nawrocki wrote:
On 03/14/2014 11:12 AM, Josh Wu wrote:
+clk = v4l2_clk_get(>dev, "mclk");
+if (IS_ERR(clk))
+return -EPROBE_DEFER;
You
On Tue, Mar 25, 2014 at 06:06:17PM -0700, David Rientjes wrote:
>
> The point is not to add new callers and new code should handle NULL
> correctly, not that we should run around changing current users to just do
> infinite retries. Checkpatch should have nothing to do with that.
My problem
On Tue, 2014-03-25 at 11:59 -0700, Eric W. Biederman wrote:
> So for the general horrible idea.
> Nacked-With-Extreme-Prejudice-by: "Eric W. Biederman"
Goody. I was surprised Peter didn't make it instantly dead.
-Mike
--
To unsubscribe from this list: send the line "unsubscribe
On Tue, 2014-03-25 at 11:59 -0700, Eric W. Biederman wrote:
So for the general horrible idea.
Nacked-With-Extreme-Prejudice-by: Eric W. Biederman ebied...@xmission.com
Goody. I was surprised Peter didn't make it instantly dead.
-Mike
--
To unsubscribe from this list: send the line
On Tue, Mar 25, 2014 at 06:06:17PM -0700, David Rientjes wrote:
The point is not to add new callers and new code should handle NULL
correctly, not that we should run around changing current users to just do
infinite retries. Checkpatch should have nothing to do with that.
My problem with
Hi, Sylwester
On 3/20/2014 10:44 PM, Sylwester Nawrocki wrote:
Hi Josh,
On 19/03/14 10:17, Josh Wu wrote:
On 3/15/2014 5:17 AM, Sylwester Nawrocki wrote:
On 03/14/2014 11:12 AM, Josh Wu wrote:
+clk = v4l2_clk_get(client-dev, mclk);
+if (IS_ERR(clk))
+return -EPROBE_DEFER;
Because trans_support be used when check i/o accelerator mode support
before its value assigned, it caused following build warning:
drivers/scsi/hpsa.c: In function ‘hpsa_init_one’:
drivers/scsi/hpsa.c:7468:20: warning: ‘trans_support’
may be used uninitialized in this function
On Wednesday, March 26, 2014 at 06:56:34 AM, Yao Yuan wrote:
On Wednesday, March 26, 2014 at 11:43:27 AM, Marek Vasut wrote:
On Wednesday, March 26, 2014 at 04:08:27 AM, Yao Yuan wrote:
[...]
+ i2c_imx-use_dma = false;
+ } else if (i2c_imx_dma_request(i2c_imx,
Hi, all
since v4l2_clk_get() WON'T return EPROBE_DEFER. So this version of patch
is invalid.
Please drop this version of the patch.
Sorry for the noise.
Best Regards,
Josh Wu
On 3/20/2014 5:01 PM, Josh Wu wrote:
Since the the v4l2_clk_get() may return a EPROBE_DEFER during async
probing. So
On Sat, Mar 15, 2014 at 02:02:39AM +0800, Chew Chiau Ee wrote:
From: Chew, Chiau Ee chiau.ee.c...@intel.com
This is to disable/enable DW_DMAC hw during late suspend/early resume.
Since DMA is providing service to other clients (eg: SPI, HSUART),
we need to ensure DMA suspends after the
On Wed, 26 Mar 2014 02:19:04 -0400 ty...@mit.edu wrote:
On Tue, Mar 25, 2014 at 06:06:17PM -0700, David Rientjes wrote:
The point is not to add new callers and new code should handle NULL
correctly, not that we should run around changing current users to just do
infinite retries.
Hi Lucas,
On Mon, Mar 24, 2014 at 10:19 PM, Lucas Stach l.st...@pengutronix.de wrote:
Hi Alexandre,
Am Montag, den 24.03.2014, 17:42 +0900 schrieb Alexandre Courbot:
Hi everyone,
[...]
A few lines of hacks (not included here) are still needed to deal with cached
mappings triggering
Hi all,
This tree still fails (more than usual) the powerpc allyesconfig build.
Changes since 20140325:
The powerpc tree still had its build failure.
Non-merge commits (relative to Linus' tree): 10769
9188 files changed, 442121 insertions(+), 216825 deletions(-)
To fix the sparse warning cast to restricted __le32 marked
rom_version to __le32 instead of unsigned int in struct ath3k_version
and added cpu_to_le32() for the expression assigning int value to
rom_version.
Successfully built the module without warnings and errors on x86 machine with
Hi Jason,
Will try this out this evening when I'm back home (and have some time
to reboot the sheevaplug).
Note that the write-back stalls seem to be nilfs2 specific (I'm getting
them on usb-storage as well while ext4 filesystem on usb-storage
does not present the stall issue).
Bruno
On Tue,
On Wed, Mar 26, 2014 at 01:03:00PM +0800, Jason Wang wrote:
Current error handling of virtqueue_kick() was wrong in two places:
- The skb were freed immediately when virtqueue_kick() fail during
xmit. This may lead double free since the skb was not detached from
the virtqueue.
-
(2014/03/22 9:49), Steven Rostedt wrote:
On Wed, 05 Mar 2014 20:59:11 +0900
Masami Hiramatsu masami.hiramatsu...@hitachi.com wrote:
diff --git a/Documentation/kprobes.txt b/Documentation/kprobes.txt
index 0cfb00f..7062631 100644
--- a/Documentation/kprobes.txt
+++
On Wednesday, March 26, 2014 at 02:27:46 PM, Marek Vasut wrote:
On Wednesday, March 26, 2014 at 06:56:34 AM, Yao Yuan wrote:
On Wednesday, March 26, 2014 at 11:43:27 AM, Marek Vasut wrote:
On Wednesday, March 26, 2014 at 04:08:27 AM, Yao Yuan wrote:
[...]
+
On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote:
From: Hongbo Zhang hongbo.zh...@freescale.com
The usage of spin_lock_irqsave() is a stronger locking mechanism than is
required throughout the driver. The minimum locking required should be used
instead. Interrupts will be
This patch modifies mfd driver to use regmap for handling interrupts.
It allows to simplify irq handling process. This modifications needed
to make small changes in function drivers, which use interrupts.
Signed-off-by: Robert Baldyga r.bald...@samsung.com
---
drivers/mfd/Kconfig
Hi Lee,
From: Lee Jones [mailto:lee.jo...@linaro.org]
Obtain IRQ number and request IRQ resource via the usual methods. We're
also registering an IRQ handler to inform us of any completed tasks.
Notice that we're starting to make use of the device struct that we
defined before. In keeping with
On Tue, Mar 25, 2014 at 06:42:13PM -0400, Mikulas Patocka wrote:
On Mon, 24 Mar 2014, Daniel Vetter wrote:
Like I've said the entire teardown sequence for legacy drm drivers is
terminally busted, so the only hope we have is to reapply this missing
duct-tape which made your X crash.
From: Lee Jones [mailto:lee.jo...@linaro.org]
Add support for clocks when, and only when, they are supplied. It is
not yet compulsory to provide the BCH and EMI clocks, as Common Clk isn't
supported Mainline yet. Until an implementation lands upstream all clocks
located on STM boards default to
Hi Surendra,
To fix the sparse warning cast to restricted __le32 marked
rom_version to __le32 instead of unsigned int in struct ath3k_version
and added cpu_to_le32() for the expression assigning int value to
rom_version.
Successfully built the module without warnings and errors on x86
-Original Message-
From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] On
Behalf Of Peter Wu
Sent: Sunday, March 23, 2014 6:00 AM
To: Gleb Natapov
Cc: Venkatesh Srinivas; Peter Zijlstra; Ingo Molnar; Andi Kleen; Linux Kernel
Developers List; H. Peter Anvin;
On Wednesday, March 26, 2014 at 08:08:28 AM, Yao Yuan wrote:
On Wednesday, March 26, 2014 at 02:27:46 PM, Marek Vasut wrote:
On Wednesday, March 26, 2014 at 06:56:34 AM, Yao Yuan wrote:
On Wednesday, March 26, 2014 at 11:43:27 AM, Marek Vasut wrote:
On Wednesday, March 26, 2014 at
On Tue, Mar 25, 2014 at 07:00:45PM -0300, Ezequiel Garcia wrote:
After taking a quick glance at the whole driver I noticed you have something
strange going on. AFAIK, the typical NAND driver probe() should be one of
these two:
* Call nand_scan() which calls nand_scan_ident() +
On Tue, Mar 11, 2014 at 01:31:26AM -0300, Erico Nunes wrote:
Mention to CONFIG_MTD_ECC_BCH in the warning message can be confusing as this
doesn't match the exact name of the configuration option.
This warning showed up once to me when I was starting to set up BCH. After
checking my .config
On Wed, 2014-03-26 at 08:10 +0100, Robert Baldyga wrote:
This patch modifies mfd driver to use regmap for handling interrupts.
It allows to simplify irq handling process. This modifications needed
to make small changes in function drivers, which use interrupts.
Signed-off-by: Robert Baldyga
fix a sparse warning.
drivers/staging/vt6655/iwctl.c:1846:35: warning: cast from restricted
gfp_t
drivers/staging/vt6655/iwctl.c:1846:35: warning: incorrect type in
argument 2 (different base types)
drivers/staging/vt6655/iwctl.c:1846:35:expected restricted gfp_t
[usertype] flags
This patch modifies mfd driver to use regmap for handling interrupts.
It allows to simplify irq handling process. This modifications needed
to make small changes in function drivers, which use interrupts.
Signed-off-by: Robert Baldyga r.bald...@samsung.com
---
drivers/mfd/Kconfig
Below is the list of build error/warning regressions/improvements in
v3.14-rc8[1] compared to v3.13[2].
Summarized:
- build errors: +10/-30
- build warnings: +93/-79
JFYI, when comparing v3.14-rc8[1] to v3.14-rc7[3], the summaries are:
- build errors: +6/-1
- build warnings: +19/-15
As
I realise that you and Alessandro are very busy (I have not often seen
Alessandro do any posting in the past fortnight), but a couple of weeks
ago I sent an RTC patch set with register.h changes ...
From: Opensource [Steve Twiss] stwiss.opensou...@diasemi.com
Add the RTC driver for
On Wed, Mar 26, 2014 at 8:47 AM, Geert Uytterhoeven
ge...@linux-m68k.org wrote:
JFYI, when comparing v3.14-rc8[1] to v3.14-rc7[3], the summaries are:
- build errors: +6/-1
+ /scratch/kisskb/src/arch/powerpc/include/asm/io.h: error:
'virt_phys_offset' undeclared (first use in this
于 2014/3/26 0:25, Satoru Takeuchi 写道:
At Tue, 25 Mar 2014 16:58:58 +0800,
Liu hua wrote:
于 2014/3/24 4:50, Satoru Takeuchi 写道:
At Sun, 23 Mar 2014 15:54:04 +0800,
Liu Hua wrote:
As sysctl_hung_task_timeout_sec is unsigned long, when this value is
larger then LONG_MAX/HZ, the function
For tsc deadline mode, when we shut down clockevent we should disarm
local-APIC timer like count-down mode.
Signed-off-by: wang, biao biao.w...@intel.com
Signed-off-by: Zhang Di di.zh...@intel.com
---
arch/x86/kernel/apic/apic.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff
This patch modifies mfd driver to use regmap for handling interrupts.
It allows to simplify irq handling process. This modifications needed
to make small changes in function drivers, which use interrupts.
Signed-off-by: Robert Baldyga r.bald...@samsung.com
Reviewed-by: Krzysztof Kozlowski
This patch modifies mfd driver to use regmap for handling interrupts.
It allows to simplify irq handling process. This modifications needed
to make small changes in function drivers, which use interrupts.
Signed-off-by: Robert Baldyga r.bald...@samsung.com
Reviewed-by: Krzysztof Kozlowski
I realise that you and Alessandro are very busy (I have not often seen
Alessandro do any posting in the past fortnight), but a couple of weeks
ago I sent an RTC patch set with register.h changes ...
From: Opensource [Steve Twiss] stwiss.opensou...@diasemi.com
Add the RTC driver for
fix an bug related to extension list.
there was the potential bug in set_cold_files function, namei.c.
Signed-off-by: Dongho Sim dh@samsung.com
---
mkfs/f2fs_format.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/mkfs/f2fs_format.c b/mkfs/f2fs_format.c
index
On Tue, Mar 25, 2014 at 07:01:10PM +0100, Sam Ravnborg wrote:
There are two things that don't work too well with this. First this
causes the build to break if the build machine doesn't have the new
public header (include/uapi/linux/dma-buf.h) installed yet. So the only
way to make this
On 26 March 2014 02:29, micky micky_ch...@realsil.com.cn wrote:
Hi Ulf,
On 03/25/2014 06:44 PM, Ulf Hansson wrote:
On 25 March 2014 10:47, micky_ch...@realsil.com.cn wrote:
From: Micky Ching micky_ch...@realsil.com.cn
Add new command type(R1 without CRC) handle, without this
patch mmc
On Thu, Feb 13, 2014 at 10:15 PM, Paul Gortmaker
paul.gortma...@windriver.com wrote:
While copy_to/from_user_page() users are uncommon, there is one in
drivers/staging/lustre/lustre/libcfs/linux/linux-curproc.c which leads
to the following:
ERROR: sparc32_cachetlb_ops
On Sat, Jan 25, 2014 at 4:23 AM, Oleg Drokin gr...@linuxhacker.ru wrote:
On Jan 24, 2014, at 3:55 AM, Geert Uytterhoeven wrote:
On Fri, Jan 24, 2014 at 6:51 AM, Oleg Drokin gr...@linuxhacker.ru wrote:
+STAGING - LUSTRE
+M: Andreas Dilger andreas.dil...@intel.com
+M: Oleg Drokin
Hi,
Huang Shijie wrote:
On Fri, Mar 21, 2014 at 11:50:17AM +0100, Lothar Waßmann wrote:
With a flash-based BBT there is no reason to move the Factory Bad
Block Marker from the data area buffer (to where it is mapped by the
GPMI NAND controller) to the OOB buffer. Thus, make this feature
(cc akpm, who still uses these scripts, while the rest of the world
moved to git ;-)
On Wed, Mar 26, 2014 at 12:49 AM, Mitchel Humpherys
mitch...@codeaurora.org wrote:
The link to the tarball for Andrew Morton's patch scripts is dead. These
scripts don't seem to be used for kernel development
On Tue, Mar 18, 2014 at 10:25:27AM -0400, Wang, Xiaoming wrote:
pcmd-parmbuf-pbuf has been allocated if command is
GEN_CMD_CODE(_Set_Drv_Extra),
and it enqueued by rtw_enqueue_cmd. rtw_cmd_thread dequeue pcmd by
rtw_dequeue_cmd.
The memory leak happened on this branch if( _FAIL ==
On Tue, 25 Mar, at 03:40:30PM, Roy Franz wrote:
Add the efi_early_call() macro to invoke functions in the efi_early
structure. Using a macro for these invocations allows the arm32/arm64
architectures to define the macro differently so that they can directly
invoke the boot services functions
On Wed, Mar 26, 2014 at 09:44:08AM +0100, Geert Uytterhoeven wrote:
Ping?
We got a new fix for this in due to https://lkml.org/lkml/2014/2/13/479
Given that lustre maintainers aren't willing to fix even the worst crap
like this we really should drop it from the staging tree.
--
To
On Tue, Mar 25, 2014 at 06:19:24PM +, j...@ringle.org wrote:
From: Jon Ringle jrin...@gridpoint.com
This patch adds the devicetree documentation for the NXP SC16IS7XX UARTs.
Signed-off-by: Jon Ringle jrin...@gridpoint.com
---
.../devicetree/bindings/serial/nxp,sc16is7xx.txt | 35
On Tue, 25 Mar, at 03:47:23PM, Roy Franz wrote:
I have sent a patch (attempted to reply using git-send-email) that
adds the macro for x86 and updates efi-stub-helper.c. If you could
add this to your series for 3.15 that would be great, as then we would
not have any x86 changes in the ARM
Hi Lee,
From: Lee Jones [mailto:lee.jo...@linaro.org]
Fetch platform specific data from Device Tree. Any functions which
are useful to other STM NAND Controllers have been separated into a
separate file so they can be easily referenced by them as they
appear.
Signed-off-by: Lee Jones
It was impossible to enumerate on a SuperSpeed (XHCI) host
with alternate setting = 1 due to the wrongly set 'bMaxBurst'
field in the SuperSpeed Endpoint Companion descriptor.
Testcase:
host modprobe -r usbtest; modprobe usbtest alt=1
device modprobe g_zero
plug device to SuperSpeed port on the
Hi Jason,
On 26/03/2014 01:30, Jason Cooper wrote:
On Tue, Mar 25, 2014 at 11:48:18PM +0100, Gregory CLEMENT wrote:
The initial binding for PMSU were wrong. It didn't take into account
all the registers from the PMSU and moreover it referred to registers
which are not part of PMSU.
The
Il 26/03/2014 08:23, Wu, Feng ha scritto:
Is there a solution for this issue right now? I also met this GPF crash.
Can you attach your .config?
Paolo
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo
During system resume, if the event buffers are not setup before
the gadget controller starts then we start with invalid context
and this can lead to bus access errors. This is especially true for
platforms that loose the controller context during system suspend.
e.g. AM437x.
The following
From: Cody P Schafer
On 03/25/2014 03:43 AM, Anton Blanchard wrote:
Hi Cody,
hv-24x7: could not obtain capabilities, error 0x
fffe, not enabling
hv-gpci: could not obtain capabilities, error 0x
fffe, not enabling
+ pr_info(could not obtain
This is another one I'd like to send off to James ASAP, can I get some
reviews?
On Mon, Mar 17, 2014 at 06:28:01AM -0700, Christoph Hellwig wrote:
Instead of letting the ULD play games with the prep_fn move back to
the model of a central prep_fn with a callback to the ULD. This
already cleans
On 03/26/2014 10:33 AM, Gregory CLEMENT wrote:
On 26/03/2014 01:30, Jason Cooper wrote:
On Tue, Mar 25, 2014 at 11:48:18PM +0100, Gregory CLEMENT wrote:
The initial binding for PMSU were wrong. It didn't take into account
all the registers from the PMSU and moreover it referred to registers
On 26/03/2014 01:42, Jason Cooper wrote:
Gregory,
On Tue, Mar 25, 2014 at 11:48:11PM +0100, Gregory CLEMENT wrote:
...
The first patch should go through ARM subsystem and should be taken by
Russell King. I made few change on it following Lorenzo advice and
now it will reuse the cpu v7
Grant Grundler [mailto:grund...@google.com]
Sent: Wednesday, March 26, 2014 4:12 AM
[...]
Hayes,
I believe this patch was dropped after the series was split.
Can you please repost this patch by itself?
There is no problem for current behavior, and I don't get the
issue of tx timeout, yet. I
On 2014/03/26 03:43 AM, Chase Southwood wrote:
There were just a handful of more while loops in this file that needed
timeouts, and this patch takes care of them. One new callback is
introduced, and all of the proper comedi_timeout() calls are then used.
Signed-off-by: Chase Southwood
This patch adds IRQ line specification for the MAX8997 chip and necessary
pinctrl group to configure pull-up and driver strength of the pin.
It's needed for proper handling of IRQs coming from MAX8997 multifunction
device (IRQs are used by PMIC, MUIC and RTC).
Signed-off-by: Robert Baldyga
On Wed, 2014-03-26 at 12:56 +0900, SeongJae Park wrote:
Fix following trivial build warning:
drivers/net/xen-netback/netback.c: In function ‘xenvif_tx_dealloc_action’:
drivers/net/xen-netback/netback.c:1585:8: warning:
format ‘%x’ expects argument of type
‘unsigned int’, but argument 3 has
In older kernels (before v3.10) ipc_rcu_hdr-refcount was non-atomic int.
There was possuble double-free bug: do_msgsnd() calls ipc_rcu_putref() under
msq-q_perm-lock and RCU, while freequeue() calls it while it holds only
'rw_mutex', so there is no sinchronization between them. Two function
Russell,
Thanks for the reply!
On Mar 26, Russell King - ARM Linux wrote:
On Tue, Mar 25, 2014 at 07:45:55PM -0300, Ezequiel Garcia wrote:
Without this patch we got the heartbeat's reboot_notifier called twice while
testing the recent hibernation patches, which was unexpected and
On Tue, Mar 25, 2014 at 10:12:14PM +, Tanmay Inamdar wrote:
Hello Liviu,
Thanks for taking a look. Please see inline.
On Tue, Mar 25, 2014 at 7:02 AM, Liviu Dudau liviu.du...@arm.com wrote:
On Wed, Mar 19, 2014 at 11:12:39PM +, Tanmay Inamdar wrote:
This patch adds the
Hi Lee,
From: Lee Jones [mailto:lee.jo...@linaro.org]
Use DMA to read and/or write a single page of data.
Signed-off-by: Lee Jones lee.jo...@linaro.org
---
drivers/mtd/nand/stm_nand_bch.c | 119
1 file changed, 119 insertions(+)
diff --git
Hi Lee,
From: Lee Jones [mailto:lee.jo...@linaro.org]
missing commit log :-)
Though $subject is self-explanatory, but you can add more description about
assumption and hardware caveats about the controller, and its use.
Signed-off-by: Lee Jones lee.jo...@linaro.org
---
Hi Gregory,
On Mar 25, Gregory CLEMENT wrote:
@@ -285,4 +290,21 @@ static struct notifier_block
armada_370_xp_cpu_pm_notifier = {
.notifier_call = armada_370_xp_cpu_pm_notify,
};
+int __init armada_370_xp_cpu_pm_init(void)
+{
+ if (!((of_find_compatible_node(NULL, NULL,
Hi Lee,
From: Lee Jones [mailto:lee.jo...@linaro.org]
Helper function for bch_mtd_read() and bch_mtd_write() to handle
multi-page or non-aligned reads and writes respectively.
Signed-off-by: Lee Jones lee.jo...@linaro.org
---
I think below code is duplicate of nand_do_read_ops() and
Dear Gregory CLEMENT,
On Tue, 25 Mar 2014 23:48:25 +0100, Gregory CLEMENT wrote:
+int __init armada_370_xp_cpu_pm_init(void)
+{
+ if (!((of_find_compatible_node(NULL, NULL,
marvell,armada-370-xp-pmsu) ||
+ of_find_compatible_node(NULL, NULL,
Hi Alexandre,
Am Mittwoch, den 26.03.2014, 15:33 +0900 schrieb Alexandre Courbot:
Hi Lucas,
On Mon, Mar 24, 2014 at 10:19 PM, Lucas Stach l.st...@pengutronix.de wrote:
Hi Alexandre,
Am Montag, den 24.03.2014, 17:42 +0900 schrieb Alexandre Courbot:
Hi everyone,
[...]
A few lines
Thank you for let me know kindly :)
On Wed, Mar 26, 2014 at 7:09 PM, Ian Campbell ian.campb...@citrix.com wrote:
On Wed, 2014-03-26 at 12:56 +0900, SeongJae Park wrote:
Fix following trivial build warning:
drivers/net/xen-netback/netback.c: In function ‘xenvif_tx_dealloc_action’:
On 01/08/2014 02:39 AM, Tony Lindgren wrote:
* Taras Kondratiuk taras.kondrat...@linaro.org [131223 10:20]:
On 23 December 2013 20:10, Taras Kondratiuk taras.kondrat...@linaro.org
wrote:
This series does trivial replacement of __raw_xxx functions with xxx_relaxed
endian-neutral variants in
On 26/03/2014 11:31, Thomas Petazzoni wrote:
Dear Gregory CLEMENT,
On Tue, 25 Mar 2014 23:48:25 +0100, Gregory CLEMENT wrote:
+int __init armada_370_xp_cpu_pm_init(void)
+{
+if (!((of_find_compatible_node(NULL, NULL,
marvell,armada-370-xp-pmsu) ||
+
I don't know if it is a valid idea, but maybe it would be ok to process
events after resume in general, and only throw away events on those
platforms that continue to log events while in standby (Samsung 5/7/9)?
But after all, it would be better to find the command to tell the EC to
stop
于 2014年03月26日 16:51, Lothar Waßmann 写道:
I don't see why this should not be supported on i.MX28 (i.MX23 doesn't
do byteswapping anyway, so this wouldn't change anything for i.MX23).
The partitions used by Linux need not necessarily be accessible for the
Boot ROM code (and vice versa).
But the
On 03/26/2014 10:28 AM, Roger Quadros wrote:
Bug was introduced by commit cf9a08ae in v3.9
I'm sorry for the trouble.
Acked-by: Sebastian Andrzej Siewior bige...@linutronix.de
Sebastian
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to
On 24/03/14 09:26, Ian Campbell wrote:
On Fri, 2014-03-21 at 17:23 +, Zoltan Kiss wrote:
This reverts commit e9275f5e2df1b2098a8cc405d87b88b9affd73e6. This commit is the
last in the netback grant mapping series, and it tries to do more aggressive
aggreagtion of unmap operations. However
On Wed, Mar 26, 2014 at 07:12:27AM -0300, Ezequiel Garcia wrote:
On Mar 26, Russell King - ARM Linux wrote:
On Tue, Mar 25, 2014 at 07:45:55PM -0300, Ezequiel Garcia wrote:
Without this patch we got the heartbeat's reboot_notifier called twice
while
testing the recent hibernation
Paul,
You have been awfully silent for this whole thread while this is a regression
caused by a patch of you
(ca2f09f2b2c6c25047cfc545d057c4edfcfe561c as clearly stated much earlier in
this thread).
The commit messages states:
net_rx_action() is the place where we could do with an accurate
hrtimer_set_expires_range() and hrtimer_set_expires_range_ns() have almost same
implementations and so we can call hrtimer_set_expires_range() from
hrtimer_set_expires_range_ns() internally, instead of duplicating code.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
Hi Thomas,
I am going through this piece of code to complete my 'cpusets.quiesce' work.
While going through code I accumulated these patches which are mostly code
cleanups and shouldn't have much functional change.
Thanks for applying yesterdays cleanups :)
Viresh Kumar (14):
hrtimer: replace
retrigger_next_event() is defined within #ifdef CONFIG_HIGH_RES_TIMERS and we
already have pointer to base available. So it makes more sense to simply use
base-hres_active instead of doing this by calling hrtimer_hres_active():
__this_cpu_read(hrtimer_bases.hres_active)
Signed-off-by:
In init_timers() we need to call init_timers_cpu() for boot CPU. For this,
currently we are emulating a call to hotplug notifier. Probably this was done
initially to get rid of code redundancy. But this sequence always called a
single routine, i.e. init_timers_cpu(), and so calling that routine
Caller of hrtimer_switch_to_hres(), i.e. hrtimer_run_pending(), has already
verified this by calling hrtimer_hres_active() and so we don't need to do it
again in hrtimer_switch_to_hres().
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
kernel/hrtimer.c | 3 ---
1 file changed, 3
clock_was_set_delayed() is called from only hrtimer.c and so should be marked
static. Along with that its declaration and dummy definition must be removed
from hrtimer.h.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
include/linux/hrtimer.h | 5 -
kernel/hrtimer.c| 3 ++-
2
We have just checked that expires_next.tv64 == cpu_base-expires_next.tv64, and
in this case we shouldn't rewrite the same value again. Rewrite code to fix
this.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
kernel/hrtimer.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Currently we are returning notifier_from_errno() from CPU_UP_PREPARE notifier
when we detect an error while calling init_timers_cpu(). notifier_from_errno()
already has enough checks within to do something similar. And so we can call it
directly without checking if there was an error or not.
High Resolution feature can be enabled/disabled from bootargs if we have a
string 'highres=' followed by 'on' or 'off'. The default value of this variable
is '1'. When 'on' is passed as bootarg, we don't have to overwrite this
variable by '1'.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
Active_bases field of struct hrtimer_cpu_base is used at only one place, i.e.
hrtimer_interrupt() and at that place too we can easily use timerqueue_getnext()
instead to achieve the same result. I don't think this will have any performance
degradation issues and so removing this field.
In hrtimers_init() we need to call init_hrtimers_cpu() for boot CPU. For this,
currently we are emulating a call to hotplug notifier. Probably this was done
initially to get rid of code redundancy. But this sequence always called a
single routine, i.e. init_hrtimers_cpu(), and so calling that
hrtimer_force_reprogram() is only called from parts of kernel which are defined
within #ifdef CONFIG_HIGH_RES_TIMERS and so its empty definition is never used.
Remove it.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
kernel/hrtimer.c | 2 --
1 file changed, 2 deletions(-)
diff --git
Currently we have a 'tab' here instead of 'space' after 'comma'. Replace it with
'space'.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
kernel/hrtimer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/hrtimer.c b/kernel/hrtimer.c
index d55092c..a1120a0 100644
Breaking format fragments into multiple lines hits readability of code. Even if
it goes over 80 column width, its better to keep them together.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
kernel/hrtimer.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
In switch_hrtimer_base() we have created a local variable basenum which is set
to base-index. This variable is used at only one place. It makes code more
readable if we remove this variable use base-index directly.
Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
---
kernel/hrtimer.c | 3 +--
On 03/26/2014 09:26 AM, Preeti U Murthy wrote:
Its possible that the tick_broadcast_force_mask contains cpus which are not
in cpu_online_mask when a broadcast tick occurs. This could happen under the
following circumstance assuming CPU1 is among the CPUs waiting for broadcast.
CPU0
On 03/26/2014 04:51 PM, Viresh Kumar wrote:
In switch_hrtimer_base() we have created a local variable basenum which is set
to base-index. This variable is used at only one place. It makes code more
readable if we remove this variable use base-index directly.
No, this doesn't look right. Note
501 - 600 of 1008 matches
Mail list logo