"Period changes take" in Audio_registers.md - NR13 paragraph

[marcie1000]

Hello,

It seams there is an unfinished sentence at the end of the paragraph NR13, it is only written "Period changes take".

[ISSOtm]

I think it's meant to say "Period changes take effect only when the previous period next expires"; cc @LIJI32 to confirm that that's right.

Also, not sure that formulation is super clear.

[LIJI32]

Yes, that's correct.

[ISSOtm]

Alright, now to find a way to phrase it that makes sense not just in retrospect.

[ISSOtm]

I wrote this in #334:

Note that the value written to this field is not re-read by the hardware until a sweep iteration completes, or the channel is (re)triggered.

Does that sound clear? If so, then it can be used as a base for the new explanation.

[marcie1000]

I think it's meant to say "Period changes take effect only when the previous period next expires"; cc @LIJI32 to confirm that that's right.

Also, not sure that formulation is super clear.

I wrote this in #334:

Note that the value written to this field is not re-read by the hardware until a sweep iteration completes, or the channel is (re)triggered.

Does that sound clear? If so, then it can be used as a base for the new explanation.

To me, both sound pretty clear!

[ISSOtm]

I'm afraid of the first one only making sense "in retrospect", i.e. if you already understand what "when the period expires" means—it's really talking about the counter used to implement that period.

Regardless, it turns out that the explanation was already added in #486, so maybe a warning-type box reading "writes to this register may not take effect immediately, see the next paragraph for more details" would do the trick?