AT91 rework: pm9261, pm9263 and pm9g45
PATCH 1/1 AT91 rework: pm9261, pm9263 and pm9g45
Hello,
I got several instances of the patches, probably you got it too, so I apology for the mess.
Here is the list of patches:
[PM9261]
http://patchwork.ozlabs.org/patch/99663/
http://patchwork.ozlabs.org/patch/99665/
http://patchwork.ozlabs.org/patch/99666/
[PM9263]
http://patchwork.ozlabs.org/patch/99662/
http://patchwork.ozlabs.org/patch/99668/
[PM9G45]
http://patchwork.ozlabs.org/patch/99664/
http://patchwork.ozlabs.org/patch/99667/
Regards,
Asen
PATCH 1/1 AT91 rework: pm9261, pm9263 and pm9g45
Dear Asen Dimov,
Quoted 4 lines
Hello,
I got several instances of the patches, probably you got it too, so I apology for the mess.
Actually, I did get none per e-Mail, I see them in patchworks only.
Quoted 5 lines
Here is the list of patches:
The empty reset_timer() function added there can obviously only solve build issues.
On any account reset_timer() must not be used anymore. As such this patch must get a NAK.
Quoted 10 lines
http://patchwork.ozlabs.org/patch/99665/
http://patchwork.ozlabs.org/patch/99666/[PM9263]
http://patchwork.ozlabs.org/patch/99662/
http://patchwork.ozlabs.org/patch/99668/[PM9G45]
http://patchwork.ozlabs.org/patch/99664/
http://patchwork.ozlabs.org/patch/99667/
I will check on those 6 remaining patches now.
Best Regards,
Reinhard
PATCH 1/1 AT91 rework: pm9261, pm9263 and pm9g45
Am 09.06.2011 12:57, schrieb Reinhard Meyer:
Quoted 11 lines
Dear Asen Dimov,
Quoted 4 lines
Hello,
I got several instances of the patches, probably you got it too, so I apology for the mess.
Actually, I did get none per e-Mail, I see them in patchworks only.
Quoted 3 lines
Here is the list of patches:
Quoted 18 lines
Quoted 2 lines
The empty
reset_timer()function added there can obviously only solve build issues.
On any accountreset_timer()must not be used anymore. As such this patch must get a NAK.
Ah, and patches should be numbered 1/n .. n/n; 0/n is only for a descriptive summary.
Best Regards,
Reinhard
PATCH 1/1 AT91 rework: pm9261, pm9263 and pm9g45
Hi Reinhard,
On 06/09/2011 02:00 PM, Reinhard Meyer wrote:
...
Quoted 7 lines
Quoted 6 lines
Quoted 3 lines
Quoted 3 lines
[PM9G45]
http://patchwork.ozlabs.org/patch/99664/
http://patchwork.ozlabs.org/patch/99667/Ah, and patches should be numbered 1/n .. n/n; 0/n is only for a descriptive summary.
should I repost them numbering with 1/n?
Quoted 2 lines
Best Regards,
Reinhard
Regards,
Asen
PATCH 1/1 AT91 rework: pm9261, pm9263 and pm9g45
Dear Asen Dimov,
Quoted 11 lines
Hi Reinhard,
On 06/09/2011 02:00 PM, Reinhard Meyer wrote:
...Quoted 7 lines
Quoted 7 lines
Quoted 6 lines
Quoted 3 lines
Quoted 3 lines
[PM9G45]
http://patchwork.ozlabs.org/patch/99664/
http://patchwork.ozlabs.org/patch/99667/Ah, and patches should be numbered 1/n .. n/n; 0/n is only for a descriptive summary.
should I repost them numbering with 1/n?
After applying, the text within [] is gone anyway, so it is ok for me.
Unless someone mandates otherwise...
Best Regards,
Reinhard
PATCH 1/1 AT91 rework: pm9261, pm9263 and pm9g45
In reply to Reinhard MeyerHello Reinhard,
On 06/09/2011 01:57 PM, Reinhard Meyer wrote:
...
Quoted 6 lines
Dear Asen Dimov,
The emptyreset_timer()function added there can obviously only solve build issues.
On any accountreset_timer()must not be used anymore. As such this patch must get a NAK.
The architectures, except AT91 are using reset_timer() to make epochs (start from zero). I don't want to break the other architectures and I need the CFI driver for pm92613 and pm9261. I can not think of another idea, except an empty reset_timer(). Any suggestions, ideas?
...
Quoted 6 lines
Best Regards,
Reinhard
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Regards,
Asen
PATCH 1/1 AT91 rework: pm9261, pm9263 and pm9g45
Dear Asen Dimov,
Quoted 10 lines
Hello Reinhard,
On 06/09/2011 01:57 PM, Reinhard Meyer wrote:
...Quoted 6 lines
Dear Asen Dimov,
The emptyreset_timer()function added there can obviously only solve build issues.
On any accountreset_timer()must not be used anymore. As such this patch must get a NAK.
Quoted 4 lines
The architectures, except AT91 are using
reset_timer()to make epochs (start from zero). I don't want to break the other architectures and I need the CFI driver for pm92613 and pm9261. I can not think of another idea, except an emptyreset_timer(). Any suggestions, ideas?
1. an empty reset_timer() will allow you to build, but at runtime it must break.
2._if_ the current CFI driver is based on reset_timer() that should actually be fixed there.
Whereby the actual impact on other architectures that have a broken get_timer() implementation and therefore _require_ reset_timer() is unclear to me. We just had a lengthy discussion about timer API etc. In essence, this discussion, whatever exact API will be implemented, resulted in NOT having any reset_timer() and only a monotonous, millisecond returning get_timer() function.
The only _interim_ solution would be to reintroduce the original reset_timer() to AT91, which I am NOT fond of.
Wolfgang?
Best Regards,
Reinhard