[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [PATCH] hw/timer/imx_gpt: Properly mask control regis

From: Michael Tokarev
Subject: Re: [Qemu-trivial] [PATCH] hw/timer/imx_gpt: Properly mask control register on software reset
Date: Wed, 14 Sep 2016 10:47:19 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.2.0

07.08.2016 09:50, Donny Yang wrote:
Signed-off-by: Donny Yang <address@hidden>
 hw/timer/imx_gpt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/timer/imx_gpt.c b/hw/timer/imx_gpt.c
index 82bc73c..02d875f 100644
--- a/hw/timer/imx_gpt.c
+++ b/hw/timer/imx_gpt.c
@@ -306,8 +306,8 @@ static void imx_gpt_reset(DeviceState *dev)
      * Soft reset doesn't touch some bits; hard reset clears them
-               GPT_CR_WAITEN|GPT_CR_DBGEN);
     s->sr = 0;
     s->pr = 0;
     s->ir = 0;

Hello.  Thank you for the patch.

Please don't send html-formatted emails, patch handling software
dislikes it.

And please include qemu-devel@ on all patches, so more people
can review your changes (Cc'd now).

This change, while one-liner, isn't obvious for anoyne who
isn't familiar with the timer code. I for one don't know what
is happening there and hence don't know if this change is ok
or not.

Maybe someone else can confirm this is right thing to do.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]