summaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
Diffstat
-rw-r--r--lib/fatfs/ff.c51
-rw-r--r--lib/fatfs/ff.h10
-rw-r--r--lib/fatfs/ffconf_template.h (renamed from lib/fatfs/ffconf.h)0
3 files changed, 53 insertions, 8 deletions
diff --git a/lib/fatfs/ff.c b/lib/fatfs/ff.c
index 9c887c4596..dd32f7b3c0 100644
--- a/lib/fatfs/ff.c
+++ b/lib/fatfs/ff.c
@@ -119,6 +119,8 @@
/ Fixed null pointer dereference on attempting to delete the root direcotry.
/---------------------------------------------------------------------------*/
+#include <string.h>
+
#include "ff.h" /* Declarations of FatFs API */
#include "diskio.h" /* Declarations of disk I/O functions */
@@ -506,7 +508,8 @@ static FATFS *FatFs[_VOLUMES]; /* Pointer to the file system objects (logical dr
static WORD Fsid; /* File system mount ID */
#if _FS_RPATH && _VOLUMES >= 2
-static BYTE CurrVol; /* Current drive */
+// dpgeorge: changed name and made non-static
+BYTE ff_CurrVol; /* Current drive */
#endif
#if _FS_LOCK
@@ -559,6 +562,12 @@ static const BYTE ExCvt[] = _EXCVT; /* Upper conversion table for extended chara
/* String functions */
/*-----------------------------------------------------------------------*/
+// dpgeorge:
+// We use stdlib version of memset and memcmp to reduce code size.
+// We use the original custom version of mem_cpy so that gcc doesn't
+// recognise the builtin function and then inline it. Allowing gcc
+// to use the builtin memcpy increases code size by 72 bytes.
+
/* Copy memory to memory */
static
void mem_cpy (void* dst, const void* src, UINT cnt) {
@@ -577,6 +586,7 @@ void mem_cpy (void* dst, const void* src, UINT cnt) {
}
/* Fill memory */
+#if 0
static
void mem_set (void* dst, int val, UINT cnt) {
BYTE *d = (BYTE*)dst;
@@ -584,8 +594,13 @@ void mem_set (void* dst, int val, UINT cnt) {
while (cnt--)
*d++ = (BYTE)val;
}
+#else
+// use stdlib
+#define mem_set memset
+#endif
/* Compare memory to memory */
+#if 0
static
int mem_cmp (const void* dst, const void* src, UINT cnt) {
const BYTE *d = (const BYTE *)dst, *s = (const BYTE *)src;
@@ -594,6 +609,10 @@ int mem_cmp (const void* dst, const void* src, UINT cnt) {
while (cnt-- && (r = *d++ - *s++) == 0) ;
return r;
}
+#else
+// use stdlib
+#define mem_cmp memcmp
+#endif
/* Check if chr is contained in the string */
static
@@ -854,6 +873,8 @@ FRESULT sync_fs ( /* FR_OK: successful, FR_DISK_ERR: failed */
/*-----------------------------------------------------------------------*/
/* Hidden API for hacks and disk tools */
+// dpgeorge: made static
+static
DWORD clust2sect ( /* !=0: Sector number, 0: Failed - invalid cluster# */
FATFS* fs, /* File system object */
DWORD clst /* Cluster# to be converted */
@@ -872,6 +893,8 @@ DWORD clust2sect ( /* !=0: Sector number, 0: Failed - invalid cluster# */
/*-----------------------------------------------------------------------*/
/* Hidden API for hacks and disk tools */
+// dpgeorge: made static
+static
DWORD get_fat ( /* 0xFFFFFFFF:Disk error, 1:Internal error, 2..0x0FFFFFFF:Cluster status */
FATFS* fs, /* File system object */
DWORD clst /* FAT item index (cluster#) to get the value */
@@ -927,6 +950,8 @@ DWORD get_fat ( /* 0xFFFFFFFF:Disk error, 1:Internal error, 2..0x0FFFFFFF:Cluste
/* Hidden API for hacks and disk tools */
#if !_FS_READONLY
+// dpgeorge: made static
+static
FRESULT put_fat (
FATFS* fs, /* File system object */
DWORD clst, /* FAT item index (cluster#) to be set */
@@ -2093,6 +2118,8 @@ FRESULT follow_path ( /* FR_OK(0): successful, !=0: error code */
/* Get logical drive number from path name */
/*-----------------------------------------------------------------------*/
+// dpgeorge: replaced with custom ff_get_ldnumber
+#if 0
static
int get_ldnumber ( /* Returns logical drive number (-1:invalid drive) */
const TCHAR** path /* Pointer to pointer to the path name */
@@ -2144,6 +2171,7 @@ int get_ldnumber ( /* Returns logical drive number (-1:invalid drive) */
}
return vol;
}
+#endif
@@ -2197,7 +2225,7 @@ FRESULT find_volume ( /* FR_OK(0): successful, !=0: any error occurred */
/* Get logical drive number from the path name */
*rfs = 0;
- vol = get_ldnumber(path);
+ vol = ff_get_ldnumber(path);
if (vol < 0) return FR_INVALID_DRIVE;
/* Check if the file system object is valid or not */
@@ -2397,7 +2425,7 @@ FRESULT f_mount (
const TCHAR *rp = path;
- vol = get_ldnumber(&rp);
+ vol = ff_get_ldnumber(&rp);
if (vol < 0) return FR_INVALID_DRIVE;
cfs = FatFs[vol]; /* Pointer to fs object */
@@ -2885,10 +2913,10 @@ FRESULT f_chdrive (
int vol;
- vol = get_ldnumber(&path);
+ vol = ff_get_ldnumber(&path);
if (vol < 0) return FR_INVALID_DRIVE;
- CurrVol = (BYTE)vol;
+ ff_CurrVol = (BYTE)vol;
return FR_OK;
}
@@ -2984,11 +3012,19 @@ FRESULT f_getcwd (
tp = buff;
if (res == FR_OK) {
#if _VOLUMES >= 2
+ // dpgeorge: change to use our volume names
+ #if 0
*tp++ = '0' + CurrVol; /* Put drive number */
*tp++ = ':';
+ #else
+ ff_get_volname(ff_CurrVol, &tp);
+ #endif
#endif
if (i == len) { /* Root-directory */
+ // dpgeorge: not needed with volume names
+ #if 0
*tp++ = '/';
+ #endif
} else { /* Sub-directroy */
do /* Add stacked path str */
*tp++ = buff[i++];
@@ -3671,7 +3707,7 @@ FRESULT f_rename (
} else {
mem_cpy(buf, djo.dir+DIR_Attr, 21); /* Save the object information except name */
mem_cpy(&djn, &djo, sizeof (DIR)); /* Duplicate the directory object */
- if (get_ldnumber(&path_new) >= 0) /* Snip drive number off and ignore it */
+ if (ff_get_ldnumber(&path_new) >= 0) /* Snip drive number off and ignore it */
res = follow_path(&djn, path_new); /* and make sure if new object name is not conflicting */
else
res = FR_INVALID_DRIVE;
@@ -3850,6 +3886,7 @@ FRESULT f_setlabel (
/* Create a volume label in directory form */
vn[0] = 0;
+ if (label[0] == '/') label++; // dpgeorge: skip '/' to handle our volume names
for (sl = 0; label[sl]; sl++) ; /* Get name length */
for ( ; sl && label[sl-1] == ' '; sl--) ; /* Remove trailing spaces */
if (sl) { /* Create volume label in directory form */
@@ -4010,7 +4047,7 @@ FRESULT f_mkfs (
/* Check mounted drive and clear work area */
if (sfd > 1) return FR_INVALID_PARAMETER;
- vol = get_ldnumber(&path);
+ vol = ff_get_ldnumber(&path);
if (vol < 0) return FR_INVALID_DRIVE;
fs = FatFs[vol];
if (!fs) return FR_NOT_ENABLED;
diff --git a/lib/fatfs/ff.h b/lib/fatfs/ff.h
index 5eaad5fef6..6f7c86debf 100644
--- a/lib/fatfs/ff.h
+++ b/lib/fatfs/ff.h
@@ -36,7 +36,7 @@ typedef struct {
BYTE pd; /* Physical drive number */
BYTE pt; /* Partition: 0:Auto detect, 1-4:Forced partition) */
} PARTITION;
-extern PARTITION VolToPart[]; /* Volume - Partition resolution table */
+extern const PARTITION VolToPart[]; /* Volume - Partition resolution table */
#define LD2PD(vol) (VolToPart[vol].pd) /* Get physical drive number */
#define LD2PT(vol) (VolToPart[vol].pt) /* Get partition index */
@@ -273,8 +273,16 @@ void ff_rel_grant (_SYNC_t sobj); /* Unlock sync object */
int ff_del_syncobj (_SYNC_t sobj); /* Delete a sync object */
#endif
+// dpgeorge: added the following 3 declarations to support our volume names
+// Current drive
+extern BYTE ff_CurrVol;
+// Returns logical drive number (-1:invalid drive)
+int ff_get_ldnumber(const TCHAR** path);
+
+// Store the volume name into dest, and advance the pointer
+void ff_get_volname(BYTE vol, TCHAR **dest);
/*--------------------------------------------------------------*/
/* Flags and offset address */
diff --git a/lib/fatfs/ffconf.h b/lib/fatfs/ffconf_template.h
index f968d7e077..f968d7e077 100644
--- a/lib/fatfs/ffconf.h
+++ b/lib/fatfs/ffconf_template.h
generated by cgit v1.2.3 (git 2.39.1) at 2025-05-21 21:29:32 +0000