> r21912 <http://svn.rockbox.org/viewvc.cgi?view=rev;revision=21912>:
> Storage API : remove undeeded target-specific functions
>
> storage_sleep, storage_spin, storage_spindown are only defined if
> #defiend (HAVE_DISK_STORAGE), not for MMC/ATA/SD
> remove already unneeded nand_disk_is_active, nand_soft_reset

I don't understand that one.

Isn't the point of an API to forget about underlying differences so that
a developer doesn't need to care/know about this differences? This
commit introduces more (imo ugly) #ifdefs with destroying that point.

The functions are not target-specific, they're (if at all) storage type
specific, which is what the API wanted to hide.

Best regards.
Received on 2009-07-17