Improve commenting of the Dataflash stub board driver file, to prevent confusion of what the dataflash selection masks need to be. Update stub driver to automatically interleave data between the chips if two Dataflash chips are present.

pull/1469/head
Dean Camera 15 years ago
parent 8b053d6ba5
commit 35b7946950

@ -67,10 +67,10 @@
#define DATAFLASH_NO_CHIP DATAFLASH_CHIPCS_MASK #define DATAFLASH_NO_CHIP DATAFLASH_CHIPCS_MASK
/** Mask for the first dataflash chip selected. */ /** Mask for the first dataflash chip selected. */
#define DATAFLASH_CHIP1 // TODO: Replace with mask to select the first Dataflash chip #define DATAFLASH_CHIP1 // TODO: Replace with mask to hold /CS of first Dataflash low, and all others high
/** Mask for the second dataflash chip selected. */ /** Mask for the second dataflash chip selected. */
#define DATAFLASH_CHIP2 // TODO: Replace with mask to select the second Dataflash chip, if available #define DATAFLASH_CHIP2 // TODO: Replace with mask to hold /CS of second Dataflash low, and all others high
/** Internal main memory page size for the board's dataflash ICs. */ /** Internal main memory page size for the board's dataflash ICs. */
#define DATAFLASH_PAGE_SIZE // TODO: Replace with the page size for the Dataflash ICs #define DATAFLASH_PAGE_SIZE // TODO: Replace with the page size for the Dataflash ICs
@ -95,8 +95,14 @@
if (PageAddress >= (DATAFLASH_PAGES * DATAFLASH_TOTALCHIPS)) if (PageAddress >= (DATAFLASH_PAGES * DATAFLASH_TOTALCHIPS))
return; return;
// TODO: If more than one dataflash chip, select the correct chip from the page address here #if (DATAFLASH_TOTALCHIPS == 2)
Dataflash_SelectChip(DATAFLASH_CHIP1); if (PageAddress & 0x01)
Dataflash_SelectChip(DATAFLASH_CHIP2);
else
Dataflash_SelectChip(DATAFLASH_CHIP1);
#else
Dataflash_SelectChip(DATAFLASH_CHIP1);
#endif
} }
/** Sends a set of page and buffer address bytes to the currently selected dataflash IC, for use with /** Sends a set of page and buffer address bytes to the currently selected dataflash IC, for use with
@ -106,9 +112,11 @@
* \param[in] BufferByte Address within the dataflash's buffer * \param[in] BufferByte Address within the dataflash's buffer
*/ */
static inline void Dataflash_SendAddressBytes(uint16_t PageAddress, const uint16_t BufferByte) static inline void Dataflash_SendAddressBytes(uint16_t PageAddress, const uint16_t BufferByte)
{ {
// TODO: If more than one dataflash chip, adjust absolute page address to be correct for the current chip, #if (DATAFLASH_TOTALCHIPS == 2)
// also the shifts may need to be altered to suit the dataflash model being used PageAddress >>= 1;
#endif
Dataflash_SendByte(PageAddress >> 5); Dataflash_SendByte(PageAddress >> 5);
Dataflash_SendByte((PageAddress << 3) | (BufferByte >> 8)); Dataflash_SendByte((PageAddress << 3) | (BufferByte >> 8));
Dataflash_SendByte(BufferByte); Dataflash_SendByte(BufferByte);

Loading…
Cancel
Save