[MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Jean-Michel Hautbois
Hi lists ! I measured the latency and the jitter of the RX and TX ethernet paths on my MPC5200 board. The RX path is quite good, but the TX path can be slow. [ 1218.976762] [mpc52xx_fec_start_xmit]Delay >30us for dma_map_single => 76364 ns [ 1219.188405] [mpc52xx_fec_tx_interrupt]Delay >30us for

Re: PHY/FEC Network adapter failed to initialize on MPC52xx Board

2010-12-01 Thread Peter kuennem...@crane-soft
Tanks to all for the reply to this thread. I got the tip from the thread Tiejun mentioned. Quotation: 'St. Strobel' in Xenomai-help "I had this problem too. In my case the problem was caused by an incorrect port multiplex configuration in U-Boot, see CONIFG_SYS_GPS_PORT_CONFIG (I use 0x xxx5xx

[PATCH] of/address: use propper endianess in get_flags

2010-12-01 Thread Sebastian Andrzej Siewior
This patch changes u32 to __be32 for all "ranges", "prop" and "addr" and such. Those variables are pointing to the device tree which containts intergers in big endian format. Most functions are doing it right because of_read_number() is doing the right thing for them. of_bus_isa_get_flags(), of_bus

Re: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Jean-Michel Hautbois
2010/12/1 Jean-Michel Hautbois : > Hi lists ! > > I measured the latency and the jitter of the RX and TX ethernet paths > on my MPC5200 board. > The RX path is quite good, but the TX path can be slow. > > [ 1218.976762] [mpc52xx_fec_start_xmit]Delay >30us for dma_map_single > => 76364 ns > [ 1219.1

[PATCH] eSPI: change the read behavior of the SPIRF

2010-12-01 Thread Mingkai Hu
The user must read N bytes of SPIRF (1 <= N <= 4) that do not exceed the amount of data in the receive FIFO, so read the SPIRF byte by byte when the data in receive FIFO is less than 4 bytes. On Simics, when read N bytes that exceed the amout of data in receive FIFO, we can't read the data out, th

Basic driver devel questions ?

2010-12-01 Thread Guillaume Dargaud
Hello all, is it OK if I ask basic driver development questions here ? Could you recommend a better forum for that maybe ? Thanks -- Guillaume Dargaud http://www.gdargaud.net/ ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozla

Re: Basic driver devel questions ?

2010-12-01 Thread Michael Ellerman
On Wed, 2010-12-01 at 11:15 +0100, Guillaume Dargaud wrote: > Hello all, > is it OK if I ask basic driver development questions here ? > Could you recommend a better forum for that maybe ? Hi Guillaume, I guess it depends how basic they are :) If they're basic _powerpc_ driver questions then thi

_extending_ platform support options?

2010-12-01 Thread Joachim Foerster
Hi all, currently I'm wondering what the preferred/recommend way of _extending_ an existing "Platform support" option is? We are working with custom design/boards based on Virtex4/5. So we are primarily using the CONFIG_XILINX_VIRTEX*_GENERIC_BOARD options. In our case we have some special n

Re: _extending_ platform support options?

2010-12-01 Thread Josh Boyer
On Wed, Dec 01, 2010 at 02:25:43PM +0100, Joachim Foerster wrote: >Hi all, > >currently I'm wondering what the preferred/recommend way of >_extending_ an existing "Platform support" option is? > >We are working with custom design/boards based on Virtex4/5. So we >are primarily using the CONFIG_XIL

Re: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Steven Rostedt
On Wed, 2010-12-01 at 09:16 +0100, Jean-Michel Hautbois wrote: > Hi lists ! > > I measured the latency and the jitter of the RX and TX ethernet paths > on my MPC5200 board. > The RX path is quite good, but the TX path can be slow. > > [ 1218.976762] [mpc52xx_fec_start_xmit]Delay >30us for dma_map

RE: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread David Laight
> A mb() is usually used if you do a write to device and read from it. > With out it, the CPU could perform the read before the write, which > would give you an incorrect result. There's no other way around that. Possibly the synchronisation functions are doing significantly more work than is re

Re: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Jean-Michel Hautbois
2010/12/1 David Laight : > >> A mb() is usually used if you do a write to device and read from it. >> With out it, the CPU could perform the read before the write, which >> would give you an incorrect result. There's no other way around that. > > Possibly the synchronisation functions are doing sig

Getting the IRQ number (Was: Basic driver devel questions ?)

2010-12-01 Thread Guillaume Dargaud
> I guess it depends how basic they are :) > > If they're basic _powerpc_ driver questions then this is probably the > right place. > > But I'd say just ask and maybe someone will be able to help, or maybe > they'll point you somewhere else. OK, here goes then: how do I get the IRQ number so tha

Re: Getting the IRQ number (Was: Basic driver devel questions ?)

2010-12-01 Thread Philipp Ittershagen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/01/2010 05:35 PM, Guillaume Dargaud wrote: > Now how do I connect the dots between the hardware definitions from the dts > and > my driver ? You can get the interrupt number from the dt by calling irq_of_parse_and_map(). Be sure to pass the no

Re: Getting the IRQ number (Was: Basic driver devel questions ?)

2010-12-01 Thread Scott Wood
On Wed, 1 Dec 2010 17:35:58 +0100 Guillaume Dargaud wrote: > OK, here goes then: how do I get the IRQ number so that I can install an > interrupt handler on it ? > > In my dts file I have: > xps_acqui_data_0: xps-acqui-d...@c980 { > compatible = "xlnx,xps

Re: [PATCH] drivers: char: hvc: add arm JTAG DCC console support

2010-12-01 Thread Daniel Walker
On Tue, 2010-11-30 at 21:30 -0800, Stephen Boyd wrote: > On 11/30/2010 11:25 AM, Daniel Walker wrote: > > @@ -682,6 +682,15 @@ config HVC_UDBG > > select HVC_DRIVER > > default n > > > > +config HVC_DCC > > + bool "ARM JTAG DCC console" > > + depends on ARM > > +

Re: [PATCH] drivers: char: hvc: add arm JTAG DCC console support

2010-12-01 Thread Greg KH
On Wed, Dec 01, 2010 at 10:54:56AM -0800, Daniel Walker wrote: > On Tue, 2010-11-30 at 21:30 -0800, Stephen Boyd wrote: > > On 11/30/2010 11:25 AM, Daniel Walker wrote: > > > @@ -682,6 +682,15 @@ config HVC_UDBG > > > select HVC_DRIVER > > > default n > > > > > > +config HVC_DCC >

[PATCH 0/4] V2 Add ability to link device blob(s) into vmlinux

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie This patch set adds the ability to link device tree blobs into vmlinux. Patch 1 implements the changes to include/asm-generic/vmlinux.lds.h and adds a generic rule for generating DTB objects to be linked vmlinux. Patch 2 implements linking a DTB into an x86 image. Patch 3

[PATCH 1/4] of: Add support for linking device tree blobs into vmlinux

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie This patch adds support for linking device tree blob(s) into vmlinux. Modifies asm-generic/vmlinux.lds.h to add linking .dtb sections into vmlinux. To maintain compatiblity with the of/fdt driver code platforms MUST copy the blob to a non-init memory location before the kerne

[PATCH 2/4] x86/of: Add building device tree blob(s) into image.

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie This patch adds linking device tree blob into vmlinux. DTB's are added by adding the blob object name to list of objects to be linked into the image. Signed-off-by: Dirk Brandewie --- arch/x86/platform/ce4100/Makefile | 10 ++ 1 files changed, 10 insertions(+), 0

[PATCH 3/4] of/powerpc: Use generic rule to build dtb's

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie Modify arch/powerpc/boot/Makefile to use dtc command in scripts/Makefile.lib Signed-off-by: Dirk Brandewie --- arch/powerpc/boot/Makefile |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile

[PATCH 4/4] microblaze/of: Use generic rule to build dtb's

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie Modify arch/powerpc/boot/Makefile to use dtc command in scripts/Makefile.lib Signed-off-by: Dirk Brandewie --- arch/microblaze/boot/Makefile | 10 ++ 1 files changed, 2 insertions(+), 8 deletions(-) diff --git a/arch/microblaze/boot/Makefile b/arch/microblaze/bo

Re: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Micha Nelissen
David Laight wrote: The in_le32() not only contains the unwanted 'sync', but also a 'twi' (trap immediate - NFI exactly what this does) and 'isync'. The 'isync' is particularly horrid and unnecessary (aborts the instruction queue and refeches the opcode bytes). I've also wondered why some time

Re: _extending_ platform support options?

2010-12-01 Thread Benjamin Herrenschmidt
On Wed, 2010-12-01 at 08:47 -0500, Josh Boyer wrote: > >Though, to me, it does not seem really OK to assign ppc_md members > >that way. The original struct machdep for "virtex" (which is defined > >in virtex.c with define_machine()) is not adjusted either. Ok, we > >could modify that one, too. > >E

Re: [PATCH] drivers: char: hvc: add arm JTAG DCC console support

2010-12-01 Thread Stephen Boyd
On 12/01/2010 10:54 AM, Daniel Walker wrote: > Are you talking about __dcc_getstatus only? I don't think adding > volatile is going to hurt anything, if not having it causes problems. > Just marking __dcc_getstatus volatile gives me 0038 : 38: ee10fe11mrc 14, 0, pc, cr0, cr1,

Re: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Scott Wood
On Wed, 1 Dec 2010 15:09:54 + David Laight wrote: > The in_le32() not only contains the unwanted 'sync', but also > a 'twi' (trap immediate - NFI exactly what this does) and 'isync'. It turns a data dependency into a flow dependency. It's basically equivalent to: lwz rX, ... cmpwr

Re: [PATCH V2 2/3] powerpc: Poll VPA for topology changes and update NUMA maps

2010-12-01 Thread Jesse Larrew
On 11/28/2010 10:44 PM, Benjamin Herrenschmidt wrote: > On Tue, 2010-11-09 at 16:25 -0700, Jesse Larrew wrote: >> From: Jesse Larrew >> >> This patch sets a timer during boot that will periodically poll the >> associativity change counters in the VPA. When a change in >> associativity is detected

Re: [PATCH V2 2/3] powerpc: Poll VPA for topology changes and update NUMA maps

2010-12-01 Thread Benjamin Herrenschmidt
On Wed, 2010-12-01 at 16:50 -0500, Jesse Larrew wrote: > > Hmmm... Good point. That would eliminate a lot of complexity, and if > we wrap the code in the timer interrupt so that it only executes on > systems with the VPHN feature, then partition migration pretty much > takes care of itself as well

[PATCH V3 0/3][RFC] Add Support for Virtual Processor Home Node (VPHN)

2010-12-01 Thread Jesse Larrew
From: Jesse Larrew The SPLPAR option allows the platform to dispatch virtual processors on physical processors that, due to the variable nature of work loads, are temporarily free, thus improving the utilization of computing resources. However, SPLPAR implies inconsistent mapping of virtual to ph

[PATCH V3 1/3] powerpc: Add VPHN firmware feature

2010-12-01 Thread Jesse Larrew
From: Jesse Larrew This simple patch adds the firmware feature for VPHN to the firmware features bitmask. Signed-off-by: Jesse Larrew --- arch/powerpc/include/asm/firmware.h |3 ++- arch/powerpc/include/asm/hvcall.h |3 ++- arch/powerpc/platforms/pseries/firmware.c |

[PATCH V3 2/3] powerpc: Poll VPA for topology changes and update NUMA maps

2010-12-01 Thread Jesse Larrew
From: Jesse Larrew This patch sets a timer during boot that will periodically poll the associativity change counters in the VPA. When a change in associativity is detected, it retrieves the new associativity domain information via the H_HOME_NODE_ASSOCIATIVITY hcall and updates the NUMA node m

[PATCH V3 3/3] powerpc: Disable VPHN polling during a suspend operation

2010-12-01 Thread Jesse Larrew
From: Jesse Larrew Tie the polling mechanism into the ibm,suspend-me rtas call to stop/restart polling before/after a suspend, hibernate, migrate, or checkpoint restart operation. This ensures that the system has a chance to disable the polling if the partition is migrated to a system that does n

[PATCH] mpc52xx: gpt: include fs.h

2010-12-01 Thread Wolfram Sang
Fix build errors like these from a randconfig: src/arch/powerpc/platforms/52xx/mpc52xx_gpt.c:549: error: dereferencing pointer to incomplete type: 1 errors in 1 logs src/arch/powerpc/platforms/52xx/mpc52xx_gpt.c:636: error: implicit declaration of function 'nonseekable_open': 1 errors in 1 logs

Re: [PATCH] of/address: use propper endianess in get_flags

2010-12-01 Thread Benjamin Herrenschmidt
On Wed, 2010-12-01 at 10:54 +0100, Sebastian Andrzej Siewior wrote: > This patch changes u32 to __be32 for all "ranges", "prop" and "addr" and > such. Those variables are pointing to the device tree which containts > intergers in big endian format. > Most functions are doing it right because of_rea