Re: [Qemu-devel] [PATCH] DM9000 network card driver

[Daniel Silverstone <dsilvers@simtec.co.uk>]

On Mon, 2008-11-10 at 18:47 +0200, Blue Swirl wrote:
> >  I appreciate that you're all busy, so if the answer is simply "You're on
> >  the list, just it'll be a while" then at least I can carry that back to
> >  my boss who wants to know how it's going getting these patches to you.
> Your patch looks very clean. I wouldn't use devices.h for the
> prototype, but instead some board file (for example arm-misc.h).

I think I was told to use devices.h quite a while back. It's not
appropriate to put it in arm-misc.h since the device itself has no
specific isolation in ARM stuff. Despite being made to put the HW in the
ARM HW bits of the Makefile.

> The device is not used by any board, so it's not possible even for the
> maintainer to test the patch.

It is used by the board I am trying to get all the patches up-to-snuff
for.

> As there is a reset function already, it should not be too difficult
> to register that for system_reset use.

Can you explain what you mean by that?

> Save/load functions would be nice, though IIRC ARM boards/devices
> don't have any yet.

Some do, some don't -- a portion of what I have already has save/load
support, but this is one device I've not done it for yet. I will
continue to look at that.

Thank you very much for your feedback. Hopefully if you can explain what
this system_reset stuff is, I can get that done and punt a fresh patch
to the list.

Regards,

Daniel.

-- 
Daniel Silverstone                              http://www.simtec.co.uk/
PGP mail accepted and encouraged.            Key Id: 2BC8 4016 2068 7895