• Status Closed
  • Percent Complete
  • Task Type Bugs
  • Category Operating System/Drivers
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by sideral - 2011-01-13
Last edited by Buschel - 2011-01-23

FS#11873 - shutdown_hw waits for disk spindown twice

SVN r29045

firmware/powermgmt.c:shutdown_hw() contains the following code snippet twice:

  while (storage_disk_is_active())

This looks bogus to me.

Closed by  Buschel
2011-01-23 12:25
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Changed with r29119.

This was introduced with r11507 when submitting  FS#3001 . To me it seems like this was not done intentionally. The latest two patches in  FS#3001  just add the second “while();” whereas all patches before removed the first section and added the
I am sure we can delete the first call.


Available keyboard shortcuts


Task Details

Task Editing