mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 1/2] fat: kill unused macros
@ 2012-08-25  8:23 Namjae Jeon
  2012-08-26 13:58 ` OGAWA Hirofumi
  0 siblings, 1 reply; 3+ messages in thread
From: Namjae Jeon @ 2012-08-25  8:23 UTC (permalink / raw)
  To: hirofumi, akpm; +Cc: linux-kernel, Namjae Jeon, Amit Sahrawat

Remove unused macros.

Signed-off-by: Namjae Jeon <linkinjeon@gmail.com>
Signed-off-by: Amit Sahrawat <amit.sahrawat83@gmail.com>
---
 include/linux/msdos_fs.h |   12 ------------
 1 file changed, 12 deletions(-)

diff --git a/include/linux/msdos_fs.h b/include/linux/msdos_fs.h
index 11cc2ac..be3dc72 100644
--- a/include/linux/msdos_fs.h
+++ b/include/linux/msdos_fs.h
@@ -9,18 +9,6 @@
  * The MS-DOS filesystem constants/structures
  */
 
-#define SECTOR_SIZE	512		/* sector size (bytes) */
-#define SECTOR_BITS	9		/* log2(SECTOR_SIZE) */
-#define MSDOS_DPB	(MSDOS_DPS)	/* dir entries per block */
-#define MSDOS_DPB_BITS	4		/* log2(MSDOS_DPB) */
-#define MSDOS_DPS	(SECTOR_SIZE / sizeof(struct msdos_dir_entry))
-#define MSDOS_DPS_BITS	4		/* log2(MSDOS_DPS) */
-#define MSDOS_LONGNAME	256		/* maximum name length */
-#define CF_LE_W(v)	le16_to_cpu(v)
-#define CF_LE_L(v)	le32_to_cpu(v)
-#define CT_LE_W(v)	cpu_to_le16(v)
-#define CT_LE_L(v)	cpu_to_le32(v)
-
 #define MSDOS_ROOT_INO	 1	/* The root inode number */
 #define MSDOS_FSINFO_INO 2	/* Used for managing the FSINFO block */
 
-- 
1.7.9.5


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2] fat: kill unused macros
  2012-08-25  8:23 [PATCH 1/2] fat: kill unused macros Namjae Jeon
@ 2012-08-26 13:58 ` OGAWA Hirofumi
  2012-08-27  1:24   ` Namjae Jeon
  0 siblings, 1 reply; 3+ messages in thread
From: OGAWA Hirofumi @ 2012-08-26 13:58 UTC (permalink / raw)
  To: Namjae Jeon; +Cc: akpm, linux-kernel, Amit Sahrawat

Namjae Jeon <linkinjeon@gmail.com> writes:

> Remove unused macros.

Those are for userland. Are you sure userland doesn't use those anymore?

> Signed-off-by: Namjae Jeon <linkinjeon@gmail.com>
> Signed-off-by: Amit Sahrawat <amit.sahrawat83@gmail.com>
> ---
>  include/linux/msdos_fs.h |   12 ------------
>  1 file changed, 12 deletions(-)
>
> diff --git a/include/linux/msdos_fs.h b/include/linux/msdos_fs.h
> index 11cc2ac..be3dc72 100644
> --- a/include/linux/msdos_fs.h
> +++ b/include/linux/msdos_fs.h
> @@ -9,18 +9,6 @@
>   * The MS-DOS filesystem constants/structures
>   */
>  
> -#define SECTOR_SIZE	512		/* sector size (bytes) */
> -#define SECTOR_BITS	9		/* log2(SECTOR_SIZE) */
> -#define MSDOS_DPB	(MSDOS_DPS)	/* dir entries per block */
> -#define MSDOS_DPB_BITS	4		/* log2(MSDOS_DPB) */
> -#define MSDOS_DPS	(SECTOR_SIZE / sizeof(struct msdos_dir_entry))
> -#define MSDOS_DPS_BITS	4		/* log2(MSDOS_DPS) */
> -#define MSDOS_LONGNAME	256		/* maximum name length */
> -#define CF_LE_W(v)	le16_to_cpu(v)
> -#define CF_LE_L(v)	le32_to_cpu(v)
> -#define CT_LE_W(v)	cpu_to_le16(v)
> -#define CT_LE_L(v)	cpu_to_le32(v)
> -
>  #define MSDOS_ROOT_INO	 1	/* The root inode number */
>  #define MSDOS_FSINFO_INO 2	/* Used for managing the FSINFO block */

-- 
OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2] fat: kill unused macros
  2012-08-26 13:58 ` OGAWA Hirofumi
@ 2012-08-27  1:24   ` Namjae Jeon
  0 siblings, 0 replies; 3+ messages in thread
From: Namjae Jeon @ 2012-08-27  1:24 UTC (permalink / raw)
  To: OGAWA Hirofumi; +Cc: akpm, linux-kernel, Amit Sahrawat

2012/8/26, OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>:
> Namjae Jeon <linkinjeon@gmail.com> writes:
>
>> Remove unused macros.
>
> Those are for userland. Are you sure userland doesn't use those anymore?
Hi. OGAWA.
I missed this point. I will repost if I can be sure.
Thanks a lot.
>
>> Signed-off-by: Namjae Jeon <linkinjeon@gmail.com>
>> Signed-off-by: Amit Sahrawat <amit.sahrawat83@gmail.com>
>> ---
>>  include/linux/msdos_fs.h |   12 ------------
>>  1 file changed, 12 deletions(-)
>>
>> diff --git a/include/linux/msdos_fs.h b/include/linux/msdos_fs.h
>> index 11cc2ac..be3dc72 100644
>> --- a/include/linux/msdos_fs.h
>> +++ b/include/linux/msdos_fs.h
>> @@ -9,18 +9,6 @@
>>   * The MS-DOS filesystem constants/structures
>>   */
>>
>> -#define SECTOR_SIZE	512		/* sector size (bytes) */
>> -#define SECTOR_BITS	9		/* log2(SECTOR_SIZE) */
>> -#define MSDOS_DPB	(MSDOS_DPS)	/* dir entries per block */
>> -#define MSDOS_DPB_BITS	4		/* log2(MSDOS_DPB) */
>> -#define MSDOS_DPS	(SECTOR_SIZE / sizeof(struct msdos_dir_entry))
>> -#define MSDOS_DPS_BITS	4		/* log2(MSDOS_DPS) */
>> -#define MSDOS_LONGNAME	256		/* maximum name length */
>> -#define CF_LE_W(v)	le16_to_cpu(v)
>> -#define CF_LE_L(v)	le32_to_cpu(v)
>> -#define CT_LE_W(v)	cpu_to_le16(v)
>> -#define CT_LE_L(v)	cpu_to_le32(v)
>> -
>>  #define MSDOS_ROOT_INO	 1	/* The root inode number */
>>  #define MSDOS_FSINFO_INO 2	/* Used for managing the FSINFO block */
>
> --
> OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-08-27  1:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-08-25  8:23 [PATCH 1/2] fat: kill unused macros Namjae Jeon
2012-08-26 13:58 ` OGAWA Hirofumi
2012-08-27  1:24   ` Namjae Jeon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome