On Tue, 21 Jul 2026 08:07:59 -0700 Pierrick Bouvier <[email protected]> wrote: > > On 7/17/2026 6:32 PM, Brian Cain wrote: > > > > diff --git a/tests/qtest/qct-qtimer-test.c b/tests/qtest/qct-qtimer-test.c > > new file mode 100644 > > index 00000000000..146d98126ac > > --- /dev/null > > +++ b/tests/qtest/qct-qtimer-test.c > > @@ -0,0 +1,213 @@ > > +/* > > + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries. > > + * SPDX-License-Identifier: GPL-2.0-or-later > > + * > > + * QTest testcase for the QCT QTimer > > + */ > > + > > +#include "qemu/osdep.h" > > +#include "libqtest-single.h" > > +#include "hw/timer/qct-qtimer.h" > > +#include "qemu/bitops.h" > > + > > +#define QTIMER_DEFAULT_FREQ_HZ 19200000ULL > > +#define QTIMER_VIEW_BASE 0xfc921000 > > Should this be properties, in case another sec maps this at a different > address/has different freq?
Yeah, makes sense, IMHO. The current QTIMER_VIEW_BASE value matches v66/v68 machines but not sa8775 (0x263a1000). (Though not sure if we want to add a qtimer device to that)
