On Thu, Jun 30, 2011 at 22:09, Peter Maydell <peter.mayd...@linaro.org> wrote:
> On 30 June 2011 20:28, Anthony PERARD <anthony.per...@citrix.com> wrote:
>> @@ -202,6 +201,11 @@ rxbufsize(uint32_t v)
>> static void
>> set_ctrl(E1000State *s, int index, uint32_t val)
>> {
>> + DBGOUT(IO, "set ctrl = %08x\n", val);
>> + if (val & E1000_CTRL_RST) {
>> + s->mac_reg[CTRL] = val;
>> + e1000_reset(s);
>> + }
>> /* RST is self clearing */
>> s->mac_reg[CTRL] = val & ~E1000_CTRL_RST;
>> }
>
> This is still not right -- you want something like:
:(, sorry, I'll resend the patch with that.
> if (val & E1000_CTRL_RST) {
> e1000_reset(s);
> return;
> }
> s->mac_reg[CTRL] = val;
Thanks,
--
Anthony PERARD
