Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-27 Thread Benjamin Herrenschmidt
On Wed, 2009-05-27 at 23:42 -0500, Kumar Gala wrote: > Ben, > > Any comments on this.. need a decision so we can have patches ready > for .31. > > Clamping the DMA mask is even worse than the additional indirection > > for us. We have valid scenarios in which we'd have 512M of outbound > >

Re: powerpc: DMA coherent allocations broken for CONFIG_NOT_COHERENT_CACHE

2009-05-27 Thread Grant Likely
On Wed, May 27, 2009 at 11:00 PM, Sean MacLennan wrote: > On Wed, 27 May 2009 21:42:18 -0600 > Grant Likely wrote: > >> Make your driver use a platform device or an of_platform device.  It's >> not at all hard. > > Here is my first shot any other fields that I need to fill in so I > don't hav

Re: powerpc: DMA coherent allocations broken for CONFIG_NOT_COHERENT_CACHE

2009-05-27 Thread Sean MacLennan
On Wed, 27 May 2009 21:42:18 -0600 Grant Likely wrote: > Make your driver use a platform device or an of_platform device. It's > not at all hard. Here is my first shot any other fields that I need to fill in so I don't have any gotchas? /* This must exist */ static void warp_device_release

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-27 Thread Kumar Gala
Ben, Any comments on this.. need a decision so we can have patches ready for .31. - k On May 19, 2009, at 8:04 AM, Kumar Gala wrote: On May 18, 2009, at 4:49 PM, Benjamin Herrenschmidt wrote: On Mon, 2009-05-18 at 08:25 -0500, Kumar Gala wrote: Part of this is how the generic swiotlb

Re: powerpc: DMA coherent allocations broken for CONFIG_NOT_COHERENT_CACHE

2009-05-27 Thread Benjamin Herrenschmidt
On Thu, 2009-05-28 at 00:11 -0400, Sean MacLennan wrote: > On Thu, 28 May 2009 13:52:29 +1000 > Benjamin Herrenschmidt wrote: > > > Can't you set ISA_DMA_THRESHOLD = ~0L from your warp.c platform file ? > > I actually set it in the driver proper since it is faster to test, but > it works. I am j

Re: powerpc: DMA coherent allocations broken for CONFIG_NOT_COHERENT_CACHE

2009-05-27 Thread Sean MacLennan
On Thu, 28 May 2009 13:52:29 +1000 Benjamin Herrenschmidt wrote: > Can't you set ISA_DMA_THRESHOLD = ~0L from your warp.c platform file ? I actually set it in the driver proper since it is faster to test, but it works. I am just wondering how kosher that is. The advantage of the platform_device

Re: powerpc: DMA coherent allocations broken for CONFIG_NOT_COHERENT_CACHE

2009-05-27 Thread Benjamin Herrenschmidt
On Wed, 2009-05-27 at 23:34 -0400, Sean MacLennan wrote: > On Mon, 25 May 2009 14:33:43 +1000 ../.. You can just make it a platform device I suppose. In the meantime... > Maybe set ISA_DMA_THRESHOLD somewhere? Some platforms seem to set it: > > ./platforms/52xx/efika.c: ISA_DMA_THRESHOL

Re: powerpc: DMA coherent allocations broken for CONFIG_NOT_COHERENT_CACHE

2009-05-27 Thread Grant Likely
On Wed, May 27, 2009 at 9:34 PM, Sean MacLennan wrote: > On Mon, 25 May 2009 14:33:43 +1000 > Benjamin Herrenschmidt wrote: > >> This is going to .30 if nobody hollers. I've done some testing here >> and it seems to be fine, but more eyes at this stage are much welcome. > > Sigh, I didn't get a c

Re: powerpc: DMA coherent allocations broken for CONFIG_NOT_COHERENT_CACHE

2009-05-27 Thread Sean MacLennan
On Mon, 25 May 2009 14:33:43 +1000 Benjamin Herrenschmidt wrote: > This is going to .30 if nobody hollers. I've done some testing here > and it seems to be fine, but more eyes at this stage are much welcome. Sigh, I didn't get a chance to look at this until tonight. I use __dma_alloc_coherent in

Re: [Solved/Patch Question] Weird 5200/mtd-ram problem

2009-05-27 Thread Jon Smirl
On Wed, May 27, 2009 at 3:54 PM, Albrecht Dreß wrote: > Hi all: > > Am 25.05.09 23:47 schrieb(en) Wolfram Sang: >>> >>> A word or long copy of 0x0055aaff with U-Boot works fine, but a byte copy >>> filled the whole ram with 0x.  The reason is apparently that the >>> chip is attached to t

Re: [PATCH] v2: Display processor virtualization resource allocations in lparcfg

2009-05-27 Thread Michael Ellerman
On Wed, 2009-05-27 at 09:41 -0500, Nathan Fontenot wrote: > This patch updates the output from /proc/ppc64/lparcfg to display the > processor virtualization resource allocations for a shared processor > partition. > > This information is already gathered via the h_get_ppp call, we just > have to m

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-27 Thread Becky Bruce
On May 27, 2009, at 3:29 PM, Ian Campbell wrote: On Wed, 2009-05-27 at 15:05 -0400, Becky Bruce wrote: On May 22, 2009, at 6:11 AM, Ian Campbell wrote: BTW do you need swiotlb_bus_to_virt to be __weak or is the fact that it is implemented in terms of swiotlb_bus_to_phys sufficient? The de

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-27 Thread Ian Campbell
On Wed, 2009-05-27 at 15:05 -0400, Becky Bruce wrote: > On May 22, 2009, at 6:11 AM, Ian Campbell wrote: > > > > > > BTW do you need swiotlb_bus_to_virt to be __weak or is the fact that > > it > > is implemented in terms of swiotlb_bus_to_phys sufficient? > > The default one in swiotlb calls phy

[PATCH] powerpc: tiny memcpy_(to|from)io optimisation

2009-05-27 Thread Albrecht Dreß
This trivial patch changes memcpy_(to|from)io as to transfer as many 32-bit words as possible in 32-bit accesses (in the current solution, the last 32-bit word was transferred as 4 byte accesses). Signed-off-by: Albrecht Dreß --- diff -urpN -X linux-2.6.29.1.orig/Documentation/dontdiff li

[Solved/Patch Question] Weird 5200/mtd-ram problem

2009-05-27 Thread Albrecht Dreß
Hi all: Am 25.05.09 23:47 schrieb(en) Wolfram Sang: A word or long copy of 0x0055aaff with U-Boot works fine, but a byte copy filled the whole ram with 0x. The reason is apparently that the chip is attached to the local bus in 16-bit mode, which is incompatible with byte accesses.

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-27 Thread Becky Bruce
On May 26, 2009, at 7:51 AM, Ian Campbell wrote: On Fri, 2009-05-22 at 19:55 -0400, Jeremy Fitzhardinge wrote: Ian Campbell wrote: On Thu, 2009-05-21 at 14:27 -0400, Becky Bruce wrote: I can work with that, but it's going to be a bit inefficient, as I actually need the dma_addr_t, not the p

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-27 Thread Becky Bruce
On May 22, 2009, at 5:51 AM, FUJITA Tomonori wrote: On Thu, 21 May 2009 13:18:54 -0700 Jeremy Fitzhardinge wrote: Becky Bruce wrote: I can work with that, but it's going to be a bit inefficient, as I actually need the dma_addr_t, not the phys_addr_t, so I'll have to convert. In every case,

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-27 Thread Becky Bruce
On May 22, 2009, at 6:11 AM, Ian Campbell wrote: BTW do you need swiotlb_bus_to_virt to be __weak or is the fact that it is implemented in terms of swiotlb_bus_to_phys sufficient? The default one in swiotlb calls phys_to_virt on the result of swiotlb_bus_to_phys, which only works on low

[PATCH v3] powerpc: add ioremap_early() for mapping IO regions before MMU_init()

2009-05-27 Thread Grant Likely
From: Grant Likely ioremap_early() is useful for things like mapping SoC internally registers and early debug output because it allows mappings to devices to be setup early in the boot process where they are needed. It also give a performance boost since BAT mapped registers don't get flushed ou

[PATCH] v2: Display processor virtualization resource allocations in lparcfg

2009-05-27 Thread Nathan Fontenot
This patch updates the output from /proc/ppc64/lparcfg to display the processor virtualization resource allocations for a shared processor partition. This information is already gathered via the h_get_ppp call, we just have to make sure that the ibm,partition-performance-parameters-level property

Re: [PATCH] Display processor virtualization resource allocations in lparcfg

2009-05-27 Thread Nathan Fontenot
Michael Ellerman wrote: + root = of_find_node_by_path("/"); + if (root) { + perf_level = of_get_property(root, + "ibm,partition-performance-parameters-level", +NULL); What if there is no such pro

Re: [alsa-devel] [PATCH V2] Modify mpc5200 AC97 driver to use V9 of spin_event_timeout()

2009-05-27 Thread Timur Tabi
On Tue, May 26, 2009 at 11:30 PM, Jon Smirl wrote: > The function signature for spin_event_timeout() has changed in version V9. > Adjust the mpc5200 AC97 driver to use the new function. > v2 - switch back to udelay() for fixed timeouts. > > Signed-off-by: Jon Smirl Acked-by: Timur Tabi -- Tim

Re: Net: ucc_geth ethernet driver optimization space

2009-05-27 Thread Li Yang
On Wed, May 27, 2009 at 1:08 PM, Liu Dave-R63238 wrote: > Guys, > > The ucc_geth ethernet driver have dozens of strong sync read/write > operation, such as in_be32/16/8, out_be32/16/8. > > all of them is sync read/write, it is very expensive for performance. > Totally agree. That's one of my con

Re: [GIT PULL] fsldma driver fixes

2009-05-27 Thread Li Yang
On Wed, May 27, 2009 at 7:59 AM, Dan Williams wrote: > On Fri, May 22, 2009 at 3:47 AM, Li Yang wrote: >> Hi Dan, >> >> Here are fixes for Freescale DMA engine driver. >> >> Thanks, >> - Leo >> >> >> The following changes since commit 5805977e63a36ad56594a623f3bd2bebcb7db233: >>  Linus Torvalds (

Re: [U-Boot] [PATCH 1/2] 83xx: Replace CONFIG_MPC83XX with CONFIG_MPC83xx

2009-05-27 Thread Stefan Roese
On Wednesday 27 May 2009 01:38:34 Kim Phillips wrote: > > > If others would prefer standardizing on uppercase, I can submit new > > > patches for 85xx, 5xxx, 86xx, etc. This would be still be inconsistent > > > with Linux though, which is a bit annoying. > > > > It seems Linux uses 8?xx with very