[PATCH] Fix physical address range check comparision and force address to be virtual address

2014-02-27 Thread Benjamin Krill
The previous code added wrong TLBs and causes machine check errors. Signed-off-by: Ralph E. Bellofatto Signed-off-by: Benjamin Krill --- arch/powerpc/mm/tlb_low_64e.S | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/mm/tlb_low_64e.S b/arch/powerpc/mm

[PATCH] MTD ofpart: Check availability of reg property instead of name property

2009-08-25 Thread Benjamin Krill
The previous implementation breaks the dts binding "mtd-physmap.txt". This implementation fixes the issue by checking the availability of the reg property instead of the name property. Signed-off-by: Benjamin Krill --- drivers/mtd/ofpart.c | 21 ++--- 1 files c

Re: [PATCH v2] Check name property to determine partition nodes.

2009-02-17 Thread Benjamin Krill
* Benjamin Herrenschmidt | 2009-02-04 15:09:02 [+1100]: >On Fri, 2009-01-23 at 17:18 +0100, Benjamin Krill wrote: >> SLOF has a further node which could not be evaluate >> by the current routine. The current routine returns >> because the node hasn't the required reg prop

Re: [PATCH 1/3] powerpc/pci: Default to dma_direct_ops for pci dma_ops

2009-02-19 Thread Benjamin Krill
* Kumar Gala | 2009-02-19 14:49:15 [-0600]: >This will allow us to remove the ppc32 specific checks in get_dma_ops() >that defaults to dma_direct_ops if the archdata is NULL. We really >should always have archdata set to something going forward. > >Signed-off-by: Kumar Gala Ack

Re: [PATCH 2/3] powerpc: setup archdata for {of_}platform via a single platform_notify

2009-02-19 Thread Benjamin Krill
jority of these devices only need the >direct_dma_ops as the platforms don't have any IOMMUs. > >In the future to enable >4G DMA support on ppc32 we can hook swiotlb ops. > >Signed-off-by: Kumar Gala Acked-by: Benjamin Krill ___

Re: [PATCH 3/3] powerpc: expect all devices calling dma ops to have archdata set

2009-02-19 Thread Benjamin Krill
er should be fixed. > >Signed-off-by: Kumar Gala Acked-by: Benjamin Krill ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

[PATCH] Add rtas rtc calls for the QPACE platform

2009-02-20 Thread Benjamin Krill
The new firmware release exports further RTC calls. This patch adds these calls to the QPACE platform setup file. Signed-off-by: Benjamin Krill --- arch/powerpc/platforms/cell/qpace_setup.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/platforms/cell

Re: [PATCH v2 1/3] powerpc/pci: Default to dma_direct_ops for pci dma_ops

2009-03-20 Thread Benjamin Krill
* Kumar Gala | 2009-03-19 08:40:50 [-0500]: >This will allow us to remove the ppc32 specific checks in get_dma_ops() >that defaults to dma_direct_ops if the archdata is NULL. We really >should always have archdata set to something going forward. > >Signed-off-by: Kumar Gala Ack

Re: [PATCH v2 2/3] powerpc: setup default archdata for {of_}platform via bus_register_notifier

2009-03-20 Thread Benjamin Krill
nable >4G DMA support on ppc32 we can hook swiotlb ops. > >Signed-off-by: Kumar Gala Acked-by: Benjamin Krill Tested on QPACE node card. ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH v2 3/3] powerpc: expect all devices calling dma ops to have archdata set

2009-03-20 Thread Benjamin Krill
er should be fixed. > >Signed-off-by: Kumar Gala Acked-by: Benjamin Krill Tested on QPACE node card. ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH v2] Check name property to determine partition nodes.

2009-03-24 Thread Benjamin Krill
>>On Fri, 2009-01-23 at 17:18 +0100, Benjamin Krill wrote: >>> SLOF has a further node which could not be evaluate >>> by the current routine. The current routine returns >>> because the node hasn't the required reg property. As >>> fix this patch ad

Re: [PATCH] powerpc: PPC_CELL_NATIVE should select PCI

2009-04-02 Thread Benjamin Krill
* Arnd Bergmann | 2009-04-02 15:15:11 [+0200]: >BenK, can you try Geert's patch on QPACE? Successfully tested on QPACE. Acked-by: Benjamin Krill ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: ofpart: Partitions at same address cannot have the same name

2009-04-22 Thread Benjamin Krill
>--- a/drivers/mtd/ofpart.c >+++ b/drivers/mtd/ofpart.c >@@ -48,7 +48,7 @@ int __devinit of_mtd_parse_partitions(struct device *dev, > > /* check if this is a partition node */ > partname = of_get_property(pp, "name", &len); >- if (strcmp(partname, "partiti

Re: ofpart: Partitions at same address cannot have the same name

2009-04-22 Thread Benjamin Krill
* Ricardo Ribalda Delgado | 2009-04-22 19:59:08 [+0200]: >> >>                if (strcmp(partname, "partition") <= 0) { > >Anything alfabetically higher than partition (like "z" will pass >the test :S) You are totally right! cheers ben ___ Linuxppc

Re: [PATCH] [MTD] ofpart: Partitions at same address cannot have the same name v3

2009-04-27 Thread Benjamin Krill
continue; > } If this is the way, how to go, you get my ack. Acked-by: Benjamin Krill ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

[PATCH] Fix wrong register read address and add interrupt acknowledge.

2009-05-05 Thread Benjamin Krill
The receive interrupt routine checks the wrong register if the receive fifo is empty. Further an explicit interrupt acknowledge write is introduced. In some circumstances another interrupt was issued. Signed-off-by: Benjamin Krill --- drivers/serial/nwpserial.c |4 +++- 1 files changed, 3

Re: [PATCH] Fix wrong register read address and add interrupt acknowledge.

2009-05-08 Thread Benjamin Krill
* Josh Boyer | 2009-05-05 19:09:19 [-0400]: >On Tue, May 05, 2009 at 02:24:04PM -0500, Timur Tabi wrote: >>On Tue, May 5, 2009 at 2:11 PM, Benjamin Krill wrote: > >That is quite true. Although resending the patch just for that seems overkill. >The maintainer can edit the sum

rtas instantiation when commandline contains mem

2009-07-16 Thread Benjamin Krill
Hi, the rtas instantiation (prom_init.c) doesn't work correctly if the kernel parameter "mem=" is used. The current code doesn't evaluate the kernel parameter which causes the issue that alloc_down allocates somewhere in the "real" memory space. So it can happen that the allocation space is above

Re: rtas instantiation when commandline contains mem

2009-07-16 Thread Benjamin Krill
>> the rtas instantiation (prom_init.c) doesn't work correctly if the >> kernel parameter "mem=" is used. The current code doesn't evaluate >> the kernel parameter which causes the issue that alloc_down >> allocates somewhere in the "real" memory space. So it can >> happen that the allocation space

[PATCH] prom_init: evaluate mem kernel parameter for early allocation

2009-07-17 Thread Benjamin Krill
Evaluate mem kernel parameter for early memory allocations. If mem is set no allocation in the region above the given boundary is allowed. The current code doesn't take care about this and allocate memory above the given mem boundary. Signed-off-by: Benjamin Krill --- arch/powerpc/k

[PATCH] [v2] prom_init: evaluate mem kernel parameter for early allocation

2009-07-28 Thread Benjamin Krill
Evaluate mem kernel parameter for early memory allocations. If mem is set no allocation in the region above the given boundary is allowed. The current code doesn't take care about this and allocate memory above the given mem boundary. Signed-off-by: Benjamin Krill --- arch/powerpc/k

Re: Spansion S25FL128-Flash and MTD

2009-08-05 Thread Benjamin Krill
Hi, >#address-cells = <1>; >#size-cells = <1>; >roo...@80 { The partition must be called partit...@... >label = "rootfs"; >reg = <0x80 0x80>; >}; >}; >}; c

Re: [PATCH] Add support for the nwp serial device

2008-11-20 Thread Benjamin Krill
Hi Josh, >What does 'NWP' stand for, and how is it different from a regular serial >port? Also, what platforms can this device be found on currently? > >Some of that should be included in the changelog, but I'm curious as well >too. NWP stands for network processor and it is part of the QPACE -

Re: [PATCH] Add support for the nwp serial device

2008-11-20 Thread Benjamin Krill
* David Gibson | 2008-11-21 11:35:45 [+1100]: > >Uh.. registers as port number 84?? What's this random number about? > It's just the next free number in serial_core.h. Ben ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailma

Re: [PATCH] Add support for the nwp serial device

2008-11-23 Thread Benjamin Krill
* Andrew Morton | 2008-11-21 13:22:44 [-0800]: >(does a powerpc allmodconfig) > >drivers/serial/nwpserial.c:443: warning: data definition has no type or >storage class >drivers/serial/nwpserial.c:443: warning: type defaults to 'int' in declaration >of 'console_initcall' >drivers/serial/nwpserial

Re: [PATCH v2] serial: add support for the cell network processor NWP device

2009-01-07 Thread Benjamin Krill
>My version is mucked up anyway - PORT_NWPSERIAL has the same value as >PORT_S3C6400. Should I send an updated version against a current tree? ben ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

[PATCH] Check name property to determine partition nodes.

2009-01-23 Thread Benjamin Krill
SLOF has a further node which could not be evaluate by the current routine. The current routine returns because the node hasn't the required reg property. As fix this patch adds a check to determine the partition child nodes. Signed-off-by: Benjamin Krill --- drivers/mtd/ofpart.c |

Re: [PATCH] Check name property to determine partition nodes.

2009-01-23 Thread Benjamin Krill
I missed a fix in this patch. I will resend it as v2. cheers ben * Benjamin Krill | 2009-01-23 16:51:42 [+0100]: >SLOF has a further node which could not be evaluate >by the current routine. The current routine returns >because the node hasn't the required reg property. As &

[PATCH v2] Check name property to determine partition nodes.

2009-01-23 Thread Benjamin Krill
ll be decreased and loop continue with the next nodes. Signed-off-by: Benjamin Krill --- drivers/mtd/ofpart.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/drivers/mtd/ofpart.c b/drivers/mtd/ofpart.c index 9e45b3f..3e164f0 100644 --- a/drivers/mtd/ofpart.c +++ b/dr