Hello, Damien. On Tue, Mar 22, 2016 at 08:22:22AM +0000, Damien Le Moal wrote: > diff --git a/include/linux/ata.h b/include/linux/ata.h > index c1a2f34..f460e7f 100644 > --- a/include/linux/ata.h > +++ b/include/linux/ata.h > @@ -371,7 +371,8 @@ enum { > SETFEATURES_AAM_ON = 0x42, > SETFEATURES_AAM_OFF = 0xC2, > > - SETFEATURES_SPINUP = 0x07, /* Spin-up drive */ > + SETFEATURES_SPINUP = 0x07, /* Spin-up drive */ > + SETFEATURES_SPINUP_TIMEOUT = 30, /* 30s timeout for drive spin-up from PUIS */ Other timeout constants are defined in msecs. Let's do the same here. Also, can you please send the email as plain-text without the base64 encoding? If you can't, please send the patch as an attachment instead. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html