________________________________________
From: Aurelien Jarno [aurel...@aurel32.net]
Sent: Friday, May 17, 2013 7:34 PM
To: Petar Jovanovic
Cc: qemu-devel@nongnu.org; Petar Jovanovic
Subject: Re: [PATCH 2/2] target-mips: fix EXTPDP and setting up pos field in 
the DSPControl reg

I am not sure it is correct there for the 64-bit mode.

The value that should be written should be DSPcontrolpos6..0 - (size
+ 1), therefore keeeping only last 6 bits doesn't seems correct to me.
Given the change above, I think it should simply be
set_DSPControl_pos(sub, env);

----------

You are correct. I have submitted a second version of the patch.

Petar

Reply via email to