]> git.kernelconcepts.de Git - karo-tx-uboot.git/commitdiff
FAT: remove cur_part_nr
authorStephen Warren <swarren@nvidia.com>
Wed, 17 Oct 2012 06:44:57 +0000 (06:44 +0000)
committerTom Rini <trini@ti.com>
Thu, 25 Oct 2012 19:07:41 +0000 (12:07 -0700)
A future patch will implement the more standard filesystem API
fat_set_blk_dev(). This API has no way to know which partition number
the partition represents. Equally, future DM rework will make the
concept of partition number harder to pass around.

So, simply remove cur_part_nr from fat.c; its only use is in a
diagnostic printf, and the context where it's printed should make it
obvious which partition is referred to anyway (since the partition ID
would come from the user command-line that caused it).

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Reviewed-by: Benoît Thébaudeau <benoit.thebaudeau@advansee.com>
fs/fat/fat.c

index 4a60a2503e7733461a8b75866b236cce77e19c81..731d3690c318d523aa216a4b0e4785e062886730 100644 (file)
@@ -46,7 +46,6 @@ static void downcase(char *str)
 }
 
 static block_dev_desc_t *cur_dev;
-static unsigned int cur_part_nr;
 static disk_partition_t cur_part_info;
 
 #define DOS_BOOT_MAGIC_OFFSET  0x1fe
@@ -77,10 +76,8 @@ int fat_register_device(block_dev_desc_t * dev_desc, int part_no)
      defined(CONFIG_SYSTEMACE) )
 
        /* Read the partition table, if present */
-       if (!get_partition_info(dev_desc, part_no, &cur_part_info)) {
+       if (!get_partition_info(dev_desc, part_no, &cur_part_info))
                cur_dev = dev_desc;
-               cur_part_nr = part_no;
-       }
 #endif
 
        /* Otherwise it might be a superfloppy (whole-disk FAT filesystem) */
@@ -92,7 +89,6 @@ int fat_register_device(block_dev_desc_t * dev_desc, int part_no)
                }
 
                cur_dev = dev_desc;
-               cur_part_nr = 1;
                cur_part_info.start = 0;
                cur_part_info.size = dev_desc->lba;
                cur_part_info.blksz = dev_desc->blksz;
@@ -1239,8 +1235,7 @@ int file_fat_detectfs(void)
        vol_label[11] = '\0';
        volinfo.fs_type[5] = '\0';
 
-       printf("Partition %d: Filesystem: %s \"%s\"\n", cur_part_nr,
-               volinfo.fs_type, vol_label);
+       printf("Filesystem: %s \"%s\"\n", volinfo.fs_type, vol_label);
 
        return 0;
 }