[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

[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(),

Re: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Jean-Michel Hautbois
2010/12/1 Jean-Michel Hautbois jhautb...@gmail.com: 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

[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,

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

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 this

_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

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

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_single

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: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Jean-Michel Hautbois
2010/12/1 David Laight david.lai...@aculab.com: 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

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 that I

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 node

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 darg...@lpsc.in2p3.fr 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 {

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 + select HVC_DRIVER

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 + bool

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

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie dirk.brande...@gmail.com 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

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

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie dirk.brande...@gmail.com 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

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

2010-12-01 Thread dirk . brandewie
From: Dirk Brandewie dirk.brande...@gmail.com 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 dirk.brande...@gmail.com --- arch/x86/platform/ce4100/Makefile | 10

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

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

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

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

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. Especially

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 hvc_dcc_get_chars: 38: ee10fe11mrc 14, 0,

Re: [MPC52xx]Latency issue with DMA on FEC

2010-12-01 Thread Scott Wood
On Wed, 1 Dec 2010 15:09:54 + David Laight david.lai...@aculab.com 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

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 jlar...@linux.vnet.ibm.com This patch sets a timer during boot that will periodically poll the associativity change counters in the VPA. When a change in

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 jlar...@linux.vnet.ibm.com 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

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

2010-12-01 Thread Jesse Larrew
From: Jesse Larrew jlar...@linux.vnet.ibm.com This simple patch adds the firmware feature for VPHN to the firmware features bitmask. Signed-off-by: Jesse Larrew jlar...@linux.vnet.ibm.com --- arch/powerpc/include/asm/firmware.h |3 ++- arch/powerpc/include/asm/hvcall.h |3

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

2010-12-01 Thread Jesse Larrew
From: Jesse Larrew jlar...@linux.vnet.ibm.com 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

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

2010-12-01 Thread Jesse Larrew
From: Jesse Larrew jlar...@linux.vnet.ibm.com 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

[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