Message ID | 1480566640-27264-1-git-send-email-baiyaowei@cmss.chinamobile.com |
---|---|
State | New |
Headers | show |
On Wed, Nov 30, 2016 at 11:30:38PM -0500, Yaowei Bai wrote: > It's timer to expire, not clock. > > Signed-off-by: Yaowei Bai <baiyaowei@cmss.chinamobile.com> > --- > include/qemu/timer.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) For the whole series: Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> PS: I suggest sending a cover letter "[PATCH 0/3]" in the future. This makes it easy for reviewers to indicate they have reviewed the whole series. Without a cover letter it's ambiguous whether my single Reviewed-by: applies to just this patch or to the whole series - and patch management tools will probably get it wrong too.
On 01/12/2016 14:50, Stefan Hajnoczi wrote: > On Wed, Nov 30, 2016 at 11:30:38PM -0500, Yaowei Bai wrote: >> It's timer to expire, not clock. >> >> Signed-off-by: Yaowei Bai <baiyaowei@cmss.chinamobile.com> >> --- >> include/qemu/timer.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > For the whole series: > > Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> > > PS: I suggest sending a cover letter "[PATCH 0/3]" in the future. This > makes it easy for reviewers to indicate they have reviewed the whole > series. Without a cover letter it's ambiguous whether my single > Reviewed-by: applies to just this patch or to the whole series - and > patch management tools will probably get it wrong too. > I've queued the series for QEMU 2.9. This kind of patch can probably be sent to qemu-trivial@nongnu.org, which will simplify their inclusion. Of course, this is not meant to diminish your contribution! "Trivial" patches are important and good comments will also help the next person studying QEMU's source code. Thanks, Paolo
On Thu, Dec 01, 2016 at 04:23:17PM +0100, Paolo Bonzini wrote: > > > On 01/12/2016 14:50, Stefan Hajnoczi wrote: > > On Wed, Nov 30, 2016 at 11:30:38PM -0500, Yaowei Bai wrote: > >> It's timer to expire, not clock. > >> > >> Signed-off-by: Yaowei Bai <baiyaowei@cmss.chinamobile.com> > >> --- > >> include/qemu/timer.h | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > > > > For the whole series: > > > > Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> > > > > PS: I suggest sending a cover letter "[PATCH 0/3]" in the future. This > > makes it easy for reviewers to indicate they have reviewed the whole > > series. Without a cover letter it's ambiguous whether my single > > Reviewed-by: applies to just this patch or to the whole series - and > > patch management tools will probably get it wrong too. > > > > I've queued the series for QEMU 2.9. This kind of patch can probably be > sent to qemu-trivial@nongnu.org, which will simplify their inclusion. > > Of course, this is not meant to diminish your contribution! "Trivial" > patches are important and good comments will also help the next person > studying QEMU's source code. Got it from both of you, will correct it in the future, thanks. > > Thanks, > > Paolo
diff --git a/include/qemu/timer.h b/include/qemu/timer.h index 309f3d0..c89ed2a 100644 --- a/include/qemu/timer.h +++ b/include/qemu/timer.h @@ -140,7 +140,7 @@ bool qemu_clock_has_timers(QEMUClockType type); * @type: the clock type * * Determines whether a clock's default timer list - * has an expired clock. + * has an expired timer. * * Returns: true if the clock's default timer list has * an expired timer
It's timer to expire, not clock. Signed-off-by: Yaowei Bai <baiyaowei@cmss.chinamobile.com> --- include/qemu/timer.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)