Re: [PATCH 02/12] clocksource: sh_mtu2: Add clk_prepare/unprepare support

2013-10-28 Thread Simon Horman
On Mon, Oct 28, 2013 at 11:49:19PM +0100, Laurent Pinchart wrote: > Prepare the clock at probe time, as there is no other appropriate place > in the driver where we're allowed to sleep. > > Cc: Daniel Lezcano > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Laurent Pinchart > --- >

Re: [PATCH 03/12] clocksource: sh_tmu: Add clk_prepare/unprepare support

2013-10-28 Thread Simon Horman
On Mon, Oct 28, 2013 at 11:49:20PM +0100, Laurent Pinchart wrote: > Prepare the clock at probe time, as there is no other appropriate place > in the driver where we're allowed to sleep. > > Cc: Daniel Lezcano > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Laurent Pinchart > --- >

Re: [PATCH 01/12] clocksource: sh_cmt: Add clk_prepare/unprepare support

2013-10-28 Thread Simon Horman
On Mon, Oct 28, 2013 at 11:49:18PM +0100, Laurent Pinchart wrote: > Prepare the clock at probe time, as there is no other appropriate place > in the driver where we're allowed to sleep. > > Cc: Daniel Lezcano > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Laurent Pinchart Thanks Laurent,

Re: [PATCH v2] arch: use ASM_NL instead of ';' for assembler new line character in the macro

2013-10-28 Thread Vineet Gupta
On 10/29/2013 07:21 AM, Chen Gang wrote: > For some assemblers, they use another character as newline in a macro > (e.g. arc uses '`'), so for generic assembly code, need use ASM_NL (a > macro) instead of ';' for it. > > Also need notice about code styles ('\t' for each line). > > >

Re: [PATCH v3 01/10] of: Add empty for_each_available_child_of_node() macro definition

2013-10-28 Thread Rob Herring
On Mon, Oct 28, 2013 at 1:12 PM, Josh Cartwright wrote: > From: Sylwester Nawrocki > > Add this empty macro definition so users can be compiled without > excluding this macro call with preprocessor directives when CONFIG_OF > is disabled. > > Signed-off-by: Sylwester Nawrocki > Signed-off-by:

Re: [PATCH] ACPI: remove unused ACPI_PROCFS Kconfig param

2013-10-28 Thread Lan Tianyu
2013/10/28 Paul Bolle : > On Mon, 2013-10-28 at 07:22 +0100, Michael Opdenacker wrote: >> [...] CONFIG_ACPI_PROCFS is no longer used anywhere, neither in >> Makefiles nor in the source code. ACPI_PROCFS isn't event used in >> Kconfig files as an intermediate dependency. >> >> What I deduce from

Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Namhyung Kim
Hi Ingo, On Sat, 26 Oct 2013 11:50:23 +0200, Ingo Molnar wrote: > * Srikar Dronamraju wrote: > >> Hi Pekka, >> >> > > >> > > You can now use it in all perf tools, such as: >> > > >> > > perf record -e libc:my_event -aR sleep 1 >> > >> > Is there a technical reason why 'perf list' could not

Re: [PATCH 3/3] sched: Aggressive balance in domains whose groups share package resources

2013-10-28 Thread Preeti U Murthy
Hi Peter, On 10/28/2013 09:23 PM, Peter Zijlstra wrote: > On Mon, Oct 21, 2013 at 05:15:02PM +0530, Vaidyanathan Srinivasan wrote: >> From: Preeti U Murthy >> >> The current logic in load balance is such that after picking the >> busiest group, the load is attempted to be moved from the busiest

Re: [patch 5/6] export efi runtime memory mapping to sysfs

2013-10-28 Thread Dave Young
On 10/27/13 at 11:47am, Dave Young wrote: > kexec kernel will need exactly same mapping for > efi runtime memory ranges. Thus here export the > runtime ranges mapping to sysfs, kexec-tools > will assemble them and pass to 2nd kernel via > setup_data. > > Introducing a new directly

Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Namhyung Kim
Hi Masami, On Tue, 29 Oct 2013 12:19:37 +0900, Masami Hiramatsu wrote: > (2013/10/29 2:48), Pekka Enberg wrote: >> For the 32-bit and 64-bit libc case, why cannot 'perf list' >> by default print out something like: >> >> $ perf list >> >>libc:setjmp [SDT marker group] >> >> and provide a

Re: State of "perf: Add a new sort order: SORT_INCLUSIVE"

2013-10-28 Thread Namhyung Kim
On Mon, 28 Oct 2013 21:10:38 -0700, Arun Sharma wrote: > On 10/28/13 8:11 PM, Namhyung Kim wrote: > > Hey Namhyung: > >>> >>> Also, what's the reasoning for --cumulate not being an option under >>> perf record -g ..,? >> >> Sorry, I cannot understand you. The 'perf record' just saves sample >>

Re: [PATCH 1/4] perf tools: Split -g and --call-graph for record command

2013-10-28 Thread Namhyung Kim
On Mon, 28 Oct 2013 12:20:53 -0600, David Ahern wrote: > On 10/28/13 11:46 AM, Arnaldo Carvalho de Melo wrote: >> Added this: >> >> --call-graph:: >> Setup and enable call-graph (stack chain/backtrace) recording, >> implies -g. >> >> Allows specifying "fp" (frame

Re: [PATCH 4/6] edac: Document Krait L1/L2 EDAC driver binding

2013-10-28 Thread Stephen Boyd
On 10/28, Mark Rutland wrote: > On Tue, Oct 29, 2013 at 12:31:28AM +, Stephen Boyd wrote: > > + > > +Optional properties: > > +- interrupt-names: Should contain the interrupt names "l1_irq" and > > + "l2_irq" > > As with my comment on the parsing code, I'd prefer that if interrupt-names was

Re: [PATCH 3/6] ARM: Add Krait L2 accessor functions

2013-10-28 Thread Stephen Boyd
On 10/28, Mark Rutland wrote: > On Tue, Oct 29, 2013 at 01:21:57AM +, Stephen Boyd wrote: > > On 10/28/13 18:19, Mark Rutland wrote: > > > It might be worth commmenting inline as to what register each of these is > > > accessing. Inevitably the commit message will become harder to find and > >

Re: [PATCH 1/7] power_supply: Add charger control properties

2013-10-28 Thread NeilBrown
On Sun, 27 Oct 2013 23:18:08 -0700 Anton Vorontsov wrote: > On Mon, Oct 28, 2013 at 03:36:36AM +, Tc, Jenny wrote: > > > But do we really want to control the chargers through the power_supply's > > > user-visible > > > interface? It makes the whole power supply thing so complicated that I'm

Re: [PATCH] mm: cma: free cma page to buddy instead of being cpu hot page

2013-10-28 Thread Minchan Kim
Hello, On Mon, Oct 28, 2013 at 07:42:49PM +0800, zhang.ming...@linaro.org wrote: > From: Mingjun Zhang > > free_contig_range frees cma pages one by one and MIGRATE_CMA pages will be > used as MIGRATE_MOVEABLE pages in the pcp list, it causes unnecessary > migration action when these pages

Re: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Scott Wood
On Mon, 2013-10-28 at 23:45 -0500, Bhushan Bharat-R65777 wrote: > > > -Original Message- > > From: Wood Scott-B07421 > > Sent: Tuesday, October 29, 2013 10:05 AM > > To: Bhushan Bharat-R65777 > > Cc: Wood Scott-B07421; Alex Williamson; Kim Phillips; Yoder Stuart-B08248; > >

RE: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Bhushan Bharat-R65777
> -Original Message- > From: Wood Scott-B07421 > Sent: Tuesday, October 29, 2013 10:05 AM > To: Bhushan Bharat-R65777 > Cc: Wood Scott-B07421; Alex Williamson; Kim Phillips; Yoder Stuart-B08248; > christoffer.d...@linaro.org; linux-kernel@vger.kernel.org; >

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-28 Thread Stefano Stabellini
ping? On Fri, 25 Oct 2013, Stefano Stabellini wrote: > Russell, > this is the only patch that needs an ack at the moment. > As you commented on it before and I have already addressed your comments > few versions ago, unless you have any complaints I am going to add it to > linux-next and I am

Re: [PATCH v2 3/3] ARM: shmobile: kzm9d: Use common clock framework

2013-10-28 Thread Simon Horman
On Wed, Oct 09, 2013 at 07:39:18PM +0900, Simon Horman wrote: > On Wed, Oct 09, 2013 at 04:29:56PM +0900, Magnus Damm wrote: > > Hi Simon, > > > > On Wed, Oct 9, 2013 at 3:54 PM, Simon Horman wrote: > > > On Wed, Oct 09, 2013 at 01:59:46PM +0900, Magnus Damm wrote: > > >> Hi Simon, > > >> > > >>

Re: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Scott Wood
On Mon, 2013-10-28 at 23:31 -0500, Bhushan Bharat-R65777 wrote: > > > -Original Message- > > From: Wood Scott-B07421 > > Sent: Tuesday, October 29, 2013 10:00 AM > > To: Bhushan Bharat-R65777 > > Cc: Wood Scott-B07421; Alex Williamson; Kim Phillips; Yoder Stuart-B08248; > >

Re: [PATCH v2] ARM: SMMU: add devices attached to the SMMU to an IOMMU group

2013-10-28 Thread Will Deacon
On Mon, Oct 28, 2013 at 04:44:22PM +, Alex Williamson wrote: > On Fri, 2013-10-18 at 17:08 +0200, Antonios Motakis wrote: > > IOMMU groups are expected by certain users of the IOMMU API, > > e.g. VFIO. Add new devices found by the SMMU driver to an IOMMU > > group to satisfy those users. > >

Re: State of "perf: Add a new sort order: SORT_INCLUSIVE"

2013-10-28 Thread Arun Sharma
On 10/28/13 8:11 PM, Namhyung Kim wrote: Hey Namhyung: Also, what's the reasoning for --cumulate not being an option under perf record -g ..,? Sorry, I cannot understand you. The 'perf record' just saves sample data (and callchains) from the ring-buffer. All the processing happens in

RE: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Bhushan Bharat-R65777
> -Original Message- > From: Wood Scott-B07421 > Sent: Tuesday, October 29, 2013 10:00 AM > To: Bhushan Bharat-R65777 > Cc: Wood Scott-B07421; Alex Williamson; Kim Phillips; Yoder Stuart-B08248; > christoffer.d...@linaro.org; linux-kernel@vger.kernel.org; >

Re: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Scott Wood
On Mon, 2013-10-28 at 22:52 -0500, Bhushan Bharat-R65777 wrote: > > > -Original Message- > > From: Wood Scott-B07421 > > Sent: Tuesday, October 29, 2013 9:11 AM > > To: Bhushan Bharat-R65777 > > Cc: Wood Scott-B07421; Alex Williamson; Kim Phillips; Yoder Stuart-B08248; > >

Re: perf: PERF_EVENT_IOC_PERIOD on ARM vs everywhere else

2013-10-28 Thread Will Deacon
On Mon, Oct 28, 2013 at 02:07:48PM +, Vince Weaver wrote: > It's also a shame this change apprently didn't hit the linux-kernel list > as far as I can tell. I do my best to try to note all of the perf > ABI-related changes there, but if things like this are going to start > getting merged

Re: [PATCH 0/5] r8152 bug fixes

2013-10-28 Thread David Miller
From: Hayes Wang Date: Mon, 28 Oct 2013 19:58:09 +0800 > These could fix some driver issues. > > Hayes Wang (5): > net/usb/r8152: fix tx/rx memory overflow > net/usb/r8152: make sure the tx checksum setting is correct > net/usb/r8152: modify the tx flow > net/usb/r8152: fix incorrect

Re: [PATCH v2 2/3] net, datagram: fix the incorrect comment in zerocopy_sg_from_iovec()

2013-10-28 Thread David Miller
From: Zhi Yong Wu Date: Mon, 28 Oct 2013 14:01:49 +0800 > From: Zhi Yong Wu > > Signed-off-by: Zhi Yong Wu Applied to net-next. -- 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

Re: [PATCH v2 3/3] net, iovec: fix the incorrect comment in memcpy_fromiovecend()

2013-10-28 Thread David Miller
From: Zhi Yong Wu Date: Mon, 28 Oct 2013 14:01:50 +0800 > From: Zhi Yong Wu > > Signed-off-by: Zhi Yong Wu Applied to net-next -- 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

Re: [PATCH v2 1/3] vxlan: silence one build warning

2013-10-28 Thread David Miller
From: Zhi Yong Wu Date: Mon, 28 Oct 2013 14:01:48 +0800 > From: Zhi Yong Wu > > drivers/net/vxlan.c: In function ‘vxlan_sock_add’: > drivers/net/vxlan.c:2298:11: warning: ‘sock’ may be used uninitialized in > this function [-Wmaybe-uninitialized] > drivers/net/vxlan.c:2275:17: note: ‘sock’

Re: [PATCH] net, mc: fix the incorrect comments in two mc-related functions

2013-10-28 Thread David Miller
From: Zhi Yong Wu Date: Mon, 28 Oct 2013 16:15:50 +0800 > From: Zhi Yong Wu > > Signed-off-by: Zhi Yong Wu Applied to net-next. -- 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

Re: [BUG] 3.12.0-rcX IPv6 panic

2013-10-28 Thread Bob Tracy
On Mon, Oct 21, 2013 at 06:40:41PM -0500, Bob Tracy wrote: > On Mon, Oct 21, 2013 at 05:52:52PM +0200, Hannes Frederic Sowa wrote: > > On Mon, Oct 21, 2013 at 08:18:46AM -0500, Bob Tracy wrote: > > > Actually, a regression: the 3.11 kernel is rock-solid stable on my > > > Alpha. > > > > > >

[PATCH 04/11 V2] Staging: bcm: Replaces UCHAR with u8 in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replace "UCHAR" with "u8" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h | 82 - 1 file changed, 41 insertions(+), 41 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index

[PATCH 00/11 V2] Staging: bcm: Cleanup Adapter.h

2013-10-28 Thread Kevin McKinney
These patches fix several issues in Adapter.h Kevin McKinney (11): Staging: bcm: Fix WARNING: space prohibited before semicolon. Staging: bcm: Remove typedef for _U_IP_ADDRESS and call directly. Staging: bcm: Replace USHORT with unsigned short in Adapter.h Staging: bcm: Replaces UCHAR

[PATCH 02/11 V2] Staging: bcm: Remove typedef for _U_IP_ADDRESS and call directly.

2013-10-28 Thread Kevin McKinney
This patch removes typedef for _U_IP_ADDRESS, and changes the name of the struct to bcm_ip_address. In addition, any calls to struct "U_IP_ADDRESS" are changed to call directly. Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h |8 1 file changed, 4 insertions(+), 4

[PATCH 09/11 V2] Staging: bcm: Replace UINT with unsigned int in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "UINT" with "unsigned int" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h | 84 - 1 file changed, 42 insertions(+), 42 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h

[PATCH 10/11 V2] Staging: bcm: Replace PVOID with void * in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "PVOID" with "void *" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index

[PATCH 07/11 V2] Staging: bcm: Replace B_UINT8 with u8 in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "B_UINT8" with "u8" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index 01788d3..f7620ab 100644 ---

RE: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Bhushan Bharat-R65777
> -Original Message- > From: Wood Scott-B07421 > Sent: Tuesday, October 29, 2013 9:11 AM > To: Bhushan Bharat-R65777 > Cc: Wood Scott-B07421; Alex Williamson; Kim Phillips; Yoder Stuart-B08248; > christoffer.d...@linaro.org; linux-kernel@vger.kernel.org; >

[PATCH 08/11 V2] Staging: bcm: Replace UINT32 with u32 in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "UINT32" with "u32" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index f7620ab..35f480a 100644 ---

[PATCH 06/11 V2] Staging: bcm: Replace B_UINT16 with u16 in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "B_UINT16" with "u16" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index a5dab86..01788d3 100644 ---

[PATCH 11/11 V2] Staging: bcm: Replace LARGE_INTEGER with u64 in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "LARGE_INTEGER" with "u64" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index 7055b83..c373bc0 100644 ---

[PATCH 05/11 V2] Staging: bcm: Replace ULONG with unsigned long or u32 in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "ULONG" with "unsigned long", or "u32" for ipv4 addresses, in Adapter.h. For ipv4 addresses, all formating is change to match u32 definition. Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h | 42 -

[PATCH 01/11 V2] Staging: bcm: Fix WARNING: space prohibited before semicolon.

2013-10-28 Thread Kevin McKinney
This patch removes a space before semicolon as specified by checkpatch.pl. Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index d6c9630..11fd7f1

[PATCH 03/11 V2] Staging: bcm: Replace USHORT with unsigned short in Adapter.h

2013-10-28 Thread Kevin McKinney
This patch replaces "USHORT" with "unsigned short" in Adapter.h Signed-off-by: Kevin McKinney --- drivers/staging/bcm/Adapter.h | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h

[PATCH] pinctrl: remove minor dead code

2013-10-28 Thread Michael Opdenacker
This removes a test whether the 'desc' variable is NULL. This possibility has already been eliminated by the below test earlier in the loop: if (desc == NULL) { dev_warn(pctldev->dev, "could not get pin desc for pin %d\n",

[PATCH] vexpress: remove declaration of vexpress_clk_of_init()

2013-10-28 Thread Kefeng Wang
After commit:6e973d2c(clk: vexpress: Add separate SP810 driver), vexpress_clk_of_init() is unnecessary and removed, so kill it. Signed-off-by: Kefeng Wang --- include/linux/vexpress.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/linux/vexpress.h b/include/linux/vexpress.h index

RE: [RFC][PATCHv5 4/4] Documentation: Add device tree bindings for Freescale FTM PWM.

2013-10-28 Thread Xiubo Li-B47053
> > This adds the Document for Freescale FTM PWM driver under > > Documentation/devicetree/bindings/pwm/. > > > > Signed-off-by: Xiubo Li > > --- > > .../devicetree/bindings/pwm/pwm-fsl-ftm.txt| 34 > ++ > > 1 file changed, 34 insertions(+) > > create mode 100644 > >

Re: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Scott Wood
On Mon, 2013-10-28 at 22:38 -0500, Bhushan Bharat-R65777 wrote: > > > -Original Message- > > From: Wood Scott-B07421 > > Sent: Monday, October 28, 2013 11:40 PM > > To: Alex Williamson > > Cc: Kim Phillips; Bhushan Bharat-R65777; Wood Scott-B07421; Yoder > > Stuart-B08248; > >

RE: [PATCH 3/4] VFIO: pci: amend vfio-pci for explicit binding via sysfs only

2013-10-28 Thread Bhushan Bharat-R65777
> -Original Message- > From: Wood Scott-B07421 > Sent: Monday, October 28, 2013 11:40 PM > To: Alex Williamson > Cc: Kim Phillips; Bhushan Bharat-R65777; Wood Scott-B07421; Yoder > Stuart-B08248; > christoffer.d...@linaro.org; linux-kernel@vger.kernel.org; >

Re: [PATCH 1/3] sched: Fix nohz_kick_needed to consider the nr_busy of the parent domain's group

2013-10-28 Thread Preeti U Murthy
Hi Peter, On 10/28/2013 07:20 PM, Peter Zijlstra wrote: > On Thu, Oct 24, 2013 at 01:37:38PM +0530, Preeti U Murthy wrote: >> kernel/sched/core.c |5 + >> kernel/sched/fair.c | 38 -- >> kernel/sched/sched.h |1 + >> 3 files changed, 26

Re: [3.11.4] Thunderbolt/PCI unplug oops in pci_pme_list_scan

2013-10-28 Thread Bjorn Helgaas
On Wed, Oct 16, 2013 at 2:21 PM, Bjorn Helgaas wrote: > On Tue, Oct 15, 2013 at 03:44:52AM +0100, Matthew Garrett wrote: >> On Mon, Oct 14, 2013 at 05:50:38PM -0600, Bjorn Helgaas wrote: >> > [+cc Rafael, Mika, Kirill, linux-pci] >> > >> > On Mon, Oct 14, 2013 at 4:47 PM, Andreas Noever >> >

[PATCH] net/cdc_ncm: fix null pointer panic at usbnet_link_change

2013-10-28 Thread Du, ChangbinX
From: "Du, Changbin" In cdc_ncm_bind() function, it call cdc_ncm_bind_common() to setup usb. But cdc_ncm_bind_common() may meet error and cause usbnet_disconnect() be called which calls free_netdev(net). Thus usbnet structure(alloced with net_device structure) will be freed,too. So we cannot

Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Masami Hiramatsu
(2013/10/29 2:48), Pekka Enberg wrote: > On 10/28/13 7:31 PM, Srikar Dronamraju wrote: But what if a system has both 32 bit libc and 64 bit libc? Wont we could end up with 2 libc:setjmp? Should we give some more intelligence into perf to choose the 64 bit libc over 32 bit one?

Re: State of "perf: Add a new sort order: SORT_INCLUSIVE"

2013-10-28 Thread Namhyung Kim
Hi Arun, On Mon, 28 Oct 2013 09:43:21 -0700, Arun Sharma wrote: > On 10/28/13 2:29 AM, Rodrigo Campos wrote: >> On Mon, Oct 28, 2013 at 06:09:30PM +0900, Namhyung Kim wrote: >>> On Mon, 28 Oct 2013 08:42:44 +, Rodrigo Campos wrote: On Mon, Oct 28, 2013 at 02:09:49PM +0900, Namhyung Kim

[PATCH 2/2] [trivial]doc:alsa: Fix typo in documentation/alsa

2013-10-28 Thread Masanari Iida
Correct spelling typo in documentation/alsa Signed-off-by: Masanari Iida --- Documentation/sound/alsa/ALSA-Configuration.txt | 2 +- Documentation/sound/alsa/Audiophile-Usb.txt | 2 +- Documentation/sound/alsa/CMIPCI.txt | 2 +- Documentation/sound/alsa/compress_offload.txt |

Re: RFC: paravirtualizing perf_clock

2013-10-28 Thread David Ahern
On 10/28/13 7:15 AM, Peter Zijlstra wrote: Any suggestions on how to do this and without impacting performance. I noticed the MSR path seems to take about twice as long as the current implementation (which I believe results in rdtsc in the VM for x86 with stable TSC). So assuming all the TSCs

Re: Re: [PATCH v4 2/3] Support for perf to probe into SDT markers:

2013-10-28 Thread Masami Hiramatsu
(2013/10/29 2:31), Srikar Dronamraju wrote: >>> >>> But what if a system has both 32 bit libc and 64 bit libc? >>> Wont we could end up with 2 libc:setjmp? >>> Should we give some more intelligence into perf to choose the 64 bit >>> libc over 32 bit one? >> >> You can just trace both of them by

linux-next: manual merge of the vfs tree with the v9fs tree

2013-10-28 Thread Stephen Rothwell
Hi Al, Today's linux-next merge of the vfs tree got conflicts in fs/9p/vfs_file.c, fs/9p/vfs_inode_dotl.c and fs/9p/vfs_inode.c between commit 94876b5bb6a8 ("9P: introduction of a new cache=mmap model") from the v9fs tree and commit 2b052ff59861 ("9p: make v9fs_cache_inode_ {get,put,set}_cookie

[PATCH v4] watchdog: w83627hf: Convert to watchdog infrastructure

2013-10-28 Thread Guenter Roeck
Signed-off-by: Guenter Roeck --- v4: Restore 'nowayout' module parameter The changes cause a trivial conflict with 'watchdog: w83627hf: Auto-detect IO address and supported chips'. Please let me know if I should re-send the entire series. drivers/watchdog/Kconfig|1 +

Re: [patch 3/6] Cleanup efi_enter_virtual_mode function

2013-10-28 Thread Dave Young
> > > > 3. Then I merged the 'efi' branch of > > git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git ontop. > > I used the patches from mailbox directly. > Fixed an conflict and merge them one by one. I remember I tried your efi branch, but the code is different from what you sent to list,

[PATCH v2] kernel/modsign_certificate.S: use real contents instead of macro GLOBAL()

2013-10-28 Thread Chen Gang
If a macro is only used within 2 times, and also its contents are within 2 lines, recommend to expand it to shrink code line. For our case, the macro is not portable either: some architectures' assembler may use another character to mark newline in a macro (e.g. '`' for arc), which will cause

Re: [PATCH] commit: Add -f, --fixes option to add Fixes: line

2013-10-28 Thread Jeff King
On Mon, Oct 28, 2013 at 11:10:13PM +0100, Thomas Rast wrote: > * In your list > > > Fixes: > > Reported-by: > > Suggested-by: > > Improved-by: > > Acked-by: > > Reviewed-by: > > Tested-by: > > Signed-off-by: > > and I might add > > Cherry-picked-from: > Reverts: > >

Re: [PATCH] commit: Add -f, --fixes option to add Fixes: line

2013-10-28 Thread Jeff King
On Mon, Oct 28, 2013 at 12:29:32PM +0100, Johan Herland wrote: > > A hook-based solution could do this. But a built-in "all-purpose" > > handler like "footer.Fixes.arg=commit", which was intended to be > > reusable, wouldn't be able to do such footer-specific extra work without > > having to

[PATCH] ARM: spear: fix return value check in spear_setup_of_timer()

2013-10-28 Thread Wei Yongjun
From: Wei Yongjun In case of error, the function clk_get_sys() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- arch/arm/mach-spear/time.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH] befs: fix return value check in befs_iget()

2013-10-28 Thread Wei Yongjun
From: Wei Yongjun In case of error, the function iget_locked() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun --- fs/befs/linuxvfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

Re: [PATCH] ARM: Support arch_irq_work_raise() via self IPIs

2013-10-28 Thread Kevin Hilman
Stephen Boyd writes: > This will allow the scheduler tick to be restarted if we're in > full NOHZ mode. > > Cc: Kevin Hilman > Cc: Frederic Weisbecker > Signed-off-by: Stephen Boyd Minor nit, but I'd prefer a more verbose changelog (I forget things quickly and like to rely on changelogs for

[PATCH v2] arch: use ASM_NL instead of ';' for assembler new line character in the macro

2013-10-28 Thread Chen Gang
For some assemblers, they use another character as newline in a macro (e.g. arc uses '`'), so for generic assembly code, need use ASM_NL (a macro) instead of ';' for it. Also need notice about code styles ('\t' for each line). Signed-off-by: Chen Gang --- arch/arc/include/asm/linkage.h |2

Re: [PATCH] 9p: unsigned/signed wrap in p9/unix modes.

2013-10-28 Thread Geyslan Gregório Bem
2013/10/28 Geyslan Gregório Bem > > 2013/10/27 Eric Van Hensbergen >> >> Looks like the right approach. The one other optional thing I mentioned was >> support for passing NULL for rdev and not trying to parse the device info >> when rdev == NULL. Its a very slight optimization in the grand

Re: [PATCH] arch: use ASM_NL instead of ';' for assembler new line character in the macro

2013-10-28 Thread Chen Gang
On 10/29/2013 09:04 AM, Chen Gang wrote: > On 10/28/2013 09:45 PM, Vineet Gupta wrote: >> On 10/28/2013 04:00 PM, Chen Gang wrote: >>> For some toolchains, they use another character as newline in a macro >>> (e.g. arc uses '`'), so for generic assembly code, need use ASM_NL (a >>> macro) instead

Re: Re: [tpmdd-devel] [PATCH] tpm: MAINTAINERS: Add myself as tpm maintainer

2013-10-28 Thread Rajiv Andrade
Hey Peter, Yes, a cleanup there is way welcome. Please remove my name indeed, as it doesn't make sense at this point, the lists should suffice. I'll keep looking at the patches and try to review them for sure. Cheers and thanks again for stepping in. On Wed, Oct 23, 2013 at 9:10 PM, Peter

Re: [PATCH 4/6] edac: Document Krait L1/L2 EDAC driver binding

2013-10-28 Thread Mark Rutland
On Tue, Oct 29, 2013 at 12:31:28AM +, Stephen Boyd wrote: > The Krait L1/L2 error reporting device is made up of two > interrupts, one per-CPU interrupt for the L1 caches and one > interrupt for the L2 cache. > > Cc: > Signed-off-by: Stephen Boyd > --- >

Re: [PATCH 2/4] wl1251: move power GPIO handling into the driver

2013-10-28 Thread Mark Brown
On Tue, Oct 29, 2013 at 12:26:26AM +0100, Sebastian Reichel wrote: > 1. The pin is named PMEN in the Nokia N900 schematics > 2. PMEN is described as "Power management enable - system shutdown" >in a crippled datasheet of the wl1253, which I found in the internet. > I don't think this is

Re: [PATCH 3/6] ARM: Add Krait L2 accessor functions

2013-10-28 Thread Mark Rutland
On Tue, Oct 29, 2013 at 01:21:57AM +, Stephen Boyd wrote: > On 10/28/13 18:19, Mark Rutland wrote: > > On Tue, Oct 29, 2013 at 12:31:27AM +, Stephen Boyd wrote: > >> Qualcomm's Krait CPUs have a handful of L2 cache controller > >> registers that live behind a cp15 based indirection

Re: [PATCH 5/6] edac: Add support for Krait CPU cache error detection

2013-10-28 Thread Mark Rutland
On Tue, Oct 29, 2013 at 12:31:29AM +, Stephen Boyd wrote: > Add support for the Krait CPU cache error detection. This is a > simplified version of the code originally written by Stepan > Moskovchenko[1] ported to the EDAC device framework. > > [1] >

Re: [PATCH 3/6] ARM: Add Krait L2 accessor functions

2013-10-28 Thread Stephen Boyd
On 10/28/13 18:19, Mark Rutland wrote: > On Tue, Oct 29, 2013 at 12:31:27AM +, Stephen Boyd wrote: >> Qualcomm's Krait CPUs have a handful of L2 cache controller >> registers that live behind a cp15 based indirection register. >> First you program the indirection register (l2cpselr) to point

Re: [PATCH 3/6] ARM: Add Krait L2 accessor functions

2013-10-28 Thread Mark Rutland
On Tue, Oct 29, 2013 at 12:31:27AM +, Stephen Boyd wrote: > Qualcomm's Krait CPUs have a handful of L2 cache controller > registers that live behind a cp15 based indirection register. > First you program the indirection register (l2cpselr) to point > the L2 'window' register (l2cpdr) at what

[PATCH v2]tty:vt: Add NULL check to return value of kzalloc()

2013-10-28 Thread Wang YanQing
In extreme situation kzalloc in con_init may return NULL, so we should check it to prevent OOPS! Signed-off-by: Wang YanQing Reported-by: RUC_SoftSec --- Changes: v1-v2: 1:fix warning and error reported by scripts/checkpatch.pl drivers/tty/vt/vc_screen.c | 6 -- drivers/tty/vt/vt.c

Re: [PATCH] arc: include: asm: remove '__init' for setup_processor() and arc_init_IRQ()

2013-10-28 Thread Chen Gang
On 10/28/2013 09:56 PM, Vineet Gupta wrote: > On 10/23/2013 07:42 AM, Chen Gang wrote: >> They haven't '__init' in definition, but has '__init' in declaration. >> And normal function start_kernel_secondary() may call setup_processor() >> which will call arc_init_IRQ(). >> >> So need remove

Re: [PATCH] arc: kernel: export symbol for pm_power_off in reset.c

2013-10-28 Thread Chen Gang
On 10/28/2013 10:09 PM, Vineet Gupta wrote: > On 10/28/2013 09:19 AM, Chen Gang wrote: >> Need export symbol for it, or can not pass compiling, the related error >> with allmodconfig: >> >> MODPOST 2994 modules >> ERROR: "pm_power_off" [drivers/mfd/retu-mfd.ko] undefined! >> ERROR:

Re: [PATCH] arc: kernel: export symbol for save_stack_trace() in stacktrace.c

2013-10-28 Thread Chen Gang
On 10/28/2013 10:06 PM, Vineet Gupta wrote: > On 10/28/2013 08:30 AM, Chen Gang wrote: >> Need export its symbol just like other architectures done, or can not >> pass compiling with allmodconfig, the related error: >> >> MODPOST 2994 modules >> ERROR: "save_stack_trace"

Re: [PATCH] arc: kernel: kgdb: add default implementation for kgdb_roundup_cpus()

2013-10-28 Thread Chen Gang
On 10/28/2013 09:55 PM, Vineet Gupta wrote: > On 10/24/2013 09:21 AM, Chen Gang wrote: >> arc supports kgdb, but need update -- add function kgdb_roundup_cpus(), >> or can not pass compiling. At present, add the simple generic one just >> like other architectures(e.g. tile, mips ...). >> >> The

[RFC/PATCH 2/2] usb: ffs/dwc3: pad epout buffer size when not aligned to maxpacketsize

2013-10-28 Thread David Cohen
DWC3 requires buffer size to be aligned to maxpacketsize of an out endpoint. ffs_epfile_io() needs to pad epout buffer to match above condition if DWC3 controller is used. This patch solves an specific situation but a more generic solution should be found. Signed-off-by: David Cohen ---

[RFC/PATCH 1/2] usb: gadget: re-introduce gadget_is_dwc3()

2013-10-28 Thread David Cohen
gadget_is_dwc3() is necessary to check whether we are running on Desineware USB3 DRD controller. This macro was previously removed by commit ed9cbda63d45638b69ce62412e3a3c7b00644835 due to it wasn't needed anymore. We're adding it again as things have changed. Signed-off-by: David Cohen ---

Re: [PATCH] arc: kernel: remove '__init' for first_lines_of_secondary()

2013-10-28 Thread Chen Gang
On 10/28/2013 09:53 PM, Vineet Gupta wrote: > On 10/23/2013 07:46 AM, Chen Gang wrote: >> first_lines_of_secondary() is a '__init' function, but it may be called >> by __cpu_up() by _cpu_up() by cpu_up() which is a normal export symbol >> function. So recommend to remove '__init'. >> >> The

[RFC/PATCH 0/2] patchset to workaround f_fs for DWC3

2013-10-28 Thread David Cohen
Hi, These patches are a proposal to workaround f_fs when using DWC3 controller. Since DWC3 requires epout buffer size to be aligned to maxpacketsize, f_fs needs to pad buffer size to match the above case. This change is necessary to make Android's adbd service to work with f_fs instead of

Re: [f2fs-dev 4/5] f2fs: Key functions to handle inline data

2013-10-28 Thread Jaegeuk Kim
Hi, 2013-10-29 (화), 01:20 +0800, Huajun Li: > On Mon, Oct 28, 2013 at 8:43 PM, Jaegeuk Kim wrote: > > Hi, > > > > 2013-10-26 (토), 00:01 +0800, Huajun Li: > >> From: Huajun Li > >> > >> Functions to implement inline data read/write, and move inline data to > >> normal data block when file size

Re: [PATCHv2 1/3] Input: twl4030-keypad - add device tree support

2013-10-28 Thread Sebastian Reichel
On Mon, Oct 28, 2013 at 01:42:47AM -0500, Kumar Gala wrote: > > +This binding is based on the matrix-keymap binding with the following > > +changes: > > Maybe be a bit more specific and say 'based on the input/matrix-keymap.txt > binding'.. OK. > > + * keypad,num-rows and keypad,num-columns

Re: [PATCH] arch: use ASM_NL instead of ';' for assembler new line character in the macro

2013-10-28 Thread Chen Gang
On 10/28/2013 09:45 PM, Vineet Gupta wrote: > On 10/28/2013 04:00 PM, Chen Gang wrote: >> For some toolchains, they use another character as newline in a macro >> (e.g. arc uses '`'), so for generic assembly code, need use ASM_NL (a >> macro) instead of ';' for it. >> >> Since "linux/export.h" are

Re: [f2fs-dev 3/5] f2fs: Add a new function: f2fs_reserve_block()

2013-10-28 Thread Jaegeuk Kim
Hi Huajun, 2013-10-29 (화), 00:53 +0800, Huajun Li: > Hi Jaegeuk, > > Thanks for your kindly review and comments. > > On Mon, Oct 28, 2013 at 8:28 PM, Jaegeuk Kim wrote: > > 2013-10-28 (월), 21:16 +0900, Jaegeuk Kim: > > Hi, > > > >> > >> 2013-10-26 (토), 00:01 +0800, Huajun Li: > >> > From:

[PATCH] tpm/tpm_ibmvtpm: fix unreachable code warning (smatch warning)

2013-10-28 Thread Peter Huewe
smatch complains: /data/data-old/linux-2.6/drivers/char/tpm/tpm_ibmvtpm.c:510 ibmvtpm_crq_process() info: ignoring unreachable code. -> The return is not necessary here, remove it Signed-off-by: Peter Huewe --- drivers/char/tpm/tpm_ibmvtpm.c |1 - 1 files changed, 0 insertions(+), 1

Re: [PATCH v6 3/3] Staging: zram: Fix decrement of variable by calling bdput()

2013-10-28 Thread Minchan Kim
On Mon, Oct 28, 2013 at 05:54:39PM +0530, Rashika Kheria wrote: > As suggested by Jerome Marchand "The code in reset_store get the block device > (bdget_disk()) but it does not put it (bdput()) when it's done using it. > The usage count is therefor incremented but never decremented." > > Hence,

Re: [PATCH v6 2/3] Staging: zram: Fix variable dereferenced before check

2013-10-28 Thread Minchan Kim
On Mon, Oct 28, 2013 at 05:53:40PM +0530, Rashika Kheria wrote: > This patch fixes the following Smatch warning in zram_drv.c- > drivers/staging/zram/zram_drv.c:899 > destroy_device() warn: variable dereferenced before check 'zram->disk' (see > line 896) > > Signed-off-by: Rashika Kheria

Re: [PATCH v6 1/3] Staging: zram: Fix access of NULL pointer

2013-10-28 Thread Minchan Kim
Hello Rashika, On Mon, Oct 28, 2013 at 05:51:38PM +0530, Rashika Kheria wrote: > This patch fixes the bug in reset_store caused by accessing NULL pointer. > Hence, It introduces a check for bdev. It also removes unnecessary check > of bdev for fsync_bdev(). It's better than old but I still want

Re: [PATCH v3 08/10] mfd: pm8x41: add support for Qualcomm 8x41 PMICs

2013-10-28 Thread Stephen Boyd
On 10/28/13 11:12, Josh Cartwright wrote: > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 914c3d1..0a288cb 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -476,6 +476,16 @@ config MFD_PM8XXX_IRQ > This is required to use certain other PM 8xxx features,

[PATCH] swiotlb-xen: fix error code returned by xen_swiotlb_map_sg_attrs

2013-10-28 Thread Stefano Stabellini
map_sg returns the number of elements mapped, not a dma_addr_t. In case of error return 0, not DMA_ERROR_CODE. Signed-off-by: Stefano Stabellini --- drivers/xen/swiotlb-xen.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/xen/swiotlb-xen.c

[PATCH 6/6] ARM: dts: msm: Add Krait edac node

2013-10-28 Thread Stephen Boyd
Cc: David Brown Signed-off-by: Stephen Boyd --- This is based on the dts updates I sent two weeks ago to add mmio architected timesr to qcom-msm8974.dtsi. arch/arm/boot/dts/qcom-msm8974.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi

[PATCH 5/6] edac: Add support for Krait CPU cache error detection

2013-10-28 Thread Stephen Boyd
Add support for the Krait CPU cache error detection. This is a simplified version of the code originally written by Stepan Moskovchenko[1] ported to the EDAC device framework. [1] https://www.codeaurora.org/cgit/quic/la/kernel/msm/tree/arch/arm/mach-msm/cache_erp.c?h=msm-3.4 Cc: Stepan

[PATCH 3/6] ARM: Add Krait L2 accessor functions

2013-10-28 Thread Stephen Boyd
Qualcomm's Krait CPUs have a handful of L2 cache controller registers that live behind a cp15 based indirection register. First you program the indirection register (l2cpselr) to point the L2 'window' register (l2cpdr) at what you want to read/write. Then you read/write the 'window' register to do

[PATCH 4/6] edac: Document Krait L1/L2 EDAC driver binding

2013-10-28 Thread Stephen Boyd
The Krait L1/L2 error reporting device is made up of two interrupts, one per-CPU interrupt for the L1 caches and one interrupt for the L2 cache. Cc: Signed-off-by: Stephen Boyd --- .../devicetree/bindings/arm/qcom,krait-cache-erp.txt | 16 1 file changed, 16 insertions(+)

  1   2   3   4   5   6   7   8   9   10   >