The comment edited in this patch was the only reference to the FAT_FIRST_ENT macro, which is not used anymore. Moreover, the commented line of code does not compile with the current code. Since the FAT_FIRST_ENT macro checks the FAT variant in a way that the patch series changes, I removed it, and instead wrote a clear explanation of what was checked. I verified that the changed comment is correct according to Microsoft FAT spec, search for "BPB_Media" in the following references: 1. Microsoft FAT specification 2005 (http://read.pudn.com/downloads77/ebook/294884/FAT32%20Spec%20%28SDA%20Contribution%29.pdf). Search for 'volume label'. 2. Microsoft Extensible Firmware Initiative, FAT32 File System Specification (https://staff.washington.edu/dittrich/misc/fatgen103.pdf). Search for 'volume label'.
Showing the changes to include/uapi/linux (1 of 2 files). Show all
Unified · Split
| @@ -58,9 +58,6 @@ · 57 unmodified lines | |||
| 58 | #define MSDOS_DOT ". " /* ".", padded to MSDOS_NAME chars */ | 58 | #define MSDOS_DOT ". " /* ".", padded to MSDOS_NAME chars */ |
| 59 | #define MSDOS_DOTDOT ".. " /* "..", padded to MSDOS_NAME chars */ | 59 | #define MSDOS_DOTDOT ".. " /* "..", padded to MSDOS_NAME chars */ |
| 60 | 60 | ||
| 61 | #define FAT_FIRST_ENT(s, x) ((MSDOS_SB(s)->fat_bits == 32 ? 0x0FFFFF00 : \ | ||
| 62 | MSDOS_SB(s)->fat_bits == 16 ? 0xFF00 : 0xF00) | (x)) | ||
| 63 | |||
| 64 | /* start of data cluster's entry (number of reserved clusters) */ | 61 | /* start of data cluster's entry (number of reserved clusters) */ |
| 65 | #define FAT_START_ENT 2 | 62 | #define FAT_START_ENT 2 |
| 66 | 63 | ||