[PATCH 3/4] 8xx: Don't touch ACCESSED when no SWAP.

2010-02-26 Thread Joakim Tjernlund
Only the swap function cares about the ACCESSED bit in the pte. Do not waste cycles updateting ACCESSED when swap is not compiled into the kernel. --- arch/powerpc/kernel/head_8xx.S |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/head_8xx.S b/arch/

[PATCH 4/4] 8xx: Use SPRG2 and DAR registers to stash r11 and cr.

2010-02-26 Thread Joakim Tjernlund
This avoids storing these registers in memory. CPU6 errata will still use the old way. Remove some G2 leftover accesses from 2.4 --- arch/powerpc/kernel/head_8xx.S | 49 +-- 1 files changed, 36 insertions(+), 13 deletions(-) diff --git a/arch/powerpc/kernel/h

Re: [PATCH 1/4] 8xx: Optimze TLB Miss handlers

2010-02-27 Thread Joakim Tjernlund
Scott Wood wrote on 2010/02/26 20:50:18: > > On Fri, Feb 26, 2010 at 09:29:40AM +0100, Joakim Tjernlund wrote: > > This removes a couple of insn's from the TLB Miss > > handlers whithout changing functionality. > > --- > > Did a quick test of the patchset, se

Re: [PATCH 1/4] 8xx: Optimze TLB Miss handlers

2010-02-27 Thread Joakim Tjernlund
Kumar Gala wrote on 2010/02/26 21:10:31: > > > On Feb 26, 2010, at 2:29 AM, Joakim Tjernlund wrote: > > >li r11, 0x00f0 > > - rlwimi r10, r11, 0, 24, 28 /* Set 24-27, clear 28 */ > > + rlwimi r10, r11, 0, 0x07f8 /* Set 24-27, clear 21-23,28 */

[PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-02 Thread Joakim Tjernlund
uot; these patches too? Joakim Tjernlund (4): 8xx: Optimze TLB Miss handlers 8xx: Avoid testing for kernel space in ITLB Miss. 8xx: Don't touch ACCESSED when no SWAP. 8xx: Use SPRG2 and DAR registers to stash r11 and cr. arch/powerpc/kernel/he

[PATCH 2/4] 8xx: Avoid testing for kernel space in ITLB Miss.

2010-03-02 Thread Joakim Tjernlund
Only modules will cause ITLB Misses as we always pin the first 8MB of kernel memory. Signed-off-by: Joakim Tjernlund --- arch/powerpc/kernel/head_8xx.S |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/kernel/head_8xx.S b/arch/powerpc/kernel/head_8xx.S

[PATCH 1/4] 8xx: Optimze TLB Miss handlers

2010-03-02 Thread Joakim Tjernlund
This removes a couple of insn's from the TLB Miss handlers whithout changing functionality. Signed-off-by: Joakim Tjernlund --- arch/powerpc/kernel/head_8xx.S | 11 +++ 1 files changed, 3 insertions(+), 8 deletions(-) diff --git a/arch/powerpc/kernel/head_8xx.S b/arch/powerpc/k

[PATCH 3/4] 8xx: Don't touch ACCESSED when no SWAP.

2010-03-02 Thread Joakim Tjernlund
Only the swap function cares about the ACCESSED bit in the pte. Do not waste cycles updateting ACCESSED when swap is not compiled into the kernel. Signed-off-by: Joakim Tjernlund --- arch/powerpc/kernel/head_8xx.S |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a

[PATCH 4/4] 8xx: Use SPRG2 and DAR registers to stash r11 and cr.

2010-03-02 Thread Joakim Tjernlund
This avoids storing these registers in memory. CPU6 errata will still use the old way. Remove some G2 leftover accesses from 2.4 Signed-off-by: Joakim Tjernlund --- arch/powerpc/kernel/head_8xx.S | 49 +-- 1 files changed, 36 insertions(+), 13 deletions

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-03 Thread Joakim Tjernlund
Heiko Schocher wrote on 2010/03/03 09:02:47: > > Hello Joakim, > > I tried your 4 patches on a MPC855M based system: Thanks a lot for testing this for me! > > -bash-3.2# cat /proc/cpuinfo > processor : 0 > cpu : 8xx > clock : 66.00MHz > revision: 0.0 (pvr

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-03 Thread Joakim Tjernlund
> > Heiko Schocher wrote on 2010/03/03 09:02:47: > > > > Hello Joakim, > > > > I tried your 4 patches on a MPC855M based system: > > Thanks a lot for testing this for me! > > > > > -bash-3.2# cat /proc/cpuinfo > > processor : 0 > > cpu : 8xx > > clock : 66.00MHz > >

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-03 Thread Joakim Tjernlund
Heiko Schocher wrote on 2010/03/03 11:10:10: > > Hello Joakim, > > Joakim Tjernlund wrote: > > Heiko Schocher wrote on 2010/03/03 09:02:47: > [...] > >> Here the results: > >> (The first 4 rows are the results for the kernel without your patches, > &

Strange OOPS in 2.6.33

2010-03-04 Thread Joakim Tjernlund
Got this OOPS a few times after coldstarting out board a few times: Unable to handle kernel paging request for unknown fault Faulting instruction address: 0xc020e2b4 Oops: Kernel access of bad area, sig: 11 [#1] TMCUTU Modules linked in: NIP: c020e2b4 LR: c020e274 CTR: REGS: c7a41b40 TRA

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-04 Thread Joakim Tjernlund
Wolfgang Denk wrote on 2010/03/04 13:16:56: > From: Wolfgang Denk > To: h...@denx.de > Cc: Joakim Tjernlund , Klaus-Jürgen > , linuxppc-...@ozlabs.org, Scott Wood > > Date: 2010/03/04 13:17 > Subject: Re: [PATCH 0/4] 8xx: Optimize TLB Miss code. > > Dear Heiko, >

Re: Strange OOPS in 2.6.33

2010-03-05 Thread Joakim Tjernlund
> > > Got this OOPS a few times after coldstarting out > board a few times: > > Unable to handle kernel paging request for unknown fault > Faulting instruction address: 0xc020e2b4 > Oops: Kernel access of bad area, sig: 11 [#1] > TMCUTU > Modules linked in: > NIP: c020e2b4 LR: c020e274 CTR: 000

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-05 Thread Joakim Tjernlund
Heiko Schocher wrote on 2010/03/04 17:30:07: > > Hello Joakim, > > Joakim Tjernlund wrote: > > Wolfgang Denk wrote on 2010/03/04 13:16:56: > >> From: Wolfgang Denk > >> To: h...@denx.de > >> Cc: Joakim Tjernlund , Klaus-Jürgen > >> , linuxp

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-07 Thread Joakim Tjernlund
Heiko Schocher wrote on 2010/03/04 17:30:07: > From: Heiko Schocher > To: Joakim Tjernlund > Cc: Wolfgang Denk , Klaus-Jürgen , > linuxppc-...@ozlabs.org, Scott Wood > Date: 2010/03/04 17:30 > Subject: Re: [PATCH 0/4] 8xx: Optimize TLB Miss code. > > Hello Joakim, &g

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-08 Thread Joakim Tjernlund
Heiko Schocher wrote on 2010/03/08 08:46:29: > > Hello Joakim, > > Joakim Tjernlund wrote: > [...] > > What would be interesting is to skip patch 3 and turn off > > MODULES add PIN_TLB and compare that against your unpatched .33 but > > with MODULES off and PIN_T

Re: [PATCH 0/4] 8xx: Optimize TLB Miss code.

2010-03-08 Thread Joakim Tjernlund
Heiko Schocher wrote on 2010/03/08 10:06:39: > > Hello Joakim, > > Joakim Tjernlund wrote: > > Heiko Schocher wrote on 2010/03/08 08:46:29: > >> Hello Joakim, > >> > >> Joakim Tjernlund wrote: > >> [...] > >>> What would be int

Re: [PATCH 2/4] 8xx: Avoid testing for kernel space in ITLB Miss.

2010-03-17 Thread Joakim Tjernlund
Benjamin Herrenschmidt wrote on 2010/03/16 22:19:36: > > On Fri, 2010-02-26 at 09:29 +0100, Joakim Tjernlund wrote: > > +#ifdef CONFIG_MODULES > > + /* Only modules will cause ITLB Misses as we always > > +* pin the first 8MB of kernel memory */ > >

Re: [PATCH 3/4] 8xx: Don't touch ACCESSED when no SWAP.

2010-03-17 Thread Joakim Tjernlund
Benjamin Herrenschmidt wrote on 2010/03/16 22:20:52: > > On Fri, 2010-02-26 at 09:29 +0100, Joakim Tjernlund wrote: > > Only the swap function cares about the ACCESSED bit in > > the pte. Do not waste cycles updateting ACCESSED when swap > > is not compiled into the

Re: Question regarding the DTLB Miss exceptions

2010-03-31 Thread Joakim Tjernlund
> > I'm tracking a problem that's leading me through DSI and DTLB miss > exceptions on an MPC8347 (e300c1 core), and I've come across an oddity > that I'm hoping someone can explain. > > When a DTLB Miss exception can't find a PTE for the virtual address being > written/read, it dummies up the SPRs

Re: FTrace on MPC8xx

2011-04-13 Thread Joakim Tjernlund
Stefan Roese wrote on 2011/04/13 16:32:21: > > Hi, > > I noticed that ftrace doesn't seem to work on MPC8xx. I'm trying to use it on > an MPC855T board, but ftrace oopses upon startup: > > [0.028785] ftrace: allocating 10312 entries in 31 pages > [0.038594] [ cut here ]

Re: FTrace on MPC8xx

2011-04-13 Thread Joakim Tjernlund
Stefan Roese wrote on 2011/04/13 17:21:33: > > Hi Joakim, > > On Wednesday 13 April 2011 16:58:21 Joakim Tjernlund wrote: > > > This is on a 2.6.38 kernel (2.6.32 fails too). Debugging shows that > > > __copy_tofrom_user() fails with return code 4 (calle

Re: FTrace on MPC8xx

2011-04-14 Thread Joakim Tjernlund
Stefan Roese wrote on 2011/04/14 17:59:30: > > Hi Joakim, > > On Wednesday 13 April 2011 17:38:03 Joakim Tjernlund wrote: > > > > How big was the size to copy_tofrom_user()? Did it mange to copy > > > > any bytes? > > > > > > The size

Re: FTrace on MPC8xx

2011-04-14 Thread Joakim Tjernlund
> > Stefan Roese wrote on 2011/04/14 17:59:30: > > > > Hi Joakim, > > > > On Wednesday 13 April 2011 17:38:03 Joakim Tjernlund wrote: > > > > > How big was the size to copy_tofrom_user()? Did it mange to copy > > > > > any bytes? >

Re: FTrace on MPC8xx

2011-04-15 Thread Joakim Tjernlund
Stefan Roese wrote on 2011/04/15 09:22:42: > Hi Joakim, > > On Thursday 14 April 2011 21:21:23 Joakim Tjernlund wrote: > > > hmm, I guess 8xx really maps kernel RO as RO :) Try > > > > > > changing in pte-8xx.h: > > > - #define _PAGE_KERNEL_RO (_

[PATCH 06/15] 8xx: Always pin kernel instruction TLB

2011-06-14 Thread Joakim Tjernlund
Various kernel asm modifies SRR0/SRR1 just before executing a rfi. If such code crosses a page boundary you risk a TLB miss which will clobber SRR0/SRR1. Avoid this by always pinning kernel instruction TLB space. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S |9 + 1

[PATCH 07/15] 8xx: Fixup DAR from buggy dcbX instructions.

2011-06-14 Thread Joakim Tjernlund
This is an assembler version to fixup DAR not being set by dcbX, icbi instructions. There are two versions, one uses selfmodifing code, the other uses a jump table but is much bigger(default). Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 149

[PATCH 02/15] 8xx: Tag DAR with 0x00f0 to catch buggy instructions.

2011-06-14 Thread Joakim Tjernlund
dcbz, dcbf, dcbi, dcbst and icbi do not set DAR when they cause a DTLB Error. Dectect this by tagging DAR with 0x00f0 at every exception exit that modifies DAR. This also fixes MachineCheck to pass DAR and DSISR as well. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 18

[PATCH 05/15] 8xx: Update TLB asm so it behaves as linux mm expects.

2011-06-14 Thread Joakim Tjernlund
space. - Free up 2 SW TLB bits in the linux pte(add back _PAGE_WRITETHRU ?) - kernel RO/user NA support. Not sure this is really needed, would save a few insn if not required. Cons: - A few more instructions in the DTLB Miss routine. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel

[PATCH 09/15] 8xx: Add missing Guarded setting in DTLB Error.

2011-06-14 Thread Joakim Tjernlund
only DTLB Miss did set this bit, DTLB Error needs too otherwise the setting is lost when the page becomes dirty. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 12 +--- 1 files changed, 9 insertions(+), 3 deletions(-) diff --git a/arch/ppc/kernel/head_8xx.S b/arch

[PATCH 15/15] 8xx: Optimize TLB Miss handlers

2011-06-14 Thread Joakim Tjernlund
Only update pte w.r.t ACCESSED if it isn't already set Wrap ACCESSED with #ifndef NO_SWAP for too ease optimization. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 11 +-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/arch/ppc/kernel/head_8x

[PATCH 03/15] 8xx: invalidate non present TLBs

2011-06-14 Thread Joakim Tjernlund
8xx sometimes need to load a invalid/non-present TLBs in it DTLB asm handler. These must be invalidated separately as 8xx MMU don't. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 12 ++-- 1 files changed, 10 insertions(+), 2 deletions(-) diff --git a/arc

[PATCH 08/15] 8xx: CPU6 errata make DTLB error too big to fit.

2011-06-14 Thread Joakim Tjernlund
branch to common code in DTLB Miss instead. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 23 ++- 1 files changed, 2 insertions(+), 21 deletions(-) diff --git a/arch/ppc/kernel/head_8xx.S b/arch/ppc/kernel/head_8xx.S index f4c3f41..44e79b3 100644 --- a

[PATCH 00/15] Backport 8xx TLB to 2.4

2011-06-14 Thread Joakim Tjernlund
very found during development. I know 2.4 is in strict maintenance mode and 8xx is obsolete but as it is still in use I wanted 8xx to age with grace. Addendum: I have now ported our 8xx custom board to 2.4.37.11 and tested these patches there. Joakim Tjernlund (15): 8xx: Use a macro to simpliy

[PATCH 14/15] 8xx: Optimize ITLBMiss handler.

2011-06-14 Thread Joakim Tjernlund
Don't check for kernel space if no modules. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/arch/ppc/kernel/head_8xx.S b/arch/ppc/kernel/head_8xx.S index a4290b5..7f9e647 100644 --- a/arch/ppc/k

[PATCH 13/15] 8xx: Use symbolic constants in TLB asm

2011-06-14 Thread Joakim Tjernlund
Use the PTE #defines where possible instead of hardcoded constants. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/ppc/kernel/head_8xx.S b/arch/ppc/kernel/head_8xx.S index d701542..a4290b5

[PATCH 10/15] 8xx: Restore _PAGE_WRITETHRU

2011-06-14 Thread Joakim Tjernlund
8xx has not had WRITETHRU due to lack of bits in the pte. After the recent rewrite of the 8xx TLB code, there are two bits left. Use one of them to WRITETHRU. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S |8 include/asm-ppc/pgtable.h |5 +++-- 2 files changed

[PATCH 04/15] 8xx: Fix CONFIG_PIN_TLB

2011-06-14 Thread Joakim Tjernlund
The wrong register was loaded into MD_RPN. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/ppc/kernel/head_8xx.S b/arch/ppc/kernel/head_8xx.S index b3aff21..9d8a1b5 100644 --- a/arch/ppc/kernel

[PATCH 12/15] 8xx: start using dcbX instructions in various copy routines

2011-06-14 Thread Joakim Tjernlund
Now that 8xx can fixup dcbX instructions, start using them where possible like every other PowerPc arch do. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/misc.S | 18 -- arch/ppc/lib/string.S | 17 - 2 files changed, 0 insertions(+), 35 deletions

[PATCH 11/15] 8xx: Set correct HW pte flags in DTLB Error too

2011-06-14 Thread Joakim Tjernlund
DTLB Error needs to adjust the HW PTE bits as DTLB Miss does. Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/arch/ppc/kernel/head_8xx.S b/arch/ppc/kernel/head_8xx.S index a29a011..d701542 100644 --- a

[PATCH 01/15] 8xx: Use a macro to simpliy CPU6 errata code.

2011-06-14 Thread Joakim Tjernlund
Signed-off-by: Joakim Tjernlund --- arch/ppc/kernel/head_8xx.S | 84 +++ 1 files changed, 22 insertions(+), 62 deletions(-) diff --git a/arch/ppc/kernel/head_8xx.S b/arch/ppc/kernel/head_8xx.S index f9a30f3..ba05a57 100644 --- a/arch/ppc/kernel

Re: [PATCH 06/15] 8xx: Always pin kernel instruction TLB

2011-06-14 Thread Joakim Tjernlund
Dan Malek wrote on 2011/06/14 18:06:45: > > > Hi Joakim. > > On Jun 14, 2011, at 6:54 AM, Joakim Tjernlund wrote: > > > Various kernel asm modifies SRR0/SRR1 just before executing > > a rfi. . > > I'm going to argue we can easily visually inspect for

Re: [PATCH 00/15] Backport 8xx TLB to 2.4

2011-06-14 Thread Joakim Tjernlund
Dan Malek wrote on 2011/06/14 18:11:51: > > > Hi Joakim. > > On Jun 14, 2011, at 6:54 AM, Joakim Tjernlund wrote: > > > I know 2.4 is in strict maintenance mode and 8xx is obsolete > > but as it is still in use I wanted 8xx to age with grace. > > Thanks

Re: [PATCH 06/15] 8xx: Always pin kernel instruction TLB

2011-06-14 Thread Joakim Tjernlund
Dan Malek wrote on 2011/06/14 20:11:18: > > > Hi Joakim. > > On Jun 14, 2011, at 11:00 AM, Joakim Tjernlund wrote: > > > I don't have a mpc850, do you? > > I have to say I do :-) Good, you get to beat the crap out of it then :) > > > Probably but tha

Re: [PATCH 06/15] 8xx: Always pin kernel instruction TLB

2011-06-15 Thread Joakim Tjernlund
Joakim Tjernlund/Transmode wrote on 2011/06/14 20:00:09: > From: Joakim Tjernlund/Transmode > > Dan Malek wrote on 2011/06/14 18:06:45: > > > > > > Hi Joakim. > > > > On Jun 14, 2011, at 6:54 AM, Joakim Tjernlund wrote: > > > > > Various kerne

Re: [PATCH 00/15] Backport 8xx TLB to 2.4

2011-06-15 Thread Joakim Tjernlund
Willy Tarreau wrote on 2011/06/14 21:31:06: > > Hi Joakim, > > On Tue, Jun 14, 2011 at 03:54:45PM +0200, Joakim Tjernlund wrote: > > This is a backport from 2.6 which I did to overcome 8xx CPU > > bugs. 8xx does not update the DAR register when taking a TLB > > error

Re: [PATCH 06/15] 8xx: Always pin kernel instruction TLB

2011-06-15 Thread Joakim Tjernlund
Dan Malek wrote on 2011/06/14 20:11:18: > > Hi Joakim. > > On Jun 14, 2011, at 11:00 AM, Joakim Tjernlund wrote: > > > I don't have a mpc850, do you? > > I have to say I do :-) > > > Probably but that is another matter. You could continue with that &

[RFC] Optimize __arch_swab32 and __arch_swab16

2011-08-11 Thread Joakim Tjernlund
PPC __arch_swab32 and __arch_swab16 generates non optimal code. It doesn't schedule very well, need to copy its input register and and swab16 needs an extra insn to clear its upper bits. I have improved these functions(see my__xx). Any problem with the new asm? If not I will send a patch. Below s

Re: [RFC] Optimize __arch_swab32 and __arch_swab16

2011-08-11 Thread Joakim Tjernlund
Andreas Schwab wrote on 2011/08/11 10:45:42: > > Joakim Tjernlund writes: > > > unsigned short my__arch_swab16(unsigned short value) > > { > >__asm__("rlwimi %0,%0,16,0x00ff" > > : "+r" (value)); > > You are creating a v

RE: [RFC] Optimize __arch_swab32 and __arch_swab16

2011-08-11 Thread Joakim Tjernlund
"David Laight" wrote on 2011/08/11 10:56:26: > > > Joakim Tjernlund writes: > > > > > unsigned short my__arch_swab16(unsigned short value) > > > { > > >__asm__("rlwimi %0,%0,16,0x00ff" > > > : "+r" (

RE: [RFC] Optimize __arch_swab32 and __arch_swab16

2011-08-11 Thread Joakim Tjernlund
"David Laight" wrote on 2011/08/11 11:29:33: > > > > > Which is a problem because the compiler could schedule > > > it be written back to real memory between the instructions. > > > > It can? There is no memory here, just registers. Even if it > > is written to memory, how would that affect the re

[PATCH] powerpc: fixup QE_General4 errata

2011-08-23 Thread Joakim Tjernlund
QE_General4 should only round up the divisor iff divisor is > 3. Rounding up lower divisors makes the error too big, causing USB on MPC832x to fail. Signed-off-by: Joakim Tjernlund --- arch/powerpc/sysdev/qe_lib/qe.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/a

[PATCH] usb: Allocate pram dynamically.

2011-08-23 Thread Joakim Tjernlund
MPC832x does not have enough MURAM to do fixed MURAM allocation. Change to dynamic allocation. Signed-off-by: Joakim Tjernlund --- drivers/usb/host/fhci-hcd.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/drivers/usb/host/fhci-hcd.c b/drivers/usb/host/fhci-hcd.c

Re: [PATCH] usb: Allocate pram dynamically.

2011-08-23 Thread Joakim Tjernlund
Anton Vorontsov wrote on 2011/08/23 15:02:53: > From: Anton Vorontsov > To: Joakim Tjernlund > Cc: linuxppc-dev@lists.ozlabs.org > Date: 2011/08/23 15:02 > Subject: Re: [PATCH] usb: Allocate pram dynamically. > > On Tue, Aug 23, 2011 at 02:38:41PM +0200, Joakim Tjernlu

RE: JFFS2 corruption when mounting filesystem with filenames oflength> 7

2010-06-26 Thread Joakim Tjernlund
> > > -Original Message- > > From: linux-mtd-boun...@lists.infradead.org > > [mailto:linux-mtd-boun...@lists.infradead.org] On Behalf Of > > Steve Deiters > > Sent: Thursday, June 24, 2010 3:02 PM > > To: linux-...@lists.infradead.org > > Subject: RE: JFFS2 corruption when mounting filesyst

Re: machine check in kernel for a mpc870 board

2010-06-29 Thread Joakim Tjernlund
> From: Shawn Jin > > I'm porting a mpc870 board to the powerpc arch. The base is the > adder-875 board. My first try to boot the cuImage.my870 is > experiencing a machine check. And I have no clue where to look. Any > help? > > => bootm 100 > ## Booting image at 0100 ... >Image Name:

CONFIG_KTIME_SCALAR=y

2010-09-10 Thread Joakim Tjernlund
Noticed that there is a CONFIG_KTIME_SCALAR knob for HIGH_RES on 32 bit which is off for ppc. I wonder not 64 bit math on 32 bit arch is good enough on ppc? Jocke ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.or

Re: [PATCH] fixes probe issue and hangup in spi_mpc8xxx.c for data bigger than PAGE_SIZE

2010-09-15 Thread Joakim Tjernlund
> > This patch applies to 2.6.34.7. It also applies to 2.6.35.4 althought > part of it is already included in 2.6.35.4 > It fixed a problem with spi_mpc8xxx.c when transmitting or receiving > data bigger than PAGE_SIZE when doing a read only or write only operation > It also fixed an issue with t

Re: [PATCH] spi_mpc8xxx: fix buffer overrun when sending only/receiving only more than PAGE_SIZE bytes

2010-09-16 Thread Joakim Tjernlund
g > only more than PAGE_SIZE bytes > Sent by: linuxppc-dev-bounces+joakim.tjernlund=transmode...@lists.ozlabs.org > > This patch applies to 2.6.34.7 and 2.6.35.4 > It fixes an issue when sending only or receiving only more than PAGE_SIZE > bytes > > Signed-off-by: chr

Re: [PATCH] spi_mpc8xxx: fix writing to adress 0

2010-09-16 Thread Joakim Tjernlund
linuxppc-dev-bounces+joakim.tjernlund=transmode...@lists.ozlabs.org > > This patch applies to 2.6.34.7 (already included in 2.6.35.4) > It fixes an issue when sending only or receiving only (mspi->tx-dma was reset > as when no tx_buf is defined, tx_dma is 0) > > Signed-off-by: chr

Re: [PATCH 05/04] *** NOT FOR RELEASE *** HACK *** Work around MII clock issue ***

2009-05-07 Thread Joakim Tjernlund
> > Signed-off-by: Wolfgang Denk > Cc: Grant Likely > Cc: John Rigby > --- > This patch is NOT intended for inclusion into mainline, but rather a > request for help. For some reason which I don't understand yet, the > Ethernet interface on the ARIA board does not work in the default > configura

Re: [PATCH 05/04] *** NOT FOR RELEASE *** HACK *** Work around MII clock issue ***

2009-05-07 Thread Joakim Tjernlund
Wolfgang Denk wrote on 07/05/2009 11:19:48: > > Dear Joakim Tjernlund, > > In message you wrote: > > > > Just a stab in the dark: Perhaps the fec->fecp->fec_mii_speed field is > > misaligned or is 16 bits ? > > Good idea. The RM documents the regist

Re: [PATCH] i2c-mpc: generate START condition after STOP caused by read i2c_msg

2009-05-14 Thread Joakim Tjernlund
> > Wolfram Sang wrote: > > On Thu, May 14, 2009 at 10:10:03AM +0200, Esben Haabendal wrote: > > > >> This fixes MAL (arbitration lost) bug caused by illegal use of > >> RSTA (repeated START) after STOP condition generated after last byte > >> of reads. > >> > >> > > > > Could you elaborate a bit,

Re: [PATCH] i2c-mpc: generate START condition after STOP caused by read i2c_msg

2009-05-14 Thread Joakim Tjernlund
Esben Haabendal wrote on 14/05/2009 12:17:48: > On Thu, May 14, 2009 at 11:58 AM, Joakim Tjernlund > wrote: > >> > >> The problem is that after the STOP condition, the following i2c_msg will be > >> attempted with a repeated START, which according to specifica

Re: [PATCH] i2c-mpc: generate START condition after STOP caused by read i2c_msg

2009-05-15 Thread Joakim Tjernlund
Esben Haabendal wrote on 15/05/2009 12:18:39: > > I have now spent a few hours trying a lot of different paths to fix > this approach, but I simply cannot find a way to get i2c read to work > without a trailing STOP condition with this controller. I found a bug which lets me remove the "fix" and

Re: [PATCH] i2c-mpc: generate START condition after STOP caused by read i2c_msg

2009-05-15 Thread Joakim Tjernlund
Esben Haabendal wrote on 15/05/2009 14:52:28: > > Your new patch also does not work. > > Have you tested it? sure, works fine. I haven't stressed it too much though. > > I already tried something very much what you sent here, I believe the > only difference was that I named the "last" variable "

Re: Net: ucc_geth ethernet driver optimization space

2009-05-26 Thread Joakim Tjernlund
linuxppc-dev-bounces+joakim.tjernlund=transmode...@ozlabs.org wrote on 27/05/2009 07:08:07: > > 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.

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

2009-05-28 Thread Joakim Tjernlund
> > 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/d

Re: [PATCH v2] i2c-mpc: generate START condition after STOP caused by read i2c_msg

2009-05-28 Thread Joakim Tjernlund
ne breaks (check > @@-line). Also, I don't really have the time to dig into the topic, so I > would only test it and give a tested-by-tag if it doesn't break anything > here. I think Joakim would be a good candidate for an acked-by . It sure looks O

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

2009-05-28 Thread Joakim Tjernlund
> Am 28.05.09 18:13 schrieb(en) Joakim Tjernlund: > > hmm, these do look a bit unoptimal anyway. Any reason not to write > > them something like below(written by me for uClibc long time ago). > > You will have to add eieio()/sync > > No (and I wasn't aware of th

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

2009-05-31 Thread Joakim Tjernlund
> > Hi Jocke: > > Am 29.05.09 08:31 schrieb(en) Joakim Tjernlund: > > > No (and I wasn't aware of the PPC pre-inc vs. post-inc stuff) - I > > just > > > > I think this is true for most RISC based CPU's. It is a pity as > > post ops a

[PATCH] [spi_mpc83xx] Always enable legacy support.

2009-07-01 Thread Joakim Tjernlund
There are out of tree boards that need this legacy support too. Signed-off-by: Joakim Tjernlund --- I hope this is OK. I just discovered that spi didn't work for me in 2.6.30 and I have no time to work on coverting my boards. I don't have a clue either on how to do that, any pointers

Re: [PATCH] [spi_mpc83xx] Always enable legacy support.

2009-07-01 Thread Joakim Tjernlund
Anton Vorontsov wrote on 01/07/2009 23:12:12: > > On Wed, Jul 01, 2009 at 09:16:12PM +0200, Joakim Tjernlund wrote: > > There are out of tree boards that need this legacy support too. > > Heh. > > > Signed-off-by: Joakim Tjernlund > > --- > > I hope this i

Re: [PATCH 0/2] Setting GPIOs simultaneously

2009-07-13 Thread Joakim Tjernlund
Anton Vorontsov wrote on 13/07/2009 17:19:11: > > Hi all, > > I've been sitting on these patches for some time, but now it appears > that the set_sync() feature is needed elsewhere. So here are the > patches. > > Joakim, I think this is what you need. Yes, it sure looks so :) I will have to look

Re: [PATCH 0/2] Setting GPIOs simultaneously

2009-07-13 Thread Joakim Tjernlund
Joakim Tjernlund/Transmode wrote on 13/07/2009 18:01:02: > > Anton Vorontsov wrote on 13/07/2009 17:19:11: > > > > Hi all, > > > > I've been sitting on these patches for some time, but now it appears > > that the set_sync() feature is needed elsewhere. So

Re: [PATCH 0/2] Setting GPIOs simultaneously

2009-07-13 Thread Joakim Tjernlund
Anton Vorontsov wrote on 13/07/2009 19:34:55: > > On Mon, Jul 13, 2009 at 06:01:02PM +0200, Joakim Tjernlund wrote: > > > > Anton Vorontsov wrote on 13/07/2009 17:19:11: > > > > > > Hi all, > > > > > > I've been sitting on these patche

Ang: Re: [PATCH 0/2] Setting GPIOs simultaneously

2009-07-14 Thread Joakim Tjernlund
-Anton Vorontsov skrev: - >Till: Joakim Tjernlund >Från: Anton Vorontsov >Datum: 07/14/2009 00:20 >Kopia: David Brownell , >linux-ker...@vger.kernel.org, linuxppc-...@ozlabs.org >Ärende: Re: [PATCH 0/2] Setting GPIOs simultaneously > > >On Mon, Jul 13, 2009

Re: Ang: Re: [PATCH 0/2] Setting GPIOs simultaneously

2009-08-04 Thread Joakim Tjernlund
Anton Vorontsov wrote on 15/07/2009 00:09:31: > > On Tue, Jul 14, 2009 at 11:20:13PM +0200, Joakim Tjernlund wrote: > [...] > > >But any users of the legacy bindings should be in-tree. > > > > ehh, it was working until you made it OF only. Why do call the native >

Re: Ang: Re: [PATCH 0/2] Setting GPIOs simultaneously

2009-08-04 Thread Joakim Tjernlund
Anton Vorontsov wrote on 04/08/2009 16:22:50: > > On Tue, Aug 04, 2009 at 03:38:40PM +0200, Joakim Tjernlund wrote: > > Anton Vorontsov wrote on 15/07/2009 00:09:31: > > > > > > On Tue, Jul 14, 2009 at 11:20:13PM +0200, Joakim Tjernlund wrote: > > > [..

Re: [PATCH] powerpc/8xx: drop unused self-modifying code alternative to FixupDAR.

2019-08-22 Thread Joakim Tjernlund
o. Reviewed-by: Joakim Tjernlund Jocke > > Signed-off-by: Christophe Leroy > --- > arch/powerpc/kernel/head_8xx.S | 24 > 1 file changed, 24 deletions(-) > > diff --git a/arch/powerpc/kernel/head_8xx.S b/arch/powerpc/kernel/head_8xx.S > index b8ca5b4

Re: [RFC PATCH 0/4] Remove some e300/MPC83xx evaluation platforms

2023-02-28 Thread Joakim Tjernlund
On Mon, 2023-02-27 at 14:48 -0600, Li Yang wrote: > On Sat, Feb 25, 2023 at 10:52 AM Paul Gortmaker > wrote: > > > > [RE: [RFC PATCH 0/4] Remove some e300/MPC83xx evaluation platforms] On > > 24/02/2023 (Fri 21:16) Leo Li wrote: > > > > > > > > > > > > -Original Message- > > > > From:

Re: fsl_espi errors on v5.7.15

2020-09-07 Thread Joakim Tjernlund
On Thu, 2020-09-03 at 23:58 +, Chris Packham wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > On 1/09/20 6:14 pm, Nicholas Piggin wrote: > > Excerpts from

Re: fsl_espi errors on v5.7.15

2020-09-07 Thread Joakim Tjernlund
[SNIP] > > > > > Would you be able to ftrace the interrupt handler function and see if you > > > can see a difference in number or timing of interrupts? I'm at a bit of > > > a loss. > > > > I tried ftrace but I really wasn't sure what I was looking for. > > Capturing a "bad" case was pretty tri

Re: [PATCH] spi: fsl-espi: Only process interrupts for expected events

2020-09-14 Thread Joakim Tjernlund
On Mon, 2020-09-14 at 12:28 +1000, Nicholas Piggin wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > Excerpts from Chris Packham's message of September 14, 2020

Re: [PATCH] i2c: cpm: Fix i2c_ram structure

2020-09-22 Thread Joakim Tjernlund
On Tue, 2020-09-22 at 11:04 +0200, nico.vi...@gmail.com wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > From: Nicolas VINCENT > > the i2c_ram structure is m

[PATCH] powerpc: Send SIGBUS from machine_check

2020-10-01 Thread Joakim Tjernlund
Embedded PPC CPU should send SIGBUS to user space when applicable. Signed-off-by: Joakim Tjernlund --- arch/powerpc/kernel/traps.c | 17 + 1 file changed, 17 insertions(+) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index 0381242920d9..12715d24141c

Re: [PATCH] powerpc: Send SIGBUS from machine_check

2020-10-22 Thread Joakim Tjernlund
ping Also Cc: sta...@vger.kernel.org On Thu, 2020-10-01 at 19:05 +0200, Joakim Tjernlund wrote: > Embedded PPC CPU should send SIGBUS to user space when applicable. > > Signed-off-by: Joakim Tjernlund > --- >  arch/powerpc/kernel/traps.c | 17 + >  1 file chan

Re: [PATCH] powerpc: Send SIGBUS from machine_check

2020-10-23 Thread Joakim Tjernlund
On Fri, 2020-10-23 at 11:57 +1100, Michael Ellerman wrote: > > > Joakim Tjernlund writes: > > Embedded PPC CPU should send SIGBUS to user space when applicable. > > Yeah, but it's not clear that it's applicable in all cases. > > At least I need some rea

RE: [PATCH 1/3] [NET] phy/fixed.c: rework to not duplicate PHY layerfunctionality

2007-12-29 Thread Joakim Tjernlund
> -Original Message- > From: > [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] > abs.org] On Behalf Of Vitaly Bordug > Sent: den 6 december 2007 23:51 > To: Paul Mackerras > Cc: [EMAIL PROTECTED]; linuxppc-dev > Subject: [PATCH 1/3] [NET] phy/fixed.c: rework to not > duplicate PHY layerfun

RE: [PATCH 0/3] UCC TDM driver for MPC83xx platforms

2008-01-18 Thread Joakim Tjernlund
On Fri, 2008-01-18 at 17:28 +0530, Aggrwal Poonam wrote: > Hello All > > The TDM driver just now does not have a proper framework. Probably the > interface cannot be generalised as such. Hence we could not decide > whether it would be right to think of a TDM framework. Infact the > interface this

BUG: ucc_geth, MDIO only works for UCC3([EMAIL PROTECTED]) on MPC8321

2008-02-14 Thread Joakim Tjernlund
Trying to use other UCC2 or UCC4 for ethernet instead of UCC3 makes MDIO unusable. UCC2 can not find its PHY if not UCC3 is present too. Seems like only UCC3 is able to drive the MDIO bus. Jocke ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org http

[PATCH] Fix ucc_geth MII master selection

2008-02-14 Thread Joakim Tjernlund
Remove bogus UCC regs range test and correct off by one error in call to ucc_set_qe_mux_mii_mng() Signed-off-by: Joakim Tjernlund <[EMAIL PROTECTED]> --- I am not 100% sure this is the correct fix, please ACK or NACK drivers/net/ucc_geth_mii.c | 26 +++--- 1 files c

Re: [PATCH] Fix ucc_geth MII master selection

2008-02-15 Thread Joakim Tjernlund
On Thu, 2008-02-14 at 18:11 +0100, Joakim Tjernlund wrote: > Remove bogus UCC regs range test and correct > off by one error in call to ucc_set_qe_mux_mii_mng() > > Signed-off-by: Joakim Tjernlund <[EMAIL PROTECTED]> > --- > > I am not 100% sure this is the corr

[Fwd: [PATCH] Fix ethernet multicast for ucc_geth.]

2007-10-18 Thread Joakim Tjernlund
No response yet, figured I should try here instead. Jocke Forwarded Message From: Joakim Tjernlund <[EMAIL PROTECTED]> Reply-To: [EMAIL PROTECTED] To: Netdev <[EMAIL PROTECTED]>, Li Yang-r58472 <[EMAIL PROTECTED]> Subject: [PATCH] Fix ethernet multicast for u

RE: [PATCH] Fix ethernet multicast for ucc_geth.

2007-10-21 Thread Joakim Tjernlund
> -Original Message- > From: Li Yang-r58472 [mailto:[EMAIL PROTECTED] > Sent: den 18 oktober 2007 16:24 > To: [EMAIL PROTECTED]; Netdev; linuxppc-dev@ozlabs.org > Subject: RE: [PATCH] Fix ethernet multicast for ucc_geth. > > > -Original Message- > >

Re: [i2c] i2c-mpc.c driver issues

2007-10-26 Thread Joakim Tjernlund
On Fri, 2007-10-26 at 11:53 +0200, Jean Delvare wrote: > Hi Jocke, > > On Wed, 24 Oct 2007 23:06:13 +0200, Tjernlund wrote: > > While browsing the i2c-mpc.c driver I noticed some things that look odd > > to me so I figured I report them. Could not find a maintainer in the > > MAINTANERS file > >

RE: [i2c] i2c-mpc.c driver issues

2007-10-27 Thread Joakim Tjernlund
> -Original Message- > From: > [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] abs.org] On Behalf Of Jean Delvare > Sent: den 26 oktober 2007 11:53 > To: Tjernlund > Cc: linuxppc-dev@ozlabs.org; [EMAIL PROTECTED] > Subject: Re: [i2c] i2c-mpc.c driver issues > > Hi Jocke, > > On Wed, 24 Oc

Re: [PATCH 2/3] [POWERPC] fsl_soc: add support for gianfar for fixed-link property

2007-11-26 Thread Joakim Tjernlund
On Mon, 2007-11-26 at 17:29 +0300, Vitaly Bordug wrote: > fixed-link says: register new "Fixed/emulated PHY", i.e. PHY that > not connected to the real MDIO bus. > > Signed-off-by: Vitaly Bordug <[EMAIL PROTECTED]> > Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> > > --- > > Documentation/p

<    1   2   3   4   5   6   7   8   >