On Mon, 18 Dec 2017, John Snow wrote:
On 12/18/2017 07:30 AM, BALATON Zoltan wrote:
On Mon, 18 Dec 2017, David Gibson wrote:
On Sat, Dec 16, 2017 at 11:42:39PM +0100, BALATON Zoltan wrote:
This is a common generic PCI SATA controller that is also used in PCs
but more importantly guests running on the Sam460ex board prefer this
card and have a driver for it (unlike for other SATA controllers
already emulated).
Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu>
I don't know enough about IDE to give this any more than a quick check
for anything obviously bogus looking.? So,
Reviewed-by: David Gibson <da...@gibson.dropbear.id.au>
John, can you take this through your tree, or should I take it through
the ppc tree?
I think he said before it can go via ppc tree:
http://lists.nongnu.org/archive/html/qemu-devel/2017-08/msg04480.html
Yep, I tend to use Acked-by to indicate that it's okay for you to stage.
I'll give it another look-over in a minute, but I'm fine with adding the
model in general, even if it's not quite complete just yet.
Maybe it's not complete yet but it should work to some degree. I've tested
it with Linux on i386-softmmu and on PPC Sam460ex with u-boot and some
guests (both on x86_64 host) and these could boot from an ide-cd model
connected to this controller so at least basic PIO and UDMA modes should
work. Maybe some more advanced features are missing but don't know if
there are any drivers using those.
On i386 the bios does not have a driver so one cannot boot from this
controller only, so I've tested it with -kernel and -initrd options but
root filesystem was then read from CD connected to this controller and it
could read it and start. I haven't tested it more than that though. Maybe
the card's original option ROM that would allow booting from BIOS might
not work with this device model but I think we could not distribute that
anyway for legal reasons so I did not try that. On the Sam460ex which is
my target for this the u-boot firmware has a driver and can boot from this
controller as well as clients can use it.
As of now this patch only adds it to ppcemb and will only be created by
default on the sam460ex machine so not very likely someone would try to
use it on something else without recompiling QEMU.
Regards,
BALATON Zoltan
