On 01/10/2012 04:11 PM, Peter Maydell wrote:
+    case UINTP:
+        s->reg[I_(UINTP)]&= ~val;
+        /* XXX: It's the assumption that it works in this way */
Can you expand this comment a little to clarify exactly what the
assumption you're making is?
It was not clear from documentation if UINTSP register bits should be cleared inside
this "case" too. Practise has showed that should. I think, this comment
message could be removed.

--
Kind regards,
Evgeny Voevodin,
Leading Software Engineer,
ASWG, Moscow R&D center, Samsung Electronics
e-mail: e.voevo...@samsung.com


Reply via email to