[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] hw/timer/pl031: Add warning "do not use as model to implement a
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH] hw/timer/pl031: Add warning "do not use as model to implement a RTC" |
Date: |
Tue, 24 Sep 2019 16:12:28 +0200 |
Peter said we should "not use the pl031 as a good model of how
to do an RTC, it has some definite flaws."
Add this information to the PL031 header for other developers.
Suggested-by: Peter Maydell <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
---
hw/timer/pl031.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/hw/timer/pl031.c b/hw/timer/pl031.c
index 2b3e261006..da453b5b04 100644
--- a/hw/timer/pl031.c
+++ b/hw/timer/pl031.c
@@ -1,6 +1,14 @@
/*
* ARM AMBA PrimeCell PL031 RTC
*
+ * Note from the maintainer:
+ *
+ * Don't use the pl031 code as a good model of how to do an RTC,
+ * it has some definite flaws. x86 or ppc RTC handling is probably
+ * a better place to look.
+ *
+ * See https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg05399.html
+ *
* Copyright (c) 2007 CodeSourcery
*
* This file is free software; you can redistribute it and/or modify
--
2.20.1
- [PATCH] hw/timer/pl031: Add warning "do not use as model to implement a RTC",
Philippe Mathieu-Daudé <=