Re: [PATCH] P1021: set IReady in QE Microcode Upload

2012-07-10 Thread Kumar Gala

On Nov 11, 2011, at 10:05 AM, Kokoris, Ioannis wrote:

 Hi,
 
 QE Microcode Initialization using qe_upload_microcode() does not work on 
 P1021 if the IRAM-Ready register is not set after the microcode upload. This 
 patch adds a definition for the I-RAM Ready register and sets it uppon 
 microcode upload completion.
 
 
 Signed-off-by: Ioannis Kokkoris ioannis.koko...@siemens-enterprise.com
 

applied to next

- k
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2012-07-02 Thread Timur Tabi
Kumar Gala wrote:
  No, I forgot all about it.  I'll try it today, assuming the lone 8323
  board in the board farm still works.

 Do you remember if you ever tested this?

Well, I tried to test it.  We have an 83xx board that has a QE UART that
needs firmware uploaded, but I don't know how to connect that QE to an
actual RS232 port.  It needs a riser which I don't think we have.

I can verify that the QE UART driver does not hang which this patch, and
that it appears to still be sending data, but I have no way of knowing for
sure.

-- 
Timur Tabi
Linux kernel developer at Freescale

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2012-06-29 Thread Kumar Gala

On Mar 16, 2012, at 11:07 AM, Timur Tabi wrote:

 Kumar Gala wrote:
 Haiying said it should be ok, but I haven't tried it yet.  I'll try it on 
 Monday.
 
 Did you ever test this?
 
 No, I forgot all about it.  I'll try it today, assuming the lone 8323
 board in the board farm still works.

Do you remember if you ever tested this?

- k
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2012-06-29 Thread Timur Tabi
Kumar Gala wrote:
 
 On Mar 16, 2012, at 11:07 AM, Timur Tabi wrote:
 
 Kumar Gala wrote:
 Haiying said it should be ok, but I haven't tried it yet.  I'll try it on 
 Monday.

 Did you ever test this?

 No, I forgot all about it.  I'll try it today, assuming the lone 8323
 board in the board farm still works.
 
 Do you remember if you ever tested this?

No, I forgot again.  Ugh, sorry.  I'll do it now.

-- 
Timur Tabi
Linux kernel developer at Freescale

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2012-03-16 Thread Kumar Gala

On Nov 24, 2011, at 7:57 AM, Tabi Timur-B04825 wrote:

 On Nov 24, 2011, at 1:55 AM, Kumar Gala ga...@kernel.crashing.org wrote:
 
 
 I'll have to check.  But this patch can't be applied as-is unless it's
 proven safe for all QE-enabled chips.
 
 Any update on trying this on a MPC8323?
 
 Haiying said it should be ok, but I haven't tried it yet.  I'll try it on 
 Monday.

Did you ever test this?

- k
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2012-03-16 Thread Timur Tabi
Kumar Gala wrote:
  Haiying said it should be ok, but I haven't tried it yet.  I'll try it on 
  Monday.

 Did you ever test this?

No, I forgot all about it.  I'll try it today, assuming the lone 8323
board in the board farm still works.

-- 
Timur Tabi
Linux kernel developer at Freescale

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2011-11-24 Thread Tabi Timur-B04825
On Nov 24, 2011, at 1:55 AM, Kumar Gala ga...@kernel.crashing.org wrote:

 
 I'll have to check.  But this patch can't be applied as-is unless it's
 proven safe for all QE-enabled chips.
 
 Any update on trying this on a MPC8323?

Haiying said it should be ok, but I haven't tried it yet.  I'll try it on 
Monday.

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2011-11-23 Thread Kumar Gala

On Nov 15, 2011, at 9:02 PM, Tabi Timur-B04825 wrote:

 On Mon, Nov 14, 2011 at 2:55 AM, Kokoris, Ioannis
 ioannis.koko...@siemens-enterprise.com wrote:
 Ready register is needed for ROM-less devices such as P1021, MPC859, MPC8306 
 etc.
 For ROM-based devices such as MCP8323 the Ready register does not exist.
 Is there a global definition for conditionally including this code?
 
 I'll have to check.  But this patch can't be applied as-is unless it's
 proven safe for all QE-enabled chips.

Any update on trying this on a MPC8323?

- k
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2011-11-15 Thread Tabi Timur-B04825
On Mon, Nov 14, 2011 at 2:55 AM, Kokoris, Ioannis
ioannis.koko...@siemens-enterprise.com wrote:
 Ready register is needed for ROM-less devices such as P1021, MPC859, MPC8306 
 etc.
 For ROM-based devices such as MCP8323 the Ready register does not exist.
 Is there a global definition for conditionally including this code?

I'll have to check.  But this patch can't be applied as-is unless it's
proven safe for all QE-enabled chips.

-- 
Timur Tabi
Linux kernel developer at Freescale
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


RE: [PATCH] P1021: set IReady in QE Microcode Upload

2011-11-14 Thread Kokoris, Ioannis
Kokoris, Ioannis wrote:
 Hi,

 QE Microcode Initialization using qe_upload_microcode() does not work
 on P1021 if the IRAM-Ready register is not set after the microcode
 upload. This patch adds a definition for the I-RAM Ready register and
 sets it uppon microcode upload completion.

Will this code still work on other QE parts, like the MPC8323?


Ready register is needed for ROM-less devices such as P1021, MPC859, MPC8306 
etc. 
For ROM-based devices such as MCP8323 the Ready register does not exist.
Is there a global definition for conditionally including this code?

Regards,
Ioannis
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] P1021: set IReady in QE Microcode Upload

2011-11-11 Thread Tabi Timur-B04825
Kokoris, Ioannis wrote:
 Hi,

 QE Microcode Initialization using qe_upload_microcode() does not work
 on P1021 if the IRAM-Ready register is not set after the microcode
 upload. This patch adds a definition for the I-RAM Ready register and
 sets it uppon microcode upload completion.

Will this code still work on other QE parts, like the MPC8323?

-- 
Timur Tabi
Linux kernel developer at Freescale
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev