[PATCH 2/9] ARC: [dts] Introduce Timer bindings

2016-02-02 Thread Vineet Gupta
ARC Timers have historically been probed directly. As precursor to start probing Timers thru DT introduce these bindings Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Cc: Rob Herring <r...@kernel.org> Cc: devicet...@vger.kernel.org Signed-off-by: Vineet Gupta <vgu.

[PATCH 1/9] ARC: [dts] Add clk feeding into timers to DTs

2016-02-02 Thread Vineet Gupta
This allows us to introduce timers in DT in next commit Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Cc: Rob Herring <r...@kernel.org> Cc: devicet...@vger.kernel.org Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/boot/dts/axc001.dtsi | 6 ++ arch/a

[PATCH 0/9] ARC clockevent/clocksource modernization

2016-02-02 Thread Vineet Gupta
: clockevent: Prepare for DT based probe Vineet Gupta (4): ARC: [dts] Add clk feeding into timers to DTs ARC: [dts] Introduce Timer bindings ARC: clockevent: DT based probe ARC: clocksource: DT based probe .../devicetree/bindings/timer/snps,arc-timer0.txt | 23 +++ .../devicetree/bindings

[PATCH 3/9] ARC: clockevent: switch to cpu notifier for clockevent setup

2016-02-02 Thread Vineet Gupta
") - Remove exposing timer function to smp.c Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Signed-off-by: Noam Camus <no...@ezchip.com> [vgupta: broken off from a bigger patch] Signed-off-by: Vineet Gupta <vgu...@synopsys.com> Signed-off-by: Vineet Gupta <vgu...@syn

[PATCH 7/9] ARC: use fixed frequencies in arc_set_early_base_baud()

2016-02-02 Thread Vineet Gupta
mework later on. Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/kernel/devtree.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/arch/arc/kernel/devtree.c b/arch/arc/kernel/devtree.c

[GIT PULL] ARC fixes for 4.5-rc6 (updated)

2016-02-24 Thread Vineet Gupta
ing updates for new hardware features - Don't force DEVTMPFS if INITRAMFS Alexey Brodkin (1): arc: get rid of DEVTMPFS dependency on INITRAMFS_SOURCE Valentin Rothberg (1): arc: SMP: CONFIG_ARC_IPI_DBG cleanup Vineet Gu

Re: ARC: leftover reference on ARC_IPI_DBG

2016-02-23 Thread Vineet Gupta
Hi Valentin, On Wednesday 24 February 2016 12:53 PM, Valentin Rothberg wrote: > Hi Vineet, > > your commit 0443ad7fb2eb ("ARC: SMP: No need for CONFIG_ARC_IPI_DBG") > has shown up in todays linux-next tree (i.e., next-20160224) and removes > the Kconfig option ARC_IPI_DBG and all referencing

[PATCH 5/5] ARC: SMP: No need for CONFIG_ARC_IPI_DBG

2016-02-23 Thread Vineet Gupta
This was more relevant during SMP bringup. The warning for bogus msg better be visible always. Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/Kconfig | 8 arch/arc/kernel/mcip.c | 9 + arch/arc/kernel/smp.c | 3 --- 3 files changed, 1 insertion(

[PATCH 1/5] ARCv2: SMP: Emulate IPI to self using software triggered interrupt

2016-02-23 Thread Vineet Gupta
pet...@infradead.org> Cc: <sta...@vger.kernel.org>[4.2] Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/include/asm/irqflags-arcv2.h | 11 +++ arch/arc/kernel/mcip.c| 15 +++ 2 files changed, 26 insertions(+) diff --git a/arch/arc/in

[PATCH 2/5] ARC: [intc-compact] Remove IPI setup from ARCompact port

2016-02-23 Thread Vineet Gupta
There is no real ARC700 based SMP SoC so remove IPI definition. EZChip's SMP ARC700 is going to use a different intc and IPI provider anyways. Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/include/asm/irq.h | 1 - arch/arc/kernel/intc-compact.c | 3 --- 2 files chan

[PATCH 3/5] ARCv2: SMP: Push IPI_IRQ into IPI provider

2016-02-23 Thread Vineet Gupta
Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/include/asm/irq.h | 1 - arch/arc/kernel/mcip.c | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arc/include/asm/irq.h b/arch/arc/include/asm/irq.h index bc5103637326..49014f0ef36d 100644 --- a/ar

[PATCH 4/5] ARCv2: Elide sending new cross core intr if receiver didn't ack prev

2016-02-23 Thread Vineet Gupta
jor...@synopsys.com> Cc: Peter Zijlstra <pet...@infradead.org> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/kernel/mcip.c | 27 ++- 1 file changed, 10 insertions(+), 17 deletions(-) diff --git a/arch/arc/kernel/mcip.c b/arch/arc/kernel/mcip.c

Re: Interesting csd deadlock on ARC

2016-02-23 Thread Vineet Gupta
>> What I actually meant was is it OK for irq_work_queue_on() to be called >> locally >> (is this a sched bug/optimization(. Further if it is OK to be called, does >> it need >> to do behave more like irq_work_queue() i.e. call arch_irq_work_raise() or >> arch_send_call_function_single_ipi() is

[PATCH 0/5] ARC IPI related fix

2016-02-23 Thread Vineet Gupta
Hi, This cam eout of debugging a csd deadlock on ARC. The fix itself is 1/5, rest of patches are additional improvements, optimizations while we are at it. Thx, -Vineet Vineet Gupta (5): ARCv2: SMP: Emulate IPI to self using software triggered interrupt ARC: [intc-compact] Remove IPI setup

Re: Interesting csd deadlock on ARC

2016-02-23 Thread Vineet Gupta
On Tuesday 23 February 2016 03:28 PM, Peter Zijlstra wrote: > On Tue, Feb 23, 2016 at 10:51:42AM +0530, Vineet Gupta wrote: >> On Friday 19 February 2016 12:17 PM, Vineet Gupta wrote: >>> Hi Peter, >>> >>> I've been debugging a csd_lock_wait(

[GIT PULL] ARC fixes for 4.5-rc6

2016-02-23 Thread Vineet Gupta
ncy on INITRAMFS_SOURCE Vineet Gupta (8): ARC: Assume multiplier is always present ARCv2: boot print Low Latency Memory ARCv2: boot report CCMs (Closely Coupled Memories) ARCv2: SMP: Emulate IPI to self using software triggered interrupt ARC: [intc-compact] Remove IPI se

Re: Interesting csd deadlock on ARC

2016-02-23 Thread Vineet Gupta
On Tuesday 23 February 2016 04:28 PM, Noam Camus wrote: >> From: Peter Zijlstra [mailto:pet...@infradead.org] >> Sent: Tuesday, February 23, 2016 12:40 PM > >> The only requirement for irq_work is that it runs after the NMI completes >> and runs from regular IRQ context. >There are no strict

Re: [PATCH] arc: make sure __delay() never gets executed with 0 loops

2016-02-23 Thread Vineet Gupta
--->8 > Calibrating delay loop... 0.77 BogoMIPS (lpj=3862) > >8 > > And on console output delays were way too long. > > Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> > Cc: Vineet Gupta <vgu...@s

[PATCH] ARC Build system tweak

2016-02-26 Thread Vineet Gupta
opencoded the pre-processor check. Thx, -Vineet Vineet Gupta (1): ARC: build: Better way to detect ISA compatible toolchain arch/arc/Makefile | 14 ++ arch/arc/include/asm/arcregs.h | 6 -- 2 files changed, 14 insertions(+), 6 deletions(-) -- 2.5.0

[PATCH] ARC: build: Better way to detect ISA compatible toolchain

2016-02-26 Thread Vineet Gupta
for an existing preprocessor macro which is again uniquely defined per ISA. Cc: Michal Marek <mma...@suse.cz> Suggested-by: Claudiu Zissulescu <claz...@synopsys.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/Makefile | 14 ++ arch/arc/inc

Re: [PATCH RESEND v9 1/2] PCI support added to ARC

2016-02-25 Thread Vineet Gupta
>> > Signed-off-by: Joao Pinto <jpi...@synopsys.com> > Did we have an ack from the ARC arch maintainer for this? I seem to > remember one, but it would help me out if you included it here so I > didn't have to track it down. > Indeed I'd acked it before. Still

Re: Interesting csd deadlock on ARC

2016-02-25 Thread Vineet Gupta
On Thursday 25 February 2016 07:36 PM, Peter Zijlstra wrote: > On Wed, Feb 24, 2016 at 10:21:25AM +0530, Vineet Gupta wrote: >>>> What I actually meant was is it OK for irq_work_queue_on() to be called >>>> locally >>>> (is this a sched bug/optimization(.

Re: [PATCH 4/5] ARCv2: Elide sending new cross core intr if receiver didn't ack prev

2016-02-26 Thread Vineet Gupta
On Tuesday 23 February 2016 02:41 PM, Vineet Gupta wrote: > ARConnect/MCIP IPI sending has a retry-wait loop in case caller had > not seen a previous such interrupt. Turns out that it is not needed at > all. Linux cross core calling allows coalescing multiple IPIs to same > receiver

Re: Interesting csd deadlock on ARC

2016-02-25 Thread Vineet Gupta
On Thursday 25 February 2016 08:00 PM, Russell King - ARM Linux wrote: > On Thu, Feb 25, 2016 at 07:53:39PM +0530, Vineet Gupta wrote: >> But then ARM CONFIG_SMP on UP hardware will still crap out because there >> is no way to send IPI to self. Same as the bug in above discussion. I

Re: Interesting csd deadlock on ARC

2016-02-22 Thread Vineet Gupta
On Friday 19 February 2016 12:17 PM, Vineet Gupta wrote: > Hi Peter, > > I've been debugging a csd_lock_wait() deadlock on SMP+PREEMPT ARC HS38x2 and > it > turned out to be lot more interesting than I'd hoped for. This is stock v4.4 > > Trouble starts with an IPI to s

Re: [PATCH] arc: get rid of DEVTMPFS dependency on INITRAMFS_SOURCE

2016-02-22 Thread Vineet Gupta
On Saturday 20 February 2016 06:35 PM, Alexey Brodkin wrote: > Even though DEVTMPFS is required when our pre-built initramfs > is used it is not the case in general. It is perfectly possible > to use initramfs with device nodes already populated or there > could be other usages, see discussion

Re: [PATCH v4 05/19] irqchip: add nps Internal and external irqchips

2016-01-25 Thread Vineet Gupta
I just located the crap: arch_do_IRQ() happily takes a hwirq (the > vector number), and uses that as a Linux IRQ. This looks a lot like ARM > pre-DT, about 10 years ago. > > Well, time to meet the 21st century. If you intend to use DT, please fix > your arch port. Otherwise, just

Re: [PATCH 07/15] dmaengine: dw: revisit data_width property

2016-01-24 Thread Vineet Gupta
On Monday 25 January 2016 12:55 AM, Mans Rullgard wrote: > From: Andy Shevchenko > > There are several changes are done here: > > - Convert the property to be in bytes > >Much more convenient than keeping encoded value. > > - Use one value for all AHB

Interesting csd deadlock on ARC

2016-02-18 Thread Vineet Gupta
Hi Peter, I've been debugging a csd_lock_wait() deadlock on SMP+PREEMPT ARC HS38x2 and it turned out to be lot more interesting than I'd hoped for. This is stock v4.4 Trouble starts with an IPI to self which doesn't get delivered as the inter-core interrupt providing h/w is not capable of IPI to

Re: [PATCH 2/9] ARC: [dts] Introduce Timer bindings

2016-02-16 Thread Vineet Gupta
On Wednesday 03 February 2016 09:09 PM, Rob Herring wrote: > On Wed, Feb 3, 2016 at 2:04 AM, Vineet Gupta > wrote: >> Hi Rob, >> >> On Wednesday 03 February 2016 03:33 AM, Rob Herring wrote: >>> On Tue, Feb 02, 2016 at 04:28:52PM +0530, Vineet Gupta

Re: [PATCH] arc: use little endian accesses

2016-03-09 Thread Vineet Gupta
igned-off-by: Lada Trimasova <ltri...@synopsys.com> Sorry NACK on this ! If you still think we need it I need more data / details on what exactly is failing in 8250 and how ! -Vineet > Cc: Alexey Brodkin <abrod...@synopsys.com> > Cc: Vineet Gupta <vgu...@synopsys.com&g

Re: [PATCH] arc: use little endian accesses

2016-03-09 Thread Vineet Gupta
On Thursday 10 March 2016 10:35 AM, Vineet Gupta wrote: > +CC Noam > > On Wednesday 09 March 2016 10:51 PM, Lada Trimasova wrote: >> > Memory access primitives should use cpu_to_le16, cpu_to_le32, le16_to_cpu >> > and le32_to_cpu because it is not really guaranteed th

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-09 Thread Vineet Gupta
On Wednesday 09 March 2016 08:21 PM, Peter Zijlstra wrote: >> But in SLUB: bit_spin_lock() + __bit_spin_unlock() is acceptable ? How so >> (ignoring the performance thing for discussion sake, which is a side effect >> of >> this implementation). > > The sort answer is: Per definition. They are

Re: [PATCH] arc: use little endian accesses

2016-03-10 Thread Vineet Gupta
On Thursday 10 March 2016 01:14 PM, Alexey Brodkin wrote: >>> For example, serial port driver doesn't work when kernel is build for >>> arc big endian architecture. >> Last I tested Big Endian on SDP with 8250 part + 8250 driver it was working >> fine. >> I presume this is the systemC model for

[PATCH 2/2] ARC: [*defconfig] No need to specify CONFIG_CROSS_COMPILE

2016-03-11 Thread Vineet Gupta
in defconfigs anyways ! Cc: Noam Camus <no...@ezchip.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Anton Kolesov <akole...@synosys.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/configs/axs101_defconfig | 1 - arch/arc/configs/axs103_defcon

[PATCH 1/2] ARC: [BE] Select correct CROSS_COMPILE prefix

2016-03-11 Thread Vineet Gupta
This allows CONFIG_CPU_BIG_ENDIAN=y to build correctly out of the box, w/o any other tweaks. Cc: Noam Camus <no...@ezchip.com> Cc: Alexey Brodkin <abrod...@synopsys.com> Cc: Anton Kolesov <akole...@synosys.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/

[PATCH 0/2] Big Endian build cleanups

2016-03-11 Thread Vineet Gupta
Vineet Gupta (2): ARC: [BE] Select correct CROSS_COMPILE prefix ARC: [*defconfig] No need to specify CONFIG_CROSS_COMPILE arch/arc/Makefile | 4 arch/arc/configs/axs101_defconfig | 1 - arch/arc/configs/axs103_defconfig | 1 - arch/arc/configs

Re: [PATCH V3] net: ezchip: adapt driver to little endian architecture

2016-03-11 Thread Vineet Gupta
On Friday 04 March 2016 03:50 AM, David Miller wrote: > From: Lada Trimasova > Date: Thu, 3 Mar 2016 17:07:46 +0300 > >> Since ezchip network driver is written with big endian EZChip platform it >> is necessary to add support for little endian architecture. >> >>

Re: [PATCH] arc: use little endian accesses

2016-03-11 Thread Vineet Gupta
On Friday 11 March 2016 06:14 PM, Vineet Gupta wrote: > @Lada I will fix up the changelog to add some of the background behind this > change, and mark this for stable backport as well. This is what I'm planning to add. Noam please give this a spin - you might have to revert those native-end

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-14 Thread Vineet Gupta
4: st_s r3,[r13,0] <--- (C) sets PG_locked (overwrites > unlock) > > Fixes ARC STAR 9000817404 (and probably more). > > Cc: sta...@vger.kernel.org > Reported-by: Vineet Gupta <vineet.gup...@synopsys.com> > Tested-by: Vineet Gupta <vineet.gup...@synops

Re: [PATCH] Remove the inclusion of pci-bridge.h and pci-dma-compat.h

2016-03-15 Thread Vineet Gupta
c/2016-March/000711.html > > This patch has the goal to fix the build problem. > > Signed-off-by: Joao Pinto <jpi...@synopsys.com> Acked-by: Vineet Gupta <vgu...@synopsys.com> @Bjorn since the ARC PCI support is still not upstream, and was slated to go via you, please pic

Re: [PATCH] ARC: Don't source drivers/pci/pcie/Kconfig ourselves

2016-03-15 Thread Vineet Gupta
On Tuesday 15 March 2016 04:47 PM, Andreas Ziegler wrote: > Commit 5f8fc43217a0 ("PCI: Include pci/pcie/Kconfig directly from > pci/Kconfig") in linux-next changed drivers/pci/Kconfig to include > drivers/pci/pcie/Kconfig itself, so that architectures do not need > to source both files themselves.

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-08 Thread Vineet Gupta
+CC linux-arch, parisc folks, PeterZ On Wednesday 09 March 2016 02:10 AM, Christoph Lameter wrote: > On Tue, 8 Mar 2016, Vineet Gupta wrote: > >> # set the bit >> 80543b8e:ld_s r2,[r13,0] <--- (A) Finds PG_locked is set >> 80543b90:or r3,r2,1

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-09 Thread Vineet Gupta
On Wednesday 09 March 2016 03:43 PM, Peter Zijlstra wrote: >> There is clearly a problem in slub code that it is pairing a >> test_and_set_bit() >> with a __clear_bit(). Latter can obviously clobber former if they are not a >> single >> instruction each unlike x86 or they use llock/scond kind of

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-09 Thread Vineet Gupta
On Wednesday 09 March 2016 05:10 PM, Peter Zijlstra wrote: > On Wed, Mar 09, 2016 at 04:30:31PM +0530, Vineet Gupta wrote: >> FWIW, could we add some background to commit log, specifically what prompted >> this. >> Something like below... > > Sure.. find below. >

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-09 Thread Vineet Gupta
est_and_set_bit() and therefore should be 'somewhat' atomic. > > Therefore the generic implementation of __clear_bit_unlock() cannot use > the fully non-atomic __clear_bit() as a default. > > If an arch is able to do better; is must provide an implementation of > __clear_bit_unlock() itse

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-09 Thread Vineet Gupta
On Wednesday 09 March 2016 04:01 PM, Peter Zijlstra wrote: > On Wed, Mar 09, 2016 at 11:13:49AM +0100, Peter Zijlstra wrote: >> --- >> Subject: bitops: Do not default to __clear_bit() for __clear_bit_unlock() >> >> __clear_bit_unlock() is a special little snowflake. While it carries the >>

Re: [PATCH] arc: use little endian accesses

2016-03-11 Thread Vineet Gupta
On Friday 11 March 2016 12:54 AM, Arnd Bergmann wrote: > On Thursday 10 March 2016, Lada Trimasova wrote: >> Driver is 8250, kernel is built for BE arc, nsim option in model >> "nsim_isa_big_endian = 1". >> >> With current "readl" and "writel" implementation for ARC we read word from >> memory

Re: [PATCH] ARC: build: Turn off -Wmaybe-uninitialized for ARC gcc 4.8

2016-03-19 Thread Vineet Gupta
On Friday 18 March 2016 03:22 PM, Arnd Bergmann wrote: > On Friday 18 March 2016 14:16:23 Vineet Gupta wrote: >> diff --git a/arch/arc/Makefile b/arch/arc/Makefile >> index fed12f39d8ce..aeb101e8e674 100644 >> --- a/arch/arc/Makefile >> +++ b/arch/arc/Makefile

Re: [PATCH v2] ARC: axs10x - add Ethernet PHY description in .dts

2016-03-19 Thread Vineet Gupta
On Thursday 17 March 2016 04:28 PM, Sergei Shtylyov wrote: > On 3/17/2016 12:41 PM, Alexey Brodkin wrote: > >> > Following commit broke DW GMAC functionality on AXS10x boards: >> > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=e34d65696d2ef13dc32f2a162556c86c461ed763 >

Re: [PATCH v2 all 04/14] arc: use of_platform_default_populate() to populate default bus

2016-03-19 Thread Vineet Gupta
On Thursday 17 March 2016 06:58 PM, Rob Herring wrote: >> Perhaps too late - but can't of_platform_default_populate just be a static >> inline >> > calling of_platform_populate in the header rather than exporting yet >> > another symbol ! > The idea is to not export the variable

[PATCH] ARC: build: Turn off -Wmaybe-uninitialized for ARC gcc 4.8

2016-03-19 Thread Vineet Gupta
fixing gcc and/or relevant kernel code. Cc: Claudiu Zissulescu <claudiu.zissule...@synopsys.com> Cc: Arnd Bergmann <a...@arndb.de> Cc: Michal Marek <mma...@suse.cz> Cc: Geert Uytterhoeven <ge...@linux-m68k.org> Cc: linux-kbu...@vger.kernel.org Cc: linux-kernel@vger.kernel.o

Re: [PATCH v2 2/9] ARC: [dts] Introduce Timer bindings

2016-03-10 Thread Vineet Gupta
On Tuesday 08 March 2016 05:56 PM, Vineet Gupta wrote: > ARC Timers have historically been probed directly. > As precursor to start probing Timers thru DT introduce these bindings > Note that to keep series bisectable, these bindings are not yet used in > code. > > Cc: Daniel Le

Re: [PATCH] arc: use little endian accesses

2016-03-10 Thread Vineet Gupta
On Thursday 10 March 2016 01:15 PM, Arnd Bergmann wrote: > On Thursday 10 March 2016, Vineet Gupta wrote: >> On Wednesday 09 March 2016 10:51 PM, Lada Trimasova wrote: >>> Memory access primitives should use cpu_to_le16, cpu_to_le32, le16_to_cpu >>> and le32_to_c

Re: [PATCH v2] ARC: axs10x - add Ethernet PHY description in .dts

2016-04-06 Thread Vineet Gupta
Hi, On Thursday 17 March 2016 03:11 PM, Alexey Brodkin wrote: > Following commit broke DW GMAC functionality on AXS10x boards: > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=e34d65696d2ef13dc32f2a162556c86c461ed763 > > That's what happens on eth0 up: >

Re: [PATCH v8 3/3] irqchip: add nps Internal and external irqchips

2016-04-11 Thread Vineet Gupta
On Monday 11 April 2016 06:23 PM, Marc Zyngier wrote: > On 03/04/16 19:14, Noam Camus wrote: >> From: Noam Camus >> >> Adding EZchip NPS400 support. >> Internal interrupts are handled by Multi Thread Manager (MTM) >> Once interrupt is serviced MTM is acked for deactivating the

Re: [PATCH 06/19] arc: get rid of superfluous __GFP_REPEAT

2016-04-11 Thread Vineet Gupta
nel uses __get_order_pte but this is obviously > always zero because BITS_FOR_PTE is not larger than 9 yet the page > size is always larger than 4K. This means that this flag has never > been actually useful here because it has always been used only for > PAGE_ALLOC_COSTLY requests. > > Cc: Vine

Re: [PATCH v8 3/3] irqchip: add nps Internal and external irqchips

2016-04-11 Thread Vineet Gupta
On Monday 11 April 2016 06:41 PM, Marc Zyngier wrote: >> >> It doesn't mean I really approve of it. The default domain thing is >> >> going to end-up biting you in the rear. But hey, better having this >> >> supported upstream than leaving these patches up in the air... Hi Marc, Aside of this -

CC in git cover letter vs patches (was Re: [PATCH 0/19] get rid of superfluous __GFP_REPORT)

2016-04-13 Thread Vineet Gupta
Trimming CC list + CC git folks Hi Michal, On Monday 11 April 2016 04:37 PM, Michal Hocko wrote: > Hi, > this is the second version of the patchset previously sent [1] I have a git question if you didn't mind w.r.t. this series. Maybe there's an obvious answer... I'm using git 2.5.0 I was

[PATCH v4 4/5] ARC: [intc-*] Do a domain lookup in primary handler for hwirq -> linux virq

2016-04-13 Thread Vineet Gupta
Zyngier <marc.zyng...@arm.com> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Noam Camus <no...@ezchip.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/Kconfig | 1 + arch/arc/kernel/intc-arcv2.c | 9 ++--- arch/arc/kernel/intc-compact.c |

[PATCH v4 2/5] ARC: clocksource: DT based probe

2016-04-13 Thread Vineet Gupta
doesn't exist, TIMER1 can take over (as it is always present) Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/kernel/mcip.c | 4 +- arch/arc/kernel/setup.c | 3 -- arch/arc/kernel/t

[PATCH v4 5/5] ARC: [intc-*] switch to linear domain

2016-04-13 Thread Vineet Gupta
Now that we have Timers probed from DT, don't need legacy domain This however requires mapping to be called explicitly for the IRQ which still can't (and probably never) be probed from DT such as IPI and SOFTIRQ Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/kernel/intc-a

[PATCH v4 0/5] Modernize ARC clocksource/clockevent/intc drivers

2016-04-13 Thread Vineet Gupta
, -Vineet [1] http://lists.infradead.org/pipermail/linux-snps-arc/2016-March/000653.html Vineet Gupta (5): ARC: clockevent: DT based probe ARC: clocksource: DT based probe ARC: irq: export some IRQs again ARC: [intc-*] Do a domain lookup in primary handler for hwirq -> linux virq

[PATCH v4 3/5] ARC: irq: export some IRQs again

2016-04-13 Thread Vineet Gupta
This will be needed for switching ti lineaser irq domain as irq_create_mapping() called by intr code, needs the IRQ numbers as well in addition to existing usage in mcip.c Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/include/asm/irq.h | 6 ++ arch/arc/kernel/mcip.c

[PATCH v4 1/5] ARC: clockevent: DT based probe

2016-04-13 Thread Vineet Gupta
g> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/include/asm/irq.h | 9 arch/arc/kernel/intc-compact.c | 2 ++ arch/arc/kernel/time.c | 49 +- 3 files changed, 36 insertions(+), 24 deletions(-) diff --git a/arch/arc/in

Re: [PATCH] ARCv2: Additional trace IRQs to support locking correctness validator

2016-04-12 Thread Vineet Gupta
Hi Evgeny, On Wednesday 23 March 2016 02:56 PM, Evgeny Voevodin wrote: > Flags should be saved in the same format in which clri instruction saves > them since they are passed directly to seti instruction over > arch_local_save_flags/arch_local_irq_restore calls. > Trace of all clri/seti assembly

[GIT PULL] ARC fixes for 4.6-rc4

2016-04-11 Thread Vineet Gupta
(1): ARC: Don't source drivers/pci/pcie/Kconfig ourselves Vineet Gupta (1): Revert "ARC: [plat-axs10x] add Ethernet PHY description in .dts" arch/arc/Kconfig | 1 - arch/arc/boot/dts/axs10x_mb.dtsi | 8 arch/arc/configs/axs103_defco

Re: [PATCH v6 2/3] clocksource: Add NPS400 timers driver

2016-04-07 Thread Vineet Gupta
On Tuesday 29 March 2016 06:51 PM, Daniel Lezcano wrote: > On 03/29/2016 02:52 PM, Vineet Gupta wrote: >> On Monday 21 March 2016 07:54 PM, Noam Camus wrote: >>> From: Noam Camus <no...@ezchip.com> >>> >>> Add internal tick generator which is shared by

Re: [PATCH] ARC: build: Turn off -Wmaybe-uninitialized for ARC gcc 4.8

2016-03-19 Thread Vineet Gupta
On Friday 18 March 2016 05:43 PM, Arnd Bergmann wrote: > I think it's dangerous to use -O3 in one architecture when nothing else > uses it. If you don't have a strong reason to use -O3, maybe just drop that > use the default -O2 -Wmaybe-uninitialized like everyone else does. I know what u mean.

Re: [PATCH v2] ARC: axs10x - add Ethernet PHY description in .dts

2016-03-19 Thread Vineet Gupta
t; Cc: David S. Miller <da...@davemloft.net> Cc: linux-kernel@vger.kernel.org Cc: net...@vger.kernel.org Cc: sta...@vger.kernel.org # 4.5 Cc: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/boot/dts/axs10x_mb.dtsi |

Re: [PATCH v2] ARC: axs10x - add Ethernet PHY description in .dts

2016-03-19 Thread Vineet Gupta
On Thursday 17 March 2016 05:29 PM, Sergei Shtylyov wrote: > Hello. > > On 3/17/2016 2:41 PM, Vineet Gupta wrote: > >>>>> >>>> Following commit broke DW GMAC functionality on AXS10x boards: >>>>> >>>> http://

Re: [PATCH] ARC: build: Turn off -Wmaybe-uninitialized for ARC gcc 4.8

2016-03-19 Thread Vineet Gupta
On Friday 18 March 2016 03:59 PM, Arnd Bergmann wrote: > On Friday 18 March 2016 15:50:11 Vineet Gupta wrote: >> Sure, but I prefer this to be only for gcc 4.8 as this warning seems to be >> healthy in small doses At least it keeps the door open for future discussion >> with

Re: [PATCH v3] ARC: [dts] Introduce Timer bindings

2016-03-20 Thread Vineet Gupta
On Sunday 20 March 2016 06:12 AM, Rob Herring wrote: > On Fri, Mar 18, 2016 at 10:56:29AM +0530, Vineet Gupta wrote: >> ARC Timers have historically been probed directly. >> As precursor to start probing Timers thru DT introduce these bindings >> Note that to keep series bisec

[GIT PULL] ARC changes for 4.6-rc1

2016-03-20 Thread Vineet Gupta
use of_platform_default_populate() to populate default bus Lada Trimasova (2): ARC: [BE] readl()/writel() to work in Big Endian CPU configuration arc: [plat-nsimosci*] use ezchip network driver Vineet Gupta (16): ARC: bitops: Remove non relevant comments ARC: [BE] Select correct CROSS_COMPILE pre

[PATCH v3] ARC: [dts] Introduce Timer bindings

2016-03-20 Thread Vineet Gupta
Cc: devicet...@vger.kernel.org Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- v3: - Renamed Node name to avoid new warnings when unit address used w/o regs [Rob] v2: - http://lists.infradead.org/pipermail/linux-snps-arc/2016-March/000653.html - snps,arc-timer[0-1] folded into sin

Re: [PATCH v2 all 04/14] arc: use of_platform_default_populate() to populate default bus

2016-03-19 Thread Vineet Gupta
On Wednesday 16 March 2016 01:03 PM, Kefeng Wang wrote: > Use helper of_platform_default_populate() in linux/of_platform > when possible, instead of calling of_platform_populate() with > the default match table. > > Acked-by: Vineet Gupta <vgu...@synopsys.com> &g

[PATCH v2 2/9] ARC: [dts] Introduce Timer bindings

2016-03-08 Thread Vineet Gupta
Cc: devicet...@vger.kernel.org Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- Changes v1 -> v2 - snps,arc-timer[0-1] folded into single snps-arc-timer[Rob] - Node name in DT example fixed: [Rob] "timer1: timer_clksrc {" ->

[PATCH v2 7/9] ARC: use fixed frequencies in arc_set_early_base_baud()

2016-03-08 Thread Vineet Gupta
mework later on. Cc: Christian Ruppert <christian.rupp...@alitech.com> Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- Changes v1 -> v2 - Base baud TB10x needs to be 166.66 not 166.67[Christian] v1: - http://li

[PATCH v2 5/9] ARC: clockevent: DT based probe

2016-03-08 Thread Vineet Gupta
ano <daniel.lezc...@linaro.org> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- Changes v1 -> v2 - Rebased on 4.5-rc6 - Fix snafu in a adapting patch from EZChip tree v1: - http://lists.infradead.org/pipermail/linux-snps-arc/2016-February/000449.html Signed-off-by: Vi

[PATCH v2 3/9] ARC: clockevent: switch to cpu notifier for clockevent setup

2016-03-08 Thread Vineet Gupta
notifier to set clockevent on non-boot CPUs. So explicit setup is done only on boot CPU (which will later be done by DT) Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Signed-off-by: Noam Camus <no...@ezchip.com> [vgupta: broken off from a bigger patch] Signed-off-by: Vinee

[PATCH v2 9/9] ARC: RIP arc_{get|set}_core_freq() clk API

2016-03-08 Thread Vineet Gupta
From: Alexey Brodkin <abrod...@synopsys.com> There are no more users of this - so RIP! Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> [vgupta: update changelog] Signed-off-by: Vineet Gupta <vgu...@synopsys.com> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> -

[PATCH v2 1/9] ARC: [dts] Add clk feeding into timers to DTs

2016-03-08 Thread Vineet Gupta
This allows us to introduce timers in DT in next commit Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Cc: Rob Herring <r...@kernel.org> Cc: devicet...@vger.kernel.org Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- arch/arc/boot/dts/axc001.dtsi | 6 ++ arch/a

[PATCH v2 8/9] ARC: [plat-axs] Don't use arc_{get|set}_core_freq() for manipulating core clk

2016-03-08 Thread Vineet Gupta
n no longer use the top level "clock-frequency". This change reduces the number of users of ARC clk hack and paves way for removal. Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> [vgupta: broken out of from bigger patch] Signed-off-by: Vineet Gupta <vgu...@synopsys

[PATCH v2 4/9] ARC: clockevent: Prepare for DT based probe

2016-03-08 Thread Vineet Gupta
From: Noam Camus <no...@ezchip.com> - call clocksource_probe() - This in turns needs of_clk_init() to be called earlier Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Signed-off-by: Noam Camus <no...@ezchip.com> [vgupta: broken off from a bigger patch] Signed-off-by:

[PATCH v2 6/9] ARC: clocksource: DT based probe

2016-03-08 Thread Vineet Gupta
doesn't exist, TIMER1 can take over (as it is always present) Cc: Rob Herring <r...@kernel.org> Cc: Noam Camus <no...@ezchip.com> Cc: Daniel Lezcano <daniel.lezc...@linaro.org> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> --- Changes v1 -> v2 - Introduce arc_of

[PATCH v2 0/9] ARC clockevent/clocksource modernization

2016-03-08 Thread Vineet Gupta
|set}_core_freq() for manipulating core clk ARC: RIP arc_{get|set}_core_freq() clk API Noam Camus (2): ARC: clockevent: switch to cpu notifier for clockevent setup ARC: clockevent: Prepare for DT based probe Vineet Gupta (4): ARC: [dts] Add clk feeding into timers to DTs ARC: [dts

[PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-08 Thread Vineet Gupta
e.com> Cc: Andrew Morton <a...@linux-foundation.org> Cc: Noam Camus <no...@ezchip.com> Cc: <sta...@vger.kernel.org> Cc: <linux...@kvack.org> Cc: <linux-kernel@vger.kernel.org> Cc: <linux-snps-...@lists.infradead.org> Signed-off-by: Vineet Gupta <vgu...@synopsys.com&

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-08 Thread Vineet Gupta
On Tuesday 08 March 2016 08:30 PM, Christoph Lameter wrote: > On Tue, 8 Mar 2016, Vineet Gupta wrote: > >> This in turn happened because slab_unlock() doesn't serialize properly >> (doesn't use atomic clear) with a concurrent running >> slab_lock()->test_and_set_bit

Re: [PATCH] ARC: build: Better way to detect ISA compatible toolchain

2016-03-04 Thread Vineet Gupta
Hi Michal, Can u please take a look at patch below ! Thx, -Vineet On Friday 26 February 2016 01:59 PM, Vineet Gupta wrote: > ARC architecture has 2 instruction sets: ARCompact/ARCv2. > While same gcc supports compiling for either (using appropriate toggles), > we can't use the same

weird build issue

2016-04-04 Thread Vineet Gupta
Hi, I'm running into a really weird build issue when building 4.6-rc1 for ARC (although this doesn't seem to be version related) I have 2 source trees and I see a build error in only one with exact same everything. ~/arc/k.org>make mrproper ~/arc/k.org>make ARCH=arc axs103_smp_defconfig O=tmp

Re: [PATCH v6 3/3] irqchip: add nps Internal and external irqchips

2016-03-29 Thread Vineet Gupta
On Monday 21 March 2016 07:54 PM, Noam Camus wrote: > From: Noam Camus > > Adding EZchip NPS400 support. > NPS internal interrupts are internally handled at > Multi Thread Manager (MTM) that is signaled for deactivating > an interrupt. > External interrupts is handled also at

Re: [PATCH v6 2/3] clocksource: Add NPS400 timers driver

2016-03-29 Thread Vineet Gupta
y same > clock source. > > Signed-off-by: Noam Camus <no...@ezchip.com> > Cc: Daniel Lezcano <daniel.lezc...@linaro.org> > Cc: Rob Herring <robh...@kernel.org> > Cc: Thomas Gleixner <t...@linutronix.de> > Cc: John Stultz <john.stu...@linaro.org> > Acked-by:

Re: [RESEND PATCH v4] clk/axs10x: Add I2S PLL clock driver

2016-04-21 Thread Vineet Gupta
On Thursday 21 April 2016 05:48 PM, Alexey Brodkin wrote: > Hi Jose, > > On Thu, 2016-04-21 at 10:51 +0100, Jose Abreu wrote: >> Hi Alexey, > > > Otherwise, I still prefer two DTS files for the two different FPGA > versions. At the least, please use ioremap for any pointers that >

Re: [RFC][PATCH 03/31] locking,arc: Implement atomic_fetch_{add,sub,and,andnot,or,xor}()

2016-04-22 Thread Vineet Gupta
On Friday 22 April 2016 03:13 PM, Peter Zijlstra wrote: > Implement FETCH-OP atomic primitives, these are very similar to the > existing OP-RETURN primitives we already have, except they return the > value of the atomic variable _before_ modification. > > This is especially useful for irreversible

Re: [PATCH 2/2 v2] ARC: [axs10x] Specify reserved memory for frame buffer

2016-04-29 Thread Vineet Gupta
; > Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> Acked-by: Vineet Gupta <vgu...@synopsys.com>

Re: [PATCH 2/2] ARC: [axs10x] Specify reserved memory for frame buffer

2016-04-28 Thread Vineet Gupta
On Thursday 28 April 2016 07:16 PM, Alexey Brodkin wrote: >> > Note that the IOC start alignment needs to follow >> > max(4k, size). What will be maximum size of frame buffer - 16M always ! > What do you mean by that? For HS38, we intend to bypass the frame buffer traffic from IOC port. So the

Re: [PATCH 2/2] ARC: [axs10x] Specify reserved memory for frame buffer

2016-04-27 Thread Vineet Gupta
On Wednesday 27 April 2016 08:05 PM, Alexey Brodkin wrote: > Allocation of a frame buffer memory in a special memory region > allows bypassing of so-called IO Coherency aperture > which is typically set as a range 0x8z-0xAz. > > I.e. all data traffic to PGU bypasses IO Coherency block > and saves

Re: Misleading hint to select CONFIG_PERF_EVENTS if driver sets PERF_PMU_CAP_NO_INTERRUPT

2016-04-27 Thread Vineet Gupta
On Wednesday 27 April 2016 08:43 PM, Lada Trimasova wrote: > On Tue, 2016-04-26 at 12:42 +0000, Vineet Gupta wrote: > > On Friday 22 April 2016 06:56 PM, Lada Trimasova wrote: > I think what we have now is sufficient - but u seem to want a prettier > failure output. > &g

Re: Misleading hint to select CONFIG_PERF_EVENTS if driver sets PERF_PMU_CAP_NO_INTERRUPT

2016-04-26 Thread Vineet Gupta
On Friday 22 April 2016 06:56 PM, Lada Trimasova wrote: > I have a question about user-space perf handling error numbers. > The problem is that PMU interrupts are not supported in arc700 > architecture and it is impossible to evaluate `perf record` command. > In our perf implementation we set

Re: [RFC][PATCH 03/31] locking,arc: Implement atomic_fetch_{add,sub,and,andnot,or,xor}()

2016-04-24 Thread Vineet Gupta
On Friday 22 April 2016 07:46 PM, Peter Zijlstra wrote: > On Fri, Apr 22, 2016 at 10:50:41AM +0000, Vineet Gupta wrote: > >>> > > +#define ATOMIC_FETCH_OP(op, c_op, asm_op) >>> > > \ >>> > > +sta

<    7   8   9   10   11   12   13   14   15   16   >