Re: [patch 06/10] 40x decrementer fixes

On Fri, 03 Aug 2007 21:10:19 +0400
Sergei Shtylyov [off-list ref] wrote:

I wrote:

quoted

quoted

Allow generic_calibrate_decr to work for 40x platforms.  Given that the hardware
behavior is identical, this also changes the set_dec function to reload the PIT

    It's not actually identical. When you're writing to the PIT reg on 40x, 
you're alos writing to the hidden PIT auto-reload reg. which is not the case 
for 440 that is true Book E.

I said the hardware _behavior_ is identical.  On both 440 and 405, if
the PIT/DEC counts down to 0 and ARE isn't set, it just doesn't tick
anymore.  The fact that writing to PIT sets the shadow reload reg on
405 doesn't matter since ARE isn't enabled.  The same is true for 440
at the moment, since DECAR is never set.

quoted

quoted

on 40x to match the behavior 44x currently has.

quoted

    Why bother doing this?! This will only warrant you imprecise decrementer 
interrupts while it should be interrupting at the precise period currently (if 
you load PIT once)...

    I.e. the error will only accumulate. NAK.

What error exactly?  I'm not following you there.  If there really is
an error, then we can fix it but it seems that it would be common error
for both 440 and 405 at the moment.

josh