On 10/30/2015 03:53 PM, Benjamin Rood wrote: > The documentation for the 8070 and 8072 SPCv chip explicitly states that > a minimum of 500ms must elapse before issuing commands, otherwise the > SPCv may not process them and the firmware may get into an unrecoverable > state requiring a reboot. While the Linux guys will probably think this > is 'racy', it is called out in the chip documentation and inserting this > delay makes power management function properly. > > Signed-off-by: Benjamin Rood <brood@xxxxxxxxxxxx> > --- > drivers/scsi/pm8001/pm8001_init.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > Can you please add a comment in the code, too? Just to clarify why this is done, lest that someone else tries to 'optimize' this away. Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxx +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
