[PATCH 1/2] perf annotate: use arch->objdump.comment_char in dec__parse()

2016-11-30 Thread Kim Phillips
Presume neglected in commit 786c1b5 "perf annotate: Start supporting cross arch annotation". This doesn't fix a bug since none of the affected arches support parsing dec/inc instructions yet. Signed-off-by: Kim Phillips Cc: Arnaldo Carvalho de Melo --- tools/perf/util/annot

[PATCH 2/2] perf annotate: AArch64 support

2016-11-30 Thread Kim Phillips
▒ Press 'h' for help on key bindings Signed-off-by: Chris Ryder Signed-off-by: Kim Phillips Cc: Pawel Moll Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Cc: Alexander Shishkin Cc: linux-perf-us...@vger.kernel.org Cc: Will Deacon

Re: [PATCH] perf c2c report: Properly check data presence in rb_tree loop

2016-10-21 Thread Kim Phillips
On Fri, 21 Oct 2016 02:17:06 +0200 Jiri Olsa wrote: > On Thu, Oct 20, 2016 at 06:02:24PM -0500, Kim Phillips wrote: > > SNIP > > > (gdb) bt > > #0 c2c_browser__update_nr_entries (hb=, hb=) > > at builtin-c2c.c:2142 > > #1 perf_c2c__hists_browse (hists=0x8

Re: [GIT PULL 00/52] New Tool: perf c2c

2016-10-20 Thread Kim Phillips
On Thu, 20 Oct 2016 12:03:44 -0300 Arnaldo Carvalho de Melo wrote: > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git > tags/perf-c2c-for-mingo-20161020 Running this as a normal user may not be the best thing to do, but I'm pretty sure the TUI c2c report shouldn't sigsegv: -

Re: [PATCH v7 6/6] perf annotate: cross arch annotate support fixes for ARM

2016-10-10 Thread Kim Phillips
On Mon, 10 Oct 2016 19:16:16 +0530 Ravi Bangoria wrote: > On Wednesday 05 October 2016 05:04 PM, Arnaldo Carvalho de Melo wrote: > > Em Wed, Sep 21, 2016 at 09:17:56PM +0530, Ravi Bangoria escreveu: > >> From: Kim Phillips > >> > >> For ARM we remove the l

Re: [PATCH] perf: remove orphan DEBUG_PERF_USE_VMALLOC

2016-09-27 Thread Kim Phillips
On Tue, 27 Sep 2016 07:53:56 -0700 Vineet Gupta wrote: > This doesn't seem to be used anywhere I thought so too, until I got this response for an equivalent patch: https://lkml.org/lkml/2016/8/31/53 Kim

Re: [PATCH v7 0/6] perf annotate: Cross arch support + few fixes

2016-09-22 Thread Kim Phillips
On Thu, 22 Sep 2016 10:48:13 +0530 Ravi Bangoria wrote: > On Thursday 22 September 2016 01:04 AM, Kim Phillips wrote: > > On Wed, 21 Sep 2016 21:17:50 +0530 > > Ravi Bangoria wrote: > > > >> Kim, I don't have arm test machine. Can you please help me to test &

Re: [PATCH 47/61] perf c2c report: Add cacheline browser

2016-09-21 Thread Kim Phillips
On Wed, 21 Sep 2016 14:55:40 +0200 Jiri Olsa wrote: > On Wed, Sep 21, 2016 at 10:21:55AM +0200, Jiri Olsa wrote: > > On Tue, Sep 20, 2016 at 03:10:07PM -0500, Kim Phillips wrote: > > > this fails to build on systems without slang: > > > > > > CC b

Re: [PATCH v7 0/6] perf annotate: Cross arch support + few fixes

2016-09-21 Thread Kim Phillips
On Wed, 21 Sep 2016 21:17:50 +0530 Ravi Bangoria wrote: > Kim, I don't have arm test machine. Can you please help me to test > this on arm. This works for me: hitting return on return instructions yields "Invalid jump offset", but I'll get that later. Thanks, Kim

perf tests kmod-path: don't fail if compressed modules aren't supported

2016-09-20 Thread Kim Phillips
__kmod_path__parse() uses is_supported_compression() to determine and parse out compressed module file extensions. On systems without zlib, this test fails and __kmod_path__parse() continues to strcmp "ko" with "gz". Don't do this on those systems. Signed-off-by: Kim

Re: [PATCH 47/61] perf c2c report: Add cacheline browser

2016-09-20 Thread Kim Phillips
On Mon, 19 Sep 2016 15:09:56 +0200 Jiri Olsa wrote: > + /* reset abort key so that it can get Ctrl-C as a key */ > + SLang_reset_tty(); > + SLang_init_tty(0, 0, 0); this fails to build on systems without slang: CC builtin-c2c.o builtin-c2c.c: In function ‘perf_c2c__browse_ca

Re: [PATCH 2/2] tools include: Add mman macros needed by perf for all arch

2016-09-16 Thread Kim Phillips
ne to 0. > > This resolves the build issue on powerpc, thanks! ditto for arm > Tested-by: Naveen N. Rao Tested-by: Kim Phillips Thanks, Kim

[PATCH 1/2] perf_event: remove unused DEBUG_PERF_USE_VMALLOC

2016-08-30 Thread Kim Phillips
f on powerpc" was trying to do with it either. Signed-off-by: Kim Phillips Cc: Peter Zijlstra Cc: Michael Ellerman --- init/Kconfig | 13 - 1 file changed, 13 deletions(-) diff --git a/init/Kconfig b/init/Kconfig index cac3f09..934a61f 100644 --- a/init/Kconfig +++ b/init/Kc

[PATCH 2/2] arm64: don't select PERF_USE_VMALLOC by default

2016-08-30 Thread Kim Phillips
Any arm64 based parts that have cache aliasing issues can set it manually. Apparently dragged in from ARM(32) defaults in commit 8c2c3df "arm64: Build infrastructure". Signed-off-by: Kim Phillips Cc: Catalin Marinas --- arch/arm64/Kconfig | 1 - 1 file changed, 1 deletion(-) di

[PATCH] perf annotate: cross arch annotate support fixes for ARM

2016-08-26 Thread Kim Phillips
For ARM we remove the list that contains non-arm insns, and instead add more maintainable branch instruction regex logic. Signed-off-by: Kim Phillips Acked-by: Ravi Bangoria Cc: Namhyung Kim --- tools/perf/util/annotate.c | 177 + 1 file changed, 67

Re: [PATCH v6 2/7] perf annotate: Add cross arch annotate support

2016-08-23 Thread Kim Phillips
On Tue, 23 Aug 2016 11:17:16 +0900 Namhyung Kim wrote: > On Tue, Aug 23, 2016 at 8:01 AM, Kim Phillips wrote: > > On Fri, 19 Aug 2016 18:29:33 +0530 > > Ravi Bangoria wrote: > > > >> Changes in v6: > >> - Instead of adding only those instructions defi

Re: [PATCH v6 2/7] perf annotate: Add cross arch annotate support

2016-08-22 Thread Kim Phillips
On Fri, 19 Aug 2016 18:29:33 +0530 Ravi Bangoria wrote: > Changes in v6: > - Instead of adding only those instructions defined in #ifdef __arm__, > add all instructions from default table to arm table. .. > +static struct ins instructions_arm[] = { > { .name = "add", .ops = &mov_op

Re: [PATCH 1/3] perf/e6500: Make event translations available in sysfs

2015-03-27 Thread Kim Phillips
crickets. How do we make progress in this area? (a) can we assume Andi's json format is acceptable? We would like to know this so we don't have to reformat our data more than once. (b) Would an acceptable interim resolution the 'download area' problem be to take Andi's "perf: Add support for fu

[PATCH 2/3] powerpc: make copyuser_power7.S 64-byte cacheline friendly

2015-03-24 Thread Kim Phillips
. We also add an e6500 copyuser to copyloops tests, which requires we copy asm/cache.h into copyloops, just to satisfy the #include: We define the define symbols manually in the testing makefile. Includes a minor spelling fix: desination->destination. Signed-off-by: Kim Phillips Cc: Shuah Kha

[PATCH 3/3] powerpc/e6500: disable POWER7 data cache prefetch and implement our own

2015-03-24 Thread Kim Phillips
20 -t TCP_CRR -P 0 -- -b 8 -D -r 64,100 Signed-off-by: Kim Phillips --- arch/powerpc/lib/copyuser_power7.S | 254 - 1 file changed, 221 insertions(+), 33 deletions(-) diff --git a/arch/powerpc/lib/copyuser_power7.S b/arch/powerpc/lib/copyuser_power7

[PATCH 1/3] powerpc/e6500: set FTR_VMX_COPY

2015-03-24 Thread Kim Phillips
This enables the VMX/ALTIVEC optimised copy-to/from-user code in arch/powerpc/lib/copyuser_power7.S. The e6500 does, and the e5500 does not, have ALTIVEC. Signed-off-by: Kim Phillips --- arch/powerpc/include/asm/cputable.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

[RFC] powerpc: e6500 optimised copy_to_user/copy_from_user

2015-03-12 Thread Kim Phillips
wer-sized transfer performance Signed-off-by: Kim Phillips --- applies to Linus' ToT today (commit 09d35919b06), since Scott's tree seems a bit outdated. arch/powerpc/include/asm/cputable.h | 2 +- arch/powerpc/lib/Makefile | 5 +- arch/powerpc/lib/copyuser_64.S |

Re: [PATCH v2 15/17] crypto: talitos - Implementation of SEC1

2015-03-09 Thread Kim Phillips
On Fri, 6 Mar 2015 17:42:26 +0100 Christophe Leroy wrote: > This patch adds talitos1.c and talitos1.h with all specificities needed > to handle the SEC1 security engine found in MPC885 and MPC8272. > > The SEC1 has several differences with its younger brother SEC2: > * Several bits in registers

Re: [PATCH 2/2] crypto: caam_rng: fix rng_unmap_ctx's DMA_UNMAP size problem

2015-03-06 Thread Kim Phillips
38388 48546ed9 6000 <0fe0> 3c62ff8f 38637fc8 48546ec5 > ---[ end trace e43fd1734d6600df ]--- > > Signed-off-by: Yanjiang Jin > --- Acked-by: Kim Phillips Thanks, Kim -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in th

[PATCH] crypto: powerpc - move files to fix build error

2015-03-06 Thread Kim Phillips
pc/crypto' failed Move the two sha1 spe files under crypto/, and whilst there, rename other powerpc crypto files with underscores to use dashes for consistency. Cc: Markus Stockhausen Signed-off-by: Kim Phillips --- applies to today's cryptodev-2.6. arch/powerpc/crypto

Re: [PATCH 1/2] crypto: caamhash: - fix uninitialized edesc->sec4_sg_bytes field

2015-03-06 Thread Kim Phillips
/0x700 > [effcbfe0] [c00455d8] irq_exit+0x108/0x120 > [effcbff0] [c000f520] call_do_irq+0x24/0x3c > [e95a3e20] [c00059b8] do_IRQ+0xc8/0x170 > [e95a3e50] [c0011bc8] ret_from_except+0x0/0x18 > > Signed-off-by: Yanjiang Jin > --- Acked-by: Kim Phillips Thanks, Kim -

Re: [PATCH 1/3] crypto: caam: fix some compile warnings

2015-03-05 Thread Kim Phillips
On Thu, 5 Mar 2015 09:12:13 +0200 Cristian Stoica wrote: > On 03/04/2015 08:34 PM, Kim Phillips wrote: > > > I don't see how, e.g., for one, dma_map_sg is I/O TLB > > implementation-dependent. > > I'll need some remedial classes on this topic, but for the mom

Re: [PATCH 2/3] crypto: caam_rng: fix rng_unmap_ctx's DMA_UNMAP size problem

2015-03-05 Thread Kim Phillips
On Thu, 5 Mar 2015 10:52:37 +0800 yjin wrote: > On 2015年03月05日 02:36, Kim Phillips wrote: > > On Wed, 4 Mar 2015 13:33:22 +0800 > > yjin wrote: > > > >> On 2015年03月04日 03:31, Kim Phillips wrote: > >>> On Tue, 3 Mar 2015 14:50:52 +0800 > >>>

Re: [PATCH 0/17] crypto: talitos - Add support for SEC1

2015-03-05 Thread Kim Phillips
On Thu, 5 Mar 2015 17:46:05 +0100 Christophe Leroy wrote: > [15/17] crypto: talitos - Implementation of SEC1 ... > [16/17] crypto: talitos - SEC1 bugs on 0 data hash > [17/17] crypto: talitos - Update DT bindings with SEC1 This patchseries doesn't apply, at least on top of Herbert's cryptodev-

Re: [PATCH 2/3] crypto: caam_rng: fix rng_unmap_ctx's DMA_UNMAP size problem

2015-03-04 Thread Kim Phillips
On Wed, 4 Mar 2015 13:33:22 +0800 yjin wrote: > On 2015年03月04日 03:31, Kim Phillips wrote: > > On Tue, 3 Mar 2015 14:50:52 +0800 > > wrote: > > > >> - dma_unmap_single(jrdev, ctx->sh_desc_dma, DESC_RNG_LEN, > >>

Re: [PATCH 1/3] crypto: caam: fix some compile warnings

2015-03-04 Thread Kim Phillips
On Wed, 4 Mar 2015 11:03:28 +0200 Cristian Stoica wrote: > On 03/04/2015 06:57 AM, yjin wrote: > > An alternative is moving the definitions to a ".c" file, but I don't > > think it will be fundamental different. > > I know I am fixing a potential error which doesn't exist now, it seems > > useles

Re: [PATCH 1/3] crypto: caam: fix some compile warnings

2015-03-03 Thread Kim Phillips
On Tue, 3 Mar 2015 14:50:51 +0800 wrote: > This commit is to avoid the below warnings: > > drivers/crypto/caam/sg_sw_sec4.h:88:12: warning: > 'dma_map_sg_chained' defined but not used [-Wunused-function] > static int dma_map_sg_chained(struct device *dev, struct scatterlist *sg, > ^

Re: [PATCH 2/3] crypto: caam_rng: fix rng_unmap_ctx's DMA_UNMAP size problem

2015-03-03 Thread Kim Phillips
On Tue, 3 Mar 2015 14:50:52 +0800 wrote: > - dma_unmap_single(jrdev, ctx->sh_desc_dma, DESC_RNG_LEN, > - DMA_TO_DEVICE); > + dma_unmap_single(jrdev, ctx->sh_desc_dma, > + desc_bytes(ctx->sh_desc), DMA_TO_DEVICE); al

Re: [PATCH v3] powerpc/mm: fix undefined reference to `.__kernel_map_pages' on FSL PPC64

2015-01-28 Thread Kim Phillips
On Wed, 28 Jan 2015 14:22:02 +1100 Michael Ellerman wrote: > On Tue, 2015-01-27 at 18:57 -0800, Andrew Morton wrote: > > On Wed, 28 Jan 2015 10:33:59 +0900 Joonsoo Kim wrote: > > > > > 2015-01-28 10:01 GMT+09:00 Michael Ellerman : > > > > On Mon, 2015-01-2

[PATCH] .mailmap: update Konstantin Khlebnikov's email address

2015-01-26 Thread Kim Phillips
dress in the wrong email address field. Cc: Konstantin Khlebnikov Signed-off-by: Kim Phillips --- .mailmap | 1 + 1 file changed, 1 insertion(+) diff --git a/.mailmap b/.mailmap index d357e1b..0d971cf 100644 --- a/.mailmap +++ b/.mailmap @@ -73,6 +73,7 @@ Juha Yrjola Juha Yrjola Kay Si

Re: [PATCH 2/2] mm: fix undefined reference to `.kernel_map_pages' on PPC builds

2015-01-26 Thread Kim Phillips
On Thu, 22 Jan 2015 10:45:51 +0900 Joonsoo Kim wrote: > From 7cb9d1ed8a785df152cb8934e187031c8ebd1bb2 Mon Sep 17 00:00:00 2001 > From: Joonsoo Kim > Date: Thu, 22 Jan 2015 10:28:58 +0900 > Subject: [PATCH] mm/debug_pagealloc: fix build failure on ppc and some other > archs &g

[PATCH v3] powerpc/mm: fix undefined reference to `.__kernel_map_pages' on FSL PPC64

2015-01-26 Thread Kim Phillips
ed reference to `.__kernel_map_pages' include/linux/mm.h:2076: undefined reference to `.__kernel_map_pages' Makefile:925: recipe for target 'vmlinux' failed make: *** [vmlinux] Error 1 Signed-off-by: Kim Phillips --- v3: - fix wording for hash_utils_64.c implementation pointed out by Mic

Re: [PATCH 2/2] mm: fix undefined reference to `.kernel_map_pages' on PPC builds

2015-01-22 Thread Kim Phillips
On Fri, 23 Jan 2015 08:49:36 +0900 Akinobu Mita wrote: > 2015-01-23 5:41 GMT+09:00 Kim Phillips : > > Thanks. Now I get this: > > > > LD init/built-in.o > > mm/built-in.o: In function `kernel_map_pages': > > include/linux/mm.h:2076: undefined referenc

[PATCH v2] powerpc/fsl_pci: Fix pci stack build bug with FRAME_WARN

2015-01-22 Thread Kim Phillips
efactoring indirect_read_config() to take hose and bus number instead of the 1344-byte struct pci_bus. Signed-off-by: Kim Phillips --- v2: refactor indirect_read_config() instead of kmalloc'ing a struct pci_bus. Scott, I will look deeper into fake_pci_bus() later. arch/powerpc/include/asm/pc

Re: [PATCH] powerpc/fsl_pci: Fix pci stack build bug with FRAME_WARN

2015-01-22 Thread Kim Phillips
On Wed, 21 Jan 2015 21:02:27 -0600 Scott Wood wrote: > On Wed, 2015-01-21 at 20:48 -0600, Kim Phillips wrote: > > On Tue, 20 Jan 2015 18:31:32 -0600 > > Scott Wood wrote: > > > > > On Tue, 2015-01-20 at 14:03 -0600, Kim Phillips wrote: > > > > Fix thi

Re: [PATCH 2/2] mm: fix undefined reference to `.kernel_map_pages' on PPC builds

2015-01-22 Thread Kim Phillips
ith ARCH_SUPPORTS_DEBUG_PAGEALLOC should not build > mm/debug-pagealloc.o. I attach the patch to remove old declaration. > I hope it will fix Kim's problem. > > -->8-- > From 7cb9d1ed8a785df152cb8934e187031c8ebd1bb2 Mon Sep 17 00:00:00 2001 > From:

Re: [PATCH] powerpc/fsl_pci: Fix pci stack build bug with FRAME_WARN

2015-01-21 Thread Kim Phillips
On Tue, 20 Jan 2015 18:31:32 -0600 Scott Wood wrote: > On Tue, 2015-01-20 at 14:03 -0600, Kim Phillips wrote: > > Fix this: > > > > CC arch/powerpc/sysdev/fsl_pci.o > > arch/powerpc/sysdev/fsl_pci.c: In function 'fsl_pcie_check_link': > > arc

[PATCH] powerpc/fsl_pci: Fix pci stack build bug with FRAME_WARN

2015-01-20 Thread Kim Phillips
erting the allocation from the stack to the heap. We use GFP_ATOMIC since this function can be called with interrupts disabled. Signed-off-by: Kim Phillips --- arch/powerpc/sysdev/fsl_pci.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/sysdev/fsl_pc

[PATCH 2/2] mm: fix undefined reference to `.kernel_map_pages' on PPC builds

2015-01-20 Thread Kim Phillips
' mm/built-in.o: In function `prep_new_page': mm/page_alloc.c:933: undefined reference to `.kernel_map_pages' mm/built-in.o: In function `map_pages': mm/compaction.c:61: undefined reference to `.kernel_map_pages' make: *** [vmlinux] Error 1 Signed-off-by: Kim Phillips --- m

[PATCH 1/2] mm/slub: fix typo

2015-01-20 Thread Kim Phillips
Signed-off-by: Kim Phillips --- mm/slub.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/slub.c b/mm/slub.c index fe376fe..a64cc1b 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -2512,7 +2512,7 @@ EXPORT_SYMBOL(kmem_cache_alloc_node_trace); #endif /* - * Slow patch

[PATCH] crypto: caam - don't emit ICV check failures to dmesg

2015-01-20 Thread Kim Phillips
ICV check failures are part of normal operation; leave user notification up to the higher levels, as is done in s/w algorithm implementations. Signed-off-by: Kim Phillips --- drivers/crypto/caam/error.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers

Re: [PATCH 01/16] crypto: caam - Remove unnecessary smp_read_barrier_depends()

2014-11-13 Thread Kim Phillips
arly circular buffer documentation, and left it in for the benefit of the doubt. Now it looks to me it's not necessary, given both sw_idx and tail are just being computed within a lock, and removing both barriers doesn't affect the compiler output, so: Reviewed-by: Kim Phillips Thanks, Kim -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH v2] crypto: caam: fix error reporting

2014-11-05 Thread Kim Phillips
On Wed, 5 Nov 2014 11:21:24 +0200 Cristian Stoica wrote: > The error code returned by hardware is four bits wide with an expected > zero MSB. A hardware error condition where the error code can get between > 0x8 and 0xf will trigger an out of bound array access on the error > message table. > Thi

Re: [PATCH] crypto: caam: fix error reporting

2014-11-04 Thread Kim Phillips
On Tue, 4 Nov 2014 10:57:57 +0200 Cristian Stoica wrote: > Hi Kim, > > >> Actually, our static code analyzer did not see this one. > > > > ok, so the patch technically isn't fixing anything broken, then. > > Are you saying the code isn't broken _because_ a static tool analyser > did not see an

Re: [PATCH] crypto: caam: fix error reporting

2014-11-03 Thread Kim Phillips
On Mon, 3 Nov 2014 11:18:36 +0200 Cristian Stoica wrote: > On 10/31/2014 08:22 PM, Kim Phillips wrote: > > On Fri, 31 Oct 2014 18:57:33 +0200 > > Cristian Stoica wrote: > > > > If this issue was brought up by h/w, the appropriate new error codes > > should be b

Re: [PATCH] crypto: caam: fix error reporting

2014-10-31 Thread Kim Phillips
On Fri, 31 Oct 2014 18:57:33 +0200 Cristian Stoica wrote: > The error code returned by hardware is four bits wide with an expected > zero MSB. A hardware error condition where the error code can get between > 0x8 and 0xf will trigger an out of bound array access on the error > message table. If

Re: [PATCH] driver core: amba: add device binding path 'driver_override'

2014-10-13 Thread Kim Phillips
ce requested by the user. > > [1] > http://lists-archives.com/linux-kernel/28030441-pci-introduce-new-device-binding-path-using-pci_dev-driver_override.html > [2] https://www.redhat.com/archives/libvir-list/2014-April/msg00382.html > > Signed-off-by: Antonios Motakis > ---

Re: [PATCH 1/3 v2] drivers/bus: Added Freescale Management Complex APIs

2014-09-24 Thread Kim Phillips
On Wed, 24 Sep 2014 21:23:59 -0500 German Rivera wrote: > On 09/23/2014 07:49 PM, Kim Phillips wrote: > > On Fri, 19 Sep 2014 17:49:39 -0500 > > "J. German Rivera" wrote: > > > >> + * Delay in microseconds between polling iterations while &g

Re: [PATCH 1/3 v2] drivers/bus: Added Freescale Management Complex APIs

2014-09-23 Thread Kim Phillips
On Fri, 19 Sep 2014 17:49:39 -0500 "J. German Rivera" wrote: > +int mc_get_version(struct fsl_mc_io *mc_io, struct mc_version *mc_ver_info) ... > + err = mc_send_command(mc_io, &cmd); > + if (err) > + return err; > + > + DPMNG_RSP_GET_VERSION(cmd, mc_ver_info); al

Re: [PATCH 0/3 v2] drivers/bus: Freescale Management Complex bus driver patch series

2014-09-22 Thread Kim Phillips
On Mon, 22 Sep 2014 12:59:21 -0500 Yoder Stuart-B08248 wrote: > > From: Kim Phillips [mailto:kim.phill...@freescale.com] > > Sent: Monday, September 22, 2014 11:53 AM > > > > On Fri, 19 Sep 2014 17:49:38 -0500 > > "J. German Rivera" wrote: > > &

Re: [PATCH 0/3 v2] drivers/bus: Freescale Management Complex bus driver patch series

2014-09-22 Thread Kim Phillips
On Fri, 19 Sep 2014 17:49:38 -0500 "J. German Rivera" wrote: > CHANGE HISTORY > > Issues pending resolution not addressed by v2: > - What to do with Doxygen comments in patch 1 It's clear they should be removed. > - Whether to move or not FSL-specific header files added in include/linux, > b

Re: [PATCH 1/4] drivers/bus: Added Freescale Management Complex APIs

2014-09-19 Thread Kim Phillips
On Fri, 19 Sep 2014 13:25:06 -0500 Yoder Stuart-B08248 wrote: > > From: Yoder Stuart-B08248 > > Sent: Thursday, September 18, 2014 7:19 PM > > > > +/** > > > >>> + * @briefDisconnects one endpoint to remove its network link > > > >>> + * > > > >>> + * @param[in] mc_ioPointer to opa

Re: [PATCH 1/4] drivers/bus: Added Freescale Management Complex APIs

2014-09-19 Thread Kim Phillips
On Fri, 19 Sep 2014 14:06:32 -0500 Yoder Stuart-B08248 wrote: > > > > +/** > > > > + * @briefManagement Complex firmware version information > > > > + */ > > > > +#define MC_VER_MAJOR 2 > > > > +#define MC_VER_MINOR 0 > > > > > > code should be adjusted to run on

Re: [PATCH 1/4] drivers/bus: Added Freescale Management Complex APIs

2014-09-19 Thread Kim Phillips
On Thu, 18 Sep 2014 22:05:03 -0500 German Rivera wrote: > On 09/18/2014 03:22 PM, Kim Phillips wrote: > > On Thu, 18 Sep 2014 15:14:03 +0200 > > >>> unnecessarily complicated error path, plus a simpler > >>>> implementation can be made if fn can re

Re: [PATCH 1/4] drivers/bus: Added Freescale Management Complex APIs

2014-09-18 Thread Kim Phillips
On Thu, 18 Sep 2014 15:14:03 +0200 Alexander Graf wrote: > > Am 18.09.2014 um 06:17 schrieb German Rivera : > > > >> On 09/15/2014 06:44 PM, Kim Phillips wrote: > >> On Thu, 11 Sep 2014 12:34:21 -0500 > >> "J. German Rivera" wrote: > >

Re: [PATCH 1/4] drivers/bus: Added Freescale Management Complex APIs

2014-09-16 Thread Kim Phillips
On Mon, 15 Sep 2014 23:31:21 -0500 Scott Wood wrote: > On Mon, 2014-09-15 at 18:44 -0500, Kim Phillips wrote: > > On Thu, 11 Sep 2014 12:34:21 -0500 > > "J. German Rivera" wrote: > > > > > +int mc_get_version(struct fsl_mc_io *mc_i

Re: [PATCH 0/4] drivers/bus: Freescale Management Complex bus driver patch series

2014-09-15 Thread Kim Phillips
On Thu, 11 Sep 2014 12:34:20 -0500 "J. German Rivera" wrote: > This patch series introduces Linux support for the Freescale > Management Complex (fsl-mc) hardware. here are the results of using some tools to check this patchseries: make C=1 CF="-D__CHECK_ENDIAN__": drivers/bus/fsl-mc/fsl_mc_sy

Re: [PATCH 1/4] drivers/bus: Added Freescale Management Complex APIs

2014-09-15 Thread Kim Phillips
On Thu, 11 Sep 2014 12:34:21 -0500 "J. German Rivera" wrote: > From: "J. German Rivera" > > APIs to access the Management Complex (MC) hardware > module of Freescale LS2 SoCs. This patch includes > APIs to check the MC firmware version and to manipulate > DPRC objects in the MC. > > Signed-off

Re: [PATCH] driver core: platform: add device binding path 'driver_override'

2014-07-07 Thread Kim Phillips
On Mon, 2 Jun 2014 21:28:42 -0700 Greg KH wrote: > On Mon, Jun 02, 2014 at 07:42:58PM -0500, Kim Phillips wrote: > > You are the platform driver core maintainer: can you apply this to > > your driver-core tree now? > > Yes, I will after this merge window ends, it's to

Re: [PATCH] driver core: platform: add device binding path 'driver_override'

2014-06-25 Thread Kim Phillips
On Mon, 2 Jun 2014 21:28:42 -0700 Greg KH wrote: > On Mon, Jun 02, 2014 at 07:42:58PM -0500, Kim Phillips wrote: > > You are the platform driver core maintainer: can you apply this to > > your driver-core tree now? > > Yes, I will after this merge window ends, it's to

[PATCH 2/2 v2] ARM: KVM: user_mem_abort: support stage 2 MMIO page mapping

2014-06-25 Thread Kim Phillips
From: Kim Phillips A userspace process can map device MMIO memory via VFIO or /dev/mem, e.g., for platform device passthrough support in QEMU. During early development, we found the PAGE_S2 memory type being used for MMIO mappings. This patch corrects that by using the more strongly ordered

[PATCH] driver core: platform: add device binding path 'driver_override'

2014-06-02 Thread Kim Phillips
PCI device binding, e.g., when in the presence of hotplug. Reviewed-by: Alex Williamson Reviewed-by: Alexander Graf Reviewed-by: Stuart Yoder Signed-off-by: Kim Phillips --- Greg, This is largely identical to the PCI version of the same that has been accepted for v3.16 and a

[RFC PATCH v5_v2 01/11] driver core: platform: add device binding path 'driver_override'

2014-05-20 Thread Kim Phillips
From: Kim Phillips Needed by platform device drivers, such as the vfio-platform driver later in series, in order to bypass the existing OF, ACPI, id_table and name string matches, and successfully be able to be bound to any device, like so: echo vfio-platform > /sys/bus/platform/devi

Re: [PATCH] checkpatch: Make --strict a default for files in drivers/net and net/

2014-05-09 Thread Kim Phillips
On Fri, 9 May 2014 13:56:15 -0700 Joe Perches wrote: > On Fri, 2014-05-09 at 15:37 -0500, Kim Phillips wrote: > > On Wed, 7 May 2014 11:13:26 -0700 > > Joe Perches wrote: > > > > > Networking files are generally more strictly conformant to > > > linux-ker

Re: [PATCH] checkpatch: Make --strict a default for files in drivers/net and net/

2014-05-09 Thread Kim Phillips
On Wed, 7 May 2014 11:13:26 -0700 Joe Perches wrote: > Networking files are generally more strictly conformant to > linux-kernel style checkpatch disagrees :) : {drivers/}net/ :~10.8 CHECKs per .[hc] file everything else:~10 CHECKs per .[hc] file no net, no drivers/sta

[PATCH] driver core: platform: add device binding path 'driver_override'

2014-04-08 Thread Kim Phillips
PCI device binding, e.g., when in the presence of hotplug. [1] http://lkml.iu.edu/hypermail/linux/kernel/1402.1/00177.html [2] http://lists-archives.com/linux-kernel/28030441-pci-introduce-new-device-binding-path-using-pci_dev-driver_override.html Suggested-by: Alex Williamson Signed-o

Re: [RFC PATCH] PCI: Introduce new device binding path using pci_dev.driver_override

2014-04-04 Thread Kim Phillips
PCI device binding, e.g., when in the presence of hotplug. [1] http://lkml.iu.edu/hypermail/linux/kernel/1402.1/00177.html [2] http://thread.gmane.org/gmane.linux.kernel.iommu/4605 Signed-off-by: Kim Phillips --- if this looks ok, should it be included in the next version of the vfio-platfor

Re: [RFC PATCH] PCI: Introduce new device binding path using pci_dev.driver_override

2014-04-02 Thread Kim Phillips
On Tue, 1 Apr 2014 17:23:24 -0700 Greg KH wrote: > On Tue, Apr 01, 2014 at 06:52:12PM -0500, Kim Phillips wrote: > > On Tue, 01 Apr 2014 10:28:54 -0600 > > Alex Williamson wrote: > > > > > The driver_override field allows us to specify the driver for a device &g

Re: [RFC PATCH] PCI: Introduce new device binding path using pci_dev.driver_override

2014-04-01 Thread Kim Phillips
On Tue, 01 Apr 2014 10:28:54 -0600 Alex Williamson wrote: > The driver_override field allows us to specify the driver for a device > rather than relying on the driver to provide a positive match of the > device. This shortcuts the existing process of looking up the vendor > and device ID, adding

Re: mechanism to allow a driver to bind to any device

2014-03-31 Thread Kim Phillips
On Fri, 28 Mar 2014 11:10:23 -0600 Alex Williamson wrote: > On Fri, 2014-03-28 at 12:58 -0400, Konrad Rzeszutek Wilk wrote: > > On Wed, Mar 26, 2014 at 04:09:21PM -0600, Alex Williamson wrote: > > > On Wed, 2014-03-26 at 10:21 -0600, Alex Williamson wrote: > > > > On Wed, 2014-03-26 at 23:06 +080

Re: mechanism to allow a driver to bind to any device

2014-03-31 Thread Kim Phillips
On Mon, 31 Mar 2014 20:23:36 + Stuart Yoder wrote: > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Monday, March 31, 2014 2:47 PM > > > > On Mon, Mar 31, 2014 at 06:47:51PM +, Stuart Yoder wrote: > > > I also, was at the point where I thought we should perhaps just > > >

[PATCH 2/2] ARM: KVM: user_mem_abort: support stage 2 MMIO page mapping

2014-03-25 Thread Kim Phillips
Use the correct memory type for device MMIO mappings: PAGE_S2_DEVICE. Signed-off-by: Kim Phillips --- arch/arm/kvm/mmu.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/arch/arm/kvm/mmu.c b/arch/arm/kvm/mmu.c index 7789857..a354610 100644 --- a/arch/arm/kvm/mmu.c

[PATCH 1/2] ARM: KVM: Enable the KVM-VFIO device

2014-03-25 Thread Kim Phillips
Used by KVM-enabled VFIO-based device passthrough support in QEMU. Signed-off-by: Kim Phillips --- This is just a couple of patches developed during platform device passthrough development in QEMU. Based on linux v3.14-rc8. arch/arm/kvm/Kconfig | 1 + arch/arm/kvm/Makefile | 2 +- 2 files

Re: [REPOST][PATCH 1/2] driver core: Add new device_driver flag to allow binding via sysfs only

2013-12-09 Thread Kim Phillips
On Thu, 5 Dec 2013 16:38:15 -0600 Scott Wood wrote: > On Thu, 2013-12-05 at 17:45 +0000, Kim Phillips wrote: > > On Tue, 03 Dec 2013 16:34:33 +0100 > > Jan Kiszka wrote: > > > > > On 2013-12-03 13:34, Kim Phillips wrote: > > > > VFIO supports pass-t

Re: [REPOST][PATCH 1/2] driver core: Add new device_driver flag to allow binding via sysfs only

2013-12-05 Thread Kim Phillips
On Tue, 03 Dec 2013 16:34:33 +0100 Jan Kiszka wrote: > On 2013-12-03 13:34, Kim Phillips wrote: > > VFIO supports pass-through of devices to user space - for sake > > of illustration, say a PCI e1000 device: > > > > - the e1000 is first unbound from the PCI e1000 driv

[REPOST][PATCH 2/2] driver core: platform: allow platform drivers to bind to any device

2013-12-03 Thread Kim Phillips
form_driver, for drivers to be able to set and thus not cause platform_match() to fail when a bind is requested. [1] http://www.spinics.net/lists/kvm/msg96701.html Signed-off-by: Kim Phillips --- rebased onto 3.13-rc2, and reposted from first submission which received no comments: https://lk

[REPOST][PATCH 1/2] driver core: Add new device_driver flag to allow binding via sysfs only

2013-12-03 Thread Kim Phillips
possible by introducing a sysfs_bind_only flag in struct device_driver. Signed-off-by: Stuart Yoder Signed-off-by: Kim Phillips --- rebased onto 3.13-rc2, and reposted from first submission which recieved no comments: https://lkml.org/lkml/2013/10/11/53 drivers/base/dd.c | 5 - include

Re: driver core patches

2013-12-03 Thread Kim Phillips
On Tue, 5 Nov 2013 18:12:47 -0800 "gre...@linuxfoundation.org" wrote: > On Tue, Nov 05, 2013 at 10:53:36PM +, Stuart Yoder wrote: > > Hi Greg, > > > > Any feedback on the driver core patches in Kim Phillips patch series > > from mid-October? > >

Re: [PATCH v7] arm: use built-in byte swap function

2013-11-05 Thread Kim Phillips
On Sat, 26 Oct 2013 22:41:34 -0400 Nicolas Pitre wrote: > On Thu, 23 May 2013, Kim Phillips wrote: > > > Enable the compiler intrinsic for byte swapping on arch ARM. This > > allows the compiler to detect and be able to optimize out byte > > swappings, and has a very m

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

2013-10-11 Thread Kim Phillips
On Fri, 11 Oct 2013 15:43:40 -0500 Scott Wood wrote: > On Fri, 2013-10-11 at 01:27 -0500, Kim Phillips wrote: > > Force the vfio-pci driver to only be bound explicitly via sysfs to avoid > > conflics with other drivers in the event of a hotplug. > > > >

[PATCH] VFIO: platform: allow the driver to bind to any device explicitly via sysfs

2013-10-10 Thread Kim Phillips
Set match_any_dev and sysfs_bind_only such that echoing a platform device ID to the driver sysfs bind file will successfully match and bind the device to the vfio-platform meta-driver in accordance to the desired semantics for vfio drivers. Signed-off-by: Kim Phillips --- this patch depends on

[PATCH 2/4] driver core: platform: allow platform drivers to bind to any device

2013-10-10 Thread Kim Phillips
form_driver, for drivers to be able to set and thus not cause platform_match() to fail when a bind is requested. [1] http://www.spinics.net/lists/kvm/msg96701.html Signed-off-by: Kim Phillips --- drivers/base/platform.c | 4 include/linux/platform_device.h | 1 + 2 files changed, 5

[PATCH 1/4] driver core: Add new device_driver flag to allow binding via sysfs only

2013-10-10 Thread Kim Phillips
possible by introducing a sysfs_bind_only flag in struct device_driver. Signed-off-by: Stuart Yoder Signed-off-by: Kim Phillips --- this patchset is available on top of the WIP exynos-iommu v10 [1] and vfio-platform v2 [2] patchsets, here: git://git.linaro.org/people/kimphill/linux.git binding

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

2013-10-10 Thread Kim Phillips
Force the vfio-pci driver to only be bound explicitly via sysfs to avoid conflics with other drivers in the event of a hotplug. Signed-off-by: Kim Phillips --- drivers/vfio/pci/vfio_pci.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/vfio/pci/vfio_pci.c b/drivers/vfio/pci

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-09 Thread Kim Phillips
On Wed, 9 Oct 2013 15:03:19 -0500 Scott Wood wrote: > On Wed, 2013-10-09 at 14:44 -0500, Yoder Stuart-B08248 wrote: > > > From: Wood Scott-B07421 > > > Sent: Wednesday, October 09, 2013 2:22 PM > > > > > > On Wed, 2013-10-09 at 14:02 -0500, Yoder Stuart-B08248 wrote: > > > > Have been thinking a

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-02 Thread Kim Phillips
r Dall [mailto:christoffer.d...@linaro.org] > > > > Sent: Wednesday, October 02, 2013 10:14 AM > > > > To: Alex Williamson > > > > Cc: Kim Phillips; gre...@linuxfoundation.org; linux- > > > > ker...@vger.kernel.org; a.mota...@virtualopensystems.com;

Re: [LNG] [PATCH] mfd: twl6030: Fix endianness problem in IRQ handler

2013-10-02 Thread Kim Phillips
On Wed, 2 Oct 2013 14:08:44 +0300 Taras Kondratiuk wrote: > From: Danke Xie > > The current TWL 6030 IRQ handler assumes little endianness. > This change makes it endian-neutral. > > Signed-off-by: Danke Xie > Signed-off-by: Taras Kondratiuk > --- this patch causes a new sparse warning: m

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-01 Thread Kim Phillips
On Tue, 1 Oct 2013 13:00:54 -0700 Greg Kroah-Hartman wrote: > On Tue, Oct 01, 2013 at 01:38:31PM -0500, Kim Phillips wrote: > > Hi, > > > > Santosh and I are having a problem figuring out how to enable binding > > (and re-binding) platform devices to a platform VFIO d

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-01 Thread Kim Phillips
On Tue, 1 Oct 2013 14:17:16 -0500 Scott Wood wrote: > On Tue, 2013-10-01 at 14:15 -0500, Scott Wood wrote: > > On Tue, 2013-10-01 at 13:38 -0500, Kim Phillips wrote: > > > Hi, > > > > > > Santosh and I are having a problem figuring out how to enable bin

Re: RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-01 Thread Kim Phillips
On Tue, 1 Oct 2013 14:15:38 -0500 Scott Wood wrote: > On Tue, 2013-10-01 at 13:38 -0500, Kim Phillips wrote: > > Hi, > > > > Santosh and I are having a problem figuring out how to enable binding > > (and re-binding) platform devices to a platform VFIO driver (see &

RFC: (re-)binding the VFIO platform driver to a platform device

2013-10-01 Thread Kim Phillips
is the name for the driver (-dt for device tree) which has already been pointed out as a misnomer and should probably be rewritten as 'vfio-platform': http://lists.linux-foundation.org/pipermail/iommu/2013-August/006284.html [2] >From 6fa383d3f7bb53eb5efbb324c07484191b29543d Mon Sep 17 00:

[PATCH linux-stable-3.10.y] ARM: KVM: Fix 64-bit coprocessor handling

2013-09-19 Thread Kim Phillips
Dall Signed-off-by: Kim Phillips Cc: Gleb Natapov Cc: Paolo Bonzini Cc: Russell King Cc: Jonghwan Choi Cc: kvm...@lists.cs.columbia.edu Cc: k...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org Cc: # 3.10.x --- resumbission of: https://lkml.org/lkml

[PATCH] i2c: s3c2410: fix clk_disable/clk_unprepare warnings

2013-09-17 Thread Kim Phillips
m Sang Signed-off-by: Kim Phillips --- drivers/i2c/busses/i2c-s3c2410.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c index 3535f3c..3747b9b 100644 --- a/drivers/i2c/busses/i2c-s3c2410.c +++ b/drivers/i2c/busses/i2c-s3

[PATCH v7] arm: use built-in byte swap function

2013-05-23 Thread Kim Phillips
8117090 7bdb62 vmlinux-imx_v6_v7 #builtin-bswap Signed-off-by: Kim Phillips Reviewed-by: Nicolas Pitre Acked-by: David Woodhouse --- resending as v6 appears to have fallen though the cracks. Russell? v7: rebased onto next-20130521, re-ran above vmlinux sizes with Linaro gcc 4.8, added

Re: [PATCH] powerpc/crypto: Add property for 'era' in SEC dts crypto node

2013-04-09 Thread Kim Phillips
On Tue, 9 Apr 2013 17:20:33 +0530 Vakul Garg wrote: > The crypto node now contains a new property 'fsl,sec-era'. > This is required so that applications can retrieve era info without > having to be able to read SEC's register space. > > Signed-off-by: Vakul Garg > --- > arch/powerpc/boot/dts/f

<    1   2   3   4   5   >