On 11 January 2018 at 04:59, David Gibson <da...@gibson.dropbear.id.au> wrote: > From: BALATON Zoltan <bala...@eik.bme.hu> > > 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> > Acked-by: John Snow <js...@redhat.com> > Signed-off-by: David Gibson <da...@gibson.dropbear.id.au>
> + case 0x1c8: > + val = d->regs[1].sien << 16; > + break; There's a similar unintended-sign-extension issue here (CID 1385145); again, a cast should fix. thanks -- PMM