On 2009-12-06 00:10:13 (+0000), Alexander Motin <[email protected]> wrote:
> Author: mav
> Date: Sun Dec 6 00:10:13 2009
> New Revision: 200171
> URL: http://svn.freebsd.org/changeset/base/200171
I think this commit broke the build for mv_sata.
sys/arm/mv/mv_sata.c currently reads:
@@ -751,10 +751,10 @@
static int
sata_channel_setmode(device_t parent, int target, int mode)
{
/* Disable EDMA before using legacy registers */
sata_edma_ctrl(parent, 0);
return (ata_sata_setmode(dev, mode));
}
This doesn't build because 'dev' isn't defined. As far as I can tell the
following patch matches what was intended, but I have no idea if it's
correct. It does at least build:
Index: sys/arm/mv/mv_sata.c
===================================================================
--- sys/arm/mv/mv_sata.c (revision 200245)
+++ sys/arm/mv/mv_sata.c (working copy)
@@ -751,10 +751,10 @@
static int
sata_channel_setmode(device_t parent, int target, int mode)
{
/* Disable EDMA before using legacy registers */
sata_edma_ctrl(parent, 0);
- return (ata_sata_setmode(dev, mode));
+ return (min(mode, ATA_UDMA5));
}
Regards,
Kristof
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"
