3 comments:
File src/mainboard/sifive/hifive-unleashed/flash.h:
Patch Set #29, Line 4: * Copyright (C) 2018 HardenedLinux
Per today's announcement about the change in how we do copyright notices, you can remove this copyright line.
Patch Set #29, Line 16: #ifndef __HIFIVE_UNLEASHED_FLASH_H__
we've stopped guarding prototypes with #ifdef it seems so you can delete the preprocessor guards here. Once this is done, you will have a file with one prototype. Can it be put in some other file?
File src/soc/sifive/fu540/spi_flash.c:
Patch Set #29, Line 52: spi_txrx(spictrl, command_enable);
There's no reason to increase code fragmentation. […]
Please follow Rudolph's suggestion. He is right. We've worked hard to avoid this kind of code fragmentation in coreboot, even at the cost of using "heavy" code. The cost is worth it.
To view, visit change 30466. To unsubscribe, or for help writing mail filters, visit settings.