On 2/21/21 9:12 PM, Peter Maydell wrote: > On Sun, 21 Feb 2021 at 20:07, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: >> >> On 2/11/21 12:43 AM, Philippe Mathieu-Daudé wrote: >>> Use the new clock_ns_to_ticks() function in cp0_timer where >>> appropriate. This allows us to remove CPUMIPSState::cp0_count_ns >>> and mips_cp0_period_set(). >>> >>> Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> >>> --- >>> Based-on: <20210209132040.5091-1-peter.mayd...@linaro.org> >>> >>> RFC because this is just a starter patch to test Peter's series >>> providing a handy function which "tells me how many times this >>> clock would tick in this length of time". >>> >>> Also because we could rethink the MIPS CP0 timer logic to avoid >>> too frequent divu128 calls (painful on 32-bit hosts). >>> >>> The style should be updated, using more variables for easier >>> review. env_archcpu() could eventually be dropped (by passing >>> MIPSCPU* instead of CPUMIPSState*). >> >> I guess it is better to wait for Peter patches to get merged >> before posting the updated patches (not much, just one previous >> patch MIPSCPU* instead of CPUMIPSState*). > > This patch is still on my to-review queue, fwiw (though I am > taking a week off work next week, so I probably won't get to > it til March.)
Thanks for the update, I'll repost during the first day of March then :) Also I'll try to rebase my "hw/char/serial: Use the Clock API to feed the UART reference clock" too. Regards, Phil.
