[flashrom] Unused functions...

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Fri Mar 26 00:31:00 CET 2010


We have a few unused functions in our codebase.

Should we
- mark then static (may cause gcc to complain if -Wunused-function is
enabled)
- put them inside #if 0 (code will bitrot)
- remove them from the source base entirely (some may be used in the future)

jedec.c:
data_polling_jedec (maybe some chip in the future won't support
toggle_jedec)

stm50flw0x0x.c:
erase_sector_stm50flw0x0x (keep?)

m29f400bt.c:
write_m29f400bt (hm, probably kill)

82802ab.c:
erase_82802ab (kill, old-style erase)

pm49fl00x.c:
lock_49fl00x (for the future)

sst49lfxxxc.c:
unlock_block_49lfxxxc (strange, don't we use that?)

spi.c:
default_spi_send_command (if one chip driver ever supports only
multicommand)

spi25.c:
spi_write_status_enable (we should use it, some chips require that)
spi_write_disable (does that make sense?)
spi_aai_write (need some write infrastructure changes for this)
spi_chip_erase_d8 (kill)

flashrom.c:
chip_writel (will we ever need it?)
chip_writen (this one might be usable for a few chip drivers)
chip_readl (will we ever need it?)
list_programmers (no idea)
register_shutdown (needed for ECs)
need_erase (needed for partial write)

internal.c:
pci_dev_find_filter (hmmm)

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the flashrom mailing list